Skip to content

Conversation

@amirh
Copy link
Contributor

@amirh amirh commented Jul 1, 2019

The engine haven't rolled for a while, creating a manual roll to include the minimal set of changes needed to revert the #35291 regression.

45b66b722e1275c47dccbe6b002c8e4cb13ea983 Revert " Roll src/third_party/dart b37aa3b036...1eb113ba27 (#9559)" (#9584)
fc1d86b7580bc65a3b816b38d3b4c114079ccdf9 Revert "Avoid a full screen overlay within virtual displays (#9343)" (#9581)
140b2aef5ed381aecc13e262d1e2bb3b70960b3b  Roll src/third_party/dart b37aa3b036...1eb113ba27 (#9559)
94bb7a7ff2a52e9e69b9b4228b5a3ec9fb1e5127 Adds API for retaining intermediate engine layers (#9461)
7d1508d989e682b7fbc1e43db8053e7ded369154 Roll fuchsia/sdk/core/mac-amd64 from cFEtOUFvy2XYSKuSmCMDaMQV8_Y7GOif521L3x86J_4C to n9fZpXVmsbgKZzS8Q2GXxB453q8AbLpfGztsPsq47pMC (#9579)
7a0bbf935599b223352312a20ceabbe2dd2eb3ac Fixed memory leak by way of accidental retain on implicit self (#9329)

Fixes: #35291

```
45b66b7 Revert " Roll src/third_party/dart b37aa3b036...1eb113ba27 (flutter#9559)" (flutter#9584)
fc1d86b Revert "Avoid a full screen overlay within virtual displays (flutter#9343)" (flutter#9581)
140b2ae  Roll src/third_party/dart b37aa3b036...1eb113ba27 (flutter#9559)
94bb7a7 Adds API for retaining intermediate engine layers (flutter#9461)
7d1508d Roll fuchsia/sdk/core/mac-amd64 from cFEtOUFvy2XYSKuSmCMDaMQV8_Y7GOif521L3x86J_4C to n9fZpXVmsbgKZzS8Q2GXxB453q8AbLpfGztsPsq47pMC (flutter#9579)
7a0bbf9 Fixed memory leak by way of accidental retain on implicit self (flutter#9329)
```
@tvolkert
Copy link
Contributor

tvolkert commented Jul 1, 2019

This is running into the same problem that @a-siva is hitting in his Dart roll.

@tvolkert
Copy link
Contributor

tvolkert commented Jul 1, 2019

(and likely the same problem that's blocking the auto-roller)

@tvolkert tvolkert requested a review from a-siva July 1, 2019 16:59
@amirh
Copy link
Contributor Author

amirh commented Jul 1, 2019

Oops, added one more commit (Siva's revert of the dart roll)

@Piinks Piinks added the engine flutter/engine related. See also e: labels. label Jul 1, 2019
@goderbauer goderbauer added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Jul 1, 2019
Copy link
Contributor

@dnfield dnfield left a comment

Choose a reason for hiding this comment

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

LGTM

@amirh amirh merged commit 2fefa8c into flutter:master Jul 1, 2019
@amirh amirh deleted the engine_roll branch July 1, 2019 18:33
johnsonmh pushed a commit to johnsonmh/flutter that referenced this pull request Jul 30, 2019
```
45b66b7 Revert " Roll src/third_party/dart b37aa3b036...1eb113ba27 (flutter#9559)" (flutter#9584)
fc1d86b Revert "Avoid a full screen overlay within virtual displays (flutter#9343)" (flutter#9581)
140b2ae  Roll src/third_party/dart b37aa3b036...1eb113ba27 (flutter#9559)
94bb7a7 Adds API for retaining intermediate engine layers (flutter#9461)
7d1508d Roll fuchsia/sdk/core/mac-amd64 from cFEtOUFvy2XYSKuSmCMDaMQV8_Y7GOif521L3x86J_4C to n9fZpXVmsbgKZzS8Q2GXxB453q8AbLpfGztsPsq47pMC (flutter#9579)
7a0bbf9 Fixed memory leak by way of accidental retain on implicit self (flutter#9329)
```
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt. engine flutter/engine related. See also e: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

WebView is sized (0,0) for some pages on Android

6 participants