Remove Mac NPAPI things

Remove Mac NPAPI things

A few Mac-specific messages were only used for NPAPI on Mac. We removed the
renderer side, so these messages do nothing now and/or are never sent. Pull the
thread on the browser side and remove corresponding things.

BUG=493212
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/8a60f62c10333ec842dab85e2bc833a21996e0bc
Cr-Commit-Position: refs/heads/master@{#385098}

[piman, 2016-04-04 22:57:44]

Description was changed from

==========
Remove Mac NPAPI things

A few Mac-specific messages were only used for NPAPI on Mac. We removed the
renderer side, so these messages do nothing now and/or are never sent. Pull the
thread on the browser side and remove corresponding things.

BUG=493212
==========

to

==========
Remove Mac NPAPI things

A few Mac-specific messages were only used for NPAPI on Mac. We removed the
renderer side, so these messages do nothing now and/or are never sent. Pull the
thread on the browser side and remove corresponding things.

BUG=493212
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[piman, 2016-04-04 23:04:49]

piman@chromium.org changed reviewers:
+ dcheng@chromium.org, mark@chromium.org

[piman, 2016-04-04 23:04:50]

mark: please review
dcheng: view_messages.h

Please review carefully with extra suspicion, I have no idea what I'm doing.

https://codereview.chromium.org/1858063002/diff/1/chrome/browser/ui/cocoa/ext...
File chrome/browser/ui/cocoa/extensions/extension_popup_controller.mm (left):

https://codereview.chromium.org/1858063002/diff/1/chrome/browser/ui/cocoa/ext...
chrome/browser/ui/cocoa/extensions/extension_popup_controller.mm:422: -
(void)windowDidResize:(NSNotification*)notification {
\noideadog, since those handlers don't do anything any more, I assume I can just
remove them?

https://codereview.chromium.org/1858063002/diff/1/third_party/mozilla/BUILD.gn
File third_party/mozilla/BUILD.gn (left):

https://codereview.chromium.org/1858063002/diff/1/third_party/mozilla/BUILD.g...
third_party/mozilla/BUILD.gn:8: "ComplexTextInputPanel.mm",
The NPAPI stuff was the only thing using those, so I removed them.

[Mark Mentovai, 2016-04-04 23:10:50]

LGTM. Great!

https://codereview.chromium.org/1858063002/diff/1/chrome/browser/ui/cocoa/ext...
File chrome/browser/ui/cocoa/extensions/extension_popup_controller.mm (left):

https://codereview.chromium.org/1858063002/diff/1/chrome/browser/ui/cocoa/ext...
chrome/browser/ui/cocoa/extensions/extension_popup_controller.mm:422: -
(void)windowDidResize:(NSNotification*)notification {
piman wrote:
> \noideadog, since those handlers don't do anything any more, I assume I can
just
> remove them?

Ought to be OK, everything in the NSWindowDelegate protocol is optional.

[dcheng, 2016-04-04 23:13:10]

rslgtm for ipc deletions

[dcheng, 2016-04-04 23:13:14]

rs lgtm even

[piman, 2016-04-04 23:14:49]

The CQ bit was checked by piman@chromium.org

[commit-bot: I haz the power, 2016-04-04 23:15:20]

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

[commit-bot: I haz the power, 2016-04-04 23:58:04]

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

[commit-bot: I haz the power, 2016-04-04 23:58:05]

Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[piman, 2016-04-05 00:42:24]

The CQ bit was checked by piman@chromium.org

[piman, 2016-04-05 00:42:25]

The patchset sent to the CQ was uploaded after l-g-t-m from mark@chromium.org,
dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/1858063002/#ps20001
(title: "build fix")

[commit-bot: I haz the power, 2016-04-05 00:43:06]

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

[commit-bot: I haz the power, 2016-04-05 01:30:04]

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

[commit-bot: I haz the power, 2016-04-05 01:30:04]

Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[piman, 2016-04-05 02:30:12]

The CQ bit was checked by piman@chromium.org

[piman, 2016-04-05 02:30:12]

The patchset sent to the CQ was uploaded after l-g-t-m from mark@chromium.org,
dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/1858063002/#ps40001
(title: "more build fixes")

[commit-bot: I haz the power, 2016-04-05 02:30:45]

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

[commit-bot: I haz the power, 2016-04-05 03:39:07]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-05 03:40:59]

Description was changed from

==========
Remove Mac NPAPI things

A few Mac-specific messages were only used for NPAPI on Mac. We removed the
renderer side, so these messages do nothing now and/or are never sent. Pull the
thread on the browser side and remove corresponding things.

BUG=493212
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Remove Mac NPAPI things

A few Mac-specific messages were only used for NPAPI on Mac. We removed the
renderer side, so these messages do nothing now and/or are never sent. Pull the
thread on the browser side and remove corresponding things.

BUG=493212
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/8a60f62c10333ec842dab85e2bc833a21996e0bc
Cr-Commit-Position: refs/heads/master@{#385098}
==========

[commit-bot: I haz the power, 2016-04-05 03:41:01]

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