Skip to content

Add benchmark/test for drawing images across frames#67176

Merged
dnfield merged 1 commit intoflutter:masterfrom
dnfield:canvaskit_images
Oct 2, 2020
Merged

Add benchmark/test for drawing images across frames#67176
dnfield merged 1 commit intoflutter:masterfrom
dnfield:canvaskit_images

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Oct 2, 2020

Would have caught the CanvasKit regression introduced in #66688

@dnfield dnfield requested a review from yjbanov October 2, 2020 23:05
@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@flutter-dashboard flutter-dashboard bot added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Oct 2, 2020
Copy link
Contributor

@yjbanov yjbanov left a comment

Choose a reason for hiding this comment

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

lgtm

dnfield added a commit to dnfield/flutter that referenced this pull request Oct 2, 2020
Changes since last time:

- Test for CanvasKit image rendering
  (flutter#67176)
- Fix CanvasKit dispose impl
  (flutter/engine#21555)
- Update internal google3 customer with a problematic ImageStream
  Listener impl (cl/335091311)

This reverts commit 473358d.
@dnfield dnfield merged commit 33fb35e into flutter:master Oct 2, 2020
@dnfield dnfield deleted the canvaskit_images branch October 2, 2020 23:41
@liyuqian liyuqian added the c: performance Relates to speed or footprint issues (see "perf:" labels) label Oct 3, 2020
@liyuqian
Copy link
Contributor

liyuqian commented Oct 3, 2020

Does this need to be run in the devicelab and report the metrics?

dnfield added a commit that referenced this pull request Oct 5, 2020
* Reland dispose images when done (#67100)

Changes since last time:

- Test for CanvasKit image rendering
  (#67176)
- Fix CanvasKit dispose impl
  (flutter/engine#21555)
- Update internal google3 customer with a problematic ImageStream
  Listener impl (cl/335091311, cl/335459002)

This reverts commit 473358d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt. c: performance Relates to speed or footprint issues (see "perf:" labels)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants