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

Unified Diff: chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc

Issue 1721633002: Added UMA collection for fullscreen / mouse lock bubble re-shows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move name to constant. Created 4 years, 10 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/ui/exclusive_access/exclusive_access_bubble.cc
diff --git a/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc b/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc
index b3d1684c1112df27cfc5571e5f76c3047ad58cbf..9681d54ff08d8943a0bc6c6a467566d957c4a33b 100644
--- a/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc
+++ b/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc
@@ -57,6 +57,9 @@ void ExclusiveAccessBubble::OnUserInput() {
// If the notification suppression timer has elapsed, re-show it.
if (!suppress_notify_timeout_.IsRunning()) {
+ // Record UMA for when the bubble is re-shown.
scheib 2016/02/23 23:52:26 Comment won't be needed when method renamed.
Matt Giuca 2016/02/24 03:00:31 Done.
+ manager_->BubbleReshown(bubble_type_);
+
ShowAndStartTimers();
return;
}

Powered by Google App Engine
This is Rietveld 408576698