Use fetch_event_id (fetchEventID in blink) for FetchEvent

Use fetch_event_id (fetchEventID in blink) for FetchEvent

Currently we have |event_finish_id| in both browser and renderer code, but they
aren't the same thing. To avoid this confusion, this CL will use |fetch_event_id|
(fetchEventID in blink) for FetchEvent handling. And will stop using
|event_finish_id| in renderer code by using AddWithID() in
ServiceWorkerContextClient::DispatchFetchEvent().

BUG=649558
Committed: https://crrev.com/160d7098f1c9f293990f0a1ee6644442ad945fc9
Cr-Commit-Position: refs/heads/master@{#425255}

[horo, 2016-10-13 10:32:26]

The CQ bit was checked by horo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-13 10:32:33]

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

[horo, 2016-10-13 10:40:21]

Description was changed from

==========
Use fetch_event_id/fetchEventID for FetchEvent

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

to

==========
Use fetch_event_id (fetchEventID in blink) for FetchEvent

Currently we have |event_finish_id| in both browser and renderer code, but they
aren't the same thing. To avoid this confusion, this CL will use
|fetch_event_id|
(fetchEventID in blink) for FetchEvent handling. And will stop using
|event_finish_id| in renderer code by using AddWithID() in
ServiceWorkerContextClient::DispatchFetchEvent().

BUG=649558
==========

[horo, 2016-10-13 10:50:46]

horo@chromium.org changed reviewers:
+ shimazu@chromium.org

[horo, 2016-10-13 10:50:47]

shimazu@
Could you please review this?

[commit-bot: I haz the power, 2016-10-13 11:45:32]

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

[commit-bot: I haz the power, 2016-10-13 11:45:33]

Dry run: This issue passed the CQ dry run.

[shimazu, 2016-10-14 01:59:24]

Looks nice refactoring, thanks!
lgtm

[horo, 2016-10-14 02:01:39]

horo@chromium.org changed reviewers:
+ keishi@chromium.org

[horo, 2016-10-14 02:01:40]

keishi@
Could you please review third_party/WebKit/Source/web/*?

[keishi, 2016-10-14 02:08:54]

LGTM

[horo, 2016-10-14 03:41:38]

horo@chromium.org changed reviewers:
+ dcheng@chromium.org

[horo, 2016-10-14 03:41:39]

dcheng@
Could you please review 
content/common/service_worker/fetch_event_dispatcher.mojom and
content/common/service_worker/service_worker_messages.h?

[dcheng, 2016-10-14 04:16:19]

ipc+mojom lgtm

[horo, 2016-10-14 04:51:57]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2016-10-14 04:52:06]

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-10-14 04:57:45]

Description was changed from

==========
Use fetch_event_id (fetchEventID in blink) for FetchEvent

Currently we have |event_finish_id| in both browser and renderer code, but they
aren't the same thing. To avoid this confusion, this CL will use
|fetch_event_id|
(fetchEventID in blink) for FetchEvent handling. And will stop using
|event_finish_id| in renderer code by using AddWithID() in
ServiceWorkerContextClient::DispatchFetchEvent().

BUG=649558
==========

to

==========
Use fetch_event_id (fetchEventID in blink) for FetchEvent

Currently we have |event_finish_id| in both browser and renderer code, but they
aren't the same thing. To avoid this confusion, this CL will use
|fetch_event_id|
(fetchEventID in blink) for FetchEvent handling. And will stop using
|event_finish_id| in renderer code by using AddWithID() in
ServiceWorkerContextClient::DispatchFetchEvent().

BUG=649558
==========

[commit-bot: I haz the power, 2016-10-14 04:57:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-14 05:00:05]

Description was changed from

==========
Use fetch_event_id (fetchEventID in blink) for FetchEvent

Currently we have |event_finish_id| in both browser and renderer code, but they
aren't the same thing. To avoid this confusion, this CL will use
|fetch_event_id|
(fetchEventID in blink) for FetchEvent handling. And will stop using
|event_finish_id| in renderer code by using AddWithID() in
ServiceWorkerContextClient::DispatchFetchEvent().

BUG=649558
==========

to

==========
Use fetch_event_id (fetchEventID in blink) for FetchEvent

Currently we have |event_finish_id| in both browser and renderer code, but they
aren't the same thing. To avoid this confusion, this CL will use
|fetch_event_id|
(fetchEventID in blink) for FetchEvent handling. And will stop using
|event_finish_id| in renderer code by using AddWithID() in
ServiceWorkerContextClient::DispatchFetchEvent().

BUG=649558

Committed: https://crrev.com/160d7098f1c9f293990f0a1ee6644442ad945fc9
Cr-Commit-Position: refs/heads/master@{#425255}
==========

[commit-bot: I haz the power, 2016-10-14 05:00:07]

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