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

Unified Diff: chrome/browser/content_settings/permission_context_base.cc

Issue 881983003: media: Invoke PlatformVerificationDialog from ProtectedMediaIdentifierPermissionContext. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: added todo + bug Created 5 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/content_settings/permission_context_base.cc
diff --git a/chrome/browser/content_settings/permission_context_base.cc b/chrome/browser/content_settings/permission_context_base.cc
index 045ac487dce1b37e219289322bdce23559ba6965..c1883776af64f114038e73a0b0f19acf94540c15 100644
--- a/chrome/browser/content_settings/permission_context_base.cc
+++ b/chrome/browser/content_settings/permission_context_base.cc
@@ -18,12 +18,6 @@
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/web_contents.h"
-#if defined(OS_CHROMEOS)
-#include "chrome/browser/chromeos/attestation/platform_verification_dialog.h"
-using chromeos::attestation::PlatformVerificationDialog;
-using chromeos::attestation::PlatformVerificationFlow;
-#endif
-
PermissionContextBase::PermissionContextBase(
Profile* profile,
const ContentSettingsType permission_type)
@@ -132,19 +126,6 @@ void PermissionContextBase::DecidePermission(
PermissionContextUmaUtil::PermissionRequested(
permission_type_, requesting_origin);
-#if defined(OS_CHROMEOS)
- // TODO(xhwang): This is to use the existing platform verification UI. Remove
- // it when the infobar/bubble UI can satisfy our requirements.
- if (permission_type_ == CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER) {
- PlatformVerificationDialog::ShowDialog(
- web_contents,
- base::Bind(&PermissionContextBase::OnPlatformVerificationResult,
- weak_factory_.GetWeakPtr(), id, requesting_origin,
- embedding_origin, callback));
- return;
- }
-#endif
-
if (PermissionBubbleManager::Enabled()) {
if (pending_bubbles_.get(id.ToString()) != NULL)
return;
@@ -248,25 +229,3 @@ void PermissionContextBase::UpdateContentSetting(const GURL& requesting_origin,
ContentSettingsPattern::FromURLNoWildcard(embedding_origin),
permission_type_, std::string(), content_setting);
}
-
-#if defined(OS_CHROMEOS)
-void PermissionContextBase::OnPlatformVerificationResult(
- const PermissionRequestID& id,
- const GURL& requesting_origin,
- const GURL& embedding_origin,
- const BrowserPermissionCallback& callback,
- chromeos::attestation::PlatformVerificationFlow::ConsentResponse response) {
- if (response == PlatformVerificationFlow::CONSENT_RESPONSE_NONE) {
- // Deny request and do not save to content settings.
- PermissionDecided(id, requesting_origin, embedding_origin, callback,
- false, // Do not save to content settings.
- false); // Do not allow the permission.
- return;
- }
-
- PermissionDecided(
- id, requesting_origin, embedding_origin, callback,
- true, // Save to content settings.
- response == PlatformVerificationFlow::CONSENT_RESPONSE_ALLOW);
-}
-#endif

Powered by Google App Engine
This is Rietveld 408576698