Reland of [ServiceWorker] Mojofy PushEvent of Service Worker.

Reland of [ServiceWorker] Mojofy PushEvent of Service Worker. (patchset #1 id:1 of https://codereview.chromium.org/2600863002/ )

Reason for revert:
Fixed the root cause of the crash.
https://crrev.com/64efa56e5bb67106cea0eb4033b866d9217d8324

Original issue's description:
> 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}

TBR=leon.han@intel.com,peter@chromium.org,tsepez@chromium.org,xiaofeng.zhang@intel.com,falken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=629701
Review-Url: https://codereview.chromium.org/2620333003
Cr-Commit-Position: refs/heads/master@{#443462}
Committed: https://chromium.googlesource.com/chromium/src/+/bf99a72b0c15458d6a852d70d6cfc52abf54c0a3

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

The CQ bit was checked by shimazu@chromium.org

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

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

[commit-bot: I haz the power, 2017-01-12 07:29:06]

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

[xiaofengzhang, 2017-01-12 07:31:10]

On 2017/01/12 07:28:51, shimazu wrote:
> Created Reland of [ServiceWorker] Mojofy PushEvent of Service Worker.

Hi shimazu,
I think this patch need to be rebased because some changes happens in some of
those files.

[commit-bot: I haz the power, 2017-01-12 09:12:30]

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

[commit-bot: I haz the power, 2017-01-12 09:12:31]

Failed to apply patch for AUTHORS:
While running git apply --index -p1;
  error: patch failed: AUTHORS:726
  error: AUTHORS: patch does not apply

Patch:       AUTHORS
Index: AUTHORS
diff --git a/AUTHORS b/AUTHORS
index
75487640d50e98b932e39c5ed9ecdade16acb903..e8201951cd6f49d55375cdfe36f556d8ad301758
100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -726,6 +726,7 @@
 William Xie <william.xie@intel.com>
 Xiang Long <xiang.long@intel.com>
 Xiangze Zhang <xiangze.zhang@intel.com>
+Xiaofeng Zhang <xiaofeng.zhang@intel.com>
 Xiaolei Yu <dreifachstein@gmail.com>
 Xiaoyin Liu <xiaoyin.l@outlook.com>
 Xinchao He <hexinchao@gmail.com>

[Tom Sepez, 2017-01-12 17:39:37]

RS LGTM

[Peter Beverloo, 2017-01-12 17:50:19]

Also lgtm, thanks for relanding this.

[shimazu, 2017-01-13 00:07:12]

The CQ bit was checked by shimazu@chromium.org

[shimazu, 2017-01-13 00:07:13]

The patchset sent to the CQ was uploaded after l-g-t-m from peter@chromium.org,
tsepez@chromium.org
Link to the patchset: https://codereview.chromium.org/2620333003/#ps220001
(title: "Rebase")

[commit-bot: I haz the power, 2017-01-13 00:08:30]

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, 2017-01-13 02:11:40]

CQ is committing da patch.

Bot data: {"patchset_id": 220001, "attempt_start_ts": 1484266032506970,
"parent_rev": "67926a91215c713e851b4fa83bf64e50907ef53e", "commit_rev":
"bf99a72b0c15458d6a852d70d6cfc52abf54c0a3"}

[commit-bot: I haz the power, 2017-01-13 02:12:15]

Description was changed from

==========
Reland of [ServiceWorker] Mojofy PushEvent of Service Worker. (patchset #1 id:1
of https://codereview.chromium.org/2600863002/ )

Reason for revert:
Fixed the root cause of the crash.
https://crrev.com/64efa56e5bb67106cea0eb4033b866d9217d8324

Original issue's description:
> 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}

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

to

==========
Reland of [ServiceWorker] Mojofy PushEvent of Service Worker. (patchset #1 id:1
of https://codereview.chromium.org/2600863002/ )

Reason for revert:
Fixed the root cause of the crash.
https://crrev.com/64efa56e5bb67106cea0eb4033b866d9217d8324

Original issue's description:
> 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}

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

Review-Url: https://codereview.chromium.org/2620333003
Cr-Commit-Position: refs/heads/master@{#443462}
Committed:
https://chromium.googlesource.com/chromium/src/+/bf99a72b0c15458d6a852d70d6cf...
==========

[commit-bot: I haz the power, 2017-01-13 02:12:20]

Committed patchset #2 (id:220001) as
https://chromium.googlesource.com/chromium/src/+/bf99a72b0c15458d6a852d70d6cf...