Skip to content

Framework support for checkerboardOffscreenLayers - #10054

Merged
jason-simmons merged 1 commit into
flutter:masterfrom
jason-simmons:save_layer_checker
May 12, 2017
Merged

jason-simmons merged 1 commit into
flutter:masterfrom
jason-simmons:save_layer_checker

Conversation

@jason-simmons

Copy link
Copy Markdown
Contributor

See #9473

@jason-simmons

Copy link
Copy Markdown
Contributor Author

@chinmaygarde

@abarth

abarth commented May 12, 2017

Copy link
Copy Markdown
Contributor

LGTM

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you include a paragraph that discusses how this differs from checkerboardRasterCacheImages?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please include all the information from the render layer here (also on checkerboardRasterCacheImages while we're at it).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this class should also have the same documentation as the other classes.

@Hixie

Hixie commented May 12, 2017

Copy link
Copy Markdown
Contributor

Is this testable?

@jason-simmons
jason-simmons merged commit 91b1a07 into flutter:master May 12, 2017
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 13, 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