SELECT popup: Don't use non-100% zoom level in popup.

SELECT popup: Don't use non-100% zoom level in popup.

This CL reverts a SELECT popup part of [1], which applied page zoom to
page popups, and we pass zoomed font-size after this CL. [1] made
multiple regressions, and it's hard to fix them in non-100% zoom
level.

* Make PagePopupClient::zoomFactor() virtual, and SELECT popups return 1.0 for it.

* Fix code and comments on screen coordinate.

* Popup border ignores device scale factor and page zoom, and is always 1px width.

[1] codereview.chromium.org/1454613002

BUG=564506, 570636, 595986
Committed: https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416
Cr-Commit-Position: refs/heads/master@{#441335}

[tkent, 2016-12-28 06:15:43]

The CQ bit was checked by tkent@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-28 06:15:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tkent, 2016-12-28 07:43:03]

The CQ bit was checked by tkent@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-28 07:43:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tkent, 2016-12-28 07:43:19]

Patchset #1 (id:1) has been deleted

[tkent, 2016-12-28 07:44:35]

tkent@chromium.org changed reviewers:
+ keishi@chromium.org

[tkent, 2016-12-28 07:44:35]

keishi@, would you review this please?

[tkent, 2016-12-28 07:46:47]

Description was changed from

==========
SELECT popup: Don't use non-100% zoom level in popup.

This CL reverts a SELECT popup part of [1], which applied page zoom to
page popups, and we pass zoomed font-size after this CL. [1] made
multiple regressions, and it's hard to fix them in non-100% zoom
level.

[1] codereview.chromium.org/1454613002

BUG=564506,570636,595986
==========

to

==========
SELECT popup: Don't use non-100% zoom level in popup.

This CL reverts a SELECT popup part of [1], which applied page zoom to
page popups, and we pass zoomed font-size after this CL. [1] made
multiple regressions, and it's hard to fix them in non-100% zoom
level.

* Make PagePopupClient::zoomFactor() virtual, and SELECT popups return 1.0 for
it.

* Fix code and comments on screen coordinate.


[1] codereview.chromium.org/1454613002

BUG=564506,570636,595986
==========

[tkent, 2016-12-28 07:58:09]

Description was changed from

==========
SELECT popup: Don't use non-100% zoom level in popup.

This CL reverts a SELECT popup part of [1], which applied page zoom to
page popups, and we pass zoomed font-size after this CL. [1] made
multiple regressions, and it's hard to fix them in non-100% zoom
level.

* Make PagePopupClient::zoomFactor() virtual, and SELECT popups return 1.0 for
it.

* Fix code and comments on screen coordinate.


[1] codereview.chromium.org/1454613002

BUG=564506,570636,595986
==========

to

==========
SELECT popup: Don't use non-100% zoom level in popup.

This CL reverts a SELECT popup part of [1], which applied page zoom to
page popups, and we pass zoomed font-size after this CL. [1] made
multiple regressions, and it's hard to fix them in non-100% zoom
level.

* Make PagePopupClient::zoomFactor() virtual, and SELECT popups return 1.0 for
it.

* Fix code and comments on screen coordinate.

* Popup border ignores device scale factor and page zoom, and is always 1px
width.


[1] codereview.chromium.org/1454613002

BUG=564506,570636,595986
==========

[commit-bot: I haz the power, 2016-12-28 09:55:00]

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

[commit-bot: I haz the power, 2016-12-28 09:55:01]

Dry run: This issue passed the CQ dry run.

[keishi, 2017-01-04 03:36:31]

LGTM

[tkent, 2017-01-04 04:21:29]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2017-01-04 04:21:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-04 07:49:03]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483503689371160,
"parent_rev": "1a5f1f46f8e4fefd3ec6b6e5f6e32c7d6e43c25c", "commit_rev":
"b0264f80959bbf4b024b138f400419205d8b31a9"}

[commit-bot: I haz the power, 2017-01-04 07:49:31]

Description was changed from

==========
SELECT popup: Don't use non-100% zoom level in popup.

This CL reverts a SELECT popup part of [1], which applied page zoom to
page popups, and we pass zoomed font-size after this CL. [1] made
multiple regressions, and it's hard to fix them in non-100% zoom
level.

* Make PagePopupClient::zoomFactor() virtual, and SELECT popups return 1.0 for
it.

* Fix code and comments on screen coordinate.

* Popup border ignores device scale factor and page zoom, and is always 1px
width.


[1] codereview.chromium.org/1454613002

BUG=564506,570636,595986
==========

to

==========
SELECT popup: Don't use non-100% zoom level in popup.

This CL reverts a SELECT popup part of [1], which applied page zoom to
page popups, and we pass zoomed font-size after this CL. [1] made
multiple regressions, and it's hard to fix them in non-100% zoom
level.

* Make PagePopupClient::zoomFactor() virtual, and SELECT popups return 1.0 for
it.

* Fix code and comments on screen coordinate.

* Popup border ignores device scale factor and page zoom, and is always 1px
width.


[1] codereview.chromium.org/1454613002

BUG=564506,570636,595986

Review-Url: https://codereview.chromium.org/2606853002
==========

[commit-bot: I haz the power, 2017-01-04 07:49:32]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2017-01-04 07:54:02]

Description was changed from

==========
SELECT popup: Don't use non-100% zoom level in popup.

This CL reverts a SELECT popup part of [1], which applied page zoom to
page popups, and we pass zoomed font-size after this CL. [1] made
multiple regressions, and it's hard to fix them in non-100% zoom
level.

* Make PagePopupClient::zoomFactor() virtual, and SELECT popups return 1.0 for
it.

* Fix code and comments on screen coordinate.

* Popup border ignores device scale factor and page zoom, and is always 1px
width.


[1] codereview.chromium.org/1454613002

BUG=564506,570636,595986

Review-Url: https://codereview.chromium.org/2606853002
==========

to

==========
SELECT popup: Don't use non-100% zoom level in popup.

This CL reverts a SELECT popup part of [1], which applied page zoom to
page popups, and we pass zoomed font-size after this CL. [1] made
multiple regressions, and it's hard to fix them in non-100% zoom
level.

* Make PagePopupClient::zoomFactor() virtual, and SELECT popups return 1.0 for
it.

* Fix code and comments on screen coordinate.

* Popup border ignores device scale factor and page zoom, and is always 1px
width.

[1] codereview.chromium.org/1454613002

BUG=564506,570636,595986

Committed: https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416
Cr-Commit-Position: refs/heads/master@{#441335}
==========

[commit-bot: I haz the power, 2017-01-04 07:54:03]

Patchset 1 (id:??) landed as
https://crrev.com/421bed44b76d7743b573b4a6b313fa17813f8416
Cr-Commit-Position: refs/heads/master@{#441335}