Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(153)

Unified Diff: chrome/browser/ui/cocoa/location_bar/location_bar_view_mac_browsertest.mm

Issue 12315069: Mac: Update zoom bubble UI (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address review comments Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/location_bar/location_bar_view_mac_browsertest.mm
diff --git a/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac_browsertest.mm b/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac_browsertest.mm
index 8795988d3323f18f69567a917bcaa7b60674b8dd..9aa1c215fd2ddef5ccac5b7b7a2bd64ae2a5d839 100644
--- a/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac_browsertest.mm
+++ b/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac_browsertest.mm
@@ -6,51 +6,11 @@
#import "chrome/browser/ui/browser_window.h"
#import "chrome/browser/ui/cocoa/browser_window_controller.h"
+#include "chrome/browser/ui/cocoa/location_bar/mock_toolbar_model.h"
#import "chrome/browser/ui/cocoa/location_bar/search_token_decoration.h"
#import "chrome/browser/ui/cocoa/location_bar/separator_decoration.h"
#include "chrome/test/base/in_process_browser_test.h"
-namespace {
-
-// Override the toolbar model to allow |GetInputInProgress| and
-// |WouldReplaceSearchURLWithSearchTerms| to be customized.
-class MockToolbarModel : public ToolbarModel {
- public:
- MockToolbarModel() : ToolbarModel(),
- input_in_progress_(false),
- would_replace_search_url_with_search_terms_(false) {}
- virtual ~MockToolbarModel() {}
-
- virtual string16 GetText(
- bool display_search_urls_as_search_terms) const OVERRIDE {
- return string16();
- }
- virtual GURL GetURL() const OVERRIDE { return GURL(); }
- virtual bool WouldReplaceSearchURLWithSearchTerms() const OVERRIDE {
- return would_replace_search_url_with_search_terms_;
- }
- virtual SecurityLevel GetSecurityLevel() const OVERRIDE { return NONE; }
- virtual int GetIcon() const OVERRIDE { return 0; }
- virtual string16 GetEVCertName() const OVERRIDE { return string16(); }
- virtual bool ShouldDisplayURL() const OVERRIDE { return false; }
- virtual void SetInputInProgress(bool value) OVERRIDE {
- input_in_progress_ = value;
- }
- virtual bool GetInputInProgress() const OVERRIDE {
- return input_in_progress_;
- }
-
- void set_would_replace_search_url_with_search_terms(bool value) {
- would_replace_search_url_with_search_terms_ = value;
- }
-
- private:
- bool input_in_progress_;
- bool would_replace_search_url_with_search_terms_;
-};
-
-} // namespace
-
class LocationBarViewMacBrowserTest : public InProcessBrowserTest {
public:
LocationBarViewMacBrowserTest() : InProcessBrowserTest(),

Powered by Google App Engine
This is Rietveld 408576698