Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

ferhatb
Copy link
Contributor

@ferhatb ferhatb commented Feb 14, 2020

drawRRect will use DomCanvas when the corners are uniform. However DomCanvas doesn't support shaders yet therefore throws an exception. This change adds a check to mark the picture recording as hasArbitraryPaint=true now and fixes the issue observed in dialogs and other rounded rects.

Fixes flutter/flutter#50010

@ferhatb
Copy link
Contributor Author

ferhatb commented Feb 14, 2020

Merging on infra fail.

@ferhatb ferhatb merged commit cd77e78 into flutter:master Feb 14, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Feb 15, 2020
jason-simmons pushed a commit to flutter/flutter that referenced this pull request Feb 18, 2020
* c0549fb Roll buildroot. (flutter/engine#16613)

* 8b0b649 improve surface state assert error messages (flutter/engine#16595)

* cd77e78 Fix drawRRect failure when shader is specified (flutter/engine#16601)

* fe63094 [web] Handle alignment correctly in Paragraph.getPositionForOffset (flutter/engine#16569)

* 65d1126 [web] Fixing launching Safari. This should solve the LUCI issue (flutter/engine#16590)

* f88f7df [web] Unskip tests that are already passing in Safari (flutter/engine#16567)

* 594f660 [shell tests] Integrate Vulkan with Shell Tests

* 400ed7c Revert "[shell tests] Integrate Vulkan with Shell Tests"

* 15e7f51 Implement Path extractPath, tangent APIs (flutter/engine#16599)

* 4941ff7 Remove usage of Dart_AllocateWithNativeFields from tonic (flutter/engine#16588)

* bb01cb7 Roll fuchsia/sdk/core/linux-amd64 from Bmq1m... to J-_s6... (flutter/engine#16592)
NoamDev pushed a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
NoamDev added a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
@ferhatb ferhatb deleted the dialog_gradient_fail branch March 10, 2020 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[web] Dialog with gradient and scroll fails
3 participants