Added Protector, hooked up DSE verification with error bubble.

chrome/browser/search_engines/util.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/browser/search_engines/util.h"
 
 #include <set>
 #include <vector>
 
 #include "base/logging.h"
@@ skipping 46 matching lines @@
       } else {
         ids.insert(prepopulate_id);
         ++i;
       }
     } else {
       ++i;
     }
   }
 }
 
+// Returns the TemplateURL with id specified from the list of TemplateURLs.
+// If not found, returns NULL.
+TemplateURL* GetTemplateURLByID(
+    const std::vector<TemplateURL*>& template_urls,
+    int64 id) {
+  for (std::vector<TemplateURL*>::const_iterator i = template_urls.begin();
+       i != template_urls.end(); ++i) {
+    if ((*i)->id() == id) {
+      return *i;
+    }
+  }
+  return NULL;
+}
+
 // Loads engines from prepopulate data and merges them in with the existing
 // engines.  This is invoked when the version of the prepopulate data changes.
 void MergeEnginesFromPrepopulateData(
     PrefService* prefs,
     WebDataService* service,
     std::vector<TemplateURL*>* template_urls,
     const TemplateURL** default_search_provider) {
   DCHECK(service == NULL || BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(template_urls);
   DCHECK(default_search_provider);
@@ skipping 78 matching lines @@
     WebDataService* service,
     PrefService* prefs,
     std::vector<TemplateURL*>* template_urls,
     const TemplateURL** default_search_provider,
     int* new_resource_keyword_version) {
   DCHECK(service == NULL || BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(template_urls);
   DCHECK(template_urls->empty());
   DCHECK(default_search_provider);
   DCHECK(*default_search_provider == NULL);
-  DCHECK(result.GetType() == KEYWORDS_RESULT);
+  DCHECK_EQ(result.GetType(), KEYWORDS_RESULT);
   DCHECK(new_resource_keyword_version);
 
   *new_resource_keyword_version = 0;
   WDKeywordsResult keyword_result = reinterpret_cast<
       const WDResult<WDKeywordsResult>*>(&result)->GetValue();
 
   template_urls->swap(keyword_result.keywords);
 
   const int resource_keyword_version =
       TemplateURLPrepopulateData::GetDataVersion(prefs);
   if (keyword_result.builtin_keyword_version != resource_keyword_version) {
     // There should never be duplicate TemplateURLs. We had a bug such that
     // duplicate TemplateURLs existed for one locale. As such we invoke
     // RemoveDuplicatePrepopulateIDs to nuke the duplicates.
     RemoveDuplicatePrepopulateIDs(template_urls, service);
   }
 
-  if (keyword_result.default_search_provider_id) {
-    // See if we can find the default search provider.
-    for (std::vector<TemplateURL*>::iterator i = template_urls->begin();
-         i != template_urls->end(); ++i) {
-      if ((*i)->id() == keyword_result.default_search_provider_id) {
-        *default_search_provider = *i;
-        break;
-      }
-    }
+  int64 default_search_provider_id = keyword_result.default_search_provider_id;
+  if (default_search_provider_id) {
+    *default_search_provider =
+        GetTemplateURLByID(*template_urls, default_search_provider_id);
   }
 
   if (keyword_result.builtin_keyword_version != resource_keyword_version) {
     MergeEnginesFromPrepopulateData(prefs, service, template_urls,
                                     default_search_provider);
     *new_resource_keyword_version = resource_keyword_version;
   }
 }
 
+bool DidDefaultSearchProviderChange(
+    const WDTypedResult& result,
+    const std::vector<TemplateURL*>& template_urls,
+    const TemplateURL** backup_default_search_provider) {
+  DCHECK(backup_default_search_provider);
+  DCHECK(*backup_default_search_provider == NULL);
+  DCHECK_EQ(result.GetType(), KEYWORDS_RESULT);
+
+  WDKeywordsResult keyword_result = reinterpret_cast<
+      const WDResult<WDKeywordsResult>*>(&result)->GetValue();
+
+  *backup_default_search_provider = GetTemplateURLByID(
+      template_urls,
+      keyword_result.default_search_provider_id_backup);
+  return keyword_result.did_default_search_provider_change;

[sky, 2011/10/21 21:34:11]

Can't we return right away if !did_default_Search_provider_change?

[whywhat, 2011/10/24 15:30:17]

Done.

+}
+