Mute non-CORS cross origin promise rejection events

Mute non-CORS cross origin promise rejection events

BUG=495801
R=domenic@chromium.org,haraken@chromium.org,yangguo@chromium.org

Committed: https://crrev.com/07cffb9ff140185613a3c8721c945420e09ad69a
Cr-Commit-Position: refs/heads/master@{#361986}

[jochen (gone - plz use gerrit), 2015-11-26 13:06:06]

ptal

depends on the next v8 roll, so no tryjobs yet

https://codereview.chromium.org/1474983004/diff/1/third_party/WebKit/Source/b...
File third_party/WebKit/Source/bindings/core/v8/V8Initializer.cpp (left):

https://codereview.chromium.org/1474983004/diff/1/third_party/WebKit/Source/b...
third_party/WebKit/Source/bindings/core/v8/V8Initializer.cpp:250: } else if
(!exception.IsEmpty() && exception->IsInt32()) {
no longer needed, the new CreateMessage() will also create a message for SMIs

[Yang, 2015-11-26 13:19:03]

On 2015/11/26 13:06:06, jochen wrote:
> ptal
> 
> depends on the next v8 roll, so no tryjobs yet
> 
>
https://codereview.chromium.org/1474983004/diff/1/third_party/WebKit/Source/b...
> File third_party/WebKit/Source/bindings/core/v8/V8Initializer.cpp (left):
> 
>
https://codereview.chromium.org/1474983004/diff/1/third_party/WebKit/Source/b...
> third_party/WebKit/Source/bindings/core/v8/V8Initializer.cpp:250: } else if
> (!exception.IsEmpty() && exception->IsInt32()) {
> no longer needed, the new CreateMessage() will also create a message for SMIs

V8 api use lgtm. Did not look at the rest.

[haraken, 2015-11-26 14:02:50]

LGTM

[jochen (gone - plz use gerrit), 2015-11-26 22:42:51]

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

[commit-bot: I haz the power, 2015-11-26 22:43:08]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1474983004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1474983004/1

[commit-bot: I haz the power, 2015-11-27 00:26:18]

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

[commit-bot: I haz the power, 2015-11-27 00:26:18]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jochen (gone - plz use gerrit), 2015-11-27 12:38:38]

The CQ bit was checked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2015-11-27 12:38:38]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/1474983004/#ps20001
(title: "updates")

[commit-bot: I haz the power, 2015-11-27 12:38:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1474983004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1474983004/20001

[commit-bot: I haz the power, 2015-11-27 12:43:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-27 12:44:12]

Description was changed from

==========
Mute non-CORS cross origin promise rejection events

BUG=495801
R=domenic@chromium.org,haraken@chromium.org,yangguo@chromium.org
==========

to

==========
Mute non-CORS cross origin promise rejection events

BUG=495801
R=domenic@chromium.org,haraken@chromium.org,yangguo@chromium.org

Committed: https://crrev.com/07cffb9ff140185613a3c8721c945420e09ad69a
Cr-Commit-Position: refs/heads/master@{#361986}
==========

[commit-bot: I haz the power, 2015-11-27 12:44:13]

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

[Michael Achenbach, 2015-11-27 14:26:30]

machenbach@chromium.org changed reviewers:
+ hablich@chromium.org, machenbach@chromium.org

[Michael Achenbach, 2015-11-27 14:26:30]

API stability nag (no action required):
https://build.chromium.org/p/chromium.fyi/builders/Linux%20V8%20API%20Stabili...

[jochen (gone - plz use gerrit), 2015-11-27 14:28:07]

meh, I landed the API yesterday, it rolled into chromium yesterday... I thought
waiting until today was good enough

[Michael Achenbach, 2015-11-27 14:30:01]

On 2015/11/27 14:28:07, jochen wrote:
> meh, I landed the API yesterday, it rolled into chromium yesterday... I
thought
> waiting until today was good enough

Not in terms of canaries. If v8 gets reverted on the canary to the last canary
version it'll break the build (but only if).