Fix the flaky test by ensuring the image used by <input type=image> is loaded.

Fix the flaky test by ensuring the image used by <input type=image> is loaded.

It looks that r186800 wrongly rebased fast/events/domactivate-sets-underlying-click-event-as-handled-expected.txt, hiding the regression.

http://src.chromium.org/viewvc/blink?view=revision&revision=186800

However, the test itself has flakiness so it occasionally outputs the originally expected result correctly. That's the reason of the flakiness.

This CL fixed the flakiness by ensuring the image is loaded before testing.
Also reverted the wrong rebase of expected.txt in r186800.

It looks there is no reliable way to know when UA displays the (default) image if <input type=image> is used without src attribute.
The spec says "The src attribute must be present".

https://html.spec.whatwg.org/multipage/forms.html#image-button-state-(type=image)

BUG=520173
Committed: https://crrev.com/e68442e7003b776748b6d749a5cc7345d2dbed4e
Cr-Commit-Position: refs/heads/master@{#351532}

[hayato, 2015-09-30 08:03:51]

hayato@chromium.org changed reviewers:
+ kochi@chromium.org, tkent@chromium.org

[hayato, 2015-09-30 08:03:51]

PTAL

[hayato, 2015-09-30 08:12:09]

https://codereview.chromium.org/1376793003/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/events/domactivate-sets-underlying-click-event-as-handled.html
(right):

https://codereview.chromium.org/1376793003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/events/domactivate-sets-underlying-click-event-as-handled.html:33:
document.querySelector("input[type=image]").addEventListener("load", function()
{
The diff is difficult to read. It just wraps the entire part with the load event
listener.

[tkent, 2015-09-30 08:12:41]

lgtm

[hayato, 2015-09-30 08:14:13]

The CQ bit was checked by hayato@chromium.org

[commit-bot: I haz the power, 2015-09-30 08:14:20]

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

[commit-bot: I haz the power, 2015-09-30 08:26:49]

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

[commit-bot: I haz the power, 2015-09-30 08:26:50]

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

[hayato, 2015-09-30 11:14:33]

The CQ bit was checked by hayato@chromium.org

[commit-bot: I haz the power, 2015-09-30 11:14:54]

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

[commit-bot: I haz the power, 2015-09-30 11:21:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-30 11:22:08]

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