Refactor ChooserBubbleUiViewDelegate

components/chooser_controller/mock_chooser_controller.h

Index: components/chooser_controller/mock_chooser_controller.h
diff --git a/components/chooser_controller/mock_chooser_controller.h b/components/chooser_controller/mock_chooser_controller.h
new file mode 100644
index 0000000000000000000000000000000000000000..958db5d346a181e5c6b68069fc636166bdbb8f5c
--- /dev/null
+++ b/components/chooser_controller/mock_chooser_controller.h
@@ -0,0 +1,36 @@
+// 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 COMPONENTS_CHOOSER_CONTROLLER_MOCK_CHOOSER_CONTROLLER_H_
+#define COMPONENTS_CHOOSER_CONTROLLER_MOCK_CHOOSER_CONTROLLER_H_
+
+#include <vector>
+
+#include "base/macros.h"
+#include "components/chooser_controller/chooser_controller.h"
+#include "testing/gmock/include/gmock/gmock.h"
+
+class MockChooserController : public ChooserController {
+ public:
+  MockChooserController(content::RenderFrameHost* owner);
+  ~MockChooserController() override;
+
+  // ChooserController:
+  size_t NumOptions() const override;
+  const base::string16& GetOption(size_t index) const override;
+  MOCK_METHOD1(Select, void(size_t index));

[msw, 2016/06/09 18:31:15]

nit: I prefer overrides with no body to MOCK_METHOD.

[msw, 2016/06/09 18:39:18]

Nvm, I see the mocks are used/counted in the second CL.

+  MOCK_METHOD0(Cancel, void());
+  MOCK_METHOD0(Close, void());
+  MOCK_CONST_METHOD0(OpenHelpCenterUrl, void());
+
+  void OptionAdded(const base::string16 option_name);
+  void OptionRemoved(const base::string16 option_name);
+
+ private:
+  std::vector<base::string16> option_names_;
+
+  DISALLOW_COPY_AND_ASSIGN(MockChooserController);
+};
+
+#endif  // COMPONENTS_CHOOSER_CONTROLLER_MOCK_CHOOSER_CONTROLLER_H_