set user agent for content browser tests

content/test/browser_test_base.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 "content/test/browser_test_base.h"
 
 #include "base/command_line.h"
 #include "base/task.h"
 #include "content/common/main_function_params.h"
 #include "sandbox/src/dep.h"
+#include "webkit/glue/user_agent.h"
+#include "webkit/glue/webkit_glue.h"
 
 #if defined(OS_MACOSX)
 #include "base/mac/mac_util.h"
 #include "base/system_monitor/system_monitor.h"
 #endif
 
 extern int BrowserMain(const MainFunctionParams&);
 
 BrowserTestBase::BrowserTestBase() {
 #if defined(OS_MACOSX)
   base::mac::SetOverrideAmIBundled(true);
   base::SystemMonitor::AllocateSystemIOPorts();
 #endif
 }
 
 BrowserTestBase::~BrowserTestBase() {
 }
 
 void BrowserTestBase::SetUp() {
   SandboxInitWrapper sandbox_wrapper;
   MainFunctionParams params(*CommandLine::ForCurrentProcess(),
                             sandbox_wrapper,
                             NULL);
   params.ui_task =
       NewRunnableMethod(this, &BrowserTestBase::ProxyRunTestOnMainThreadLoop);
 
+  std::string product("Chrome/15.16.17.18");

[Paweł Hajdan Jr., 2011/10/27 08:48:52]

Why not 0.0.0.0 as for DRT? Chrome/15 might suggest a real thing, I think it's
currently in stable.

+  webkit_glue::SetUserAgent(webkit_glue::BuildUserAgentFromProduct(product),

[Paweł Hajdan Jr., 2011/10/27 08:48:52]

I don't like it anyway. We're calling BrowserMain, why not let it do it?
BrowserTestBase::SetUp should be kept extremely simple.

+                            false);
   SetUpInProcessBrowserTestFixture();
   BrowserMain(params);
   TearDownInProcessBrowserTestFixture();
 }
 
 void BrowserTestBase::TearDown() {
 }
 
 void BrowserTestBase::ProxyRunTestOnMainThreadLoop() {
   RunTestOnMainThreadLoop();
 }