Move requests from Show() argument to PermissionPrompt::Delegate

chrome/browser/permissions/permission_prompt_android.h

 // Copyright 2016 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_PERMISSIONS_PERMISSION_PROMPT_ANDROID_H_
 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_PROMPT_ANDROID_H_
 
 #include <vector>
 
 #include "base/strings/string16.h"
 #include "chrome/browser/ui/permission_bubble/permission_prompt.h"
 #include "components/content_settings/core/common/content_settings_types.h"
 
-class PermissionRequest;
-
 namespace content {
 class WebContents;
 }
 
 class PermissionPromptAndroid : public PermissionPrompt {
  public:
   explicit PermissionPromptAndroid(content::WebContents* web_contents);
   ~PermissionPromptAndroid() override;
 
   // PermissionPrompt:
   void SetDelegate(Delegate* delegate) override;
-  void Show(const std::vector<PermissionRequest*>& requests,
-            const std::vector<bool>& accept_state) override;
+  void Show() override;
   bool CanAcceptRequestUpdate() override;
   bool HidesAutomatically() override;
   void Hide() override;
   void UpdateAnchorPosition() override;
   gfx::NativeWindow GetNativeWindow() override;
 
   void Closing();
   void ToggleAccept(int index, bool value);
   void Accept();
   void Deny();
 
-  size_t permission_count() const { return requests_.size(); }
+  size_t permission_count() const { return delegate_->Requests().size(); }

[raymes, 2017/05/11 01:17:54]

nit: we should probably move the implementation into the cc file:
size_t PermissionCount

[Timothy Loh, 2017/05/11 04:42:08]

Done.

   ContentSettingsType GetContentSettingType(size_t position) const;
   int GetIconIdForPermission(size_t position) const;
   base::string16 GetMessageTextFragment(size_t position) const;
 
  private:
   // PermissionPromptAndroid is owned by PermissionRequestManager, so it should
   // be safe to hold a raw WebContents pointer here because this class is
   // destroyed before the WebContents.
   content::WebContents* web_contents_;
   // |delegate_| is the PermissionRequestManager, which owns this object.
   Delegate* delegate_;
-  // The current request being displayed (if any).
-  std::vector<PermissionRequest*> requests_;
 
   DISALLOW_COPY_AND_ASSIGN(PermissionPromptAndroid);
 };
 
 #endif  // CHROME_BROWSER_PERMISSIONS_PERMISSION_PROMPT_ANDROID_H_