| Index: chrome/browser/ui/browser_init.cc
|
| ===================================================================
|
| --- chrome/browser/ui/browser_init.cc (revision 66367)
|
| +++ chrome/browser/ui/browser_init.cc (working copy)
|
| @@ -961,11 +961,10 @@
|
| expected_tab_count =
|
| std::max(1, static_cast<int>(command_line.args().size()));
|
| }
|
| - if (!CreateAutomationProvider<TestingAutomationProvider>(
|
| + CreateAutomationProvider<TestingAutomationProvider>(
|
| testing_channel_id,
|
| profile,
|
| - static_cast<size_t>(expected_tab_count)))
|
| - return false;
|
| + static_cast<size_t>(expected_tab_count));
|
| }
|
| }
|
|
|
| @@ -982,13 +981,11 @@
|
| silent_launch = true;
|
|
|
| if (command_line.HasSwitch(switches::kChromeFrame)) {
|
| - if (!CreateAutomationProvider<ChromeFrameAutomationProvider>(
|
| - automation_channel_id, profile, expected_tabs))
|
| - return false;
|
| + CreateAutomationProvider<ChromeFrameAutomationProvider>(
|
| + automation_channel_id, profile, expected_tabs);
|
| } else {
|
| - if (!CreateAutomationProvider<AutomationProvider>(
|
| - automation_channel_id, profile, expected_tabs))
|
| - return false;
|
| + CreateAutomationProvider<AutomationProvider>(automation_channel_id,
|
| + profile, expected_tabs);
|
| }
|
| }
|
|
|
| @@ -1023,20 +1020,16 @@
|
| }
|
|
|
| template <class AutomationProviderClass>
|
| -bool BrowserInit::CreateAutomationProvider(const std::string& channel_id,
|
| +void BrowserInit::CreateAutomationProvider(const std::string& channel_id,
|
| Profile* profile,
|
| size_t expected_tabs) {
|
| scoped_refptr<AutomationProviderClass> automation =
|
| new AutomationProviderClass(profile);
|
| -
|
| - if (!automation->InitializeChannel(channel_id))
|
| - return false;
|
| + automation->ConnectToChannel(channel_id);
|
| automation->SetExpectedTabCount(expected_tabs);
|
|
|
| AutomationProviderList* list =
|
| g_browser_process->InitAutomationProviderList();
|
| DCHECK(list);
|
| list->AddProvider(automation);
|
| -
|
| - return true;
|
| }
|
|
|