Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 8 Jun 2018 14:10:05 +0000 (UTC)
From:      Jan Beich <jbeich@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r472001 - in head/www/waterfox: . files
Message-ID:  <201806081410.w58EA5Kb087733@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jbeich
Date: Fri Jun  8 14:10:04 2018
New Revision: 472001
URL: https://svnweb.freebsd.org/changeset/ports/472001

Log:
  www/waterfox: restore FF61 fix after r471887

Modified:
  head/www/waterfox/Makefile   (contents, props changed)
  head/www/waterfox/files/patch-bug1453127   (contents, props changed)

Modified: head/www/waterfox/Makefile
==============================================================================
--- head/www/waterfox/Makefile	Fri Jun  8 13:43:25 2018	(r472000)
+++ head/www/waterfox/Makefile	Fri Jun  8 14:10:04 2018	(r472001)
@@ -3,7 +3,7 @@
 PORTNAME=	waterfox
 DISTVERSION=	56.2.0-31
 DISTVERSIONSUFFIX=	-gf435a827f82ac
-PORTREVISION=	3
+PORTREVISION=	4
 CATEGORIES=	www ipv6
 
 MAINTAINER=	jbeich@FreeBSD.org

Modified: head/www/waterfox/files/patch-bug1453127
==============================================================================
--- head/www/waterfox/files/patch-bug1453127	Fri Jun  8 13:43:25 2018	(r472000)
+++ head/www/waterfox/files/patch-bug1453127	Fri Jun  8 14:10:04 2018	(r472001)
@@ -1,3 +1,39 @@
+commit 7f8f5d958ed6
+Author: Bryce Van Dyk <bvandyk@mozilla.com>
+Date:   Wed Apr 18 15:30:57 2018 -0400
+
+    Bug 1453127 - Do not use iterators in MediaStreamTrack when removing listeners. r=pehrsons a=lizzard
+    
+    --HG--
+    extra : source : 6b3aaee40f7507e240da08d6e073cff3c53971f4
+---
+ dom/media/MediaStreamTrack.cpp | 12 ++++++++----
+ 1 file changed, 8 insertions(+), 4 deletions(-)
+
+diff --git dom/media/MediaStreamTrack.cpp dom/media/MediaStreamTrack.cpp
+index 010373e9086dc..af6c6014cf02a 100644
+--- dom/media/MediaStreamTrack.cpp
++++ dom/media/MediaStreamTrack.cpp
+@@ -166,11 +166,15 @@ MediaStreamTrack::Destroy()
+     mPrincipalHandleListener->Forget();
+     mPrincipalHandleListener = nullptr;
+   }
+-  for (auto l : mTrackListeners) {
+-    RemoveListener(l);
++  // Remove all listeners -- avoid iterating over the list we're removing from
++  const nsTArray<RefPtr<MediaStreamTrackListener>> trackListeners(mTrackListeners);
++  for (auto listener : trackListeners) {
++    RemoveListener(listener);
+   }
+-  for (auto l : mDirectTrackListeners) {
+-    RemoveDirectListener(l);
++  // Do the same as above for direct listeners
++  const nsTArray<RefPtr<DirectMediaStreamTrackListener>> directTrackListeners(mDirectTrackListeners);
++  for (auto listener : directTrackListeners) {
++    RemoveDirectListener(listener);
+   }
+ }
+ 
 commit 890e77744a2a
 Author: Andreas Pehrson <pehrsons@mozilla.com>
 Date:   Tue May 29 10:13:14 2018 +0200



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201806081410.w58EA5Kb087733>