I took a quick peek at the crash since that crashtest is using Grid. I think it's an unrelated crash in Graphics code. When I run the test locally with the patch applied then I see:
> [GFX3-]: Surface size too large (exceeds extent limit)!
in the terminal but no crash. I suspect that the OOM on Android is related to that though.
So I think we can disable this crashtest on Android and file a follow-up bug (in Web Painting) on that, and then re-land this fix (which looks correct to me).
Bug 1584890 Comment 19 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
I took a quick peek at the crash since that crashtest is using Grid. I think it's an unrelated crash in Graphics code. When I run the test locally with the patch applied then I see:
> [GFX3-]: Surface size too large (exceeds extent limit)!
in the terminal but no crash. I suspect that the OOM on Android is related to that though.
So I think we can disable this crashtest on Android and file a follow-up bug (in Web Painting) on that, and then re-land this fix (which looks correct to me).