media: Fix permission request/check for EME on Android.

content/browser/media/cdm/browser_cdm_manager.h

 // Copyright 2014 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 CONTENT_BROWSER_MEDIA_CDM_BROWSER_CDM_MANAGER_H_
 #define CONTENT_BROWSER_MEDIA_CDM_BROWSER_CDM_MANAGER_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 126 matching lines @@
               int cdm_id,
               const std::string& key_system,
               const GURL& security_origin);
 
   // Removes all CDMs associated with |render_frame_id|.
   void RemoveAllCdmForFrame(int render_frame_id);
 
   // Removes the CDM with the specified id.
   void RemoveCdm(uint64 id);
 
-  // Requests permission for the given protected-media session (infobar).
-  void RequestSessionPermission(
+  using PermissionStatusCB = base::Callback<void(bool)>;
+
+  // Checks protected media identifier permission for the given
+  // |render_frame_id| and |cdm_id|.
+  void CheckPermissionStatus(int render_frame_id,
+                             int cdm_id,
+                             const PermissionStatusCB& permission_status_cb);
+
+  // Checks permission status on Browser UI thread. Runs |permission_status_cb|
+  // on the |task_runner_| with the permission status.
+  void CheckPermissionStatusOnUIThread(
       int render_frame_id,
       const GURL& security_origin,
-      int cdm_id,
-      media::EmeInitDataType init_data_type,
-      const std::vector<uint8>& init_data,
-      scoped_ptr<media::NewSessionCdmPromise> promise);
+      const PermissionStatusCB& permission_status_cb);
 
-  // If |permitted| is false, it does nothing but send
-  // |CdmMsg_LegacySessionError| IPC message.
-  // The primary use case is infobar permission callback, i.e., when infobar
-  // can decide user's intention either from interacting with the actual info
-  // bar or from the saved preference.
-  void GenerateRequestIfPermitted(
+  // Calls CreateSessionAndGenerateRequest() on the CDM if |permission_allowed|
+  // is true. Otherwise rejects the |promise|.
+  void CreateSessionAndGenerateRequestIfPermitted(
       int render_frame_id,
       int cdm_id,
-      media::EmeInitDataType init_data_type,
+      media::EmeInitDataType eme_init_data_type,

[ddorwin, 2015/03/17 20:09:59]

nit: "eme_" probably isn't necessary. Also, it's not named this in the .cc file.

[xhwang, 2015/03/17 20:16:02]

Done.

       const std::vector<uint8>& init_data,
       scoped_ptr<media::NewSessionCdmPromise> promise,
-      PermissionStatus permission);
+      bool permission_allowed);
 
   const int render_process_id_;
 
   // TaskRunner to dispatch all CDM messages to. If it's NULL, all messages are
   // dispatched to the browser UI thread.
   scoped_refptr<base::TaskRunner> task_runner_;
 
   // The key in the following maps is a combination of |render_frame_id| and
   // |cdm_id|.
 
   // Map of managed BrowserCdms.
   typedef base::ScopedPtrHashMap<uint64, media::BrowserCdm> CdmMap;
   CdmMap cdm_map_;
 
   // Map of CDM's security origin.
   std::map<uint64, GURL> cdm_security_origin_map_;
 
-  // Map of callbacks to cancel the permission request.
-  std::map<uint64, base::Closure> cdm_cancel_permission_map_;
-
   base::WeakPtrFactory<BrowserCdmManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserCdmManager);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_MEDIA_CDM_BROWSER_CDM_MANAGER_H_