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

Issue 10025037: Merge 114191 - [chromium] Remove viewport memory restrictions (Closed)

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

Description

Merge 114191 - [chromium] Remove viewport memory restrictions https://bugs.webkit.org/show_bug.cgi?id=83316 Reviewed by Adrienne Walker. We remove the memory restriction on web pages based on viewport size on desktop. This is causing pages that have complex use of layers to end up hitting our memory limits, and moreso when the window is small. Page complexity is not really a function of the viewport size. We drop the mininum memory limit entirely, and just evict all unprotected textures instead. Covered by existing tests. * platform/graphics/chromium/TextureManager.cpp: (WebCore::TextureManager::highLimitBytes): (WebCore::TextureManager::reclaimLimitBytes): * platform/graphics/chromium/TextureManager.h: (TextureManager): * platform/graphics/chromium/cc/CCLayerTreeHost.cpp: (WebCore::CCLayerTreeHost::beginCommitOnImplThread): (WebCore::CCLayerTreeHost::setVisible): (WebCore::CCLayerTreeHost::didBecomeInvisibleOnImplThread): (WebCore::CCLayerTreeHost::updateLayers): TBR=danakj@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=114642

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -1783 lines) Patch
M Source/WebCore/ChangeLog View 1 chunk +0 lines, -1760 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/TextureManager.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/TextureManager.cpp View 3 chunks +11 lines, -16 lines 0 comments Download
M Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHost.cpp View 4 chunks +6 lines, -5 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
danakj
8 years, 8 months ago (2012-04-19 15:44:08 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698