Add implementations of mojo::{Run,Terminate}[Main]Application() for "chromium".

Add implementations of mojo::{Run,Terminate}[Main]Application() for "chromium".

And convert a bunch of uses of ApplicationRunnerChromium (and
ApplicationDelegate). Mostly, I converted stuff until I got bored.

But:
* I still have to convert some Mozart-related base classes.
* I also have to fix the tracing library to not take an
  ApplicationImpl*.
* I have to add the ability to set the message loop type.
* I made Run[Main]Application() return MOJO_RESULT_OK by default (e.g.,
  if the message loop was quit via something other than
  Terminate[Main]Application()), since it's what
  ApplicationRunner[Chromium] did (and some tests rely on this). We
  should fix this.

R=vardhan@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/9030792b19912f334f058261e23c2b36bc091ce0

[viettrungluu, 2016-05-25 17:41:23]

ping

[vardhan, 2016-05-25 17:59:55]

lgtm

https://codereview.chromium.org/2005103003/diff/1/mojo/application/run_applic...
File mojo/application/run_application.cc (right):

https://codereview.chromium.org/2005103003/diff/1/mojo/application/run_applic...
mojo/application/run_application.cc:34: // |ApplicationRunnerChromium| always
returned |MOJO_RESULT_OK|.
would anything break if we changed it now?

[viettrungluu, 2016-05-25 18:10:07]

Thanks.

https://codereview.chromium.org/2005103003/diff/1/mojo/application/run_applic...
File mojo/application/run_application.cc (right):

https://codereview.chromium.org/2005103003/diff/1/mojo/application/run_applic...
mojo/application/run_application.cc:34: // |ApplicationRunnerChromium| always
returned |MOJO_RESULT_OK|.
On 2016/05/25 17:59:54, vardhan wrote:
> would anything break if we changed it now?

Yes, see CL description.

[viettrungluu, 2016-05-25 18:25:12]

Description was changed from

==========
Add implementations of mojo::{Run,Terminate}[Main]Application() for "chromium".

And convert a bunch of uses of ApplicationRunnerChromium (and
ApplicationDelegate). Mostly, I converted stuff until I got bored.

But:
* I still have to convert some Mozart-related base classes.
* I also have to fix the tracing library to not take an
  ApplicationImpl*.
* I have to add the ability to set the message loop type.
* I made Run[Main]Application() return MOJO_RESULT_OK by default (e.g.,
  if the message loop was quit via something other than
  Terminate[Main]Application()), since it's what
  ApplicationRunner[Chromium] did (and some tests rely on this). We
  should fix this.

R=vardhan@google.com
==========

to

==========
Add implementations of mojo::{Run,Terminate}[Main]Application() for "chromium".

And convert a bunch of uses of ApplicationRunnerChromium (and
ApplicationDelegate). Mostly, I converted stuff until I got bored.

But:
* I still have to convert some Mozart-related base classes.
* I also have to fix the tracing library to not take an
  ApplicationImpl*.
* I have to add the ability to set the message loop type.
* I made Run[Main]Application() return MOJO_RESULT_OK by default (e.g.,
  if the message loop was quit via something other than
  Terminate[Main]Application()), since it's what
  ApplicationRunner[Chromium] did (and some tests rely on this). We
  should fix this.

R=vardhan@google.com

Committed:
https://chromium.googlesource.com/external/mojo/+/9030792b19912f334f058261e23...
==========

[viettrungluu, 2016-05-25 18:25:14]

Committed patchset #1 (id:1) manually as
9030792b19912f334f058261e23c2b36bc091ce0 (presubmit successful).