Remove Metro pinning, and relaunch menu items and strings

Remove Metro pinning, and relaunch menu items and strings

Also removes the Metro infobar for blocked plugins. NPAPI plugins.
Which I don't know why we had at all any more anyway.

R=grt, lfg, bauerb, sky
TBR=jschuh
BUG=558054
Committed: https://crrev.com/363b5904e283b09391552fd398ce73f82387f78e
Cr-Commit-Position: refs/heads/master@{#369456}

[scottmg, 2016-01-13 20:23:55]

Description was changed from

==========
Remove Metro pinning, and relaunch menu items

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

to

==========
Remove Metro pinning, and relaunch menu items and strings

Also removes the Metro infobar for blocked plugins. NPAPI plugins.
Which I don't know why we had at all any more anyway. 

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

[scottmg, 2016-01-13 20:28:15]

scottmg@chromium.org changed reviewers:
+ bauerb@chromium.org, grt@chromium.org, lfg@chromium.org, sky@chromium.org

[scottmg, 2016-01-13 20:28:15]

chrome/app: grt@
chrome/browser/guest_view: lfg@
chrome/browser/plugins: bauerb@
chrome/browser/ui, chrome/renderer: sky@

[lfg, 2016-01-13 20:40:12]

chrome/browser/guest_view lgtm

[sky, 2016-01-13 21:19:48]

LGTM

[grt (UTC plus 2), 2016-01-14 02:21:18]

chrome/app lgtm, but i wonder why remove snap handling from Chrome without also
removing it from chrome_app_view.cc? is it already no longer possible to run in
metro mode?

[Bernhard Bauer, 2016-01-14 10:10:43]

LGTM!

[scottmg, 2016-01-14 17:26:04]

On 2016/01/14 02:21:18, grt wrote:
> chrome/app lgtm, but i wonder why remove snap handling from Chrome without
also
> removing it from chrome_app_view.cc? is it already no longer possible to run
in
> metro mode?

Yes, it's already not possible to run in Metro. All of win8/metro_driver
(including chrome_app_view.cc) will be nuked shortly, but the CLs were just
getting unwieldy so this is sort of an arbitrary sharding.

[scottmg, 2016-01-14 17:26:08]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2016-01-14 17:26:50]

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

[commit-bot: I haz the power, 2016-01-14 17:37:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-14 17:37:37]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[scottmg, 2016-01-14 17:48:24]

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

[scottmg, 2016-01-14 17:48:49]

tbr jschuh for removing a message in render_messages.h

[scottmg, 2016-01-14 17:49:24]

Description was changed from

==========
Remove Metro pinning, and relaunch menu items and strings

Also removes the Metro infobar for blocked plugins. NPAPI plugins.
Which I don't know why we had at all any more anyway. 

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

to

==========
Remove Metro pinning, and relaunch menu items and strings

Also removes the Metro infobar for blocked plugins. NPAPI plugins.
Which I don't know why we had at all any more anyway. 

R=grt, lfg, bauerb, sky
TBR=jschuh
BUG=558054
==========

[scottmg, 2016-01-14 17:49:34]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2016-01-14 17:50:21]

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

[commit-bot: I haz the power, 2016-01-14 17:56:03]

Description was changed from

==========
Remove Metro pinning, and relaunch menu items and strings

Also removes the Metro infobar for blocked plugins. NPAPI plugins.
Which I don't know why we had at all any more anyway. 

R=grt, lfg, bauerb, sky
TBR=jschuh
BUG=558054
==========

to

==========
Remove Metro pinning, and relaunch menu items and strings

Also removes the Metro infobar for blocked plugins. NPAPI plugins.
Which I don't know why we had at all any more anyway. 

R=grt, lfg, bauerb, sky
TBR=jschuh
BUG=558054
==========

[commit-bot: I haz the power, 2016-01-14 17:56:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-14 17:58:02]

Description was changed from

==========
Remove Metro pinning, and relaunch menu items and strings

Also removes the Metro infobar for blocked plugins. NPAPI plugins.
Which I don't know why we had at all any more anyway. 

R=grt, lfg, bauerb, sky
TBR=jschuh
BUG=558054
==========

to

==========
Remove Metro pinning, and relaunch menu items and strings

Also removes the Metro infobar for blocked plugins. NPAPI plugins.
Which I don't know why we had at all any more anyway.

R=grt, lfg, bauerb, sky
TBR=jschuh
BUG=558054

Committed: https://crrev.com/363b5904e283b09391552fd398ce73f82387f78e
Cr-Commit-Position: refs/heads/master@{#369456}
==========

[commit-bot: I haz the power, 2016-01-14 17:58:03]

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