Send onLaunched to apps that don't listen to onRestarted.

Send onLaunched to apps that don't listen to onRestarted.

BUG=230667, 162057
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=195698

[not at google - send to devlin, 2013-04-19 14:16:52]

lgtm

https://codereview.chromium.org/14348015/diff/1/chrome/browser/extensions/api...
File chrome/browser/extensions/api/app_runtime/app_runtime_api.cc (right):

https://codereview.chromium.org/14348015/diff/1/chrome/browser/extensions/api...
chrome/browser/extensions/api/app_runtime/app_runtime_api.cc:40: extension_id);
1 line?

https://codereview.chromium.org/14348015/diff/1/chrome/browser/extensions/api...
chrome/browser/extensions/api/app_runtime/app_runtime_api.cc:45: extension_id);
1 line?

https://codereview.chromium.org/14348015/diff/1/chrome/browser/extensions/pla...
File chrome/browser/extensions/platform_app_launcher.cc (right):

https://codereview.chromium.org/14348015/diff/1/chrome/browser/extensions/pla...
chrome/browser/extensions/platform_app_launcher.cc:430:
LaunchPlatformApp(profile, extension, NULL, base::FilePath());
Should this be LaunchPlatformAppWithNoData?

[koz (OOO until 15th September), 2013-04-21 23:33:03]

Thanks!

https://codereview.chromium.org/14348015/diff/1/chrome/browser/extensions/api...
File chrome/browser/extensions/api/app_runtime/app_runtime_api.cc (right):

https://codereview.chromium.org/14348015/diff/1/chrome/browser/extensions/api...
chrome/browser/extensions/api/app_runtime/app_runtime_api.cc:40: extension_id);
On 2013/04/19 14:16:52, kalman wrote:
> 1 line?

Done.

https://codereview.chromium.org/14348015/diff/1/chrome/browser/extensions/api...
chrome/browser/extensions/api/app_runtime/app_runtime_api.cc:45: extension_id);
On 2013/04/19 14:16:52, kalman wrote:
> 1 line?

Done.

https://codereview.chromium.org/14348015/diff/1/chrome/browser/extensions/pla...
File chrome/browser/extensions/platform_app_launcher.cc (right):

https://codereview.chromium.org/14348015/diff/1/chrome/browser/extensions/pla...
chrome/browser/extensions/platform_app_launcher.cc:430:
LaunchPlatformApp(profile, extension, NULL, base::FilePath());
On 2013/04/19 14:16:52, kalman wrote:
> Should this be LaunchPlatformAppWithNoData?

Done.

[commit-bot: I haz the power, 2013-04-21 23:33:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/14348015/5001

[commit-bot: I haz the power, 2013-04-21 23:36:18]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura...

[commit-bot: I haz the power, 2013-04-22 00:05:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/14348015/17001

[commit-bot: I haz the power, 2013-04-22 01:12:03]

Retried try job too often on mac_rel for step(s) base_unittests, browser_tests,
cacheinvalidation_unittests, chromedriver2_unittests, components_unittests,
content_browsertests, content_unittests, crypto_unittests, gpu_unittests,
interactive_ui_tests, ipc_tests, jingle_unittests, media_unittests,
net_unittests, ppapi_unittests, printing_unittests, remoting_unittests,
sql_unittests, sync_integration_tests, sync_unit_tests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-04-22 01:45:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/14348015/17001

[commit-bot: I haz the power, 2013-04-22 03:00:21]

Retried try job too often on mac_rel for step(s) base_unittests, browser_tests,
cacheinvalidation_unittests, chromedriver2_unittests, components_unittests,
content_browsertests, content_unittests, crypto_unittests, gpu_unittests,
interactive_ui_tests, ipc_tests, jingle_unittests, media_unittests,
net_unittests, ppapi_unittests, printing_unittests, remoting_unittests,
sql_unittests, sync_integration_tests, sync_unit_tests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-04-23 00:13:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/14348015/17001

[commit-bot: I haz the power, 2013-04-23 03:06:16]

Change committed as 195698