OOP PDF: Add whether a resource is embedded to StreamInfo.

OOP PDF: Add whether a resource is embedded to StreamInfo.

Previously, the out-of-process PDF extension managed page zoom for all
PDFs. This is not correct for embedded PDFs, which should not affect or
be affected by page zoom. This CL makes that information available to
the extension and uses it to decide not manage page zoom for embedded
PDFs.

BUG=433713
TEST=Run with --out-of-process-pdf. Open a page embedding a PDF. Zooming
should affect the page, including the size of the PDF viewport, but not
the zoom level of the PDF.

Committed: https://crrev.com/3ae8287447b46a6bceeec089cebb52e79633f916
Cr-Commit-Position: refs/heads/master@{#305394}

[Sam McNally, 2014-11-17 02:50:49]

Patchset #1 (id:1) has been deleted

[Sam McNally, 2014-11-17 02:54:03]

Patchset #1 (id:20001) has been deleted

[Sam McNally, 2014-11-17 03:01:48]

sammc@chromium.org changed reviewers:
+ raymes@chromium.org

[raymes, 2014-11-17 04:09:38]

lgtm

https://codereview.chromium.org/726343002/diff/60001/chrome/browser/renderer_...
File chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
(right):

https://codereview.chromium.org/726343002/diff/60001/chrome/browser/renderer_...
chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc:641:
info->GetResourceType() != content::RESOURCE_TYPE_MAIN_FRAME));
nit: maybe just split this out into a separate statement for clarity.

[Sam McNally, 2014-11-17 04:31:29]

sammc@chromium.org changed reviewers:
+ zork@chromium.org

[Sam McNally, 2014-11-17 04:31:29]

+zork for chrome/browser/extensions/api/streams_private/

https://codereview.chromium.org/726343002/diff/60001/chrome/browser/renderer_...
File chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
(right):

https://codereview.chromium.org/726343002/diff/60001/chrome/browser/renderer_...
chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc:641:
info->GetResourceType() != content::RESOURCE_TYPE_MAIN_FRAME));
On 2014/11/17 04:09:38, raymes wrote:
> nit: maybe just split this out into a separate statement for clarity.

Done.

[Sam McNally, 2014-11-19 04:46:52]

zork: ping

[Zachary Kuznia, 2014-11-19 21:43:14]

lgtm

[Sam McNally, 2014-11-19 23:45:17]

sammc@chromium.org changed reviewers:
+ benwells@chromium.org, jam@chromium.org

[Sam McNally, 2014-11-19 23:45:18]

+benwells for chrome/common/extensions/api/streams_private.idl 
+jam for
chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc

[jam, 2014-11-20 21:08:31]

lgtm

[benwells, 2014-11-23 23:55:13]

lgtm

[Sam McNally, 2014-11-23 23:55:33]

The CQ bit was checked by sammc@chromium.org

[commit-bot: I haz the power, 2014-11-23 23:56:05]

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

[commit-bot: I haz the power, 2014-11-24 01:45:01]

Committed patchset #3 (id:80001)

[commit-bot: I haz the power, 2014-11-24 01:46:55]

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