Bug 1177406, don't rely on owner to be set only on main thread, r=khuey
authorOlli Pettay <Olli.Pettay@helsinki.fi>
Thu, 25 Jun 2015 09:26:09 -0700
changeset 250199 1c071a0d3421544bf127cf1d7b7a3a35c18eb807
parent 250198 7085b1090e929f2344933565b0111577399f827a
child 250200 a8473627ca5f30a8bba16b9bb0a2a99d9d59f3a5
push id61474
push useropettay@mozilla.com
push dateThu, 25 Jun 2015 16:44:03 +0000
treeherdermozilla-inbound@1c071a0d3421 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskhuey
bugs1177406
milestone41.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1177406, don't rely on owner to be set only on main thread, r=khuey
dom/events/Event.cpp
--- a/dom/events/Event.cpp
+++ b/dom/events/Event.cpp
@@ -59,17 +59,17 @@ Event::Event(nsPIDOMWindow* aParent)
 }
 
 void
 Event::ConstructorInit(EventTarget* aOwner,
                        nsPresContext* aPresContext,
                        WidgetEvent* aEvent)
 {
   SetOwner(aOwner);
-  mIsMainThreadEvent = mOwner || NS_IsMainThread();
+  mIsMainThreadEvent = NS_IsMainThread();
 
   if (mIsMainThreadEvent && !sReturnHighResTimeStampIsSet) {
     Preferences::AddBoolVarCache(&sReturnHighResTimeStamp,
                                  "dom.event.highrestimestamp.enabled",
                                  sReturnHighResTimeStamp);
     sReturnHighResTimeStampIsSet = true;
   }