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

Issue 8491030: Merge 98392 - LayerRendererChromium::setVisible called needlessly without a visibility change (Closed)

Created:
9 years, 1 month ago by Vangelis Kokkevis
Modified:
9 years, 1 month ago
Reviewers:
Vangelis Kokkevis
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/912/
Visibility:
Public.

Description

Merge 98392 - LayerRendererChromium::setVisible called needlessly without a visibility change https://bugs.webkit.org/show_bug.cgi?id=70766 Patch by Michal Mocny <mmocny@google.com>; on 2011-10-25 Reviewed by Kenneth Russell. Check that visibility has actually changed before calling LayerRendererChromium::setVisible * platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp: (WebCore::CCLayerTreeHostImpl::CCLayerTreeHostImpl): (WebCore::CCLayerTreeHostImpl::setVisible): * platform/graphics/chromium/cc/CCLayerTreeHostImpl.h: TBR=commit-queue@webkit.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=99504

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostImpl.h View 1 chunk +1 line, -0 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp View 2 chunks +5 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Vangelis Kokkevis
9 years, 1 month ago (2011-11-08 00:43:37 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698