Use testharness.js in image load tests.

Use testharness.js in image load tests.

This test uses testharness.js to test whether image
load event is fired even if the image is the last one we
loaded successfully.

BUG=637859
.

Committed: https://crrev.com/eb35d1f2baf4c0e4b42f8b4d4a33f543c2185597
Cr-Commit-Position: refs/heads/master@{#412806}

[sivag, 2016-08-18 10:18:50]

Description was changed from

==========
Use testharness.js in image load tests.

This test uses tstharness.js to test whether image
load event is fired even if the image is the last one we
loaded successfully.

BUG=637859.
==========

to

==========
Use testharness.js in image load tests.

This test uses testharness.js to test whether image
load event is fired even if the image is the last one we
loaded successfully.

BUG=637859.
==========

[sivag, 2016-08-18 10:19:08]

siva.gunturi@samsung.com changed reviewers:
+ fs@opera.com, srirama.m@samsung.com

[sivag, 2016-08-18 10:19:54]

@fs, srirama ptal..

[fs, 2016-08-18 10:44:26]

LGTM w/ nits

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-interrupted-null-src-load.html
(right):

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-interrupted-null-src-load.html:13:
img.src= resetImage;
Nit: Space before =

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-interrupted-null-src-load.html:17:
img.src= resetImage;
Nit: Space before =

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-successful-image-load.html
(right):

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-successful-image-load.html:10:
// In this test we load an image then we immediately launch an other request
Nit: s/an other/another/

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-successful-image-load.html:14:
img.src= resetImage;
Nit: Space before =

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-successful-image-load.html:18:
img.src= resetImage;
Nit: Space before =

[sivag, 2016-08-18 10:53:20]

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-interrupted-null-src-load.html
(right):

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-interrupted-null-src-load.html:13:
img.src= resetImage;
On 2016/08/18 10:44:26, fs wrote:
> Nit: Space before =

Done.

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-interrupted-null-src-load.html:17:
img.src= resetImage;
On 2016/08/18 10:44:26, fs wrote:
> Nit: Space before =

Done.

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-successful-image-load.html
(right):

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-successful-image-load.html:10:
// In this test we load an image then we immediately launch an other request
On 2016/08/18 10:44:26, fs wrote:
> Nit: s/an other/another/

Done.

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-successful-image-load.html:14:
img.src= resetImage;
On 2016/08/18 10:44:26, fs wrote:
> Nit: Space before =

Done.

https://codereview.chromium.org/2251413002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/images/onload-event-when-reloading-image-after-successful-image-load.html:18:
img.src= resetImage;
On 2016/08/18 10:44:26, fs wrote:
> Nit: Space before =

Done.

[sivag, 2016-08-18 10:54:05]

The CQ bit was checked by siva.gunturi@samsung.com

[sivag, 2016-08-18 10:54:05]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com
Link to the patchset: https://codereview.chromium.org/2251413002/#ps20001
(title: "Addressed review comments.")

[commit-bot: I haz the power, 2016-08-18 10:54:21]

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-08-18 12:19:19]

Description was changed from

==========
Use testharness.js in image load tests.

This test uses testharness.js to test whether image
load event is fired even if the image is the last one we
loaded successfully.

BUG=637859.
==========

to

==========
Use testharness.js in image load tests.

This test uses testharness.js to test whether image
load event is fired even if the image is the last one we
loaded successfully.

BUG=637859.
==========

[commit-bot: I haz the power, 2016-08-18 12:19:20]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-18 12:23:05]

Description was changed from

==========
Use testharness.js in image load tests.

This test uses testharness.js to test whether image
load event is fired even if the image is the last one we
loaded successfully.

BUG=637859.
==========

to

==========
Use testharness.js in image load tests.

This test uses testharness.js to test whether image
load event is fired even if the image is the last one we
loaded successfully.

BUG=637859.

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

[commit-bot: I haz the power, 2016-08-18 12:23:09]

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