permissions: switch from explicitly passing queue controller to callbacks

chrome/browser/media/protected_media_identifier_infobar_delegate.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 CHROME_BROWSER_MEDIA_PROTECTED_MEDIA_IDENTIFIER_INFOBAR_DELEGATE_H_
 #define CHROME_BROWSER_MEDIA_PROTECTED_MEDIA_IDENTIFIER_INFOBAR_DELEGATE_H_
 
 #include <string>
 
+#include "base/callback.h"

[mlamouri (slow - plz ping), 2015/09/15 12:55:59]

Use callback_forward.h

[Lalit Maganti, 2015/09/15 13:16:53]

ATM I can't because I store a callback but if I do add the subclassing of
PermissionInfobarDelegate then I could do this I guess.

[Lalit Maganti, 2015/09/15 14:19:24]

Done.

 #include "chrome/browser/permissions/permission_request_id.h"
 #include "components/infobars/core/confirm_infobar_delegate.h"
 #include "url/gurl.h"
 
-class PermissionQueueController;
 class InfoBarService;
 
 // TODO(toyoshim): Much more code can be shared with GeolocationInfoBarDelegate.
 // http://crbug.com/266743
 
 class ProtectedMediaIdentifierInfoBarDelegate : public ConfirmInfoBarDelegate {
  public:
   // Creates a protected media identifier infobar and delegate and adds the
   // infobar to |infobar_service|.  Returns the infobar if it was successfully
   // added.
-  static infobars::InfoBar* Create(InfoBarService* infobar_service,
-                                   PermissionQueueController* controller,
-                                   const PermissionRequestID& id,
-                                   const GURL& requesting_frame,
-                                   const std::string& display_languages);
+  static infobars::InfoBar* Create(
+    InfoBarService* infobar_service,
+    const PermissionRequestID& id,
+    const GURL& requesting_frame,
+    const std::string& display_languages,
+    const base::Callback<void(bool, bool)>& callback);
 
  protected:
-  ProtectedMediaIdentifierInfoBarDelegate(PermissionQueueController* controller,
-                                          const PermissionRequestID& id,
-                                          const GURL& requesting_frame,
-                                          const std::string& display_languages);
+  ProtectedMediaIdentifierInfoBarDelegate(
+      const PermissionRequestID& id,
+      const GURL& requesting_frame,
+      const std::string& display_languages,
+      const base::Callback<void(bool, bool)>& callback);
   ~ProtectedMediaIdentifierInfoBarDelegate() override;
 
   // Calls back to the controller to inform it of the user's decision.
   void SetPermission(bool update_content_setting, bool allowed);
 
  private:
   // ConfirmInfoBarDelegate:
   Type GetInfoBarType() const override;
   int GetIconId() const override;
   void InfoBarDismissed() override;
   base::string16 GetMessageText() const override;
   base::string16 GetButtonLabel(InfoBarButton button) const override;
   bool Accept() override;
   bool Cancel() override;
   base::string16 GetLinkText() const override;
   bool LinkClicked(WindowOpenDisposition disposition) override;
 
-  PermissionQueueController* controller_;
   const PermissionRequestID id_;
   GURL requesting_frame_;
   std::string display_languages_;
+  const base::Callback<void(bool, bool)> callback_;
 
   DISALLOW_COPY_AND_ASSIGN(ProtectedMediaIdentifierInfoBarDelegate);
 };
 
 #endif  // CHROME_BROWSER_MEDIA_PROTECTED_MEDIA_IDENTIFIER_INFOBAR_DELEGATE_H_