mozilla-central: changeset 288957:ffdeaea4ceae

author	Nick Fitzgerald <fitzgen@gmail.com>
	Mon, 14 Mar 2016 11:44:00 +0100
changeset 288957	ffdeaea4ceae
parent 288956	3a99126a3b55
child 288958	e109baaf9e01
push id	30093
push user	kwierso@gmail.com
push date	2016-03-16 23:33 +0000
treeherder	mozilla-central@20218dd51efa [default view] [failures only]
perfherder	[talos] [build metrics] [platform microbench] (compared to previous push)
reviewers	vporof
bugs	1253516
milestone	48.0a1
first release with	nightly linux32 fefe2bc3e54f / 48.0a1 / 20160317030235 / files nightly linux64 fefe2bc3e54f / 48.0a1 / 20160317030235 / files nightly mac fefe2bc3e54f / 48.0a1 / 20160317030235 / files nightly win32 fefe2bc3e54f / 48.0a1 / 20160317030235 / files nightly win64 fefe2bc3e54f / 48.0a1 / 20160317030235 / files
last release without	nightly linux32 5e14887312d4 / 48.0a1 / 20160315030230 / files nightly linux64 5e14887312d4 / 48.0a1 / 20160315030230 / files nightly mac 5e14887312d4 / 48.0a1 / 20160315030230 / files nightly win32 5e14887312d4 / 48.0a1 / 20160315030230 / files nightly win64 5e14887312d4 / 48.0a1 / 20160315030230 / files

Bug 1253516 - Do not acquire a lock that protects data that is not used; r=vporof `sMutex` protects the `mMarkersStores` data member, but neither `TimelineConsumers::PopMarkers` nor `ObservedDocShell::PopMarkers` use that data member. Removing this lock acquisition also fixes a dead lock where we re-entrantly attempt to acquire the lock when triggering a GC when allocating JS objects inside `ObservedDocShell::PopMarkers`.

docshell/base/timeline/TimelineConsumers.cpp

file | annotate | diff | comparison | revisions

--- a/docshell/base/timeline/TimelineConsumers.cpp
+++ b/docshell/base/timeline/TimelineConsumers.cpp
@@ -295,17 +295,16 @@ TimelineConsumers::AddMarkerForAllObserv
 }
 
 void
 TimelineConsumers::PopMarkers(nsDocShell* aDocShell,
                               JSContext* aCx,
                               nsTArray<dom::ProfileTimelineMarker>& aStore)
 {
   MOZ_ASSERT(NS_IsMainThread());
-  StaticMutexAutoLock lock(sMutex); // for `mMarkersStores`.
 
   if (!aDocShell || !aDocShell->mObserved) {
     return;
   }
 
   aDocShell->mObserved->PopMarkers(aCx, aStore);
 }