base: Fix the TODO in ListValue::Remove().

chrome/browser/extensions/extension_web_ui.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/extensions/extension_web_ui.h"
 
 #include <set>
 #include <vector>
 
 #include "base/string_util.h"
@@ skipping 314 matching lines @@
         continue;
     }
     // Insert the override at the front of the list.  Last registered override
     // wins.
     page_overrides->Insert(0, new StringValue(iter->second.spec()));
   }
 }
 
 // static
 void ExtensionWebUI::UnregisterAndReplaceOverride(const std::string& page,
-    Profile* profile, ListValue* list, Value* override) {
-  int index = list->Remove(*override);
-  if (index == 0) {
+                                                  Profile* profile,
+                                                  ListValue* list,
+                                                  Value* override) {
+  size_t index = 0;
+  bool found = list->Remove(*override, &index);
+  if (found && index == 0) {
     // This is the active override, so we need to find all existing
     // tabs for this override and get them to reload the original URL.
     for (TabContentsIterator iterator; !iterator.done(); ++iterator) {
       TabContents* tab = (*iterator)->tab_contents();
       Profile* tab_profile =
           Profile::FromBrowserContext(tab->browser_context());
       if (tab_profile != profile)
         continue;
 
       GURL url = tab->GetURL();
@@ skipping 50 matching lines @@
 }
 
 // static
 void ExtensionWebUI::GetFaviconForURL(Profile* profile,
     FaviconService::GetFaviconRequest* request, const GURL& page_url) {
   // tracker deletes itself when done.
   ExtensionWebUIImageLoadingTracker* tracker =
       new ExtensionWebUIImageLoadingTracker(profile, request, page_url);
   tracker->Init();
 }