Revert of [ServiceWorker] Mojofy PushEvent of Service Worker.

Revert of [ServiceWorker] Mojofy PushEvent of Service Worker. (patchset #3 id:40001 of https://codereview.chromium.org/2569993002/ )

Reason for revert:
Causes crashes on canary. See https://crbug.com/676984

0x5eb4a4fa	(chrome.dll + 0x000fa4fa )	base::Callback<void ,1,1>::Callback<void ,1,1>(base::Callback<void ,1,1> const &)
0x6003ec7d	(chrome.dll -service_worker_version.cc:1095 )	content::ServiceWorkerVersion::OnSimpleEventFinished(int,content::ServiceWorkerStatusCode,base::Time)
0x5ff64f33	(chrome.dll -bind_internal.h:214 )	base::internal::FunctorTraits<void ( content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),void>::Invoke<scoped_refptr<content::ServiceWorkerVersion> const &,int const &,content::ServiceWorkerStatusCode,base::Time>(void ( content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),scoped_refptr<content::ServiceWorkerVersion> const &,int const &,content::ServiceWorkerStatusCode &&,base::Time &&)
0x5ff50241	(chrome.dll -bind_internal.h:285 )	base::internal::InvokeHelper<0,void>::MakeItSo<void ( content::ServiceWorkerVersion::*const &)(int,blink::WebServiceWorkerEventResult,base::Time),scoped_refptr<content::ServiceWorkerVersion> const &,int,blink::WebServiceWorkerEventResult,base::Time>(void ( content::ServiceWorkerVersion::*const &)(int,blink::WebServiceWorkerEventResult,base::Time),scoped_refptr<content::ServiceWorkerVersion> const &,int &&,blink::WebServiceWorkerEventResult &&,base::Time &&)
0x5ff655a6	(chrome.dll -bind_internal.h:339 )	base::internal::Invoker<base::internal::BindState<void ( content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),scoped_refptr<content::ServiceWorkerVersion>,int>,void >::Run(base::internal::BindStateBase *,content::ServiceWorkerStatusCode &&,base::Time &&)
0x5fbe1b09	(chrome.dll -callback.h:85 )	base::internal::RunMixin<base::Callback<void ,1,1> >::Run(filesystem::mojom::FileError,__int64)
0x5fba0984	(chrome.dll -service_worker_event_dispatcher.mojom.cc:225 )	content::mojom::ServiceWorkerEventDispatcher_DispatchPushEvent_ForwardToCallback::Accept(mojo::Message *)
0x6037605b	(chrome.dll -interface_endpoint_client.cc:336 )	mojo::InterfaceEndpointClient::HandleValidatedMessage(mojo::Message *)

Original issue's description:
> [ServiceWorker] Mojofy PushEvent of Service Worker.
>
> This CL converts push event IPCs into mojo interface:
>   ServiceWorkerMsg_PushEvent
>   ServiceWorkerHostMsg_PushEventFinished
>
> BUG=629701
> TEST=content_unittests
>
> Committed: https://crrev.com/35842853527f50a04662a6dfe2c3cd1b10cf5915
> Cr-Commit-Position: refs/heads/master@{#439739}

TBR=leon.han@intel.com,peter@chromium.org,shimazu@chromium.org,tsepez@chromium.org,xiaofeng.zhang@intel.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=629701
Committed: https://crrev.com/024112aedb31806f7c825e3591f6be1ec7474aa9
Cr-Commit-Position: refs/heads/master@{#440693}

[falken, 2016-12-26 03:55:59]

The CQ bit was checked by falken@chromium.org

[falken, 2016-12-26 03:55:59]

Created Revert of [ServiceWorker] Mojofy PushEvent of Service Worker.

[commit-bot: I haz the power, 2016-12-26 03:56:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-26 05:49:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482724559307280, "parent_rev":
"af7c9bce1cee0958c0f7490c834dd8454bbd23d2", "commit_rev":
"dbba44d015a83d785865c4403f898d08e7cea8ff"}

[commit-bot: I haz the power, 2016-12-26 05:50:42]

Description was changed from

==========
Revert of [ServiceWorker] Mojofy PushEvent of Service Worker. (patchset #3
id:40001 of https://codereview.chromium.org/2569993002/ )

Reason for revert:
Causes crashes on canary. See https://crbug.com/676984

0x5eb4a4fa	(chrome.dll + 0x000fa4fa )	base::Callback<void ,1,1>::Callback<void
,1,1>(base::Callback<void ,1,1> const &)
0x6003ec7d	(chrome.dll -service_worker_version.cc:1095
)	content::ServiceWorkerVersion::OnSimpleEventFinished(int,content::ServiceWorkerStatusCode,base::Time)
0x5ff64f33	(chrome.dll -bind_internal.h:214 )	base::internal::FunctorTraits<void
(
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),void>::Invoke<scoped_refptr<content::ServiceWorkerVersion>
const &,int const &,content::ServiceWorkerStatusCode,base::Time>(void (
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int const &,content::ServiceWorkerStatusCode &&,base::Time &&)
0x5ff50241	(chrome.dll -bind_internal.h:285
)	base::internal::InvokeHelper<0,void>::MakeItSo<void (
content::ServiceWorkerVersion::*const
&)(int,blink::WebServiceWorkerEventResult,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int,blink::WebServiceWorkerEventResult,base::Time>(void (
content::ServiceWorkerVersion::*const
&)(int,blink::WebServiceWorkerEventResult,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int &&,blink::WebServiceWorkerEventResult &&,base::Time &&)
0x5ff655a6	(chrome.dll -bind_internal.h:339
)	base::internal::Invoker<base::internal::BindState<void (
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),scoped_refptr<content::ServiceWorkerVersion>,int>,void
>::Run(base::internal::BindStateBase *,content::ServiceWorkerStatusCode
&&,base::Time &&)
0x5fbe1b09	(chrome.dll -callback.h:85
)	base::internal::RunMixin<base::Callback<void ,1,1>
>::Run(filesystem::mojom::FileError,__int64)
0x5fba0984	(chrome.dll -service_worker_event_dispatcher.mojom.cc:225
)	content::mojom::ServiceWorkerEventDispatcher_DispatchPushEvent_ForwardToCallback::Accept(mojo::Message
*)
0x6037605b	(chrome.dll -interface_endpoint_client.cc:336
)	mojo::InterfaceEndpointClient::HandleValidatedMessage(mojo::Message *)





Original issue's description:
> [ServiceWorker] Mojofy PushEvent of Service Worker.
> 
> This CL converts push event IPCs into mojo interface:
>   ServiceWorkerMsg_PushEvent
>   ServiceWorkerHostMsg_PushEventFinished
> 
> BUG=629701
> TEST=content_unittests
> 
> Committed: https://crrev.com/35842853527f50a04662a6dfe2c3cd1b10cf5915
> Cr-Commit-Position: refs/heads/master@{#439739}

TBR=leon.han@intel.com,peter@chromium.org,shimazu@chromium.org,tsepez@chromiu...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=629701
==========

to

==========
Revert of [ServiceWorker] Mojofy PushEvent of Service Worker. (patchset #3
id:40001 of https://codereview.chromium.org/2569993002/ )

Reason for revert:
Causes crashes on canary. See https://crbug.com/676984

0x5eb4a4fa	(chrome.dll + 0x000fa4fa )	base::Callback<void ,1,1>::Callback<void
,1,1>(base::Callback<void ,1,1> const &)
0x6003ec7d	(chrome.dll -service_worker_version.cc:1095
)	content::ServiceWorkerVersion::OnSimpleEventFinished(int,content::ServiceWorkerStatusCode,base::Time)
0x5ff64f33	(chrome.dll -bind_internal.h:214 )	base::internal::FunctorTraits<void
(
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),void>::Invoke<scoped_refptr<content::ServiceWorkerVersion>
const &,int const &,content::ServiceWorkerStatusCode,base::Time>(void (
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int const &,content::ServiceWorkerStatusCode &&,base::Time &&)
0x5ff50241	(chrome.dll -bind_internal.h:285
)	base::internal::InvokeHelper<0,void>::MakeItSo<void (
content::ServiceWorkerVersion::*const
&)(int,blink::WebServiceWorkerEventResult,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int,blink::WebServiceWorkerEventResult,base::Time>(void (
content::ServiceWorkerVersion::*const
&)(int,blink::WebServiceWorkerEventResult,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int &&,blink::WebServiceWorkerEventResult &&,base::Time &&)
0x5ff655a6	(chrome.dll -bind_internal.h:339
)	base::internal::Invoker<base::internal::BindState<void (
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),scoped_refptr<content::ServiceWorkerVersion>,int>,void
>::Run(base::internal::BindStateBase *,content::ServiceWorkerStatusCode
&&,base::Time &&)
0x5fbe1b09	(chrome.dll -callback.h:85
)	base::internal::RunMixin<base::Callback<void ,1,1>
>::Run(filesystem::mojom::FileError,__int64)
0x5fba0984	(chrome.dll -service_worker_event_dispatcher.mojom.cc:225
)	content::mojom::ServiceWorkerEventDispatcher_DispatchPushEvent_ForwardToCallback::Accept(mojo::Message
*)
0x6037605b	(chrome.dll -interface_endpoint_client.cc:336
)	mojo::InterfaceEndpointClient::HandleValidatedMessage(mojo::Message *)





Original issue's description:
> [ServiceWorker] Mojofy PushEvent of Service Worker.
> 
> This CL converts push event IPCs into mojo interface:
>   ServiceWorkerMsg_PushEvent
>   ServiceWorkerHostMsg_PushEventFinished
> 
> BUG=629701
> TEST=content_unittests
> 
> Committed: https://crrev.com/35842853527f50a04662a6dfe2c3cd1b10cf5915
> Cr-Commit-Position: refs/heads/master@{#439739}

TBR=leon.han@intel.com,peter@chromium.org,shimazu@chromium.org,tsepez@chromiu...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=629701

Review-Url: https://codereview.chromium.org/2600863002
==========

[commit-bot: I haz the power, 2016-12-26 05:50:43]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-26 05:52:46]

Description was changed from

==========
Revert of [ServiceWorker] Mojofy PushEvent of Service Worker. (patchset #3
id:40001 of https://codereview.chromium.org/2569993002/ )

Reason for revert:
Causes crashes on canary. See https://crbug.com/676984

0x5eb4a4fa	(chrome.dll + 0x000fa4fa )	base::Callback<void ,1,1>::Callback<void
,1,1>(base::Callback<void ,1,1> const &)
0x6003ec7d	(chrome.dll -service_worker_version.cc:1095
)	content::ServiceWorkerVersion::OnSimpleEventFinished(int,content::ServiceWorkerStatusCode,base::Time)
0x5ff64f33	(chrome.dll -bind_internal.h:214 )	base::internal::FunctorTraits<void
(
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),void>::Invoke<scoped_refptr<content::ServiceWorkerVersion>
const &,int const &,content::ServiceWorkerStatusCode,base::Time>(void (
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int const &,content::ServiceWorkerStatusCode &&,base::Time &&)
0x5ff50241	(chrome.dll -bind_internal.h:285
)	base::internal::InvokeHelper<0,void>::MakeItSo<void (
content::ServiceWorkerVersion::*const
&)(int,blink::WebServiceWorkerEventResult,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int,blink::WebServiceWorkerEventResult,base::Time>(void (
content::ServiceWorkerVersion::*const
&)(int,blink::WebServiceWorkerEventResult,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int &&,blink::WebServiceWorkerEventResult &&,base::Time &&)
0x5ff655a6	(chrome.dll -bind_internal.h:339
)	base::internal::Invoker<base::internal::BindState<void (
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),scoped_refptr<content::ServiceWorkerVersion>,int>,void
>::Run(base::internal::BindStateBase *,content::ServiceWorkerStatusCode
&&,base::Time &&)
0x5fbe1b09	(chrome.dll -callback.h:85
)	base::internal::RunMixin<base::Callback<void ,1,1>
>::Run(filesystem::mojom::FileError,__int64)
0x5fba0984	(chrome.dll -service_worker_event_dispatcher.mojom.cc:225
)	content::mojom::ServiceWorkerEventDispatcher_DispatchPushEvent_ForwardToCallback::Accept(mojo::Message
*)
0x6037605b	(chrome.dll -interface_endpoint_client.cc:336
)	mojo::InterfaceEndpointClient::HandleValidatedMessage(mojo::Message *)





Original issue's description:
> [ServiceWorker] Mojofy PushEvent of Service Worker.
> 
> This CL converts push event IPCs into mojo interface:
>   ServiceWorkerMsg_PushEvent
>   ServiceWorkerHostMsg_PushEventFinished
> 
> BUG=629701
> TEST=content_unittests
> 
> Committed: https://crrev.com/35842853527f50a04662a6dfe2c3cd1b10cf5915
> Cr-Commit-Position: refs/heads/master@{#439739}

TBR=leon.han@intel.com,peter@chromium.org,shimazu@chromium.org,tsepez@chromiu...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=629701

Review-Url: https://codereview.chromium.org/2600863002
==========

to

==========
Revert of [ServiceWorker] Mojofy PushEvent of Service Worker. (patchset #3
id:40001 of https://codereview.chromium.org/2569993002/ )

Reason for revert:
Causes crashes on canary. See https://crbug.com/676984

0x5eb4a4fa	(chrome.dll + 0x000fa4fa )	base::Callback<void ,1,1>::Callback<void
,1,1>(base::Callback<void ,1,1> const &)
0x6003ec7d	(chrome.dll -service_worker_version.cc:1095
)	content::ServiceWorkerVersion::OnSimpleEventFinished(int,content::ServiceWorkerStatusCode,base::Time)
0x5ff64f33	(chrome.dll -bind_internal.h:214 )	base::internal::FunctorTraits<void
(
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),void>::Invoke<scoped_refptr<content::ServiceWorkerVersion>
const &,int const &,content::ServiceWorkerStatusCode,base::Time>(void (
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int const &,content::ServiceWorkerStatusCode &&,base::Time &&)
0x5ff50241	(chrome.dll -bind_internal.h:285
)	base::internal::InvokeHelper<0,void>::MakeItSo<void (
content::ServiceWorkerVersion::*const
&)(int,blink::WebServiceWorkerEventResult,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int,blink::WebServiceWorkerEventResult,base::Time>(void (
content::ServiceWorkerVersion::*const
&)(int,blink::WebServiceWorkerEventResult,base::Time),scoped_refptr<content::ServiceWorkerVersion>
const &,int &&,blink::WebServiceWorkerEventResult &&,base::Time &&)
0x5ff655a6	(chrome.dll -bind_internal.h:339
)	base::internal::Invoker<base::internal::BindState<void (
content::ServiceWorkerVersion::*)(int,content::ServiceWorkerStatusCode,base::Time),scoped_refptr<content::ServiceWorkerVersion>,int>,void
>::Run(base::internal::BindStateBase *,content::ServiceWorkerStatusCode
&&,base::Time &&)
0x5fbe1b09	(chrome.dll -callback.h:85
)	base::internal::RunMixin<base::Callback<void ,1,1>
>::Run(filesystem::mojom::FileError,__int64)
0x5fba0984	(chrome.dll -service_worker_event_dispatcher.mojom.cc:225
)	content::mojom::ServiceWorkerEventDispatcher_DispatchPushEvent_ForwardToCallback::Accept(mojo::Message
*)
0x6037605b	(chrome.dll -interface_endpoint_client.cc:336
)	mojo::InterfaceEndpointClient::HandleValidatedMessage(mojo::Message *)

Original issue's description:
> [ServiceWorker] Mojofy PushEvent of Service Worker.
>
> This CL converts push event IPCs into mojo interface:
>   ServiceWorkerMsg_PushEvent
>   ServiceWorkerHostMsg_PushEventFinished
>
> BUG=629701
> TEST=content_unittests
>
> Committed: https://crrev.com/35842853527f50a04662a6dfe2c3cd1b10cf5915
> Cr-Commit-Position: refs/heads/master@{#439739}

TBR=leon.han@intel.com,peter@chromium.org,shimazu@chromium.org,tsepez@chromiu...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=629701

Committed: https://crrev.com/024112aedb31806f7c825e3591f6be1ec7474aa9
Cr-Commit-Position: refs/heads/master@{#440693}
==========

[commit-bot: I haz the power, 2016-12-26 05:52:47]

Patchset 1 (id:??) landed as
https://crrev.com/024112aedb31806f7c825e3591f6be1ec7474aa9
Cr-Commit-Position: refs/heads/master@{#440693}

[shimazu, 2017-01-12 07:28:50]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2620333003/ by shimazu@chromium.org.

The reason for reverting is: Fixed the root cause of the crash.
https://crrev.com/64efa56e5bb67106cea0eb4033b866d9217d8324.