Ignore the default zoom level for the PDF extension and print preview.

Ignore the default zoom level for the PDF extension and print preview.

Previously, the PDF extension UI was zoomed to the default zoom level
and the PDF was inversely affected (due to various calculations using
zoomed and not zoomed sizes). This resets the zoom level for extensions
used as mime handlers on mime handler WebContents creation and for print
preview (as the PDF extension zoom as controlled by the embedding print
preview dialog WebContents.

BUG=450910
Committed: https://crrev.com/335f3d635b8403ad2a2ec2e3a346f0436db40b46
Cr-Commit-Position: refs/heads/master@{#313444}

[Sam McNally, 2015-01-27 23:44:25]

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

[raymes, 2015-01-28 00:39:28]

lgtm

https://codereview.chromium.org/881553003/diff/1/extensions/browser/guest_vie...
File extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
(right):

https://codereview.chromium.org/881553003/diff/1/extensions/browser/guest_vie...
extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc:132:
content::HostZoomMap::Get(guest_site_instance)
Could you add a comment about why we disable zoom (and maybe in the other file
too)

[Sam McNally, 2015-01-28 01:35:13]

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

[Sam McNally, 2015-01-28 01:35:15]

+vitalybuka for chrome/browser/printing/print_preview_dialog_controller.cc

https://codereview.chromium.org/881553003/diff/1/extensions/browser/guest_vie...
File extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
(right):

https://codereview.chromium.org/881553003/diff/1/extensions/browser/guest_vie...
extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc:132:
content::HostZoomMap::Get(guest_site_instance)
On 2015/01/28 00:39:28, raymes wrote:
> Could you add a comment about why we disable zoom (and maybe in the other file
> too)

Done.

[Vitaly Buka (NO REVIEWS), 2015-01-28 01:40:16]

lgtm

[Sam McNally, 2015-01-28 01:52:43]

The CQ bit was checked by sammc@chromium.org

[commit-bot: I haz the power, 2015-01-28 01:53:36]

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

[commit-bot: I haz the power, 2015-01-28 04:24:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-28 04:25:47]

Patchset 2 (id:??) landed as
https://crrev.com/335f3d635b8403ad2a2ec2e3a346f0436db40b46
Cr-Commit-Position: refs/heads/master@{#313444}