Block platform verification and file IO in the CDM adapter if the CDM configuration disallows them.

media/blink/webcontentdecryptionmodule_impl.h

 // Copyright 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 MEDIA_BLINK_WEBCONTENTDECRYPTIONMODULE_IMPL_H_
 #define MEDIA_BLINK_WEBCONTENTDECRYPTIONMODULE_IMPL_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/string16.h"
@@ skipping 11 matching lines @@
 namespace media {
 
 class CdmContext;
 class CdmFactory;
 class CdmSessionAdapter;
 class WebContentDecryptionModuleSessionImpl;
 
 class MEDIA_EXPORT WebContentDecryptionModuleImpl
     : public blink::WebContentDecryptionModule {
  public:
   static void Create(CdmFactory* cdm_factory,

[ddorwin, 2015/03/06 21:42:54]

Why is this non-const param first?

[sandersd (OOO until July 31), 2015/03/06 22:36:58]

Presumably for symmetry with CdmSessionAdapter::Initialize(). Ideally it would
be const, but CdmFactory::Create is not a const method (I can't imagine why it
would need to be mutable though).

I can move it down in this CL, but the reality is that it's treated as const,
and if it were const it is already in the right place.

+                     const base::string16& key_system,
+                     bool allow_distinctive_identifier,
+                     bool allow_persistent_state,
                      const blink::WebSecurityOrigin& security_origin,
-                     const base::string16& key_system,
                      blink::WebContentDecryptionModuleResult result);
 
   virtual ~WebContentDecryptionModuleImpl();
 
   // blink::WebContentDecryptionModule implementation.
   virtual blink::WebContentDecryptionModuleSession* createSession();
 
   virtual void setServerCertificate(
       const uint8* server_certificate,
       size_t server_certificate_length,
@@ skipping 15 matching lines @@
 
 // Allow typecasting from blink type as this is the only implementation.
 inline WebContentDecryptionModuleImpl* ToWebContentDecryptionModuleImpl(
     blink::WebContentDecryptionModule* cdm) {
   return static_cast<WebContentDecryptionModuleImpl*>(cdm);
 }
 
 }  // namespace media
 
 #endif  // MEDIA_BLINK_WEBCONTENTDECRYPTIONMODULE_IMPL_H_