Clear media query rulesets on page zoom changes.

Clear media query rulesets on page zoom changes.

Otherwise, resolution media queries won't update properly when page
zoom changes. Changing the deviceScaleFactor already did this. Also,
matchMedia listeners already worked.

BUG=617095
Committed: https://crrev.com/82a43d5e94ad2b873fa14f11a4c5f34c98863069
Cr-Commit-Position: refs/heads/master@{#397697}

[rune, 2016-06-03 09:44:05]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-06-03 09:44:12]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2038793002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2038793002/1

[rune, 2016-06-03 09:49:06]

Description was changed from

==========
Clear media query rulesets on page zoom changes.

Otherwise, resolution media queries won't update properly when page
zoom changes. Changing the deviceScaleFactor already did this.

BUG=617095
==========

to

==========
Clear media query rulesets on page zoom changes.

Otherwise, resolution media queries won't update properly when page
zoom changes. Changing the deviceScaleFactor already did this. Also,
matchMedia listeners already worked.

BUG=617095
==========

[rune, 2016-06-03 09:49:30]

rune@opera.com changed reviewers:
+ yoav@yoav.ws

[rune, 2016-06-03 09:49:31]

PTAL

[Yoav Weiss, 2016-06-03 10:15:49]

LGTM

https://codereview.chromium.org/2038793002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/fast/media/resolution-change-on-page-zoom.html
(right):

https://codereview.chromium.org/2038793002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/media/resolution-change-on-page-zoom.html:2:
<script src="../../resources/js-test.js"></script>
Could you rewrite the test using testharness? It's cleaner and would make the
expected file redundant

[rune, 2016-06-03 11:09:08]

https://codereview.chromium.org/2038793002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/fast/media/resolution-change-on-page-zoom.html
(right):

https://codereview.chromium.org/2038793002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/media/resolution-change-on-page-zoom.html:2:
<script src="../../resources/js-test.js"></script>
On 2016/06/03 10:15:49, Yoav Weiss wrote:
> Could you rewrite the test using testharness? It's cleaner and would make the
> expected file redundant

Done.

[rune, 2016-06-03 11:31:53]

The CQ bit was checked by rune@opera.com

[rune, 2016-06-03 11:31:54]

The patchset sent to the CQ was uploaded after l-g-t-m from yoav@yoav.ws
Link to the patchset: https://codereview.chromium.org/2038793002/#ps20001
(title: "Use testharness.js")

[commit-bot: I haz the power, 2016-06-03 11:32:01]

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

[commit-bot: I haz the power, 2016-06-03 13:23:35]

Description was changed from

==========
Clear media query rulesets on page zoom changes.

Otherwise, resolution media queries won't update properly when page
zoom changes. Changing the deviceScaleFactor already did this. Also,
matchMedia listeners already worked.

BUG=617095
==========

to

==========
Clear media query rulesets on page zoom changes.

Otherwise, resolution media queries won't update properly when page
zoom changes. Changing the deviceScaleFactor already did this. Also,
matchMedia listeners already worked.

BUG=617095
==========

[commit-bot: I haz the power, 2016-06-03 13:23:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-03 13:24:41]

Description was changed from

==========
Clear media query rulesets on page zoom changes.

Otherwise, resolution media queries won't update properly when page
zoom changes. Changing the deviceScaleFactor already did this. Also,
matchMedia listeners already worked.

BUG=617095
==========

to

==========
Clear media query rulesets on page zoom changes.

Otherwise, resolution media queries won't update properly when page
zoom changes. Changing the deviceScaleFactor already did this. Also,
matchMedia listeners already worked.

BUG=617095

Committed: https://crrev.com/82a43d5e94ad2b873fa14f11a4c5f34c98863069
Cr-Commit-Position: refs/heads/master@{#397697}
==========

[commit-bot: I haz the power, 2016-06-03 13:24:43]

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