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

Issue 827163002: Fix surfaces scheduler so apps can draw (Closed)

Created:
5 years, 11 months ago by jamesr
Modified:
5 years, 11 months ago
Reviewers:
abarth-chromium
CC:
mojo-reviews_chromium.org, yzshen+watch_chromium.org
Base URL:
git@github.com:domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Fix surfaces scheduler so apps can draw The cc scheduler expects to go through at least one commit before the output surface is ready to draw, so even though we don't commit in the surfaces service we need to pretend we do to get the scheduler into a drawable state. This fixes the regression from https://codereview.chromium.org/822713002/. TBR=abarth@chromium.org Committed: https://chromium.googlesource.com/external/mojo/+/2bee8151adeaf2fc942b6f854816ebcbbb818d85

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M services/surfaces/surfaces_scheduler.cc View 2 chunks +3 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jamesr
This is another argument for not using the cc scheduler, perhaps, but I think we ...
5 years, 11 months ago (2014-12-30 00:36:49 UTC) #1
jamesr
Committed patchset #1 (id:1) manually as 2bee8151adeaf2fc942b6f854816ebcbbb818d85 (presubmit successful).
5 years, 11 months ago (2014-12-30 00:44:22 UTC) #2
abarth-chromium
5 years, 11 months ago (2014-12-30 00:53:30 UTC) #3
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698