Closed Bug 584537 Opened 14 years ago Closed 14 years ago

Don't try to prestart content process on non-IPC build

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mbrubeck, Assigned: mbrubeck)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
I accidentally built with --disable-ipc and discovered that ensureContentProcess throws an exception.  This prevents WeaveGlue.init from being called on the next line.

We could also use a try/catch block, not sure which is better.
Attachment #462972 - Flags: review?(doug.turner)
Comment on attachment 462972 [details] [diff] [review]
patch

or we could make ensureContentProcess just return NS_OK?
Attachment #462972 - Flags: review?(doug.turner) → review+
pushed: http://hg.mozilla.org/mobile-browser/rev/fb7d2025cbec
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: