Fix the extension activity log UI.

Fix the extension activity log UI.

BUG=
TEST=


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

[Eric Dingle, 2012-05-29 20:13:34]

On 2012/05/28 20:54:36, Eric Dingle wrote:

Ping.

[Evan Stade, 2012-05-29 22:12:38]

http://codereview.chromium.org/10449010/diff/2001/chrome/browser/resources/ex...
File chrome/browser/resources/extensions/extension_activity.html (right):

http://codereview.chromium.org/10449010/diff/2001/chrome/browser/resources/ex...
chrome/browser/resources/extensions/extension_activity.html:10: <link
rel="stylesheet" href="../uber/uber_shared.css">
why did you add this? the activity page doesn't load in an iframe in the uber
page does it?

http://codereview.chromium.org/10449010/diff/2001/chrome/browser/ui/webui/ext...
File chrome/browser/ui/webui/extensions/extension_settings_handler.cc (right):

http://codereview.chromium.org/10449010/diff/2001/chrome/browser/ui/webui/ext...
chrome/browser/ui/webui/extensions/extension_settings_handler.cc:679: void
ExtensionSettingsHandler::HandleActivityMessage(const ListValue* args) {
GenericHandler kind of already does this.

[Eric Dingle, 2012-05-30 14:13:37]

http://codereview.chromium.org/10449010/diff/2001/chrome/browser/resources/ex...
File chrome/browser/resources/extensions/extension_activity.html (right):

http://codereview.chromium.org/10449010/diff/2001/chrome/browser/resources/ex...
chrome/browser/resources/extensions/extension_activity.html:10: <link
rel="stylesheet" href="../uber/uber_shared.css">
On 2012/05/29 22:12:38, Evan Stade wrote:
> why did you add this? the activity page doesn't load in an iframe in the uber
> page does it?

That was leftover from trying to match styles to the extensions UI. Removed.

http://codereview.chromium.org/10449010/diff/2001/chrome/browser/ui/webui/ext...
File chrome/browser/ui/webui/extensions/extension_settings_handler.cc (right):

http://codereview.chromium.org/10449010/diff/2001/chrome/browser/ui/webui/ext...
chrome/browser/ui/webui/extensions/extension_settings_handler.cc:679: void
ExtensionSettingsHandler::HandleActivityMessage(const ListValue* args) {
On 2012/05/29 22:12:38, Evan Stade wrote:
> GenericHandler kind of already does this.

Done.

[Evan Stade, 2012-05-30 21:02:11]

lgtm

[commit-bot: I haz the power, 2012-05-31 13:50:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ericdingle@chromium.org/10449010/4003

[commit-bot: I haz the power, 2012-05-31 16:46:50]

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

[commit-bot: I haz the power, 2012-05-31 17:09:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ericdingle@chromium.org/10449010/4003

[commit-bot: I haz the power, 2012-05-31 19:18:38]

Change committed as 139836