Skip to content

Conversation

@liyuqian
Copy link
Contributor

@liyuqian liyuqian commented Aug 4, 2018

ecbb2b2 Call drawPaint instead of drawPath if there's clip (flutter/engine#5937)
597a508 Roll src/third_party/skia cdfa16d086b8..4c3b63e289c5 (9 commits) (flutter/engine#5936)
715b64e Roll src/third_party/skia 1400d38e0863..cdfa16d086b8 (10 commits) (flutter/engine#5934)
ffbafc8 Flush all embedded Android views on hot restart. (flutter/engine#5929)
9fe6a57 Roll src/third_party/skia 578ef2847b72..1400d38e0863 (7 commits) (flutter/engine#5933)
3b66f20 Don't drop MotionEvents with unknown tool type. (flutter/engine#5931)
391ac2f Roll src/third_party/skia 64cc576b1fa7..578ef2847b72 (20 commits) (flutter/engine#5930)

Goldens:
64b7a3a Add updated golden files
cb1fa8a Update golden files of PhysicalModel and PhysicalShape
3baed8d Add a goldens package for the embedded Android views integration test.

@liyuqian liyuqian force-pushed the engine_roll_97aea093e6bc0437eede2a0340c3474f2085fc61 branch from 6fe1931 to ebcd526 Compare August 6, 2018 16:01
@amirh
Copy link
Contributor

amirh commented Aug 6, 2018

nit: I missed that this is a goldens roll as well, maybe include in the PR title roll engine and goldens?
nit2: add a git rev-list for the goldens roll as well to the description

Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

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

LGTM

@liyuqian liyuqian changed the title Roll engine to 97aea093e6bc0437eede2a0340c3474f2085fc61 Update goldens and roll engine to 97aea093e6bc0437eede2a0340c3474f2085fc61 Aug 6, 2018
@liyuqian
Copy link
Contributor Author

liyuqian commented Aug 6, 2018

@amirh Done, thanks!

ecbb2b2 Call drawPaint instead of drawPath if there's clip (flutter/engine#5937)
597a508 Roll src/third_party/skia cdfa16d086b8..4c3b63e289c5 (9 commits) (flutter/engine#5936)
715b64e Roll src/third_party/skia 1400d38e0863..cdfa16d086b8 (10 commits) (flutter/engine#5934)
ffbafc8 Flush all embedded Android views on hot restart. (flutter/engine#5929)
9fe6a57 Roll src/third_party/skia 578ef2847b72..1400d38e0863 (7 commits) (flutter/engine#5933)
3b66f20 Don't drop MotionEvents with unknown tool type. (flutter/engine#5931)
391ac2f Roll src/third_party/skia 64cc576b1fa7..578ef2847b72 (20 commits) (flutter/engine#5930)

Goldens:
64b7a3a Add updated golden files
cb1fa8a Update golden files of PhysicalModel and PhysicalShape
3baed8d Add a goldens package for the embedded Android views integration test.
@liyuqian liyuqian force-pushed the engine_roll_97aea093e6bc0437eede2a0340c3474f2085fc61 branch from ebcd526 to f01f69c Compare August 6, 2018 17:04
@liyuqian liyuqian changed the title Update goldens and roll engine to 97aea093e6bc0437eede2a0340c3474f2085fc61 Update goldens and roll engine to ecbb2b28d48a3854aba846da38053cf381cced25 Aug 6, 2018
@liyuqian liyuqian merged commit 8b0a3be into flutter:master Aug 6, 2018
@Hixie
Copy link
Contributor

Hixie commented Aug 7, 2018

This roll caused major performance regressions.

@liyuqian
Copy link
Contributor Author

liyuqian commented Aug 7, 2018

@Hixie yes, it should be fixed by flutter/engine#5952 . I'll discuss the tests with you tomorrow.

@liyuqian liyuqian deleted the engine_roll_97aea093e6bc0437eede2a0340c3474f2085fc61 branch August 27, 2018 23:18
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants