Fix layout test for Event.scoped

Fix layout test for Event.scoped

Fix the layout test related to this CL : https://codereview.chromium.org/1586563005/
Changes are made so that the test will handle the cases where events do not happen.
BUG=531990, 575042
Committed: https://crrev.com/c27e76c7d430f4ed19298d25fde0db4ebd5cb7f4
Cr-Commit-Position: refs/heads/master@{#372627}

[yuzuchan, 2016-01-28 05:52:43]

Description was changed from

==========
Fix layout test for Event.scoped

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

to

==========
Fix layout test for Event.scoped

Fix the layout test related to this CL :
https://codereview.chromium.org/1586563005/
Changes are made so that the test will handle the cases where events do not
happen. 
BUG=531990, 575042
==========

[yuzuchan, 2016-01-28 06:16:37]

yuzus@chromium.org changed reviewers:
+ hayato@chromium.org, kochi@chromium.org, kojii@chromium.org

[kochi, 2016-01-28 07:01:12]

https://codereview.chromium.org/1641573002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/dom/shadow/scoped-events-by-ua-stopped.html
(right):

https://codereview.chromium.org/1641573002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/shadow/scoped-events-by-ua-stopped.html:5:
<img id="img" src="../../images/resources/test-load.jpg"
onload="loadScopeTest()" onerror="imgLoadTest()">
There's still a chance that these handlers are never called.

https://codereview.chromium.org/1641573002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/shadow/scoped-events-by-ua-stopped.html:18:
}, 'A new events scoped value should be set to false by default.');
FYI - you can quote a single quote in a string like '\'' or use
double-quotes to enclose the string, if you want to use unquoted single quote.

(The latter is probably not specified in any style guide, but I believe
it is generally allowed.)

https://codereview.chromium.org/1641573002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/shadow/scoped-events-by-ua-stopped.html:27:
assert_equals(event.scoped, true);
Where is this |event| coming from?

s/loadScopedTest()/loadScopedTest(event)/

(same for the event handlers below)

https://codereview.chromium.org/1641573002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/shadow/scoped-events-by-ua-stopped.html:45:
test(function() {
I don't think this test addresses original concern.
Let's discuss offline.

[yuzuchan, 2016-02-01 08:42:49]

https://codereview.chromium.org/1641573002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/dom/shadow/scoped-events-by-ua-stopped.html
(right):

https://codereview.chromium.org/1641573002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/shadow/scoped-events-by-ua-stopped.html:5:
<img id="img" src="../../images/resources/test-load.jpg"
onload="loadScopeTest()" onerror="imgLoadTest()">
On 2016/01/28 07:01:12, kochi wrote:
> There's still a chance that these handlers are never called.

Done.

https://codereview.chromium.org/1641573002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/shadow/scoped-events-by-ua-stopped.html:18:
}, 'A new events scoped value should be set to false by default.');
On 2016/01/28 07:01:12, kochi wrote:
> FYI - you can quote a single quote in a string like '\'' or use
> double-quotes to enclose the string, if you want to use unquoted single quote.
> 
> (The latter is probably not specified in any style guide, but I believe
> it is generally allowed.)

Done.

https://codereview.chromium.org/1641573002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/shadow/scoped-events-by-ua-stopped.html:27:
assert_equals(event.scoped, true);
On 2016/01/28 07:01:12, kochi wrote:
> Where is this |event| coming from?
> 
> s/loadScopedTest()/loadScopedTest(event)/
> 
> (same for the event handlers below)

Done.

https://codereview.chromium.org/1641573002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/shadow/scoped-events-by-ua-stopped.html:45:
test(function() {
On 2016/01/28 07:01:12, kochi wrote:
> I don't think this test addresses original concern.
> Let's discuss offline.

Done.

[hayato, 2016-02-01 09:04:55]

lgtm

[yuzusan, 2016-02-01 09:05:16]

The CQ bit was checked by yuzus@google.com

[commit-bot: I haz the power, 2016-02-01 09:05:56]

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

[commit-bot: I haz the power, 2016-02-01 09:48:45]

Description was changed from

==========
Fix layout test for Event.scoped

Fix the layout test related to this CL :
https://codereview.chromium.org/1586563005/
Changes are made so that the test will handle the cases where events do not
happen. 
BUG=531990, 575042
==========

to

==========
Fix layout test for Event.scoped

Fix the layout test related to this CL :
https://codereview.chromium.org/1586563005/
Changes are made so that the test will handle the cases where events do not
happen. 
BUG=531990, 575042
==========

[commit-bot: I haz the power, 2016-02-01 09:48:45]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-02-01 09:50:05]

Description was changed from

==========
Fix layout test for Event.scoped

Fix the layout test related to this CL :
https://codereview.chromium.org/1586563005/
Changes are made so that the test will handle the cases where events do not
happen. 
BUG=531990, 575042
==========

to

==========
Fix layout test for Event.scoped

Fix the layout test related to this CL :
https://codereview.chromium.org/1586563005/
Changes are made so that the test will handle the cases where events do not
happen.
BUG=531990, 575042

Committed: https://crrev.com/c27e76c7d430f4ed19298d25fde0db4ebd5cb7f4
Cr-Commit-Position: refs/heads/master@{#372627}
==========

[commit-bot: I haz the power, 2016-02-01 09:50:06]

Patchset 4 (id:??) landed as
https://crrev.com/c27e76c7d430f4ed19298d25fde0db4ebd5cb7f4
Cr-Commit-Position: refs/heads/master@{#372627}