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

Issue 788143005: Disable MSAA for Intel BayTrail on Android (Closed)

Created:
6 years ago by Yang Gu
Modified:
6 years ago
CC:
chromium-reviews, piman+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Disable MSAA for Intel BayTrail on Android The GFX (GEN 7) in Intel BayTrail has poor performance with MSAA, due to some hardware architectural limitation. We tend to think this GFX is not suitable for MSAA, so this CL would disable it. See more details in related issue. BUG=443517 Committed: https://crrev.com/b5070888983a515c4f93e87269fe2c960ba773f5 Cr-Commit-Position: refs/heads/master@{#309389}

Patch Set 1 #

Total comments: 1

Patch Set 2 : Update version to 7.13 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -1 line) Patch
M gpu/config/gpu_driver_bug_list_json.cc View 1 2 chunks +14 lines, -1 line 0 comments Download

Messages

Total messages: 9 (2 generated)
Yang Gu
Please have a review, thanks!
6 years ago (2014-12-18 08:19:14 UTC) #2
Zhenyao Mo
https://codereview.chromium.org/788143005/diff/1/gpu/config/gpu_driver_bug_list_json.cc File gpu/config/gpu_driver_bug_list_json.cc (right): https://codereview.chromium.org/788143005/diff/1/gpu/config/gpu_driver_bug_list_json.cc#newcode22 gpu/config/gpu_driver_bug_list_json.cc:22: "version": "7.12", Please update the version to 7.13
6 years ago (2014-12-18 17:44:26 UTC) #3
Yang Gu
On 2014/12/18 17:44:26, Zhenyao Mo wrote: > https://codereview.chromium.org/788143005/diff/1/gpu/config/gpu_driver_bug_list_json.cc > File gpu/config/gpu_driver_bug_list_json.cc (right): > > https://codereview.chromium.org/788143005/diff/1/gpu/config/gpu_driver_bug_list_json.cc#newcode22 ...
6 years ago (2014-12-19 01:52:10 UTC) #4
Zhenyao Mo
On 2014/12/19 01:52:10, Yang Gu wrote: > On 2014/12/18 17:44:26, Zhenyao Mo wrote: > > ...
6 years ago (2014-12-19 14:54:03 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/788143005/20001
6 years ago (2014-12-22 01:42:03 UTC) #7
commit-bot: I haz the power
Committed patchset #2 (id:20001)
6 years ago (2014-12-22 03:33:45 UTC) #8
commit-bot: I haz the power
6 years ago (2014-12-22 03:34:36 UTC) #9
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/b5070888983a515c4f93e87269fe2c960ba773f5
Cr-Commit-Position: refs/heads/master@{#309389}

Powered by Google App Engine
This is Rietveld 408576698