Add crash keys for ServiceWorkerDispatcherHost crashes

Add crash keys for ServiceWorkerDispatcherHost crashes

BUG=619294, 630495, 630496
Committed: https://crrev.com/587e8127d8c3fda4d66fea1da6fec3ff9890588e
Cr-Commit-Position: refs/heads/master@{#408561}

[horo, 2016-07-27 06:54:42]

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

[commit-bot: I haz the power, 2016-07-27 06:54:55]

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

[horo, 2016-07-27 06:56:45]

Patchset #1 (id:1) has been deleted

[horo, 2016-07-27 06:57:00]

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

[horo, 2016-07-27 06:57:01]

nhiroki@
Could you please review this?

[nhiroki, 2016-07-27 07:24:52]

lgtm with a nit

https://codereview.chromium.org/2190513002/diff/20001/chrome/app/chrome_crash...
File chrome/app/chrome_crash_reporter_client_win.cc (right):

https://codereview.chromium.org/2190513002/diff/20001/chrome/app/chrome_crash...
chrome/app/chrome_crash_reporter_client_win.cc:164: {
"swdh_get_registration_cannot_document_url", crash_keys::kLargeSize },
Looks strange, but there are no spaces after '{' and before '}' only on this
file.

[horo, 2016-07-27 07:32:15]

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

[horo, 2016-07-27 07:32:16]

https://codereview.chromium.org/2190513002/diff/20001/chrome/app/chrome_crash...
File chrome/app/chrome_crash_reporter_client_win.cc (right):

https://codereview.chromium.org/2190513002/diff/20001/chrome/app/chrome_crash...
chrome/app/chrome_crash_reporter_client_win.cc:164: {
"swdh_get_registration_cannot_document_url", crash_keys::kLargeSize },
On 2016/07/27 07:24:51, nhiroki wrote:
> Looks strange, but there are no spaces after '{' and before '}' only on this
> file.

Done.

[horo, 2016-07-27 07:32:38]

sky@
Please review this.

[horo, 2016-07-27 14:58:06]

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

[horo, 2016-07-27 14:58:06]

wez@
Please review blimp/engine/app/blimp_engine_crash_keys.cc.

[sky, 2016-07-27 15:37:37]

sky@chromium.org changed reviewers:
+ ananta@chromium.org

[sky, 2016-07-27 15:37:37]

+ananta, who should be an owner of these files.

[ananta, 2016-07-27 21:02:41]

chrome_crash_reporter_client_win.cc LGTM

[sky, 2016-07-27 21:57:32]

Rubber stamp LGTM

[horo, 2016-07-29 00:02:29]

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

[horo, 2016-07-29 00:02:29]

rsesek@
Could you please review blimp/engine/app/blimp_engine_crash_keys.cc?

[Wez, 2016-07-29 00:20:31]

blimp/ LGTM

[horo, 2016-07-29 01:41:28]

The CQ bit was checked by horo@chromium.org

[horo, 2016-07-29 01:41:28]

The patchset sent to the CQ was uploaded after l-g-t-m from nhiroki@chromium.org
Link to the patchset: https://codereview.chromium.org/2190513002/#ps40001
(title: "remove space")

[commit-bot: I haz the power, 2016-07-29 01:41:56]

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-07-29 02:50:05]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-07-29 02:51:38]

Description was changed from

==========
Add crash keys for ServiceWorkerDispatcherHost crashes

BUG=619294,630495,630496
==========

to

==========
Add crash keys for ServiceWorkerDispatcherHost crashes

BUG=619294,630495,630496

Committed: https://crrev.com/587e8127d8c3fda4d66fea1da6fec3ff9890588e
Cr-Commit-Position: refs/heads/master@{#408561}
==========

[commit-bot: I haz the power, 2016-07-29 02:51:40]

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