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

Issue 1115433002: Increase output quality of scaler from FAST to GOOD (Closed)

Created:
5 years, 7 months ago by Mathieu
Modified:
5 years, 7 months ago
Reviewers:
danakj, hubbe, luken, jbauman
CC:
chromium-reviews, jam, sievers+watch_chromium.org, jbauman+watch_chromium.org, darin-cc_chromium.org, kalyank, piman+watch_chromium.org, danakj+watch_chromium.org, miu+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Increase output quality of scaler from FAST to GOOD The android implementation already uses the GOOD quality, rather than FAST. This positively affects the quality of the thumbnails shown on the New Tab Page. Description of GOOD, from code comments: Bilinear upscale + N * 50% bilinear downscales. This is still fast enough for most purposes and Image quality is nearly as good as the BEST option. BUG=481978 Committed: https://crrev.com/f3f6f746e858929c2a2da07b48e509348f84d2c7 Cr-Commit-Position: refs/heads/master@{#327377}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M content/browser/compositor/delegated_frame_host.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 12 (4 generated)
Mathieu
Hello, please have a look, thanks.
5 years, 7 months ago (2015-04-28 14:30:53 UTC) #2
Mathieu
On 2015/04/28 14:30:53, Mathieu Perreault wrote: > Hello, please have a look, thanks. +luken who ...
5 years, 7 months ago (2015-04-28 14:36:27 UTC) #4
danakj
+hubbe
5 years, 7 months ago (2015-04-28 17:05:42 UTC) #6
hubbe
LGTM I'm not an OWNER, but this seems like a good idea to me. GOOD ...
5 years, 7 months ago (2015-04-28 17:27:16 UTC) #7
jbauman
lgtm
5 years, 7 months ago (2015-04-28 19:16:08 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1115433002/1
5 years, 7 months ago (2015-04-28 20:44:00 UTC) #10
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 7 months ago (2015-04-28 22:24:54 UTC) #11
commit-bot: I haz the power
5 years, 7 months ago (2015-04-28 22:25:39 UTC) #12
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/f3f6f746e858929c2a2da07b48e509348f84d2c7
Cr-Commit-Position: refs/heads/master@{#327377}

Powered by Google App Engine
This is Rietveld 408576698