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

chrome/browser/extensions/external_policy_extension_loader_unittest.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 <string>
 
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/values.h"
 #include "base/version.h"
@@ skipping 67 matching lines @@
 
     // Provider returns the correct location when asked.
     Extension::Location location1;
     scoped_ptr<Version> version1;
     provider_->GetExtensionDetails(id, &location1, &version1);
     EXPECT_EQ(Extension::EXTERNAL_POLICY_DOWNLOAD, location1);
     EXPECT_FALSE(version1.get());
 
     // Remove the extension from our list.
     StringValue ext_str(id + ";" + update_url.spec());
-    EXPECT_NE(-1, remaining_extensions->Remove(ext_str));
+    EXPECT_NE(false, remaining_extensions->Remove(ext_str, NULL));
   }
 
   virtual void OnExternalProviderReady() {
     EXPECT_TRUE(provider_->IsReady());
   }
 
  private:
   ListValue* remaining_extensions;
 
   scoped_ptr<TestingProfile> profile_;
@@ skipping 38 matching lines @@
       "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa;http#//www.example.com/crx"));
   forced_extensions.Append(Value::CreateStringValue(
       "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"));
   forced_extensions.Append(Value::CreateStringValue(
       "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaahttp#//www.example.com/crx"));
 
   MockExternalPolicyExtensionProviderVisitor mv;
   std::set<std::string> empty;
   mv.Visit(&forced_extensions, &valid_extensions, empty);
 }