Closed Bug 1174803 Opened 10 years ago Closed 10 years ago

Fix assertion in RuntimeService::WorkerPrefChanged

Categories

(Core :: DOM: Workers, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox41 --- affected
firefox42 --- fixed

People

(Reporter: birunthan, Assigned: birunthan)

Details

Attachments

(1 file)

The `MOZ_ASSERT(key != WORKERPREF_COUNT)` assertion doesn't actually check if the key was not handled because key is never actually changed.
Attachment #8622576 - Flags: review?(bent.mozilla)
Comment on attachment 8622576 [details] [diff] [review] Fix assertion in RuntimeService::WorkerPrefChanged Review of attachment 8622576 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, thanks!
Attachment #8622576 - Flags: review?(bent.mozilla) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: