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

Issue 2616773002: Propagate kForceGpuMemAvailableMb flag to the renderer. (Closed)

Created:
3 years, 11 months ago by vmpstr
Modified:
3 years, 11 months ago
CC:
achuith+watch_chromium.org, alemate+watch_chromium.org, chromium-reviews, creis+watch_chromium.org, darin-cc_chromium.org, davemoore+watch_chromium.org, jam, nasko+codewatch_chromium.org, oshima+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Propagate kForceGpuMemAvailableMb flag to the renderer. This flag is used in the renderer to override the memory limit for tiles. The code to actually override the memory is already there, but the flag isn't currently being propagated to the renderer. This patch fixes this. R=enne@chromium.org Review-Url: https://codereview.chromium.org/2616773002 Cr-Commit-Position: refs/heads/master@{#443033} Committed: https://chromium.googlesource.com/chromium/src/+/dd0efaefeb00163421f31ed74880d0d9a5c5b241

Patch Set 1 #

Patch Set 2 : fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M chrome/browser/chromeos/login/chrome_restart_request.cc View 1 1 chunk +1 line, -0 lines 0 comments Download
M content/browser/renderer_host/render_process_host_impl.cc View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 16 (7 generated)
vmpstr
Please take a look.
3 years, 11 months ago (2017-01-05 00:20:32 UTC) #2
ccameron
lgtm
3 years, 11 months ago (2017-01-05 00:42:30 UTC) #3
enne (OOO)
lgtm
3 years, 11 months ago (2017-01-05 01:16:45 UTC) #4
vmpstr
achuithb@ could you take a look at chrome/browser/chromeos/login/chrome_restart_request.cc please?
3 years, 11 months ago (2017-01-05 18:58:03 UTC) #5
achuithb
On 2017/01/05 18:58:03, vmpstr wrote: > achuithb@ could you take a look at > chrome/browser/chromeos/login/chrome_restart_request.cc ...
3 years, 11 months ago (2017-01-10 14:54:04 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2616773002/1
3 years, 11 months ago (2017-01-10 19:29:55 UTC) #8
commit-bot: I haz the power
Try jobs failed on following builders: chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_chromium_compile_only_ng/builds/261985) linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, ...
3 years, 11 months ago (2017-01-10 19:45:07 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2616773002/20001
3 years, 11 months ago (2017-01-11 18:58:36 UTC) #13
commit-bot: I haz the power
3 years, 11 months ago (2017-01-11 22:53:26 UTC) #16
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/dd0efaefeb00163421f31ed74880...

Powered by Google App Engine
This is Rietveld 408576698