Repository navigation
Conversation
…load The map was only reported as ready on the first 'tilesloaded' event, which never fires if a tile request stalls or fails. Also accept the first 'idle' event once the map div is laid out. Integration tests now time out waiting for the map instead of hanging until the driver fails with an opaque DriverError. Fixes flutter/flutter#193452
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request modifies the web implementation of the Google Maps Flutter plugin to ensure that onMapCreated is called even if a map tile fails to load. It updates _attachMapEvents in GoogleMapController to trigger the map-ready event using either the first tilesloaded event or the first idle event after the map's div is connected and has a positive width. Additionally, integration tests are updated to use a new helper function, waitForMap, which adds a 30-second timeout to map-creation futures to prevent test hangs. Feedback suggests also checking offsetHeight in addition to offsetWidth to guarantee the map is fully laid out and usable.
Merged
11 tasks done
Records map lifecycle events, layout, pending tiles and Maps network activity; dumps them when waitForMap times out, and adds a stress test that measures idle vs tilesloaded across many fresh maps. For flutter/flutter#193452
mdebbar
changed the title
onMapCreated when tiles fail to load
Contributor
Author
|
To check whether this actually removes the hang rather than just making it rarer, I ran the Results: 59 builds, 309 cold-start build-equivalents, 2,880 instrumented Caveat: the historical failures cluster by date (3 on 2026-09-25), so the builds are not fully independent and this is an upper bound on confidence. I also never caught a stalled All builds: All 59 shard_1 builds
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The web plugin reported a map as ready (triggering
onMapCreated) only on the firsttilesloadedevent. That event requires every visible tile to download, so a single stalled or failed tile request leavesonMapCreatedpending forever. On CI, this hangsprojection_test,overlays_test, andmarker_clustering_testuntilflutter drivefails with an opaqueDriverErrorafter 20 minutes.Example CI failures:
https://ci.chromium.org/b/8669392206413648801
https://ci.chromium.org/b/8669672566864354113
Report the map as ready on the first
tilesloadedor the firstidleevent once the map<div>is attached and has a non-zero size, when the projection and bounds are available.Re-enable the tests skipped in [google_maps_flutter] Disable flaky web tests blocking latest flutter -> packages roll #13047.
Fixes flutter/flutter#193452