WeakMaps constructor doesn't always get initialized on the global object

RESOLVED FIXED in mozilla12

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: bholley, Assigned: bholley)

Tracking

unspecified
mozilla12
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

My spidey senses tell me that we could use some consolidation here. But in the mean time, here's the patch.
Created attachment 588528 [details] [diff] [review]
patch v1
Attachment #588528 - Flags: review?(jwalden+bmo)
Comment on attachment 588528 [details] [diff] [review]
patch v1

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

::: js/src/vm/GlobalObject.cpp
@@ +49,5 @@
>  #include "frontend/BytecodeEmitter.h"
>  #include "vm/GlobalObject-inl.h"
>  
>  #include "jsobjinlines.h"
> +#include "jsweakmap.h"

Could you insert this in the js* non-inlines list above?  The current inclusions look a disordered mess, but that's no reason to compound the problem.
Attachment #588528 - Flags: review?(jwalden+bmo) → review+
Pushed to inbound: http://hg.mozilla.org/integration/mozilla-inbound/rev/5bc8dcf80474
Assignee: general → bobbyholley+bmo
Target Milestone: --- → mozilla12
https://hg.mozilla.org/mozilla-central/rev/5bc8dcf80474
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Duplicate of this bug: 697928
You need to log in before you can comment on or make changes to this bug.