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(
-      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);
+  // Checks protected media identifier permission for the given
+  // |render_frame_id|, |security_origin| and |cdm_id|.
+  // |promise| will always be resolved or rejected on the |task_runner_|.
+  void CheckPermissionStatus(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);
 
-  // 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(
+  // If |permitted_status| is not "granted", |promise| will be rejected.
+  // Otherwise, call CreateSessionAndGenerateRequest() on the CDM.
+  // |promise| will always be resolved or rejected on the |task_runner_|.
+  void CreateSessionAndGenerateRequest(
       int render_frame_id,
       int cdm_id,
       media::EmeInitDataType init_data_type,
       const std::vector<uint8>& init_data,
       scoped_ptr<media::NewSessionCdmPromise> promise,
-      PermissionStatus permission);
+      PermissionStatus permission_status);
 
   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_