permissions: handle request ids for permissions in permission manager

chromecast/browser/cast_permission_manager.cc

 // Copyright 2015 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.
 
 #include "chromecast/browser/cast_permission_manager.h"
 
 #include "base/callback.h"
 #include "base/logging.h"
 #include "content/public/browser/permission_type.h"
 
 namespace chromecast {
 namespace shell {
 
 CastPermissionManager::CastPermissionManager()
     : content::PermissionManager() {
 }
 
 CastPermissionManager::~CastPermissionManager() {
 }
 
-void CastPermissionManager::RequestPermission(
+int CastPermissionManager::RequestPermission(
     content::PermissionType permission,
     content::RenderFrameHost* render_frame_host,
-    int request_id,
     const GURL& origin,
     bool user_gesture,
     const base::Callback<void(content::PermissionStatus)>& callback) {
   LOG(INFO) << __FUNCTION__ << ": " << static_cast<int>(permission);
   callback.Run(content::PermissionStatus::PERMISSION_STATUS_GRANTED);
+  return -1;

[mlamouri (slow - plz ping), 2015/09/16 14:42:56]

use the constant

[Lalit Maganti, 2015/09/16 16:41:10]

Done.

 }
 
-void CastPermissionManager::CancelPermissionRequest(
-    content::PermissionType permission,
-    content::RenderFrameHost* render_frame_host,
-    int request_id,
-    const GURL& origin) {
+void CastPermissionManager::CancelPermissionRequest(int request_id) {
 }
 
 void CastPermissionManager::ResetPermission(
     content::PermissionType permission,
     const GURL& requesting_origin,
     const GURL& embedding_origin) {
 }
 
 content::PermissionStatus CastPermissionManager::GetPermissionStatus(
     content::PermissionType permission,
@@ skipping 16 matching lines @@
     const base::Callback<void(content::PermissionStatus)>& callback) {
   return -1;
 }
 
 void CastPermissionManager::UnsubscribePermissionStatusChange(
     int subscription_id) {
 }
 
 }  // namespace shell
 }  // namespace chromecast