Originally created by: KirtiRamchandani
The global context search sample still suggested that selecting a country switched to that country's Google domain. As noted in [#1653], country-specific Google domains no longer provide a useful visible difference for this sample.
The sample encoded locale choice as Google Search hostnames such as google.ca or google.co.jp, so the UI described obsolete domain switching instead of an explicit search-region behavior.
Use https://www.google.com/search for all searches and add the Google Search cr=countryXX country restriction parameter based on the selected context-menu item. Rename the sample UI/storage identifiers from TLDs to countries and refresh the README/manifest copy to describe country-restricted results.
npx eslint --no-warn-ignored api-samples/contextMenus/global_context_search/background.js api-samples/contextMenus/global_context_search/popup.js --quietnpx prettier --check api-samples/contextMenus/global_context_search/background.js api-samples/contextMenus/global_context_search/locales.js api-samples/contextMenus/global_context_search/popup.js api-samples/contextMenus/global_context_search/popup.html api-samples/contextMenus/global_context_search/manifest.json api-samples/contextMenus/global_context_search/README.mdgit diff --check HEAD~1..HEADchrome.* APIs verifying menu creation, cr=countryCA search URLs, and country storage updatesFixes [#1653]
Originally posted by: KirtiRamchandani
Closing this in favor of the cleaned-up follow-up in [#1697], which keeps the same issue focused but removes the extra unrelated file touch and has the latest verification. Thanks.
Related
Tickets: #1697
Ticket changed by: KirtiRamchandani