testrequest is ignored. Also added wow64 attribute.

chrome/browser/component_updater/test/component_updater_service_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/component_updater/test/component_updater_service_unittest.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/run_loop.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
@@ skipping 73 matching lines @@
 
 GURL TestConfigurator::UpdateUrl() {
   return GURL(POST_INTERCEPT_SCHEME "://"
               POST_INTERCEPT_HOSTNAME POST_INTERCEPT_PATH);
 }
 
 GURL TestConfigurator::PingUrl() {
   return UpdateUrl();
 }
 
-const char* TestConfigurator::ExtraRequestParams() { return "extra=foo"; }
+std::string TestConfigurator::ExtraRequestParams() { return "extra=\"foo\""; }
 
 size_t TestConfigurator::UrlSizeLimit() { return 256; }
 
 net::URLRequestContextGetter* TestConfigurator::RequestContext() {
   return context_.get();
 }
 
 // Don't use the utility process to run code out-of-process.
 bool TestConfigurator::InProcess() { return true; }
 
@@ skipping 361 matching lines @@
       << post_interceptor_->GetRequestsAsString();
 
   EXPECT_NE(string::npos, post_interceptor_->GetRequests()[2].find(
       "<app appid=\"jebgalgnebhfojomionfpkfelancnnkf\" version=\"1.0\">"
       "<updatecheck /></app>"));
   EXPECT_NE(string::npos, post_interceptor_->GetRequests()[2].find(
       "<app appid=\"abagagagagagagagagagagagagagagag\" version=\"2.2\">"
       "<updatecheck /></app>"))
       << post_interceptor_->GetRequestsAsString();
 
+  // Test the protocol version is correct and the extra request attributes
+  // are included in the request.
+  EXPECT_NE(string::npos, post_interceptor_->GetRequests()[0].find(
+      "request protocol=\"3.0\" extra=\"foo\""))
+      << post_interceptor_->GetRequestsAsString();

[cpu_(ooo_6.6-7.5), 2013/12/17 00:51:05]

nice

+
   component_updater()->Stop();
 }
 
 // This test checks that the "prodversionmin" value is handled correctly. In
 // particular there should not be an install because the minimum product
 // version is much higher than of chrome.
 TEST_F(ComponentUpdaterTest, ProdVersionCheck) {
   EXPECT_TRUE(post_interceptor_->ExpectRequest(new PartialMatch(
       "updatecheck"), test_file("updatecheck_reply_2.xml")));
 
@@ skipping 887 matching lines @@
   EXPECT_EQ(1, post_interceptor_->GetHitCount());
   EXPECT_EQ(0, static_cast<TestInstaller*>(com.installer)->error());
   EXPECT_EQ(0, static_cast<TestInstaller*>(com.installer)->install_count());
 
   component_updater()->Stop();
 }
 
 
 }  // namespace component_updater