Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(457)

Side by Side Diff: chrome/browser/browser_main_unittest.cc

Issue 7779040: Start moving code from BrowserMain to content, so that it can be reused by all embedders of conte... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix windows unittest Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/browser_main.h" 5 #include "chrome/browser/browser_main.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/memory/scoped_ptr.h"
11 #include "chrome/browser/chrome_content_browser_client.h"
10 #include "chrome/common/chrome_switches.h" 12 #include "chrome/common/chrome_switches.h"
11 #include "chrome/test/base/testing_pref_service.h" 13 #include "chrome/test/base/testing_pref_service.h"
14 #include "content/browser/content_browser_client.h"
12 #include "content/common/main_function_params.h" 15 #include "content/common/main_function_params.h"
13 #include "content/common/sandbox_init_wrapper.h" 16 #include "content/common/sandbox_init_wrapper.h"
14 #include "net/socket/client_socket_pool_base.h" 17 #include "net/socket/client_socket_pool_base.h"
15 #include "testing/gtest/include/gtest/gtest.h" 18 #include "testing/gtest/include/gtest/gtest.h"
16 19
17 class BrowserMainTest : public testing::Test { 20 class BrowserMainTest : public testing::Test {
18 public: 21 public:
19 BrowserMainTest() : command_line_(CommandLine::NO_PROGRAM) {} 22 BrowserMainTest() : command_line_(CommandLine::NO_PROGRAM) {}
20 protected: 23 protected:
21 virtual void SetUp() { 24 virtual void SetUp() {
22 sandbox_init_wrapper_.reset(new SandboxInitWrapper()); 25 sandbox_init_wrapper_.reset(new SandboxInitWrapper());
23 } 26 }
24 27
25 scoped_ptr<SandboxInitWrapper> sandbox_init_wrapper_; 28 scoped_ptr<SandboxInitWrapper> sandbox_init_wrapper_;
26 TestingPrefService pref_service_; 29 TestingPrefService pref_service_;
27 CommandLine command_line_; 30 CommandLine command_line_;
28 }; 31 };
29 32
30 TEST_F(BrowserMainTest, WarmConnectionFieldTrial_WarmestSocket) { 33 TEST_F(BrowserMainTest, WarmConnectionFieldTrial_WarmestSocket) {
31 command_line_.AppendSwitchASCII(switches::kSocketReusePolicy, "0"); 34 command_line_.AppendSwitchASCII(switches::kSocketReusePolicy, "0");
32 35
33 scoped_ptr<MainFunctionParams> params( 36 scoped_ptr<MainFunctionParams> params(
34 new MainFunctionParams(command_line_, *sandbox_init_wrapper_, NULL)); 37 new MainFunctionParams(command_line_, *sandbox_init_wrapper_, NULL));
35 scoped_ptr<BrowserMainParts> bw(BrowserMainParts::CreateBrowserMainParts( 38 scoped_ptr<content::BrowserMainParts> bw(
36 *params)); 39 content::GetContentClient()->browser()->CreateBrowserMainParts(
40 *params));
37 41
38 bw->WarmConnectionFieldTrial(); 42 ChromeBrowserMainParts* cbw = static_cast<ChromeBrowserMainParts*>(bw.get());
43 cbw->WarmConnectionFieldTrial();
39 44
40 EXPECT_EQ(0, net::GetSocketReusePolicy()); 45 EXPECT_EQ(0, net::GetSocketReusePolicy());
41 } 46 }
42 47
43 TEST_F(BrowserMainTest, WarmConnectionFieldTrial_Random) { 48 TEST_F(BrowserMainTest, WarmConnectionFieldTrial_Random) {
44 scoped_ptr<MainFunctionParams> params( 49 scoped_ptr<MainFunctionParams> params(
45 new MainFunctionParams(command_line_, *sandbox_init_wrapper_, NULL)); 50 new MainFunctionParams(command_line_, *sandbox_init_wrapper_, NULL));
46 scoped_ptr<BrowserMainParts> bw(BrowserMainParts::CreateBrowserMainParts( 51 scoped_ptr<content::BrowserMainParts> bw(
47 *params)); 52 content::GetContentClient()->browser()->CreateBrowserMainParts(
53 *params));
54 ChromeBrowserMainParts* cbw = static_cast<ChromeBrowserMainParts*>(bw.get());
48 55
49 const int kNumRuns = 1000; 56 const int kNumRuns = 1000;
50 for (int i = 0; i < kNumRuns; i++) { 57 for (int i = 0; i < kNumRuns; i++) {
51 bw->WarmConnectionFieldTrial(); 58 cbw->WarmConnectionFieldTrial();
52 int val = net::GetSocketReusePolicy(); 59 int val = net::GetSocketReusePolicy();
53 EXPECT_LE(val, 2); 60 EXPECT_LE(val, 2);
54 EXPECT_GE(val, 0); 61 EXPECT_GE(val, 0);
55 } 62 }
56 } 63 }
57 64
58 TEST_F(BrowserMainTest, WarmConnectionFieldTrial_Invalid) { 65 TEST_F(BrowserMainTest, WarmConnectionFieldTrial_Invalid) {
59 command_line_.AppendSwitchASCII(switches::kSocketReusePolicy, "100"); 66 command_line_.AppendSwitchASCII(switches::kSocketReusePolicy, "100");
60 67
61 scoped_ptr<MainFunctionParams> params( 68 scoped_ptr<MainFunctionParams> params(
62 new MainFunctionParams(command_line_, *sandbox_init_wrapper_, NULL)); 69 new MainFunctionParams(command_line_, *sandbox_init_wrapper_, NULL));
63 scoped_ptr<BrowserMainParts> bw(BrowserMainParts::CreateBrowserMainParts( 70 // This test ends up launching a new process, and that doesn't initialize the
64 *params)); 71 // ContentClient interfaces.
65 72 scoped_ptr<content::BrowserMainParts> bw;
66 EXPECT_DEBUG_DEATH(bw->WarmConnectionFieldTrial(), 73 if (content::GetContentClient()) {
74 bw.reset(content::GetContentClient()->browser()->CreateBrowserMainParts(
75 *params));
76 } else {
77 chrome::ChromeContentBrowserClient ccbc;
78 bw.reset(ccbc.CreateBrowserMainParts(*params));
79 }
80 ChromeBrowserMainParts* cbw = static_cast<ChromeBrowserMainParts*>(bw.get());
81 EXPECT_DEBUG_DEATH(cbw->WarmConnectionFieldTrial(),
67 "Not a valid socket reuse policy group"); 82 "Not a valid socket reuse policy group");
68 } 83 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698