Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use software renderer in embedder unittests #6928

Merged
merged 1 commit into from
Nov 26, 2018

Conversation

cbracken
Copy link
Member

Reduces spurious error log messages in GLContextMakeCurrent() attempting
set up the GR context:

[ERROR:flutter/shell/gpu/gpu_surface_gl.cc(42)] Could not make the context current to setup the gr context.

Reduces spurious error log messages in GLContextMakeCurrent() attempting
set up the GR context:

  [ERROR:flutter/shell/gpu/gpu_surface_gl.cc(42)] Could not make the context current to setup the gr context.
@cbracken
Copy link
Member Author

Primary motivation is killing off the one remaining spurious log message during the embedder_unittests after the fix patch lands, but technically, I suppose this also reduces total code by one line :P

This isn't strictly necessary; tests will pass without it. Any downside to this?

Copy link
Member

@chinmaygarde chinmaygarde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No downsides.

@cbracken cbracken merged commit 25d99cd into flutter:master Nov 26, 2018
@cbracken cbracken deleted the software-renderer branch November 26, 2018 18:48
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 26, 2018
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Nov 26, 2018
) (#24749)

flutter/engine@99e73d8...25d99cd

git log 99e73d8..25d99cd --no-merges --oneline
25d99cd Use software renderer in embedder unittests (flutter/engine#6928)


The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants