Added UMA collection for fullscreen / mouse lock bubble re-shows.

chrome/browser/ui/exclusive_access/exclusive_access_controller_base.cc

 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/exclusive_access/exclusive_access_controller_base.h"
 
+#include "base/metrics/histogram_macros.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/exclusive_access/exclusive_access_manager.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/web_contents.h"
 
 using content::WebContents;
 
 ExclusiveAccessControllerBase::ExclusiveAccessControllerBase(
     ExclusiveAccessManager* manager)
-    : manager_(manager), tab_with_exclusive_access_(nullptr) {
-}
+    : manager_(manager) {}
 
 ExclusiveAccessControllerBase::~ExclusiveAccessControllerBase() {
 }
 
 GURL ExclusiveAccessControllerBase::GetExclusiveAccessBubbleURL() const {
   return manager_->GetExclusiveAccessBubbleURL();
 }
 
 GURL ExclusiveAccessControllerBase::GetURLForExclusiveAccessBubble() const {
   if (tab_with_exclusive_access_)
@@ skipping 28 matching lines @@
 void ExclusiveAccessControllerBase::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   DCHECK_EQ(content::NOTIFICATION_NAV_ENTRY_COMMITTED, type);
   if (content::Details<content::LoadCommittedDetails>(details)
           ->is_navigation_to_different_page())
     ExitExclusiveAccessIfNecessary();
 }
 
+void ExclusiveAccessControllerBase::BubbleReshown() {
+  ++bubble_reshow_count_;
+}
+
+void ExclusiveAccessControllerBase::Exiting() {
+  // Record the number of bubble reshows during this session. Only if simplified
+  // fullscreen is enabled.
+  if (ExclusiveAccessManager::IsSimplifiedFullscreenUIEnabled()) {
+    std::string histogram_name = "ExclusiveAccess.";
+    histogram_name += HistogramPrefix();
+    histogram_name += "BubbleReshowsPerSession";

[dominickn, 2016/02/23 00:05:27]

Nit: Please rename this to
ExclusiveAccess.BubbleReshowsPerSession.(Fullscreen|Mouselock) so that it's one
less concatenation here.

[Matt Giuca, 2016/02/23 02:50:28]

Done.

+    UMA_HISTOGRAM_COUNTS_100(histogram_name, bubble_reshow_count_);
+  }
+
+  bubble_reshow_count_ = 0;
+}
+
 void ExclusiveAccessControllerBase::SetTabWithExclusiveAccess(
     WebContents* tab) {
   // Tab should never be replaced with another tab, or
   // UpdateNotificationRegistrations would need updating.
   DCHECK(tab_with_exclusive_access_ == tab ||
          tab_with_exclusive_access_ == nullptr || tab == nullptr);
   tab_with_exclusive_access_ = tab;
   UpdateNotificationRegistrations();
 }
 
 void ExclusiveAccessControllerBase::UpdateNotificationRegistrations() {
   if (tab_with_exclusive_access_ && registrar_.IsEmpty()) {
     registrar_.Add(this, content::NOTIFICATION_NAV_ENTRY_COMMITTED,
                    content::Source<content::NavigationController>(
                        &tab_with_exclusive_access_->GetController()));
   } else if (!tab_with_exclusive_access_ && !registrar_.IsEmpty()) {
     registrar_.RemoveAll();
   }
 }