Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(191)

Side by Side Diff: chrome/browser/chrome_content_browser_client.cc

Issue 1965153002: Disable WebUSB chooser service on chrome extension (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: removed redundant include file Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chrome_content_browser_client.h" 5 #include "chrome/browser/chrome_content_browser_client.h"
6 6
7 #include <map> 7 #include <map>
8 #include <set> 8 #include <set>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 2756 matching lines...) Expand 10 before | Expand all | Expand 10 after
2767 #if defined(ENABLE_MOJO_MEDIA) 2767 #if defined(ENABLE_MOJO_MEDIA)
2768 registry->AddService( 2768 registry->AddService(
2769 base::Bind(&OutputProtectionImpl::Create, render_frame_host)); 2769 base::Bind(&OutputProtectionImpl::Create, render_frame_host));
2770 #endif // defined(ENABLE_MOJO_MEDIA) 2770 #endif // defined(ENABLE_MOJO_MEDIA)
2771 } 2771 }
2772 2772
2773 void ChromeContentBrowserClient::RegisterRenderFrameMojoServices( 2773 void ChromeContentBrowserClient::RegisterRenderFrameMojoServices(
2774 content::ServiceRegistry* registry, 2774 content::ServiceRegistry* registry,
2775 content::RenderFrameHost* render_frame_host) { 2775 content::RenderFrameHost* render_frame_host) {
2776 // WebUSB is an experimental web API. It will only work if the experiment 2776 // WebUSB is an experimental web API. It will only work if the experiment
2777 // is enabled and WebUSB feature is enabled. 2777 // is enabled and WebUSB feature is enabled and it is not used from chrome
2778 // extension.
Reilly Grant (use Gerrit) 2016/05/11 00:22:44 Make this a separate sentence: "It should also no
juncai 2016/05/11 00:32:36 Done.
2778 if (base::CommandLine::ForCurrentProcess()->HasSwitch( 2779 if (base::CommandLine::ForCurrentProcess()->HasSwitch(
2779 switches::kEnableExperimentalWebPlatformFeatures) && 2780 switches::kEnableExperimentalWebPlatformFeatures) &&
2780 base::FeatureList::IsEnabled(features::kWebUsb)) { 2781 base::FeatureList::IsEnabled(features::kWebUsb)
2782 #if defined(ENABLE_EXTENSIONS)
2783 &&
2784 !render_frame_host->GetSiteInstance()->GetSiteURL().SchemeIs(
2785 extensions::kExtensionScheme)
2786 #endif
2787 ) {
2781 registry->AddService( 2788 registry->AddService(
2782 base::Bind(&CreateUsbDeviceManager, render_frame_host)); 2789 base::Bind(&CreateUsbDeviceManager, render_frame_host));
2783 registry->AddService( 2790 registry->AddService(
2784 base::Bind(&CreateWebUsbChooserService, render_frame_host)); 2791 base::Bind(&CreateWebUsbChooserService, render_frame_host));
2785 } 2792 }
2786 2793
2787 // Register mojo CredentialManager service only for main frame. 2794 // Register mojo CredentialManager service only for main frame.
2788 if (!render_frame_host->GetParent()) { 2795 if (!render_frame_host->GetParent()) {
2789 registry->AddService( 2796 registry->AddService(
2790 base::Bind(&ChromePasswordManagerClient::BindCredentialManager, 2797 base::Bind(&ChromePasswordManagerClient::BindCredentialManager,
(...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
2969 if (channel <= kMaxDisableEncryptionChannel) { 2976 if (channel <= kMaxDisableEncryptionChannel) {
2970 static const char* const kWebRtcDevSwitchNames[] = { 2977 static const char* const kWebRtcDevSwitchNames[] = {
2971 switches::kDisableWebRtcEncryption, 2978 switches::kDisableWebRtcEncryption,
2972 }; 2979 };
2973 to_command_line->CopySwitchesFrom(from_command_line, 2980 to_command_line->CopySwitchesFrom(from_command_line,
2974 kWebRtcDevSwitchNames, 2981 kWebRtcDevSwitchNames,
2975 arraysize(kWebRtcDevSwitchNames)); 2982 arraysize(kWebRtcDevSwitchNames));
2976 } 2983 }
2977 } 2984 }
2978 #endif // defined(ENABLE_WEBRTC) 2985 #endif // defined(ENABLE_WEBRTC)
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698