Skip to content

Speed up hot reload in the framework. - #7647

Merged
Hixie merged 1 commit into
flutter:masterfrom
Hixie:hotreloadspeed
Jan 26, 2017
Merged

Hixie merged 1 commit into
flutter:masterfrom
Hixie:hotreloadspeed

Conversation

@Hixie

@Hixie Hixie commented Jan 26, 2017

Copy link
Copy Markdown
Contributor

cc @abarth for review

cc @johnmccutchan after this, a profile of hot reload doesn't really have any framework hotspots, the framework side is really just the time it takes to walk the whole tree marking it dirty, build the whole tree, and lay out the whole tree. There's other things on the profile, though. For example top thing on the profile is fatstat, and second thing is "[Stub] Subtype1TestCache" which I don't know what to do with. The timeline shows that we spend time roughly in the ratios 40:50:15:10 for walking the tree marking it dirty, rebuilding it, laying it out, and painting it. About half the first part is walking the element tree and the other half is walking the render tree.

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.

You've got more ( and ) than you need here.

@abarth

abarth commented Jan 26, 2017

Copy link
Copy Markdown
Contributor

LGTM

@Hixie
Hixie merged commit 46792dc into flutter:master Jan 26, 2017
@Hixie
Hixie deleted the hotreloadspeed branch January 26, 2017 01:54
@sethladd

Copy link
Copy Markdown
Contributor

thanks for this!

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