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

Issue 12406009: Increase the resolution of low-res tiles on Nexus7. (Closed)

Created:
7 years, 9 months ago by Tom Hudson
Modified:
7 years, 9 months ago
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, apatrick_chromium, srikanth, Sami
Visibility:
Public.

Description

Increase the resolution of low-res tiles on Nexus7. This was previously done for Nexus10. Although we'd like to avoid hardcoding it on many platforms, this change is intended to address quality complaints on N7. BUG=177027 R=aelias@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=186945

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M content/gpu/gpu_info_collector_android.cc View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 11 (0 generated)
Tom Hudson
Alexandre, in our meeting early today Srikanth asked me to prepare this patch for MTV ...
7 years, 9 months ago (2013-03-05 17:27:20 UTC) #1
aelias_OOO_until_Jul13
[+wangxianzhu] Xianzhu, could you try cherry-picking this locally and see how it looks?
7 years, 9 months ago (2013-03-05 19:06:39 UTC) #2
Xianzhu
On 2013/03/05 19:06:39, aelias wrote: > [+wangxianzhu] Xianzhu, could you try cherry-picking this locally and ...
7 years, 9 months ago (2013-03-05 19:20:25 UTC) #3
Tom Hudson
After testing the change, there's a clear difference, but it's so transient during flings that ...
7 years, 9 months ago (2013-03-07 18:28:10 UTC) #4
nduca
lgtm. We should probably chat offline about how to get the cc-side of the low ...
7 years, 9 months ago (2013-03-07 19:11:37 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tomhudson@chromium.org/12406009/1
7 years, 9 months ago (2013-03-08 11:14:40 UTC) #6
commit-bot: I haz the power
Presubmit check for 12406009-1 failed and returned exit status 1. INFO:root:Found 1 file(s). Running presubmit ...
7 years, 9 months ago (2013-03-08 11:14:51 UTC) #7
Tom Hudson
Antoine, Gregg, can we get OWNERS from one of you?
7 years, 9 months ago (2013-03-08 11:18:31 UTC) #8
greggman
lgtm
7 years, 9 months ago (2013-03-08 11:29:49 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tomhudson@chromium.org/12406009/1
7 years, 9 months ago (2013-03-08 11:43:56 UTC) #10
commit-bot: I haz the power
7 years, 9 months ago (2013-03-08 13:42:11 UTC) #11
Message was sent while issue was closed.
Change committed as 186945

Powered by Google App Engine
This is Rietveld 408576698