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

Unified Diff: chrome/browser/views/first_run_search_engine_view.h

Issue 2934011: New first run sequence for Windows. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Added support for control of all import via master_preferences Created 10 years, 5 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/views/first_run_search_engine_view.h
===================================================================
--- chrome/browser/views/first_run_search_engine_view.h (revision 52523)
+++ chrome/browser/views/first_run_search_engine_view.h (working copy)
@@ -7,7 +7,7 @@
#include <vector>
-#include "chrome/browser/views/keyword_editor_view.h"
+#include "chrome/browser/search_engines/template_url_model.h"
#include "gfx/size.h"
#include "views/controls/button/native_button.h"
#include "views/view.h"
@@ -73,14 +73,6 @@
DISALLOW_COPY_AND_ASSIGN(SearchEngineChoice);
};
-// This class receives a callback when the search engine dialog closes.
-class SearchEngineSelectionObserver {
- public:
- virtual ~SearchEngineSelectionObserver() {}
- // Called when the user has chosen a search engine.
- virtual void SearchEngineChosen(const TemplateURL* default_search) = 0;
-};
-
// This class displays a large search engine choice dialog view during
// initial first run import.
class FirstRunSearchEngineView
@@ -89,13 +81,9 @@
public views::WindowDelegate,
public TemplateURLModelObserver {
public:
- // |observer| is the FirstRunView that waits for us to pass back a search
- // engine choice.
- // |profile| allows us to get the set of imported search engines, and
- // display the KeywordEditorView on demand.
+ // |profile| allows us to get the set of imported search engines.
// |randomize| is true if logos are to be displayed in random order.
- FirstRunSearchEngineView(SearchEngineSelectionObserver* observer,
- Profile* profile, bool randomize);
+ FirstRunSearchEngineView(Profile* profile, bool randomize);
virtual ~FirstRunSearchEngineView();
@@ -135,9 +123,6 @@
// The profile associated with this import process.
Profile* profile_;
- // Gets called back when one of the choice buttons is pressed.
- SearchEngineSelectionObserver* observer_;
-
bool text_direction_is_rtl_;
// Image of browser search box with grey background and bubble arrow.
« no previous file with comments | « chrome/browser/views/first_run_customize_view.cc ('k') | chrome/browser/views/first_run_search_engine_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698