Add image loading spec related tests

Add image loading spec related tests

Added the following layout tests:
* Setting order of crossOrigin and src attributes in JS doesn't change outgoing request
* Setting order of srcset and src attributes in JS doesn't change fetched resources
* Setting innerHTML with an <img> inside triggers a resource load
* Setting src of a cached image exposes its width immediately

BUG=

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=166657

[Yoav Weiss, 2014-02-05 21:34:27]

Following a discussion on blink-dev [1] and on #whatwg [2], I've realized that
there are several aspects of image loading that are either not implemented
perspec, or not covered by tests (and therefore likely to silently break when
attempting to fix the above).

Since I'd like to align the implementation with the spec (to facilitate the
implementation of picture), I've added these tests.

[1]
https://groups.google.com/a/chromium.org/d/msg/blink-dev/hKj2OsvZl1g/clCrVkQq...
[2] http://krijnhoetmer.nl/irc-logs/whatwg/20140204#l-537

[Yoav Weiss, 2014-02-05 21:39:51]

On 2014/02/05 21:34:27, Yoav Weiss wrote:
> Following a discussion on blink-dev [1] and on #whatwg [2], I've realized that
> there are several aspects of image loading that are either not implemented
> perspec, or not covered by tests (and therefore likely to silently break when
> attempting to fix the above).
> 
> Since I'd like to align the implementation with the spec (to facilitate the
> implementation of picture), I've added these tests.
> 
> [1]
>
https://groups.google.com/a/chromium.org/d/msg/blink-dev/hKj2OsvZl1g/clCrVkQq...
> [2] http://krijnhoetmer.nl/irc-logs/whatwg/20140204#l-537

Some of these tests don't yet pass and added to TestExpectations.
I've added bugs RE them at:
http://crbug.com/341172
http://crbug.com/341047

[abarth-chromium, 2014-02-06 01:11:18]

LGTM

https://codereview.chromium.org/136693020/diff/40001/LayoutTests/TestExpectat...
File LayoutTests/TestExpectations (right):

https://codereview.chromium.org/136693020/diff/40001/LayoutTests/TestExpectat...
LayoutTests/TestExpectations:113: crbug.com/341172
virtual/gpu/fast/hidpi/image-srcset-change-dynamically-src-first-2x.html [
Failure ]
Rather than making these as failures, we should check in expectations that match
our current behavior and have a comment in the test or a bug on file that
explains what results they should have.

https://codereview.chromium.org/136693020/diff/40001/LayoutTests/fast/dom/HTM...
File LayoutTests/fast/dom/HTMLImageElement/image-dynamic-width-expected.txt
(right):

https://codereview.chromium.org/136693020/diff/40001/LayoutTests/fast/dom/HTM...
LayoutTests/fast/dom/HTMLImageElement/image-dynamic-width-expected.txt:13: TEST
COMPLETE
It's strange that this prints TEST COMPLETE twice.  That's normally a bug in the
test.

[Yoav Weiss, 2014-02-06 20:19:57]

The CQ bit was checked by yoav@yoav.ws

[Yoav Weiss, 2014-02-06 20:21:27]

The CQ bit was unchecked by yoav@yoav.ws

[Yoav Weiss, 2014-02-06 22:03:26]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2014-02-06 22:03:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoav@yoav.ws/136693020/330001

[commit-bot: I haz the power, 2014-02-06 23:13:00]

Change committed as 166657