Page popups should respect zoom factor of the anchor element.

Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the following popups:
 - Calendar popups for input[type=date], input[type=month], input[type=
   datetime-local], and input[type=week]
 - Suggestion popups for them and input[type=time].
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects the zoom level.

Note: Updates of existing tests:

* popup-menu-appearance-styled.html
  Because we pass a font-size in float, not int.

* popup-menu-appearance-zoom.html
  The popup border is correctly zoomed.
  The scrollbar is unexpected and I couldn't find the reason why
  the scrollbar was shown.  However, chrome and content_shell
  doesn't show such scrollbar.  So, this is not a production
  issue.

* popup-menu-load-webfont-after-open.html
  Because we pass a font-size in float, not int.
  The popup position in -expected.png is an error of my local
  machine, I think.

BUG=555326
Committed: https://crrev.com/575d1deb85ba41095599809f83dc08ffce9fd443
Cr-Commit-Position: refs/heads/master@{#361756}

[tkent, 2015-11-17 05:27:39]

Description was changed from

==========
Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the follwoing popups:
 - Calendar popups for input[type=date], input[type=month],
input[type=datetime-local]
 - Suggestion popups for them.
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects to the zoom level.

BUG=555326
==========

to

==========
Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the follwoing popups:
 - Calendar popups for input[type=date], input[type=month], input[type=
   datetime-local], and input[type=week]
 - Suggestion popups for them.
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects to the zoom level.

BUG=555326
==========

[tkent, 2015-11-17 05:30:31]

Description was changed from

==========
Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the follwoing popups:
 - Calendar popups for input[type=date], input[type=month], input[type=
   datetime-local], and input[type=week]
 - Suggestion popups for them.
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects to the zoom level.

BUG=555326
==========

to

==========
Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the follwoing popups:
 - Calendar popups for input[type=date], input[type=month], input[type=
   datetime-local], and input[type=week]
 - Suggestion popups for them and input[type=time].
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects to the zoom level.

BUG=555326
==========

[tkent, 2015-11-18 06:28:36]

Description was changed from

==========
Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the follwoing popups:
 - Calendar popups for input[type=date], input[type=month], input[type=
   datetime-local], and input[type=week]
 - Suggestion popups for them and input[type=time].
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects to the zoom level.

BUG=555326
==========

to

==========
Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the following popups:
 - Calendar popups for input[type=date], input[type=month], input[type=
   datetime-local], and input[type=week]
 - Suggestion popups for them and input[type=time].
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects to the zoom level.

Note updates of existing tests:

* popup-menu-appearance-styled.html
  Because we pass a font-size in float, not int.

* popup-menu-appearance-zoom.html
  The popup border is correctly zoomed.
  The scrollbar is unexpected and I couldn't find the reason why
  the scrollbar was shown.  However, chrome and content_shell
  doesn't show such scrollbar.  So, this is not a production
  issue.

* popup-menu-load-webfont-after-open.html
  Because we pass a font-size in float, not int.
  The popup position in -expected.png is an error of my local
  machine, I think.

BUG=555326
==========

[tkent, 2015-11-18 06:29:04]

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

[tkent, 2015-11-18 06:29:05]

keishi, oshima, please review this.

[keishi, 2015-11-18 07:31:20]

Should we have tests with fractional scale factors like 1.25 used on some cros
devices?

[tkent, 2015-11-18 07:34:26]

Description was changed from

==========
Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the following popups:
 - Calendar popups for input[type=date], input[type=month], input[type=
   datetime-local], and input[type=week]
 - Suggestion popups for them and input[type=time].
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects to the zoom level.

Note updates of existing tests:

* popup-menu-appearance-styled.html
  Because we pass a font-size in float, not int.

* popup-menu-appearance-zoom.html
  The popup border is correctly zoomed.
  The scrollbar is unexpected and I couldn't find the reason why
  the scrollbar was shown.  However, chrome and content_shell
  doesn't show such scrollbar.  So, this is not a production
  issue.

* popup-menu-load-webfont-after-open.html
  Because we pass a font-size in float, not int.
  The popup position in -expected.png is an error of my local
  machine, I think.

BUG=555326
==========

to

==========
Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the following popups:
 - Calendar popups for input[type=date], input[type=month], input[type=
   datetime-local], and input[type=week]
 - Suggestion popups for them and input[type=time].
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects the zoom level.

Note: Updates of existing tests:

* popup-menu-appearance-styled.html
  Because we pass a font-size in float, not int.

* popup-menu-appearance-zoom.html
  The popup border is correctly zoomed.
  The scrollbar is unexpected and I couldn't find the reason why
  the scrollbar was shown.  However, chrome and content_shell
  doesn't show such scrollbar.  So, this is not a production
  issue.

* popup-menu-load-webfont-after-open.html
  Because we pass a font-size in float, not int.
  The popup position in -expected.png is an error of my local
  machine, I think.

BUG=555326
==========

[tkent, 2015-11-18 08:30:36]

On 2015/11/18 at 07:31:20, keishi wrote:
> Should we have tests with fractional scale factors like 1.25 used on some cros
devices?

That's a good idea.  What are important scale factor values in desktop
platforms?

[tkent, 2015-11-24 02:19:30]

> > Should we have tests with fractional scale factors like 1.25 used on some
cros devices?
> 
> That's a good idea.  What are important scale factor values in desktop
platforms?

Oshima-san, do you know popular device scale factors of CrOS devices?

[oshima, 2015-11-24 02:31:29]

On 2015/11/24 02:19:30, tkent wrote:
> > > Should we have tests with fractional scale factors like 1.25 used on some
> cros devices?
> > 
> > That's a good idea.  What are important scale factor values in desktop
> platforms?
> 
> Oshima-san, do you know popular device scale factors of CrOS devices?

We currently have 1, 1.25 and 2x scale factors.

[tkent, 2015-11-25 04:38:12]

Patchset #1 (id:1) has been deleted

[tkent, 2015-11-25 04:38:21]

Patchset #1 (id:20001) has been deleted

[tkent, 2015-11-25 04:39:03]

New patch set contains tests for both of 2.00 and 1.25.
Please take another look.

[keishi, 2015-11-25 07:42:42]

LGTM

[tkent, 2015-11-25 22:54:46]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2015-11-25 22:58:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1454613002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1454613002/60001

[commit-bot: I haz the power, 2015-11-26 00:29:01]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2015-11-26 00:31:01]

