refactor to Introduce AUDIO_CAPTURE and VIDEO_CAPTURE permissions.

content/browser/permissions/permission_service_impl.cc

 // Copyright 2014 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/browser/permissions/permission_service_impl.h"
 
 #include "base/bind.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/permission_manager.h"
 #include "content/public/browser/permission_type.h"
@@ skipping 11 matching lines @@
     case PERMISSION_NAME_PUSH_NOTIFICATIONS:
       return PermissionType::PUSH_MESSAGING;
     case PERMISSION_NAME_MIDI:
       return PermissionType::MIDI;
     case PERMISSION_NAME_MIDI_SYSEX:
       return PermissionType::MIDI_SYSEX;
     case PERMISSION_NAME_PROTECTED_MEDIA_IDENTIFIER:
       return PermissionType::PROTECTED_MEDIA_IDENTIFIER;
     case PERMISSION_NAME_DURABLE_STORAGE:
       return PermissionType::DURABLE_STORAGE;
+    case PERMISSION_NAME_AUDIO_CAPTURE:
+      return PermissionType::AUDIO_CAPTURE;
+    case PERMISSION_NAME_VIDEO_CAPTURE:
+      return PermissionType::VIDEO_CAPTURE;
   }
 
   NOTREACHED();

[palmer, 2015/09/01 18:31:04]

Put this in the default:.

[guoweis_left_chromium, 2015/09/01 19:36:34]

same here.

   return PermissionType::NUM;
 }
 
 } // anonymous namespace
 
 PermissionServiceImpl::PendingRequest::PendingRequest(
     PermissionType permission,
     const GURL& origin,
     const PermissionStatusCallback& callback)
     : permission(permission),
@@ skipping 240 matching lines @@
 
   PermissionStatusCallback callback = subscription->callback;
 
   subscription->callback.reset();
   pending_subscriptions_.Remove(pending_subscription_id);
 
   callback.Run(status);
 }
 
 }  // namespace content