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

Issue 12212143: Distribute extra memory evenly among visible clients (Closed)

Created:
7 years, 10 months ago by ccameron
Modified:
7 years, 10 months ago
Reviewers:
nduca, piman
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, apatrick_chromium
Visibility:
Public.

Description

Distribute extra memory evenly among visible clients After computing all clients' memory budgets, take any extra memory left over and distribute it amongst the visible clients (so that, if their memory requirments suddenly jump, they don't need to wait for a roundtrip to the GPU process to get extra memory). Disallow keeping around backgrounded tabs' contents on Android, to keep the maximum for the main tab. Explicitly limit the memory to use for prepainting by specifying NiceToHave on Mac to avoid performance problems. BUG=175125 TBR=nduca, piman Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182083

Patch Set 1 #

Patch Set 2 : One more missed test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+111 lines, -37 lines) Patch
M content/common/gpu/gpu_memory_manager.h View 1 chunk +1 line, -0 lines 0 comments Download
M content/common/gpu/gpu_memory_manager.cc View 4 chunks +59 lines, -0 lines 0 comments Download
M content/common/gpu/gpu_memory_manager_unittest.cc View 1 10 chunks +51 lines, -37 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ccameron@chromium.org/12212143/1
7 years, 10 months ago (2013-02-12 19:58:53 UTC) #1
commit-bot: I haz the power
Failed to trigger a try job on android_dbg HTTP Error 400: Bad Request
7 years, 10 months ago (2013-02-12 22:23:01 UTC) #2
commit-bot: I haz the power
7 years, 10 months ago (2013-02-12 22:23:07 UTC) #3

Powered by Google App Engine
This is Rietveld 408576698