Skip to content

Cleanup DevFS support and add unit test - #4960

Closed
johnmccutchan wants to merge 1 commit into
flutter:masterfrom
johnmccutchan:devfs_cleanup_and_tests
Closed

johnmccutchan wants to merge 1 commit into
flutter:masterfrom
johnmccutchan:devfs_cleanup_and_tests

Conversation

@johnmccutchan

Copy link
Copy Markdown
Contributor
  • Move devfs support into separate file
  • Adapt to new interface
  • Add unit test

@johnmccutchan

Copy link
Copy Markdown
Contributor Author

@devoncarew @sethladd

@johnmccutchan

Copy link
Copy Markdown
Contributor Author

Mostly done but still some WIP as I need to add more tests and fix a performance problem.

@devoncarew

devoncarew commented Jul 18, 2016 •

Copy link
Copy Markdown
Contributor

@johnmccutchan, can you merge to master - I committed changes in this area this morning - and reformat. It looks like you have some odd indents going on in these files. I'll review once you're ready.

@johnmccutchan

Copy link
Copy Markdown
Contributor Author

I'm going to close this pull request. I ended up getting hot reloading working on Android devices. Will send a more complete pull request tomorrow.

aam added a commit to aam/flutter that referenced this pull request Apr 11, 2018
Changes since last roll:
```
ed303c6 Roll dart sdk again. Previous roll required 23ae4fa098 revert. (flutter#4966)
8cd2727 Revert "Roll dart to 7764e6962e22afcf4b58c4e3cef3147330f3c884. (flutter#4960)" (flutter#4965)
9199b40 Revert "Support multiple shells in a single process. (flutter#4932)" (flutter#4964)
6baff4c Support multiple shells in a single process. (flutter#4932)
31c5bb4 Roll dart to 7764e6962e22afcf4b58c4e3cef3147330f3c884. (flutter#4960)
c8e4c69 Avoid copying the contents of large platform message responses (flutter#4947)
5ff5272 Update to use new vulkan GrBackendRenderTarget ctor. (flutter#4962)
0c8993a Update to use new vulkan GrBackendRenderTarget ctor (part 2) (flutter#4963)
132ebdd Revert "Roll src/third_party/skia/ 9874bf1bc..52e16d984 (135 commits) (flutter#4958)" (flutter#4961)
11882ab Roll src/third_party/skia/ 9874bf1bc..52e16d984 (135 commits) (flutter#4958)
```

Add consts
aam added a commit that referenced this pull request Apr 11, 2018
Changes since last roll:
```
ed303c6 Roll dart sdk again. Previous roll required 23ae4fa098 revert. (#4966)
8cd2727 Revert "Roll dart to 7764e6962e22afcf4b58c4e3cef3147330f3c884. (#4960)" (#4965)
9199b40 Revert "Support multiple shells in a single process. (#4932)" (#4964)
6baff4c Support multiple shells in a single process. (#4932)
31c5bb4 Roll dart to 7764e6962e22afcf4b58c4e3cef3147330f3c884. (#4960)
c8e4c69 Avoid copying the contents of large platform message responses (#4947)
5ff5272 Update to use new vulkan GrBackendRenderTarget ctor. (#4962)
0c8993a Update to use new vulkan GrBackendRenderTarget ctor (part 2) (#4963)
132ebdd Revert "Roll src/third_party/skia/ 9874bf1bc..52e16d984 (135 commits) (#4958)" (#4961)
11882ab Roll src/third_party/skia/ 9874bf1bc..52e16d984 (135 commits) (#4958)
```

Add consts
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request May 14, 2018
Changes since last roll:
```
ed303c6 Roll dart sdk again. Previous roll required 23ae4fa098 revert. (flutter#4966)
8cd2727 Revert "Roll dart to 7764e6962e22afcf4b58c4e3cef3147330f3c884. (flutter#4960)" (flutter#4965)
9199b40 Revert "Support multiple shells in a single process. (flutter#4932)" (flutter#4964)
6baff4c Support multiple shells in a single process. (flutter#4932)
31c5bb4 Roll dart to 7764e6962e22afcf4b58c4e3cef3147330f3c884. (flutter#4960)
c8e4c69 Avoid copying the contents of large platform message responses (flutter#4947)
5ff5272 Update to use new vulkan GrBackendRenderTarget ctor. (flutter#4962)
0c8993a Update to use new vulkan GrBackendRenderTarget ctor (part 2) (flutter#4963)
132ebdd Revert "Roll src/third_party/skia/ 9874bf1bc..52e16d984 (135 commits) (flutter#4958)" (flutter#4961)
11882ab Roll src/third_party/skia/ 9874bf1bc..52e16d984 (135 commits) (flutter#4958)
```

Add consts
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 15, 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.

2 participants