Copy SkMetaData before painting printed plugin contents.

Copy SkMetaData before painting printed plugin contents.

The SkMetaData is necessary for the printing code to determine what PDF is being printed.

See:

https://code.google.com/p/chromium/codesearch#chromium/src/content/renderer/pepper/pepper_plugin_instance_impl.cc&sq=package:chromium&type=cs&l=1980&rcl=1433159508

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

[chrishtr, 2015-06-01 23:08:32]

chrishtr@chromium.org changed reviewers:
+ pdr@chromium.org

[chrishtr, 2015-06-01 23:08:32]

I will need to work more on testing, and not even sure how to do it yet. I'd
like to not block this patch on that.

[pdr., 2015-06-02 01:13:41]

LGTM

We've all spent hours fighting with our untested plugin integration. Could you
try to make a test for this?

[chrishtr, 2015-06-04 17:00:14]

Patchset #5 (id:80001) has been deleted

[chrishtr, 2015-06-04 17:25:59]

Generalized to work with slimming paint. This required putting the metadata on
GraphicsContext, since we need it to propagate into drawing recorders.

Note that crbug.conm/496728 means only the top-level PDF printing works anyway,
SP or non-SP.

I'm looking at testing now.

[pdr., 2015-06-04 18:03:56]

LGTM

Do we need the skia/ext include? It seems to be allowed but looks a little
strange.

[chrishtr, 2015-06-05 18:39:33]

Added a test!!!

Also, this CL depends on https://codereview.chromium.org/1167063003.

[pdr., 2015-06-05 18:44:37]

lgtm

[chrishtr, 2015-06-05 19:41:05]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2015-06-05 19:41:18]

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

[commit-bot: I haz the power, 2015-06-05 22:18:23]

Committed patchset #9 (id:180001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196606