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

Side by Side Diff: trunk/src/chrome/browser/search_engines/template_url_service.cc

Issue 259093003: Revert 266615 "Store default search provider data in dictionary ..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/search_engines/template_url_service.h" 5 #include "chrome/browser/search_engines/template_url_service.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/auto_reset.h" 10 #include "base/auto_reset.h"
(...skipping 271 matching lines...) Expand 10 before | Expand all | Expand 10 after
282 loaded_(false), 282 loaded_(false),
283 load_failed_(false), 283 load_failed_(false),
284 load_handle_(0), 284 load_handle_(0),
285 default_search_provider_(NULL), 285 default_search_provider_(NULL),
286 is_default_search_managed_(false), 286 is_default_search_managed_(false),
287 next_id_(kInvalidTemplateURLID + 1), 287 next_id_(kInvalidTemplateURLID + 1),
288 time_provider_(&base::Time::Now), 288 time_provider_(&base::Time::Now),
289 models_associated_(false), 289 models_associated_(false),
290 processing_syncer_changes_(false), 290 processing_syncer_changes_(false),
291 pending_synced_default_search_(false), 291 pending_synced_default_search_(false),
292 dsp_change_origin_(DSP_CHANGE_OTHER), 292 dsp_change_origin_(DSP_CHANGE_OTHER) {
293 default_search_manager_(new DefaultSearchManager(GetPrefs())) {
294 DCHECK(profile_); 293 DCHECK(profile_);
295 Init(NULL, 0); 294 Init(NULL, 0);
296 } 295 }
297 296
298 TemplateURLService::TemplateURLService(const Initializer* initializers, 297 TemplateURLService::TemplateURLService(const Initializer* initializers,
299 const int count) 298 const int count)
300 : provider_map_(new SearchHostToURLsMap), 299 : provider_map_(new SearchHostToURLsMap),
301 profile_(NULL), 300 profile_(NULL),
302 loaded_(false), 301 loaded_(false),
303 load_failed_(false), 302 load_failed_(false),
(...skipping 368 matching lines...) Expand 10 before | Expand all | Expand 10 after
672 return !is_default_search_managed() && 671 return !is_default_search_managed() &&
673 !IsExtensionControlledDefaultSearch() && 672 !IsExtensionControlledDefaultSearch() &&
674 (url != GetDefaultSearchProvider()) && 673 (url != GetDefaultSearchProvider()) &&
675 url->url_ref().SupportsReplacement() && 674 url->url_ref().SupportsReplacement() &&
676 (url->GetType() == TemplateURL::NORMAL); 675 (url->GetType() == TemplateURL::NORMAL);
677 } 676 }
678 677
679 void TemplateURLService::SetUserSelectedDefaultSearchProvider( 678 void TemplateURLService::SetUserSelectedDefaultSearchProvider(
680 TemplateURL* url) { 679 TemplateURL* url) {
681 SetDefaultSearchProvider(url); 680 SetDefaultSearchProvider(url);
682 if (default_search_manager_) {
683 if (url)
684 default_search_manager_->SetUserSelectedDefaultSearchEngine(url->data());
685 else
686 default_search_manager_->ClearUserSelectedDefaultSearchEngine();
687 }
688 } 681 }
689 682
690 TemplateURL* TemplateURLService::GetDefaultSearchProvider() { 683 TemplateURL* TemplateURLService::GetDefaultSearchProvider() {
691 if (loaded_ && !load_failed_) 684 if (loaded_ && !load_failed_)
692 return default_search_provider_; 685 return default_search_provider_;
693 // We're not loaded, rely on the default search provider stored in prefs. 686 // We're not loaded, rely on the default search provider stored in prefs.
694 return initial_default_search_provider_.get(); 687 return initial_default_search_provider_.get();
695 } 688 }
696 689
697 bool TemplateURLService::IsSearchResultsPageFromDefaultSearchProvider( 690 bool TemplateURLService::IsSearchResultsPageFromDefaultSearchProvider(
(...skipping 2056 matching lines...) Expand 10 before | Expand all | Expand 10 after
2754 new_dse = *i; 2747 new_dse = *i;
2755 break; 2748 break;
2756 } 2749 }
2757 } 2750 }
2758 } 2751 }
2759 } 2752 }
2760 if (!new_dse) 2753 if (!new_dse)
2761 new_dse = FindNewDefaultSearchProvider(); 2754 new_dse = FindNewDefaultSearchProvider();
2762 SetDefaultSearchProviderNoNotify(new_dse); 2755 SetDefaultSearchProviderNoNotify(new_dse);
2763 } 2756 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698