Remove direct reference to GetBlockingPool() in c/b/google_apis.

chrome/browser/google_apis/base_requests_unittest.cc

 // Copyright (c) 2012 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/google_apis/base_requests.h"
 
 #include "base/bind.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "base/values.h"
@@ skipping 27 matching lines @@
 };
 
 }  // namespace
 
 class BaseRequestsTest : public testing::Test {
  public:
   virtual void SetUp() OVERRIDE {
     profile_.reset(new TestingProfile);
     sender_.reset(new RequestSender(profile_.get(),
                                     NULL /* url_request_context_getter */,
+                                    message_loop_.message_loop_proxy(),
                                     std::vector<std::string>() /* scopes */,
                                     std::string() /* custom user agent */));
     sender_->Initialize();
   }
 
   base::MessageLoop message_loop_;
   scoped_ptr<TestingProfile> profile_;
   scoped_ptr<RequestSender> sender_;
 };
 
 TEST_F(BaseRequestsTest, ParseValidJson) {
   scoped_ptr<base::Value> json;
-  ParseJson(kValidJsonString,
+  ParseJson(message_loop_.message_loop_proxy(),
+            kValidJsonString,
             base::Bind(test_util::CreateCopyResultCallback(&json)));
   // Should wait for a blocking pool task, as the JSON parsing is done in the
   // blocking pool.
   test_util::RunBlockingPoolTask();

[satorux1, 2013/07/08 03:54:22]

this looks unnecessary?

 
   DictionaryValue* root_dict = NULL;
   ASSERT_TRUE(json);
   ASSERT_TRUE(json->GetAsDictionary(&root_dict));
 
   int int_value = 0;
   ASSERT_TRUE(root_dict->GetInteger("test", &int_value));
   EXPECT_EQ(123, int_value);
 }
 
 TEST_F(BaseRequestsTest, ParseInvalidJson) {
   // Initialize with a valid pointer to verify that null is indeed assigned.
   scoped_ptr<base::Value> json(base::Value::CreateNullValue());
-  ParseJson(kInvalidJsonString,
+  ParseJson(message_loop_.message_loop_proxy(),
+            kInvalidJsonString,
             base::Bind(test_util::CreateCopyResultCallback(&json)));
   // Should wait for a blocking pool task, as the JSON parsing is done in the
   // blocking pool.
   test_util::RunBlockingPoolTask();
 
   EXPECT_FALSE(json);
 }
 
 TEST_F(BaseRequestsTest, GetDataRequestParseValidResponse) {
   GDataErrorCode error = GDATA_OTHER_ERROR;
@@ skipping 23 matching lines @@
   get_data_request->ParseResponse(HTTP_SUCCESS, kInvalidJsonString);
   // Should wait for a blocking pool task, as the JSON parsing is done in the
   // blocking pool.
   test_util::RunBlockingPoolTask();
 
   EXPECT_EQ(GDATA_PARSE_ERROR, error);
   EXPECT_FALSE(value);
 }
 
 }  // namespace google_apis