Add reload item to default developer-mode platform app context menu.

Add reload to default developer-mode platform app context menu.

Without this, the only way to reload a platform app is to close the window and
open it again, losing devtools context.

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=125075

[jeremya, 2012-03-01 23:53:39]

benwells- you wrote this code :)
brettw- chrome/browser/tab_contents owner
mihaip- chrome/browser/extensions owner

I added this because I found while developing a test platform app, it was really
inconvenient to close & re-open my app every time I updated the JS. (In
addition, it's not possible to navigate using window.location.reload(), since
navigation is disabled in platform apps).

Thanks!
Jeremy

[benwells, 2012-03-02 00:13:18]

lgtm with or without the change suggested, up to you.

http://codereview.chromium.org/9569042/diff/1/chrome/browser/extensions/platf...
File chrome/browser/extensions/platform_app_browsertest.cc (right):

http://codereview.chromium.org/9569042/diff/1/chrome/browser/extensions/platf...
chrome/browser/extensions/platform_app_browsertest.cc:138: ASSERT_EQ(2,
menu->menu_model().GetItemCount());
These tests are a bit painful as they are written (my bad, I wrote them).

If its not complicated could you change these to look for the presence / absence
of specific menu items instead of the absolute count? That way they won't need
to keep changing whenever something is added.

[Mihai Parparita -not on Chrome, 2012-03-02 01:15:49]

LGTM

[brettw, 2012-03-02 05:40:58]

lgtm

[commit-bot: I haz the power, 2012-03-05 05:14:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jeremya@chromium.org/9569042/1

[commit-bot: I haz the power, 2012-03-05 06:15:36]

Try job failure for 9569042-1 (retry) on linux_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2012-03-06 00:22:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jeremya@chromium.org/9569042/12001

[commit-bot: I haz the power, 2012-03-06 02:23:19]

Change committed as 125075