Add ChromeBrowserMainExtraParts for non main parts.

chrome/browser/chrome_browser_main_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 <vector>
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "chrome/browser/chrome_browser_main.h"
@@ skipping 12 matching lines @@
  protected:
   TestingPrefService pref_service_;
   CommandLine command_line_;
 };
 
 TEST_F(BrowserMainTest, WarmConnectionFieldTrial_WarmestSocket) {
   command_line_.AppendSwitchASCII(switches::kSocketReusePolicy, "0");
 
   scoped_ptr<content::MainFunctionParams> params(
       new content::MainFunctionParams(command_line_));
-  ScopedVector<content::BrowserMainParts> bwv;
-  content::GetContentClient()->browser()->CreateBrowserMainParts(
-      *params, &(bwv.get()));
-  ChromeBrowserMainParts* cbw = NULL;
-  if (bwv.size() >= 1)
-    cbw = static_cast<ChromeBrowserMainParts*>(bwv[0]);
+  scoped_ptr<content::BrowserMainParts> bw(
+      content::GetContentClient()->browser()->CreateBrowserMainParts(*params));
+  ChromeBrowserMainParts* cbw = static_cast<ChromeBrowserMainParts*>(bw.get());
   EXPECT_TRUE(cbw);
   if (cbw) {
     cbw->WarmConnectionFieldTrial();
     EXPECT_EQ(0, net::GetSocketReusePolicy());
   }
 }
 
 TEST_F(BrowserMainTest, WarmConnectionFieldTrial_Random) {
   scoped_ptr<content::MainFunctionParams> params(
       new content::MainFunctionParams(command_line_));
-  ScopedVector<content::BrowserMainParts> bwv;
-  content::GetContentClient()->browser()->CreateBrowserMainParts(
-      *params, &(bwv.get()));
-  ChromeBrowserMainParts* cbw = NULL;
-  if (bwv.size() >= 1)
-    cbw = static_cast<ChromeBrowserMainParts*>(bwv[0]);
+  scoped_ptr<content::BrowserMainParts> bw(
+      content::GetContentClient()->browser()->CreateBrowserMainParts(*params));
+  ChromeBrowserMainParts* cbw = static_cast<ChromeBrowserMainParts*>(bw.get());
   EXPECT_TRUE(cbw);
   if (cbw) {
     const int kNumRuns = 1000;
     for (int i = 0; i < kNumRuns; i++) {
       cbw->WarmConnectionFieldTrial();
       int val = net::GetSocketReusePolicy();
       EXPECT_LE(val, 2);
       EXPECT_GE(val, 0);
     }
   }
 }
 
 TEST_F(BrowserMainTest, WarmConnectionFieldTrial_Invalid) {
   command_line_.AppendSwitchASCII(switches::kSocketReusePolicy, "100");
 
   scoped_ptr<content::MainFunctionParams> params(
       new content::MainFunctionParams(command_line_));
   // This test ends up launching a new process, and that doesn't initialize the
   // ContentClient interfaces.
-  ScopedVector<content::BrowserMainParts> bwv;
+  scoped_ptr<content::BrowserMainParts> bw;
   if (content::GetContentClient()) {
-    content::GetContentClient()->browser()->CreateBrowserMainParts(
-        *params, &(bwv.get()));
+    bw.reset(content::GetContentClient()->browser()->CreateBrowserMainParts(
+        *params));
   } else {
     chrome::ChromeContentBrowserClient ccbc;
-    ccbc.CreateBrowserMainParts(*params, &(bwv.get()));
+    bw.reset(ccbc.CreateBrowserMainParts(*params));
   }
-  ChromeBrowserMainParts* cbw = NULL;
-  if (bwv.size() >= 1)
-    cbw = static_cast<ChromeBrowserMainParts*>(bwv[0]);
+  ChromeBrowserMainParts* cbw = static_cast<ChromeBrowserMainParts*>(bw.get());
   EXPECT_TRUE(cbw);
   if (cbw) {
 #if defined(NDEBUG) && defined(DCHECK_ALWAYS_ON)
     EXPECT_DEATH(cbw->WarmConnectionFieldTrial(),
                  "Not a valid socket reuse policy group");
 #else
     EXPECT_DEBUG_DEATH(cbw->WarmConnectionFieldTrial(),
                        "Not a valid socket reuse policy group");
 #endif
   }
 }