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; |
} |