Fixed the sizing bug for PDFs opened from links with target=_blank.

Fixed the sizing bug for PDFs opened from links with target=_blank.

BUG=462677, 469652
Committed: https://crrev.com/78ed69f066f58c82923a0cd34f1d5a3ab98c5d53
Cr-Commit-Position: refs/heads/master@{#322277}

[paulmeyer, 2015-03-25 19:20:13]

paulmeyer@chromium.org changed reviewers:
+ fsamuel@chromium.org

[paulmeyer, 2015-03-25 19:20:14]

+fsamuel@

[Fady Samuel, 2015-03-25 20:30:10]

Does this work with a PDF in a <webview>?

https://codereview.chromium.org/1034803002/diff/1/extensions/browser/guest_vi...
File extensions/browser/guest_view/guest_view_base.cc (right):

https://codereview.chromium.org/1034803002/diff/1/extensions/browser/guest_vi...
extensions/browser/guest_view/guest_view_base.cc:791: auto embedder_size =
embedder_web_contents()->GetRenderWidgetHostView()->
Use owner_web_contents() instead. embedder_web_contents() is not available until
attachment.

[paulmeyer, 2015-03-25 20:58:36]

I tried it out and it appears to work fine in webview as well.

https://codereview.chromium.org/1034803002/diff/1/extensions/browser/guest_vi...
File extensions/browser/guest_view/guest_view_base.cc (right):

https://codereview.chromium.org/1034803002/diff/1/extensions/browser/guest_vi...
extensions/browser/guest_view/guest_view_base.cc:791: auto embedder_size =
embedder_web_contents()->GetRenderWidgetHostView()->
On 2015/03/25 20:30:10, Fady Samuel wrote:
> Use owner_web_contents() instead. embedder_web_contents() is not available
until
> attachment.

Done.

[Fady Samuel, 2015-03-25 21:23:17]

Cool! lgtm

[paulmeyer, 2015-03-25 21:23:34]

The CQ bit was checked by paulmeyer@chromium.org

[commit-bot: I haz the power, 2015-03-25 21:24:05]

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

[commit-bot: I haz the power, 2015-03-25 21:34:09]

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

[commit-bot: I haz the power, 2015-03-25 21:34:12]

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

[paulmeyer, 2015-03-25 23:18:30]

On 2015/03/25 21:34:12, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

Whoops, looks like changes from a different patch got melded into this one. I
took out the files that were not meant to be with this CL.

[paulmeyer, 2015-03-25 23:18:38]

The CQ bit was checked by paulmeyer@chromium.org

[paulmeyer, 2015-03-25 23:18:39]

The patchset sent to the CQ was uploaded after l-g-t-m from fsamuel@chromium.org
Link to the patchset: https://codereview.chromium.org/1034803002/#ps40001
(title: "Took out unrelated files.")

[commit-bot: I haz the power, 2015-03-25 23:19:16]

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

[commit-bot: I haz the power, 2015-03-26 00:39:08]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-26 00:39:44]

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