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

Unified Diff: chrome/browser/extensions/api/web_navigation/web_navigation_api_helpers.cc

Issue 1236493004: Final batch adding real histogram values for extension events. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: android compile Created 5 years, 5 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
Index: chrome/browser/extensions/api/web_navigation/web_navigation_api_helpers.cc
diff --git a/chrome/browser/extensions/api/web_navigation/web_navigation_api_helpers.cc b/chrome/browser/extensions/api/web_navigation/web_navigation_api_helpers.cc
index ed92ea81a3dad4e46bd0caa8681162562e86d55d..3b04125a35373f2826840b85c5a3914624cbc65d 100644
--- a/chrome/browser/extensions/api/web_navigation/web_navigation_api_helpers.cc
+++ b/chrome/browser/extensions/api/web_navigation/web_navigation_api_helpers.cc
@@ -39,6 +39,7 @@ double MilliSecondsFromTime(const base::Time& time) {
// Dispatches events to the extension message service.
void DispatchEvent(content::BrowserContext* browser_context,
+ events::HistogramValue histogram_value,
const std::string& event_name,
scoped_ptr<base::ListValue> args,
const GURL& url) {
@@ -49,7 +50,7 @@ void DispatchEvent(content::BrowserContext* browser_context,
EventRouter* event_router = EventRouter::Get(profile);
if (profile && event_router) {
scoped_ptr<Event> event(
- new Event(events::UNKNOWN, event_name, args.Pass()));
+ new Event(histogram_value, event_name, args.Pass()));
event->restrict_to_browser_context = profile;
event->filter_info = info;
event_router->BroadcastEvent(event.Pass());
@@ -83,14 +84,15 @@ void DispatchOnBeforeNavigate(content::WebContents* web_contents,
args->Append(dict);
DispatchEvent(web_contents->GetBrowserContext(),
- web_navigation::OnBeforeNavigate::kEventName,
- args.Pass(),
+ events::WEB_NAVIGATION_ON_BEFORE_NAVIGATE,
+ web_navigation::OnBeforeNavigate::kEventName, args.Pass(),
validated_url);
}
// Constructs and dispatches an onCommitted or onReferenceFragmentUpdated
// event.
-void DispatchOnCommitted(const std::string& event_name,
+void DispatchOnCommitted(events::HistogramValue histogram_value,
+ const std::string& event_name,
content::WebContents* web_contents,
content::RenderFrameHost* frame_host,
const GURL& url,
@@ -122,8 +124,8 @@ void DispatchOnCommitted(const std::string& event_name,
dict->SetDouble(keys::kTimeStampKey, MilliSecondsFromTime(base::Time::Now()));
args->Append(dict);
- DispatchEvent(web_contents->GetBrowserContext(), event_name, args.Pass(),
- url);
+ DispatchEvent(web_contents->GetBrowserContext(), histogram_value, event_name,
+ args.Pass(), url);
}
// Constructs and dispatches an onDOMContentLoaded event.
@@ -141,8 +143,8 @@ void DispatchOnDOMContentLoaded(content::WebContents* web_contents,
args->Append(dict);
DispatchEvent(web_contents->GetBrowserContext(),
- web_navigation::OnDOMContentLoaded::kEventName,
- args.Pass(),
+ events::WEB_NAVIGATION_ON_DOM_CONTENT_LOADED,
+ web_navigation::OnDOMContentLoaded::kEventName, args.Pass(),
url);
}
@@ -161,8 +163,8 @@ void DispatchOnCompleted(content::WebContents* web_contents,
args->Append(dict);
DispatchEvent(web_contents->GetBrowserContext(),
- web_navigation::OnCompleted::kEventName,
- args.Pass(), url);
+ events::WEB_NAVIGATION_ON_COMPLETED,
+ web_navigation::OnCompleted::kEventName, args.Pass(), url);
}
// Constructs and dispatches an onCreatedNavigationTarget event.
@@ -193,9 +195,9 @@ void DispatchOnCreatedNavigationTarget(
args->Append(dict);
DispatchEvent(browser_context,
+ events::WEB_NAVIGATION_ON_CREATED_NAVIGATION_TARGET,
web_navigation::OnCreatedNavigationTarget::kEventName,
- args.Pass(),
- target_url);
+ args.Pass(), target_url);
}
// Constructs and dispatches an onErrorOccurred event.
@@ -215,8 +217,8 @@ void DispatchOnErrorOccurred(content::WebContents* web_contents,
args->Append(dict);
DispatchEvent(web_contents->GetBrowserContext(),
- web_navigation::OnErrorOccurred::kEventName,
- args.Pass(), url);
+ events::WEB_NAVIGATION_ON_ERROR_OCCURRED,
+ web_navigation::OnErrorOccurred::kEventName, args.Pass(), url);
}
// Constructs and dispatches an onTabReplaced event.
@@ -234,10 +236,8 @@ void DispatchOnTabReplaced(
dict->SetDouble(keys::kTimeStampKey, MilliSecondsFromTime(base::Time::Now()));
args->Append(dict);
- DispatchEvent(browser_context,
- web_navigation::OnTabReplaced::kEventName,
- args.Pass(),
- GURL());
+ DispatchEvent(browser_context, events::WEB_NAVIGATION_ON_TAB_REPLACED,
+ web_navigation::OnTabReplaced::kEventName, args.Pass(), GURL());
}
} // namespace web_navigation_api_helpers

Powered by Google App Engine
This is Rietveld 408576698