Add a new metric for how many times permissions prompts are shown.

chrome/browser/chrome_quota_permission_context.cc

Index: chrome/browser/chrome_quota_permission_context.cc
diff --git a/chrome/browser/chrome_quota_permission_context.cc b/chrome/browser/chrome_quota_permission_context.cc
index 0de3b2b12d3ae3130d61b60f306ef406eac5f339..6959d46d92701edcc4dd34c217180b3c7f2b1bc4 100644
--- a/chrome/browser/chrome_quota_permission_context.cc
+++ b/chrome/browser/chrome_quota_permission_context.cc
@@ -52,6 +52,7 @@ class QuotaPermissionRequest : public PermissionBubbleRequest {
 
   ~QuotaPermissionRequest() override;
 
+ private:

[tsergeant, 2016/04/15 00:54:10]

Why does everything go from public to private here (and in several of the other
classes you're changing)?

[benwells, 2016/04/18 01:02:06]

As discussed offline, this means users of these methods have to know of
instances of this class as a PermissionBubbleRequest, not a
QuotaPermissionRequest, which is more abstract and in general better as it means
it is being treated generally.

This is something I've been asked many times and seems to be a convention in
chromium. Note that if you google around some people think it is a bad idea.

   // PermissionBubbleRequest:
   int GetIconId() const override;
   base::string16 GetMessageText() const override;
@@ -61,8 +62,8 @@ class QuotaPermissionRequest : public PermissionBubbleRequest {
   void PermissionDenied() override;
   void Cancelled() override;
   void RequestFinished() override;
+  PermissionBubbleType GetPermissionBubbleType() const override;
 
- private:
   scoped_refptr<ChromeQuotaPermissionContext> context_;
   GURL origin_url_;
   int64_t requested_quota_;
@@ -131,6 +132,10 @@ void QuotaPermissionRequest::RequestFinished() {
   delete this;
 }
 
+PermissionBubbleType QuotaPermissionRequest::GetPermissionBubbleType() const {
+  return PermissionBubbleType::QUOTA;
+}
+
 #if defined(OS_ANDROID)
 // RequestQuotaInfoBarDelegate ------------------------------------------------