Description was changed from

==========
Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the following popups:
 - Calendar popups for input[type=date], input[type=month], input[type=
   datetime-local], and input[type=week]
 - Suggestion popups for them and input[type=time].
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects the zoom level.

Note: Updates of existing tests:

* popup-menu-appearance-styled.html
  Because we pass a font-size in float, not int.

* popup-menu-appearance-zoom.html
  The popup border is correctly zoomed.
  The scrollbar is unexpected and I couldn't find the reason why
  the scrollbar was shown.  However, chrome and content_shell
  doesn't show such scrollbar.  So, this is not a production
  issue.

* popup-menu-load-webfont-after-open.html
  Because we pass a font-size in float, not int.
  The popup position in -expected.png is an error of my local
  machine, I think.

BUG=555326
==========

to

==========
Page popups should respect zoom factor of the anchor element.

If the element is in non-100% zoom level, popups associated to the element
should be in the same zoom level.

This CL affects the following popups:
 - Calendar popups for input[type=date], input[type=month], input[type=
   datetime-local], and input[type=week]
 - Suggestion popups for them and input[type=time].
 - Color suggestion popup

Note that the behavior of select popups on Windows, Linux, and CrOs should not
be changed by this CL because it already respects the zoom level.

Note: Updates of existing tests:

* popup-menu-appearance-styled.html
  Because we pass a font-size in float, not int.

* popup-menu-appearance-zoom.html
  The popup border is correctly zoomed.
  The scrollbar is unexpected and I couldn't find the reason why
  the scrollbar was shown.  However, chrome and content_shell
  doesn't show such scrollbar.  So, this is not a production
  issue.

* popup-menu-load-webfont-after-open.html
  Because we pass a font-size in float, not int.
  The popup position in -expected.png is an error of my local
  machine, I think.

BUG=555326

Committed: https://crrev.com/575d1deb85ba41095599809f83dc08ffce9fd443
Cr-Commit-Position: refs/heads/master@{#361756}
==========

[commit-bot: I haz the power, 2015-11-26 00:31:02]

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