Move active_downloads_ui.* to chrome/browser/ui/webui/chromeos

Move active_downloads_ui.* to chrome/browser/ui/webui/chromeos

BUG=none
TEST=unit tests

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=126638

[achuithb, 2012-03-10 03:55:57]

Please also run this against the linux_chromeos_gtk trybot. LGTM assuming you
get a green run on that bot.

http://codereview.chromium.org/9668039/diff/4001/chrome/browser/ui/webui/chro...
File chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc (right):

http://codereview.chromium.org/9668039/diff/4001/chrome/browser/ui/webui/chro...
chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc:70: #include
"chrome/browser/ui/webui/chromeos/active_downloads_ui.h"
alphabetical please

[asanka, 2012-03-10 05:10:30]

Thanks! Will do a CrOS GTK try run.

http://codereview.chromium.org/9668039/diff/4001/chrome/browser/ui/webui/chro...
File chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc (right):

http://codereview.chromium.org/9668039/diff/4001/chrome/browser/ui/webui/chro...
chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc:70: #include
"chrome/browser/ui/webui/chromeos/active_downloads_ui.h"
On 2012/03/10 03:55:58, achuith.bhandarkar wrote:
> alphabetical please

Done.

[asanka, 2012-03-12 16:15:58]

+sky for chrome/browser/ui/OWNERS

active_downloads is only used on ChromeOS. This CL is to move it to a ChromeOS
specific directory.

[sky, 2012-03-12 17:11:23]

I'm not a webui owner, but the other chrome/browser/ui changes LGTM

[asanka, 2012-03-12 17:16:15]

On 2012/03/12 17:11:23, sky wrote:
> I'm not a webui owner, but the other chrome/browser/ui changes LGTM

Thanks!

I added other reviewers for chrome/browser/ui/webui (estade) and
chrome/browser/ui/webui/chromeos (achuith). So I think we have OWNER coverage
for chrome/browser/ui now.

[Evan Stade, 2012-03-12 20:16:30]

lgtm

[asanka, 2012-03-12 21:06:18]

On 2012/03/12 20:16:30, Evan Stade wrote:
> lgtm

Thanks!

[commit-bot: I haz the power, 2012-03-12 21:08:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asanka@chromium.org/9668039/4012

[commit-bot: I haz the power, 2012-03-12 21:51:23]

Try job failure for 9668039-4012 (retry) on linux_clang for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_clan...

[commit-bot: I haz the power, 2012-03-12 23:08:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asanka@chromium.org/9668039/4012

[commit-bot: I haz the power, 2012-03-12 23:27:49]

Try job failure for 9668039-4012 (retry) on linux_chromeos for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2012-03-13 14:15:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asanka@chromium.org/9668039/4012

[commit-bot: I haz the power, 2012-03-13 16:54:12]

Try job failure for 9668039-4012 (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-03-13 18:13:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asanka@chromium.org/9668039/4012

[commit-bot: I haz the power, 2012-03-13 20:46:52]

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

[achuithb, 2012-03-13 21:00:55]

On 2012/03/13 20:46:52, I haz the power (commit-bot) wrote:
> Try job failure for 9668039-4012 (retry) on win_rel for step "ui_tests".
> It's a second try, previously, steps "ui_tests, installer_util_unittests"
> failed.
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

Wow! I think it's time for gcl commit

[asanka, 2012-03-13 21:07:46]

On 2012/03/13 21:00:55, achuith.bhandarkar wrote:
> On 2012/03/13 20:46:52, I haz the power (commit-bot) wrote:
> > Try job failure for 9668039-4012 (retry) on win_rel for step "ui_tests".
> > It's a second try, previously, steps "ui_tests, installer_util_unittests"
> > failed.
> >
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...
> 
> Wow! I think it's time for gcl commit

Yup. I'll do so later in the evening when I have time to babysit the commit.