Fix for MIME handler bug.

Fix for MIME handler bug. The bug resulted from a previous bugfix (https://codereview.chromium.org/939443002/) that added a conversion from logical to physical units to fix a similar sizing bug with other guestviews. However, MIME handler guests never use logical pixels, so this conversion broke their sizing when they are created in a zoomed embedder.

This patch adds a more robust method of checking whether sizes are provided in logical or physical units, and all cases are handled appropriately.

BUG=462194
Committed: https://crrev.com/18f5a136e4ab830a3303c6a6f1d87c7f34eb7fd9
Cr-Commit-Position: refs/heads/master@{#318948}

[paulmeyer, 2015-03-03 17:46:46]

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

[paulmeyer, 2015-03-03 17:46:46]

+fsamuel@

[Fady Samuel, 2015-03-03 17:50:58]

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

https://codereview.chromium.org/972193003/diff/1/extensions/browser/guest_vie...
extensions/browser/guest_view/guest_view_base.cc:761: if
(params.HasKey(guestview::kElementSizeIsLogical)) {
This check seems unnecessary. Just call GetBoolean and set the default value
above?

[paulmeyer, 2015-03-03 20:35:55]

ptal.

[paulmeyer, 2015-03-03 20:36:23]

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

https://codereview.chromium.org/972193003/diff/1/extensions/browser/guest_vie...
extensions/browser/guest_view/guest_view_base.cc:761: if
(params.HasKey(guestview::kElementSizeIsLogical)) {
On 2015/03/03 17:50:58, Fady Samuel wrote:
> This check seems unnecessary. Just call GetBoolean and set the default value
> above?

Done.

[Fady Samuel, 2015-03-03 20:57:10]

lgtm

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

The CQ bit was checked by paulmeyer@chromium.org

[commit-bot: I haz the power, 2015-03-03 20:59:19]

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

[commit-bot: I haz the power, 2015-03-03 21:56:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-03 21:56:58]

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