Remove HostDesktopType from c/b/ui/views/apps/app_info_dialog

Remove HostDesktopType from c/b/ui/views/apps/app_info_dialog

R=sashab@chromium.org
BUG=558054
Committed: https://crrev.com/2ebd17bbd50e961d2a31c00ae4c209a8e2443dd3
Cr-Commit-Position: refs/heads/master@{#376820}

[scottmg, 2016-02-16 01:16:09]

Description was changed from

==========
Remove HostDesktopType from c/b/ui/views/apps/app_info_dialog


BUG=558054
==========

to

==========
Remove HostDesktopType from c/b/ui/views/apps/app_info_dialog

R=sashab@chromium.org
BUG=558054
==========

[scottmg, 2016-02-16 01:16:09]

scottmg@chromium.org changed reviewers:
+ sashab@chromium.org

[scottmg, 2016-02-19 22:26:20]

scottmg@chromium.org changed reviewers:
+ tapted@chromium.org
- sashab@chromium.org

[scottmg, 2016-02-19 22:26:20]

up to higher level owners -> tapted

[tapted, 2016-02-22 03:01:49]

lgtm %nit

https://codereview.chromium.org/1702523002/diff/20001/chrome/browser/ui/views...
File chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc
(right):

https://codereview.chromium.org/1702523002/diff/20001/chrome/browser/ui/views...
chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc:10:
#include "chrome/browser/ui/host_desktop.h"
nit: can this include go?

[scottmg, 2016-02-22 20:53:34]

The CQ bit was checked by scottmg@chromium.org

[scottmg, 2016-02-22 20:53:34]

The patchset sent to the CQ was uploaded after l-g-t-m from tapted@chromium.org
Link to the patchset: https://codereview.chromium.org/1702523002/#ps40001
(title: "include")

[commit-bot: I haz the power, 2016-02-22 20:55:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1702523002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1702523002/40001

[commit-bot: I haz the power, 2016-02-22 21:52:04]

Description was changed from

==========
Remove HostDesktopType from c/b/ui/views/apps/app_info_dialog

R=sashab@chromium.org
BUG=558054
==========

to

==========
Remove HostDesktopType from c/b/ui/views/apps/app_info_dialog

R=sashab@chromium.org
BUG=558054
==========

[commit-bot: I haz the power, 2016-02-22 21:52:04]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-22 21:53:41]

Description was changed from

==========
Remove HostDesktopType from c/b/ui/views/apps/app_info_dialog

R=sashab@chromium.org
BUG=558054
==========

to

==========
Remove HostDesktopType from c/b/ui/views/apps/app_info_dialog

R=sashab@chromium.org
BUG=558054

Committed: https://crrev.com/2ebd17bbd50e961d2a31c00ae4c209a8e2443dd3
Cr-Commit-Position: refs/heads/master@{#376820}
==========

[commit-bot: I haz the power, 2016-02-22 21:53:41]

Patchset 3 (id:??) landed as
https://crrev.com/2ebd17bbd50e961d2a31c00ae4c209a8e2443dd3
Cr-Commit-Position: refs/heads/master@{#376820}

[scottmg, 2016-02-25 20:01:39]

(Oops, forgot to mail.)

https://codereview.chromium.org/1702523002/diff/20001/chrome/browser/ui/views...
File chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc
(right):

https://codereview.chromium.org/1702523002/diff/20001/chrome/browser/ui/views...
chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc:10:
#include "chrome/browser/ui/host_desktop.h"
On 2016/02/22 03:01:49, tapted wrote:
> nit: can this include go?

Done.