Make the test NPAPI plugin not handle png

Make the test NPAPI plugin not handle png

This is not a feature we have/need in PPAPI

BUG=493212
Committed: https://crrev.com/af5d661c72f723eebb145b9df57dddc745d059c7
Cr-Commit-Position: refs/heads/master@{#383437}

[piman, 2016-03-25 21:46:12]

piman@chromium.org changed reviewers:
+ dcheng@chromium.org

[piman, 2016-03-25 21:46:12]

WDYT?
In theory, an <embed> can handle image types in 2 ways:
1- if a plugin supports that type, it loads the plugin to handle the src url
2- if no plugins support that type, blink handles it directly, similar to an
<img>.

We don't have support for plugin handling of images in PPAPI (path 1), nor do we
have the need for it (Flash/PDF don't, NaCl can't). I believe this was added to
WebKit to have the QuickTime plugin handle images, which we don't support.

We have several layout tests that use <embed src="something.png">, or <embed
type="image/png>. On linux they all take path (2), and on windows/mac they take
path (1) with the test plugin, although it doesn't really handle the image type
so you get blank instead of the image.

The intent of this CL is to make everything take path (2) since that's the
useful one. It's hard to know if the test intended to explicitly test path (1)
or (2). There's a couple of tests where the baseline seems to indicate they
expect path (1), so I switched those to using the ppapi plugin.

[dcheng, 2016-03-25 22:10:03]

lgtm

[piman, 2016-03-25 22:17:28]

The CQ bit was checked by piman@chromium.org

[commit-bot: I haz the power, 2016-03-25 22:17:55]

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

[commit-bot: I haz the power, 2016-03-25 23:28:00]

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

[commit-bot: I haz the power, 2016-03-25 23:28:01]

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

[piman, 2016-03-26 00:48:30]

The CQ bit was checked by piman@chromium.org

[piman, 2016-03-26 00:48:31]

The patchset sent to the CQ was uploaded after l-g-t-m from dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/1829293002/#ps40001
(title: "rebase; request rebaseline on linux")

[commit-bot: I haz the power, 2016-03-26 00:48:53]

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

[piman, 2016-03-26 00:50:34]

FYI, requesting rebaseline on linux for those 2 tests that I touched. I was
wondering why I didn't get any failure, but that's because linux_blink_rel
doesn't run layout tests (linux_blink_oilpan_rel does).

[commit-bot: I haz the power, 2016-03-26 02:21:47]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-26 02:23:13]

Description was changed from

==========
Make the test NPAPI plugin not handle png

This is not a feature we have/need in PPAPI

BUG=493212
==========

to

==========
Make the test NPAPI plugin not handle png

This is not a feature we have/need in PPAPI

BUG=493212

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

[commit-bot: I haz the power, 2016-03-26 02:23:14]

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