Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(118)

Issue 2363553005: Enable zoom-for-dsf on Linux by default (Closed)

Created:
4 years, 3 months ago by malaykeshav
Modified:
4 years, 2 months ago
CC:
chromium-reviews, darin-cc_chromium.org, jam
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Enable zoom-for-dsf on Linux by default BUG=640256 COMPONENT=HiDPI Displays, Content Switch, Linux Committed: https://crrev.com/cfa4d68b021ddec1b1535d37179b16af29101331 Cr-Commit-Position: refs/heads/master@{#422266}

Patch Set 1 #

Patch Set 2 : Rebaseline Layouttests #

Patch Set 3 : Sync with ToT and resolved conflicts #

Patch Set 4 : Enable zoom-for-dsf on Linux by default #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -4 lines) Patch
M content/common/content_switches_internal.cc View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/LayoutTests/TestExpectations View 1 2 3 2 chunks +27 lines, -3 lines 0 comments Download

Messages

Total messages: 36 (20 generated)
malaykeshav
PTAL before I can update the layout test expectations.
4 years, 2 months ago (2016-09-27 00:03:57 UTC) #6
malaykeshav
ping
4 years, 2 months ago (2016-09-27 21:42:50 UTC) #7
oshima
don't you have to change TestExpectation to rebaseline the expectations?
4 years, 2 months ago (2016-09-27 23:30:29 UTC) #8
malaykeshav
On 2016/09/27 at 23:30:29, oshima wrote: > don't you have to change TestExpectation to rebaseline ...
4 years, 2 months ago (2016-09-28 17:37:33 UTC) #9
oshima
lgtm
4 years, 2 months ago (2016-09-28 21:17:10 UTC) #10
malaykeshav
PTAL
4 years, 2 months ago (2016-09-29 06:23:15 UTC) #16
malaykeshav
4 years, 2 months ago (2016-09-29 17:34:03 UTC) #18
malaykeshav
Ping
4 years, 2 months ago (2016-09-30 01:43:43 UTC) #19
nasko
LGTM
4 years, 2 months ago (2016-09-30 20:26:31 UTC) #20
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2363553005/20001
4 years, 2 months ago (2016-09-30 20:34:36 UTC) #23
commit-bot: I haz the power
Try jobs failed on following builders: ios-device on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds/78360) ios-simulator on master.tryserver.chromium.mac (JOB_FAILED, ...
4 years, 2 months ago (2016-09-30 20:37:26 UTC) #25
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2363553005/40001
4 years, 2 months ago (2016-09-30 20:48:33 UTC) #28
commit-bot: I haz the power
Try jobs failed on following builders: ios-device on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds/78374) ios-simulator on master.tryserver.chromium.mac (JOB_FAILED, ...
4 years, 2 months ago (2016-09-30 20:51:19 UTC) #30
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2363553005/60001
4 years, 2 months ago (2016-09-30 22:05:09 UTC) #33
commit-bot: I haz the power
Committed patchset #4 (id:60001)
4 years, 2 months ago (2016-10-01 01:54:47 UTC) #34
commit-bot: I haz the power
4 years, 2 months ago (2016-10-01 01:58:02 UTC) #36
Message was sent while issue was closed.
Patchset 4 (id:??) landed as
https://crrev.com/cfa4d68b021ddec1b1535d37179b16af29101331
Cr-Commit-Position: refs/heads/master@{#422266}

Powered by Google App Engine
This is Rietveld 408576698