[DevTools] Introduce typed events.

[DevTools] Introduce typed events.

These are now type-checked by compiler.

BUG=none

Committed: https://crrev.com/78b49dc97ebe44a435b12c946e7bfbf1870afab3
Cr-Commit-Position: refs/heads/master@{#439054}

[dgozman, 2016-12-15 03:10:19]

dgozman@chromium.org changed reviewers:
+ lushnikov@chromium.org, pfeldman@chromium.org

[dgozman, 2016-12-15 03:10:19]

Could you please take a look?

[lushnikov, 2016-12-15 04:29:00]

lgtm

https://codereview.chromium.org/2580673002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/resources/DatabaseModel.js
(right):

https://codereview.chromium.org/2580673002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/resources/DatabaseModel.js:189:
Resources.DatabaseModel.DatabaseAddedEvent = class {
maybe we can keep a list of events - it was extremely convenient

[pfeldman, 2016-12-16 02:31:18]

lgtm

[dgozman, 2016-12-16 04:17:46]

https://codereview.chromium.org/2580673002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/resources/DatabaseModel.js
(right):

https://codereview.chromium.org/2580673002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/resources/DatabaseModel.js:189:
Resources.DatabaseModel.DatabaseAddedEvent = class {
On 2016/12/15 04:28:59, lushnikov wrote:
> maybe we can keep a list of events - it was extremely convenient

JSCompiler didn't like it :-(

[dgozman, 2016-12-16 04:17:51]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-12-16 04:18:39]

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-16 04:49:07]

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

[commit-bot: I haz the power, 2016-12-16 04:49:08]

Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[dgozman, 2016-12-16 05:16:19]

The CQ bit was checked by dgozman@chromium.org

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

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-16 07:02:59]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481865379066760,
"parent_rev": "1e46b78ad70b159a6ff4356d796daa1d1112f184", "commit_rev":
"498c4c0a2cc918df8fdc18bb3dbd2be1fe29bfbf"}

[commit-bot: I haz the power, 2016-12-16 07:03:58]

Description was changed from

==========
[DevTools] Introduce typed events.

These are now type-checked by compiler.

BUG=none
==========

to

==========
[DevTools] Introduce typed events.

These are now type-checked by compiler.

BUG=none

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

[commit-bot: I haz the power, 2016-12-16 07:03:59]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-16 07:07:20]

Description was changed from

==========
[DevTools] Introduce typed events.

These are now type-checked by compiler.

BUG=none

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

to

==========
[DevTools] Introduce typed events.

These are now type-checked by compiler.

BUG=none

Committed: https://crrev.com/78b49dc97ebe44a435b12c946e7bfbf1870afab3
Cr-Commit-Position: refs/heads/master@{#439054}
==========

[commit-bot: I haz the power, 2016-12-16 07:07:21]

Patchset 3 (id:??) landed as
https://crrev.com/78b49dc97ebe44a435b12c946e7bfbf1870afab3
Cr-Commit-Position: refs/heads/master@{#439054}