ServiceWorker: Remove unused MessageEvent implementation from ServiceWorkerGlobalScope

ServiceWorker: Remove unused MessageEvent implementation from ServiceWorkerGlobalScope

This is a follow-up patch for issue 598608. After ExtendableMessageEvent is
shipped, MessageEvent on ServiceWorkerGlobalScope is no longer used. This patch
removes implementation and UMA for the event and adds a new UMA for
ExtendableMessageEvent instead.

BUG=570820, 602924
Committed: https://crrev.com/9fc9575ca4928dcca5d6ca5b8c36ffe87466ab46
Cr-Commit-Position: refs/heads/master@{#388136}

[nhiroki, 2016-04-13 06:14:00]

Description was changed from

==========
ServiceWorker: Remove deprecated MessageEvent from ServiceWorkerGlobalScope

BUG=
==========

to

==========
ServiceWorker: Remove unused MessageEvent implementation from
ServiceWorkerGlobalScope

This is a follow-up patch for issue 598608. After ExtendableMessageEvent is
shipped, MessageEvent on ServiceWorkerGlobalScope is no longer used.

BUG=570820, 602924
==========

[nhiroki, 2016-04-13 06:24:50]

Description was changed from

==========
ServiceWorker: Remove unused MessageEvent implementation from
ServiceWorkerGlobalScope

This is a follow-up patch for issue 598608. After ExtendableMessageEvent is
shipped, MessageEvent on ServiceWorkerGlobalScope is no longer used.

BUG=570820, 602924
==========

to

==========
ServiceWorker: Remove unused MessageEvent implementation from
ServiceWorkerGlobalScope

This is a follow-up patch for issue 598608. After ExtendableMessageEvent is
shipped, MessageEvent on ServiceWorkerGlobalScope is no longer used. This patch
removes implementation and UMA for the event and adds a new UMA for
ExtendableMessageEvent.

BUG=570820, 602924
==========

[nhiroki, 2016-04-13 06:26:24]

Description was changed from

==========
ServiceWorker: Remove unused MessageEvent implementation from
ServiceWorkerGlobalScope

This is a follow-up patch for issue 598608. After ExtendableMessageEvent is
shipped, MessageEvent on ServiceWorkerGlobalScope is no longer used. This patch
removes implementation and UMA for the event and adds a new UMA for
ExtendableMessageEvent.

BUG=570820, 602924
==========

to

==========
ServiceWorker: Remove unused MessageEvent implementation from
ServiceWorkerGlobalScope

This is a follow-up patch for issue 598608. After ExtendableMessageEvent is
shipped, MessageEvent on ServiceWorkerGlobalScope is no longer used. This patch
removes implementation and UMA for the event and adds a new UMA for
ExtendableMessageEvent instead.

BUG=570820, 602924
==========

[nhiroki, 2016-04-13 06:27:08]

Patchset #1 (id:1) has been deleted

[nhiroki, 2016-04-14 04:35:58]

Patchset #1 (id:20001) has been deleted

[nhiroki, 2016-04-14 04:36:08]

nhiroki@chromium.org changed reviewers:
+ falken@chromium.org

[nhiroki, 2016-04-14 04:36:09]

PTAL, thanks!

[falken, 2016-04-14 17:32:27]

lgtm, great

[nhiroki, 2016-04-15 00:36:28]

nhiroki@chromium.org changed reviewers:
+ isherman@chromium.org, mkwst@chromium.org

[nhiroki, 2016-04-15 00:37:12]

+mkwst@, can you review following files?

  * content/common/service_worker/service_worker_messages.h
  * third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in
  * third_party/WebKit/Source/web/*
  * third_party/WebKit/public/web/WebRuntimeFeatures.h

+isherman@, can you review histograms.xml?

Thanks!

[Ilya Sherman, 2016-04-15 00:57:55]

histograms lgtm

[Mike West, 2016-04-18 20:36:23]

LGTM

[nhiroki, 2016-04-19 00:52:26]

The CQ bit was checked by nhiroki@chromium.org

[nhiroki, 2016-04-19 00:52:26]

The patchset sent to the CQ was uploaded after l-g-t-m from falken@chromium.org,
mkwst@chromium.org, isherman@chromium.org
Link to the patchset: https://codereview.chromium.org/1877283002/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2016-04-19 00:52:39]

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

[commit-bot: I haz the power, 2016-04-19 04:43:13]

Description was changed from

==========
ServiceWorker: Remove unused MessageEvent implementation from
ServiceWorkerGlobalScope

This is a follow-up patch for issue 598608. After ExtendableMessageEvent is
shipped, MessageEvent on ServiceWorkerGlobalScope is no longer used. This patch
removes implementation and UMA for the event and adds a new UMA for
ExtendableMessageEvent instead.

BUG=570820, 602924
==========

to

==========
ServiceWorker: Remove unused MessageEvent implementation from
ServiceWorkerGlobalScope

This is a follow-up patch for issue 598608. After ExtendableMessageEvent is
shipped, MessageEvent on ServiceWorkerGlobalScope is no longer used. This patch
removes implementation and UMA for the event and adds a new UMA for
ExtendableMessageEvent instead.

BUG=570820, 602924
==========

[commit-bot: I haz the power, 2016-04-19 04:43:14]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-04-19 04:44:57]

Description was changed from

==========
ServiceWorker: Remove unused MessageEvent implementation from
ServiceWorkerGlobalScope

This is a follow-up patch for issue 598608. After ExtendableMessageEvent is
shipped, MessageEvent on ServiceWorkerGlobalScope is no longer used. This patch
removes implementation and UMA for the event and adds a new UMA for
ExtendableMessageEvent instead.

BUG=570820, 602924
==========

to

==========
ServiceWorker: Remove unused MessageEvent implementation from
ServiceWorkerGlobalScope

This is a follow-up patch for issue 598608. After ExtendableMessageEvent is
shipped, MessageEvent on ServiceWorkerGlobalScope is no longer used. This patch
removes implementation and UMA for the event and adds a new UMA for
ExtendableMessageEvent instead.

BUG=570820, 602924

Committed: https://crrev.com/9fc9575ca4928dcca5d6ca5b8c36ffe87466ab46
Cr-Commit-Position: refs/heads/master@{#388136}
==========

[commit-bot: I haz the power, 2016-04-19 04:44:59]

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