Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(85)

Unified Diff: chrome/renderer/external_extension.cc

Issue 1407163002: Add feature use counting for window.external.AddSearchProvider (Blink side). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: way too much contention on this enum Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/frame/UseCounter.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/external_extension.cc
diff --git a/chrome/renderer/external_extension.cc b/chrome/renderer/external_extension.cc
index 014d388f581c7150bd237603c2fcc42dd813aff3..ef02cac8703f7b597daca8ef82eb37a5d7a912e5 100644
--- a/chrome/renderer/external_extension.cc
+++ b/chrome/renderer/external_extension.cc
@@ -116,6 +116,7 @@ void ExternalExtensionWrapper::AddSearchProvider(
GURL osdd_url = GURL(webframe->document().url()).Resolve(osdd_string);
if (!osdd_url.is_empty() && osdd_url.is_valid()) {
+ webframe->didCallAddSearchProvider();
render_view->Send(new ChromeViewHostMsg_PageHasOSDD(
render_view->GetRoutingID(), webframe->document().url(), osdd_url,
search_provider::EXPLICIT_PROVIDER));
@@ -144,11 +145,10 @@ void ExternalExtensionWrapper::IsSearchProviderInstalled(
search_provider::InstallState install = search_provider::DENIED;
GURL inquiry_url = GURL(webframe->document().url()).Resolve(name);
if (!inquiry_url.is_empty()) {
- render_view->Send(new ChromeViewHostMsg_GetSearchProviderInstallState(
- render_view->GetRoutingID(),
- webframe->document().url(),
- inquiry_url,
- &install));
+ webframe->didCallIsSearchProviderInstalled();
+ render_view->Send(new ChromeViewHostMsg_GetSearchProviderInstallState(
+ render_view->GetRoutingID(), webframe->document().url(), inquiry_url,
+ &install));
}
if (install == search_provider::DENIED) {
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/frame/UseCounter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698