Closed Bug 1154952 Opened 5 years ago Closed 5 years ago

Fix Gecko shutdown assertions, bugs, etc

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 40
Tracking Status
firefox40 --- fixed

People

(Reporter: jchen, Assigned: jchen)

Details

Attachments

(3 files)

Right now we run into some assertions, bugs, etc. when Gecko is shutting down.
The two free() calls free the same memory. That memory was also passed in as the data argument to GeckoStart, and the caller will again try to free that memory. We should just let the caller do the freeing.
Attachment #8593530 - Flags: review?(snorp)
AndroidBridge is never deleted right now. We should be good citizens and clean up.
Attachment #8593532 - Flags: review?(snorp)
The main thread could get stuck trying to wait for a nonexistent Gecko thread.
Attachment #8593533 - Flags: review?(snorp)
Comment on attachment 8593533 [details] [diff] [review]
Don't wait for sync event when Gecko is shutting down (v1)

Review of attachment 8593533 [details] [diff] [review]:
-----------------------------------------------------------------

Woo!
Attachment #8593533 - Flags: review?(snorp) → review+
Attachment #8593532 - Flags: review?(snorp) → review+
Comment on attachment 8593530 [details] [diff] [review]
Remove extra free() calls on shutdown (v1)

Review of attachment 8593530 [details] [diff] [review]:
-----------------------------------------------------------------

Yeah, I found this too a while back when working on the headless patch. Thanks.
Attachment #8593530 - Flags: review?(snorp) → review+
You need to log in before you can comment on or make changes to this bug.