This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Remove root_surface_transformation from PaintContext #6213
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Author
|
@chinmaygarde @jason-simmons I probably don't have time to explore how to add a unit test for this until November. Can you please verify the correctness using the real device? Otherwise, I can mark this PR as [WIP] until November. |
jason-simmons
approved these changes
Sep 10, 2018
It should be sufficient to provide the matrix to preroll.
liyuqian
force-pushed
the
rootmatrix
branch
from
4c2e714 to
601286b
Compare
Member
|
Tested this by applying a surface transformation in the Desktop Embedding example app. LGTM. |
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Sep 11, 2018
flutter/engine@9a173a8...7ac3345 git log 9a173a8..7ac3345 --no-merges --oneline 7ac3345 Remove root_surface_transformation from PaintContext (flutter/engine#6213) 565a194 Flutter roll for Dart. (flutter/engine#6227) 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.
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Sep 11, 2018
flutter/engine@9a173a8...838eb3d git log 9a173a8..838eb3d --no-merges --oneline 838eb3d Improve performance of performance overlay by caching. (flutter/engine#6225) 7ac3345 Remove root_surface_transformation from PaintContext (flutter/engine#6213) 565a194 Flutter roll for Dart. (flutter/engine#6227) 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.
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Sep 12, 2018
flutter/engine@9a173a8...4057327 git log 9a173a8..4057327 --no-merges --oneline 4057327 Roll src/third_party/skia bd6595544171..1b5ece0f06f4 (22 commits) (flutter/engine#6229) 838eb3d Improve performance of performance overlay by caching. (flutter/engine#6225) 7ac3345 Remove root_surface_transformation from PaintContext (flutter/engine#6213) 565a194 Flutter roll for Dart. (flutter/engine#6227) 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.
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Sep 12, 2018
flutter/engine@e65beb8...0ded891 git log e65beb8..0ded891 --no-merges --oneline 0ded891 Roll src/third_party/skia 1b5ece0f06f4..e70aed7066c6 (1 commits) (flutter/engine#6231) 4057327 Roll src/third_party/skia bd6595544171..1b5ece0f06f4 (22 commits) (flutter/engine#6229) 838eb3d Improve performance of performance overlay by caching. (flutter/engine#6225) 7ac3345 Remove root_surface_transformation from PaintContext (flutter/engine#6213) 565a194 Flutter roll for Dart. (flutter/engine#6227) 9a173a8 Roll buldroot to a11c4fd (flutter/engine#6224) 51b26f6 Revert "Roll buildroot to eba79bb (flutter#6215)" (flutter/engine#6223) 7be4462 Roll src/third_party/skia 2bf7a7bcc67f..bd6595544171 (4 commits) (flutter/engine#6222) 54fe12b Roll src/third_party/skia 82bf31003c66..2bf7a7bcc67f (1 commits) (flutter/engine#6221) 35ddf87 Roll buildroot to eba79bb (flutter/engine#6215) 6ed00a8 Roll src/third_party/skia 5518e65d90d7..82bf31003c66 (1 commits) (flutter/engine#6220) 8d9e20c Roll src/third_party/skia 3c4d533d8ebd..5518e65d90d7 (1 commits) (flutter/engine#6219) 8f39cac Roll src/third_party/skia 7891994e89a3..3c4d533d8ebd (1 commits) (flutter/engine#6218) e3133e0 Reapply "Some cleanups enabled by removing support for Dart 1." (flutter/engine#6216) 1fb01f3 Roll src/third_party/skia a2bc1ca21bbc..7891994e89a3 (9 commits) (flutter/engine#6217) f19ee56 Roll freetype2 to 6581fd3e9c8645f01c0d51e4f53893f5391f2bf3 (flutter/engine#6214) 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.
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Sep 12, 2018
flutter/engine@e65beb8...6f459e2 git log e65beb8..6f459e2 --no-merges --oneline 6f459e2 Revert "Reapply "Some cleanups enabled by removing support for Dart 1" (flutter#6216)" (flutter/engine#6232) 0ded891 Roll src/third_party/skia 1b5ece0f06f4..e70aed7066c6 (1 commits) (flutter/engine#6231) 4057327 Roll src/third_party/skia bd6595544171..1b5ece0f06f4 (22 commits) (flutter/engine#6229) 838eb3d Improve performance of performance overlay by caching. (flutter/engine#6225) 7ac3345 Remove root_surface_transformation from PaintContext (flutter/engine#6213) 565a194 Flutter roll for Dart. (flutter/engine#6227) 9a173a8 Roll buldroot to a11c4fd (flutter/engine#6224) 51b26f6 Revert "Roll buildroot to eba79bb (flutter#6215)" (flutter/engine#6223) 7be4462 Roll src/third_party/skia 2bf7a7bcc67f..bd6595544171 (4 commits) (flutter/engine#6222) 54fe12b Roll src/third_party/skia 82bf31003c66..2bf7a7bcc67f (1 commits) (flutter/engine#6221) 35ddf87 Roll buildroot to eba79bb (flutter/engine#6215) 6ed00a8 Roll src/third_party/skia 5518e65d90d7..82bf31003c66 (1 commits) (flutter/engine#6220) 8d9e20c Roll src/third_party/skia 3c4d533d8ebd..5518e65d90d7 (1 commits) (flutter/engine#6219) 8f39cac Roll src/third_party/skia 7891994e89a3..3c4d533d8ebd (1 commits) (flutter/engine#6218) e3133e0 Reapply "Some cleanups enabled by removing support for Dart 1." (flutter/engine#6216) 1fb01f3 Roll src/third_party/skia a2bc1ca21bbc..7891994e89a3 (9 commits) (flutter/engine#6217) f19ee56 Roll freetype2 to 6581fd3e9c8645f01c0d51e4f53893f5391f2bf3 (flutter/engine#6214) 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.
amirh
pushed a commit
to amirh/engine
that referenced
this pull request
Sep 21, 2018
It should be sufficient to provide the matrix to preroll.
amirh
pushed a commit
to amirh/engine
that referenced
this pull request
Sep 21, 2018
It should be sufficient to provide the matrix to preroll.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
It should be sufficient to provide the matrix to preroll.