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

Issue 14588004: Merge https://codereview.chromium.org/14896002/ to M27: Always use a separate front texture on Mac … (Closed)

Created:
7 years, 7 months ago by Ken Russell (switch to Gerrit)
Modified:
7 years, 7 months ago
Reviewers:
jamesr
CC:
blink-reviews, kareng
Visibility:
Public.

Description

Merge https://codereview.chromium.org/14896002/ to M27: Always use a separate front texture on Mac OS to work around graphics driver bugs. Tested manually with scenarios from bug report. BUG=234428 TEST=load 3 copies each of the pages in https://code.google.com/p/chromium/issues/detail?id=234428#c10 (15 tabs total) on Macs with GeForce 320M (MacBook Air), GeForce GT 330M (MacBook Pro), and GeForce GT 650M (MacBook Pro Retina Display), and cycle among them until they all load R=jamesr@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=149790

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -0 lines) Patch
M WebCore/platform/graphics/chromium/DrawingBufferChromium.cpp View 1 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Ken Russell (switch to Gerrit)
Please review the M27 merge for this driver bug workaround.
7 years, 7 months ago (2013-05-03 03:12:52 UTC) #1
jamesr
Merges to 27 are up to Karen, but this looks correct.
7 years, 7 months ago (2013-05-03 04:58:19 UTC) #2
Ken Russell (switch to Gerrit)
7 years, 7 months ago (2013-05-06 20:23:43 UTC) #3
Message was sent while issue was closed.
Committed patchset #2 manually as r149790 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698