Refactor: Make MediaCaptureDevicesDispatcher have pluggable handlers.

extensions/common/extension.h

 // Copyright (c) 2013 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.
 
 #ifndef EXTENSIONS_COMMON_EXTENSION_H_
 #define EXTENSIONS_COMMON_EXTENSION_H_
 
 #include <algorithm>
 #include <iosfwd>
 #include <map>
@@ skipping 336 matching lines @@
   }
 
   // Type-related queries.
   bool is_app() const;
   bool is_platform_app() const;
   bool is_hosted_app() const;
   bool is_legacy_packaged_app() const;
   bool is_extension() const;
   bool is_shared_module() const;
   bool is_theme() const;
+  bool is_media_request_white_listed() const;

[Sergey Ulanov, 2015/05/07 01:05:49]

I don't think the whitelist belongs in the extensions. Please keep it in
chrome/browser/media. You can move the whitelist to ExtensionMediaAccessHandler
if you address my other comments, particularly about CheckMediaAccessPermission

[changbin, 2015/05/22 05:10:59]

Thanks, i think it's done now:)

 
   bool can_be_incognito_enabled() const;
 
   void AddWebExtentPattern(const URLPattern& pattern);
   const URLPatternSet& web_extent() const { return extent_; }
 
  private:
   friend class base::RefCountedThreadSafe<Extension>;
 
   // Chooses the extension ID for an extension based on a variety of criteria.
@@ skipping 210 matching lines @@
 
   UpdatedExtensionPermissionsInfo(
       const Extension* extension,
       const PermissionSet* permissions,
       Reason reason);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_COMMON_EXTENSION_H_