Moved chrome/browser/ui/extensions apps code to chrome/browser/ui/apps

Moved chrome/browser/ui/extensions apps code to chrome/browser/ui/apps

This is part of the ongoing effort to extract apps code from extensions.

TBR=sky
BUG=159366
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=217512

[tapted, 2013-08-13 07:45:21]

lgtm, unless you want to do the comment, in which case lgtm with nits

https://codereview.chromium.org/22992003/diff/1/chrome/browser/ui/apps/apps_m...
File chrome/browser/ui/apps/apps_metro_handler_win.h (right):

https://codereview.chromium.org/22992003/diff/1/chrome/browser/ui/apps/apps_m...
chrome/browser/ui/apps/apps_metro_handler_win.h:10: namespace chrome {
I think the current trend is for stuff under a 'browser/ui' path to go in the
global namespace. Should we change that here?

E.g. AppMetroInfoBarDelegateWin was moved out in http://crrev.com/19759006

[benwells, 2013-08-13 08:25:45]

https://codereview.chromium.org/22992003/diff/1/chrome/browser/ui/apps/apps_m...
File chrome/browser/ui/apps/apps_metro_handler_win.h (right):

https://codereview.chromium.org/22992003/diff/1/chrome/browser/ui/apps/apps_m...
chrome/browser/ui/apps/apps_metro_handler_win.h:10: namespace chrome {
On 2013/08/13 07:45:22, tapted wrote:
> I think the current trend is for stuff under a 'browser/ui' path to go in the
> global namespace. Should we change that here?
> 
> E.g. AppMetroInfoBarDelegateWin was moved out in http://crrev.com/19759006

Good catch, done.

[benwells, 2013-08-13 08:27:10]

+TBR sky for refactoring fallout in
chrome/browser/ui/browser_command_controller.cc

[commit-bot: I haz the power, 2013-08-13 08:27:23]

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

[commit-bot: I haz the power, 2013-08-13 08:42:36]

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=android_db...

[commit-bot: I haz the power, 2013-08-13 21:58:29]

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

[commit-bot: I haz the power, 2013-08-13 23:12:50]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-08-14 06:43:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benwells@chromium.org/22992003/48001

[commit-bot: I haz the power, 2013-08-14 07:49:42]

Retried try job too often on linux_rel for step(s) check_deps
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-08-14 08:55:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benwells@chromium.org/22992003/48001

[commit-bot: I haz the power, 2013-08-14 09:53:57]

Change committed as 217512