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

Issue 8890079: Merge 102611 - [chromium] Remove assumption that empty surface is always at end of list (Closed)

Created:
9 years ago by jamesr
Modified:
8 years, 9 months ago
Reviewers:
jamesr
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/963/
Visibility:
Public.

Description

Merge 102611 - [chromium] Remove assumption that empty surface is always at end of list https://bugs.webkit.org/show_bug.cgi?id=74037 Patch by Shawn Singh <shawnsingh@chromium.org>; on 2011-12-12 Reviewed by James Robinson. Source/WebCore: Test case added to CCLayerTreeHostCommonTest.cpp, which reproduces a crash reported in http://code.google.com/p/chromium/issues/detail?id=106734 This patch fixes the crash in a less risky way to be merged into m17, but the root of the issue needs to be addressed in a follow-up patch. * platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp: (WebCore::calculateDrawTransformsAndVisibilityInternal): Source/WebKit/chromium: * tests/CCLayerTreeHostCommonTest.cpp: (WebCore::TEST): BUG=106734 TBR=jamesr@chromium.org

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3202 lines, -3 lines) Patch
M Source/WebCore/ChangeLog View 1 chunk +1883 lines, -0 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp View 1 chunk +10 lines, -3 lines 0 comments Download
M Source/WebKit/chromium/ChangeLog View 1 chunk +1236 lines, -0 lines 0 comments Download
M Source/WebKit/chromium/tests/CCLayerTreeHostCommonTest.cpp View 1 chunk +73 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
jamesr
9 years ago (2011-12-13 18:22:52 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698