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

Unified Diff: content/renderer/media/webcontentdecryptionmodule_impl.cc

Issue 211473005: Remove unused createContentDecryptionModule() code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 9 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: content/renderer/media/webcontentdecryptionmodule_impl.cc
diff --git a/content/renderer/media/webcontentdecryptionmodule_impl.cc b/content/renderer/media/webcontentdecryptionmodule_impl.cc
index 0c4a901f74699a0c95c4910edb4624297f445a86..a9b201fbc13278ef319526c73ae64eb7f7edff4f 100644
--- a/content/renderer/media/webcontentdecryptionmodule_impl.cc
+++ b/content/renderer/media/webcontentdecryptionmodule_impl.cc
@@ -28,18 +28,12 @@ class WebFrame;
namespace content {
WebContentDecryptionModuleImpl* WebContentDecryptionModuleImpl::Create(
- const base::string16& key_system) {
- blink::WebSecurityOrigin no_origin;
- return WebContentDecryptionModuleImpl::Create(NULL, no_origin, key_system);
-}
-
-WebContentDecryptionModuleImpl* WebContentDecryptionModuleImpl::Create(
blink::WebFrame* frame,
const blink::WebSecurityOrigin& security_origin,
const base::string16& key_system) {
// TODO(jrummell): Use |security_origin| rather than using the document URL.
- // TODO(jrummell): Once the other Create() method is removed, validate |frame|
- // and |security_origin|.
+ DCHECK(frame);
+ DCHECK(!security_origin.isNull());
DCHECK(!key_system.empty());
// TODO(ddorwin): Guard against this in supported types check and remove this.
« no previous file with comments | « content/renderer/media/webcontentdecryptionmodule_impl.h ('k') | content/renderer/renderer_webkitplatformsupport_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698