content: Terminate early if the toolkit initialization fails.

content/browser/browser_main_runner.cc

Index: content/browser/browser_main_runner.cc
diff --git a/content/browser/browser_main_runner.cc b/content/browser/browser_main_runner.cc
index 1f644121a06df750f04f344d91ecb6128750cd38..16309dec8460af490e891e47ace869c483d258c5 100644
--- a/content/browser/browser_main_runner.cc
+++ b/content/browser/browser_main_runner.cc
@@ -84,7 +84,8 @@ class BrowserMainRunnerImpl : public BrowserMainRunner {
       main_loop_->EarlyInitialization();
 
       // Must happen before we try to use a message loop or display any UI.
-      main_loop_->InitializeToolkit();
+      if (!main_loop_->InitializeToolkit())
+        return 1;

[jam, 2014/04/25 05:05:53]

the comment says 0 or negative means initialization failed, but you're returning
1?

[sadrul, 2014/04/25 10:43:35]

I messed up the comment (I think I started with 'zero or positive' and decided
halfway through that 'non-negative' would be better, and just messed up the
comment). Whoops. Fixed.

 
       main_loop_->MainMessageLoopStart();