Check that an img element with src set to a data URI with an invalid mediatype renders

Check that an img element with src set to a data URI with an invalid mediatype renders

We have been accepting a data URI with an invalid mediatype.

Image test data in Base64 is copied from fast/images/image-load-event-crash.html.

Depends on https://codereview.chromium.org/555383003/

R=hiroshige
BUG=412479

[tyoshino (SeeGerritForStatus), 2014-10-02 03:41:56]

tyoshino@chromium.org changed reviewers:
+ hiroshige@chromium.org

[tyoshino (SeeGerritForStatus), 2014-10-02 03:42:13]

Hiroshige, could you please take a look?

[hiroshige, 2014-10-06 08:43:08]

lgtm.

https://codereview.chromium.org/579673002/diff/20001/LayoutTests/fast/images/...
File LayoutTests/fast/images/image-from-data-uri-with-invalid-mediatype.html
(right):

https://codereview.chromium.org/579673002/diff/20001/LayoutTests/fast/images/...
LayoutTests/fast/images/image-from-data-uri-with-invalid-mediatype.html:4:
top-level type. without a slash and subtype) for backward compatibility.
> (only top-level type. without a slash and subtype)
Does this description corresponds only to media type = 'image'?
It would be nice if we add reference also to 'image\png'.

And is the period in 'type.' a typo? (e.g. comma or remove?)

[tyoshino (SeeGerritForStatus), 2014-10-16 09:27:54]

https://codereview.chromium.org/579673002/diff/20001/LayoutTests/fast/images/...
File LayoutTests/fast/images/image-from-data-uri-with-invalid-mediatype.html
(right):

https://codereview.chromium.org/579673002/diff/20001/LayoutTests/fast/images/...
LayoutTests/fast/images/image-from-data-uri-with-invalid-mediatype.html:4:
top-level type. without a slash and subtype) for backward compatibility.
On 2014/10/06 08:43:08, hiroshige wrote:
> > (only top-level type. without a slash and subtype)
> Does this description corresponds only to media type = 'image'?
> It would be nice if we add reference also to 'image\png'.
> 

Good point. Updated to explain both.

> And is the period in 'type.' a typo? (e.g. comma or remove?)

Removed.

[tyoshino (SeeGerritForStatus), 2014-10-16 09:30:34]

The CQ bit was checked by tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2014-10-16 09:30:36]

The CQ bit was unchecked by tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2014-10-16 09:30:50]

The CQ bit was checked by tyoshino@chromium.org

[commit-bot: I haz the power, 2014-10-16 09:30:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/579673002/40001

[commit-bot: I haz the power, 2014-10-16 11:01:32]

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

[commit-bot: I haz the power, 2014-10-16 11:01:32]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/27269)

[hiroshige, 2014-10-17 04:31:19]

I tested on Mac and the position of the image was shown a little above from the
expectation of this CL.

[tyoshino (SeeGerritForStatus), 2014-10-17 04:46:09]

On 2014/10/17 04:31:19, hiroshige wrote:
> I tested on Mac and the position of the image was shown a little above from
the
> expectation of this CL.

I'll investigate

https://storage.googleapis.com/chromium-layout-test-archives/win_blink_rel/32...