Collecting statistics on iframe permissions use.

content/child/permissions/permission_dispatcher.cc

 // Copyright 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 "content/child/permissions/permission_dispatcher.h"
 
 #include "base/callback.h"
 #include "content/child/worker_task_runner.h"
 #include "content/public/common/service_registry.h"
 #include "third_party/WebKit/public/platform/WebURL.h"
@@ skipping 38 matching lines @@
 }
 
 blink::WebPermissionStatus GetWebPermissionStatus(PermissionStatus status) {
   switch (status) {
     case PERMISSION_STATUS_GRANTED:
       return blink::WebPermissionStatusGranted;
     case PERMISSION_STATUS_DENIED:
       return blink::WebPermissionStatusDenied;
     case PERMISSION_STATUS_ASK:
       return blink::WebPermissionStatusPrompt;
+    case PERMISSION_STATUS_NUM:
+      break;

[jww, 2015/07/07 18:41:46]

Again, shouldn't this be a NOTREACHED (and of course add a comment to all these
functions specifying that PERMISSION_STATUS_NUM should never be passed in)

[keenanb, 2015/07/07 21:02:59]

Done.

   }
 
   NOTREACHED();
   return blink::WebPermissionStatusDenied;
 }
 
 const int kNoWorkerThread = 0;
 
 }  // anonymous namespace
 
@@ skipping 208 matching lines @@
       origin,
       current_status,
       base::Bind(&PermissionDispatcher::OnPermissionChanged,
                  base::Unretained(this),
                  type,
                  origin,
                  base::Unretained(observer)));
 }
 
 }  // namespace content