[Mojo] Make javascript enums extensible.

[Mojo] Make javascript enums extensible.

BUG=581390
TEST=mojo_js_unittests, mojo_js_integration_tests

Original review at: https://codereview.chromium.org/2380303003/

Committed: https://crrev.com/3c19074450571b0fdd16063353e726cbe07c8dcc
Cr-Commit-Position: refs/heads/master@{#423302}

[mbjorge, 2016-10-05 18:53:54]

Description was changed from

==========
[Mojo] Make javascript enums extensible.

BUG=581390
TEST=mojo_js_unittests, mojo_js_integration_tests
==========

to

==========
[Mojo] Make javascript enums extensible.

BUG=581390
TEST=mojo_js_unittests, mojo_js_integration_tests

Original review at: https://codereview.chromium.org/2380303003/
==========

[mbjorge, 2016-10-05 18:55:06]

mbjorge@chromium.org changed reviewers:
+ rockot@chromium.org

[mbjorge, 2016-10-05 18:55:06]

I don't know what I did to my local checkout, but I goof'd something up and my
original CL wasn't patching on the CQ properly

[Ken Rockot(use gerrit already), 2016-10-05 20:41:41]

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

[commit-bot: I haz the power, 2016-10-05 20:42:29]

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

[Ken Rockot(use gerrit already), 2016-10-05 20:42:43]

LGTM, but I've never seen that happen so there might actually be something
subtle wrong with the patch.

[Ken Rockot(use gerrit already), 2016-10-05 20:49:12]

On 2016/10/05 at 20:42:43, Ken Rockot wrote:
> LGTM, but I've never seen that happen so there might actually be something
subtle wrong with the patch.

Ah well, dry run seems to be green so far /shrug :)

[Ken Rockot(use gerrit already), 2016-10-05 20:49:22]

The CQ bit was unchecked by rockot@chromium.org

[mbjorge, 2016-10-05 20:51:09]

On 2016/10/05 at 20:42:43, rockot wrote:
> LGTM, but I've never seen that happen so there might actually be something
subtle wrong with the patch.

I'm pretty sure it was just an issue with my checkout. I was using my chromecast
checkout which normally pins chromium/src at an older revision so it builds
correctly against internal code. And then some combination of uploading,
rebasing, checking out, fetching, and/or pulling got something messed up
somewhere. I'm going to just get a separate chromium checkout to work on mojo
stuff in the future :)

[mbjorge, 2016-10-05 20:51:48]

The CQ bit was checked by mbjorge@chromium.org

[commit-bot: I haz the power, 2016-10-05 20:52:33]

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-05 21:45:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-05 21:47:30]

Description was changed from

==========
[Mojo] Make javascript enums extensible.

BUG=581390
TEST=mojo_js_unittests, mojo_js_integration_tests

Original review at: https://codereview.chromium.org/2380303003/
==========

to

==========
[Mojo] Make javascript enums extensible.

BUG=581390
TEST=mojo_js_unittests, mojo_js_integration_tests

Original review at: https://codereview.chromium.org/2380303003/

Committed: https://crrev.com/3c19074450571b0fdd16063353e726cbe07c8dcc
Cr-Commit-Position: refs/heads/master@{#423302}
==========

[commit-bot: I haz the power, 2016-10-05 21:47:31]

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