Permissions: Rename PermissionBubbleView to PermissionPromptInterface

chrome/browser/download/download_request_limiter_unittest.cc

Index: chrome/browser/download/download_request_limiter_unittest.cc
diff --git a/chrome/browser/download/download_request_limiter_unittest.cc b/chrome/browser/download/download_request_limiter_unittest.cc
index 4dab572a0b607583be976a7a5ad4ef3f50a4efcb..7e90a7ea43f987a5e92cf50ffc486311f80c0d59 100644
--- a/chrome/browser/download/download_request_limiter_unittest.cc
+++ b/chrome/browser/download/download_request_limiter_unittest.cc
@@ -28,7 +28,7 @@
 #else
 #include "chrome/browser/download/download_permission_request.h"
 #include "chrome/browser/permissions/permission_request_manager.h"
-#include "chrome/browser/ui/website_settings/mock_permission_bubble_factory.h"
+#include "chrome/browser/ui/website_settings/mock_permission_prompt_factory.h"
 #endif
 
 using content::WebContents;
@@ -99,41 +99,41 @@ class TestingDelegate {
  public:
   void SetUp(WebContents* web_contents) {
     PermissionRequestManager::CreateForWebContents(web_contents);
-    mock_permission_bubble_factory_.reset(new MockPermissionBubbleFactory(
+    mock_permission_prompt_factory_.reset(new MockPermissionPromptFactory(
         PermissionRequestManager::FromWebContents(web_contents)));
     PermissionRequestManager::FromWebContents(web_contents)
         ->DisplayPendingRequests();
   }
 
-  void TearDown() { mock_permission_bubble_factory_.reset(); }
+  void TearDown() { mock_permission_prompt_factory_.reset(); }
 
   void LoadCompleted(WebContents* web_contents) {
-    mock_permission_bubble_factory_->DocumentOnLoadCompletedInMainFrame();
+    mock_permission_prompt_factory_->DocumentOnLoadCompletedInMainFrame();
   }
 
-  void ResetCounts() { mock_permission_bubble_factory_->ResetCounts(); }
+  void ResetCounts() { mock_permission_prompt_factory_->ResetCounts(); }
 
-  int AllowCount() { return mock_permission_bubble_factory_->show_count(); }
+  int AllowCount() { return mock_permission_prompt_factory_->show_count(); }
 
   void UpdateExpectations(TestingAction action) {
     // Set expectations for PermissionRequestManager.
     if (action == ACCEPT) {
-      mock_permission_bubble_factory_->set_response_type(
+      mock_permission_prompt_factory_->set_response_type(

[Lei Zhang, 2016/08/02 03:59:22]

Use a switch statement to calculate an AutoResponseType value and call
set_response_type() in just 1 place?

[tsergeant, 2016/08/02 08:05:16]

Done.

           PermissionRequestManager::ACCEPT_ALL);
     } else if (action == CANCEL) {
-      mock_permission_bubble_factory_->set_response_type(
+      mock_permission_prompt_factory_->set_response_type(
           PermissionRequestManager::DENY_ALL);
     } else if (action == WAIT) {
-      mock_permission_bubble_factory_->set_response_type(
+      mock_permission_prompt_factory_->set_response_type(
           PermissionRequestManager::NONE);
     } else {
-      mock_permission_bubble_factory_->set_response_type(
+      mock_permission_prompt_factory_->set_response_type(
           PermissionRequestManager::DISMISS);
     }
   }
 
  private:
-  std::unique_ptr<MockPermissionBubbleFactory> mock_permission_bubble_factory_;
+  std::unique_ptr<MockPermissionPromptFactory> mock_permission_prompt_factory_;
 };
 #endif
 }  // namespace