db9095d6cd2db33c72ba6fd7bf2793da4ea4d60d: Bug 1347031 - Move the MediaCache off of opening its temporary file fd synchronously in the content process. r?jwwang draft
Chris Pearce <cpearce@mozilla.com> - Fri, 17 Mar 2017 09:54:23 +1300 - rev 500266
Push 49667 by bmo:cpearce@mozilla.com at Thu, 16 Mar 2017 23:27:30 +0000
Bug 1347031 - Move the MediaCache off of opening its temporary file fd synchronously in the content process. r?jwwang In bug 1346987 we're attempting to remove uses of the NS_OpenAnonymousTemporaryFile() in the content process as it sends a synchronous IPC to the parent process on the main thread, which can cause UI jank. This patch makes the MediaCache use the async anonymous temporary file creation function added in bug 1346987. The file descriptor is held by the FileBlockCache. This object buffers data passed to it in memory, and defers writing of said data to another thread. I added the async wait for the file descriptor to be inside that async "defer to other thread" step. This means that while the content process is waiting for the file descriptor to come down from the parent process, we'll buffer media data being streamed in memory. Given that our MSE implementation will buffer up to 100MB of media data in memory anyway, it seems that more buffering in the src=url case while we wait for an async IPC to do a round trip to the main process is acceptable. MozReview-Commit-ID: 3OTBTWw5pr0
8d03e6c502755138edba399c425cc50200d6ee2a: Bug 1346987 - Fixup thread safety issues in patches in bug 1346987. draft
Chris Pearce <cpearce@mozilla.com> - Thu, 16 Mar 2017 17:22:37 +1300 - rev 500265
Push 49667 by bmo:cpearce@mozilla.com at Thu, 16 Mar 2017 23:27:30 +0000
Bug 1346987 - Fixup thread safety issues in patches in bug 1346987. MozReview-Commit-ID: 1Ryy1Fz4Tvv
e908261e3edd747391197b45818801a168224476: Bug 1345511 - pt 3 - start using IPC call for stun addrs in PCMedia. r=bwc draft
Michael Froman <mfroman@mozilla.com> - Thu, 16 Mar 2017 13:04:44 -0500 - rev 500264
Push 49666 by bmo:mfroman@nostrum.com at Thu, 16 Mar 2017 22:57:29 +0000
Bug 1345511 - pt 3 - start using IPC call for stun addrs in PCMedia. r=bwc Two new calls are added to NrIceCtx. 1) A static call to allow StunAddrsRequestParent to get stun addrs from the main process. 2) A call to allow StunAddrsRequestChild to pass the new stun addrs back to PeerConnectionMedia on the content process. PeerConnectionMedia, when running in e10s mode, sets up the StunAddrsRequestChild and makes the async request to get the stun addrs. When they are returned, it sets the stun addrs in NrIceCtx avoid the network calls that would otherwise cause a further restricted sandbox to fail. MozReview-Commit-ID: C2hYBzm6WNv
82b515e75abe7a1fda98fa3f5d923516cf45858f: Bug 1345511 - pt 2 - add IPC mechanism for getting stun addrs on main process. r=bwc,jduell draft
Michael Froman <mfroman@mozilla.com> - Thu, 16 Mar 2017 12:48:38 -0500 - rev 500263
Push 49666 by bmo:mfroman@nostrum.com at Thu, 16 Mar 2017 22:57:29 +0000
Bug 1345511 - pt 2 - add IPC mechanism for getting stun addrs on main process. r=bwc,jduell PStunAddrsRequest.ipdl defines the new IPC protocol to get stun addrs on the main process. StunAddrsRequestChild requests the stun addrs from the parent. StunAddrsRequestParent uses a static method on NrIceCtx to get the stun addrs from the STS thead and sends the addrs back to the child process. NrIceStunAddr (nricestunaddr.{cpp|h}) wraps nr_local_addr and makes it easier to serialize/deserialize over IPC. NrIceStunAddrMessageUtils follows the pattern used by other Necko IPC classes to define top-level serialization/deserialization calls used by the IPC framework. Modifications under netwerk/ipc are to connect the new IPC protocol to get stun addrs to PNecko since it is a network related IPC protocol. MozReview-Commit-ID: GyEapBe5krl
2101857b6ce425b3fb15c248600ec9a76dd11ce5: Bug 1345511 - pt 1 - nICEr changes to support stun addr gathering from main process. r=bwc draft
Michael Froman <mfroman@mozilla.com> - Thu, 16 Mar 2017 12:06:09 -0500 - rev 500262
Push 49666 by bmo:mfroman@nostrum.com at Thu, 16 Mar 2017 22:57:29 +0000
Bug 1345511 - pt 1 - nICEr changes to support stun addr gathering from main process. r=bwc Expose a tweaked version of nr_ice_get_local_addresses to allow callers to provide pre-fetched stun addrs if they are available. By default, the normal call to nr_ice_gather calls this with no pre-fetched stun addrs (read non-e10s). In e10s, the stun addrs are discovered on the main process and provided to nr_ice_get_local_addreses. When nr_ice_gather is called from the content process the local addresses have already been gathered. Because we're now serializing nr_local_addr (wrapped by NrIceStunAddr), we can't assume that certain pointer references in the source nr_local_addr are correct when calling nr_local_addr_copy. New non-pointer-arithmetic version of setting up the pointer on the copied nr_local_addr is used. Also easier to understand when walking up to it the first time. MozReview-Commit-ID: KVRFl4dfr7J
1ed8c8e719b190ead91bdf029b1cbe2413ed741a: Bug 1347589: Implementation of Marionette Set Window Rect. r?ato draft
David Burns <dburns@mozilla.com> - Thu, 16 Mar 2017 22:30:01 +0000 - rev 500261
Push 49665 by bmo:dburns@mozilla.com at Thu, 16 Mar 2017 22:44:54 +0000
Bug 1347589: Implementation of Marionette Set Window Rect. r?ato This implements Set Window Rect from the W3C WebDriver specification to allow us to change the size and position of a window in one call MozReview-Commit-ID: KxwJyyjg1VU
ce6950fca431196f49c748f426f54313f698c52f: Bug 1318532 - Force remote='true' and noisolation='true' to WebExtensions mozbrowser iframes. r=smaug draft
Tim Nguyen <ntim.bugs@gmail.com> - Thu, 16 Mar 2017 22:41:09 +0000 - rev 500260
Push 49664 by bmo:ntim.bugs@gmail.com at Thu, 16 Mar 2017 22:41:42 +0000
Bug 1318532 - Force remote='true' and noisolation='true' to WebExtensions mozbrowser iframes. r=smaug MozReview-Commit-ID: K6AC918A4Zh
f2ef9f132f2d179d2a448b7e20c1c7968686c752: Bug 1318532 - Support <iframe mozbrowser> in WebExtension documents. r=smaug draft
Tim Nguyen <ntim.bugs@gmail.com> - Thu, 16 Mar 2017 16:53:53 +0000 - rev 500259
Push 49664 by bmo:ntim.bugs@gmail.com at Thu, 16 Mar 2017 22:41:42 +0000
Bug 1318532 - Support <iframe mozbrowser> in WebExtension documents. r=smaug MozReview-Commit-ID: DksnSiWpleI
82517bc71b416c2e115657bdca2e817b317387f3: Bug 1197420 Part 4 Tests for optional permissions draft
Andrew Swan <aswan@mozilla.com> - Thu, 16 Mar 2017 14:36:26 -0700 - rev 500258
Push 49663 by aswan@mozilla.com at Thu, 16 Mar 2017 22:30:37 +0000
Bug 1197420 Part 4 Tests for optional permissions MozReview-Commit-ID: 8uViXB1Jgz3
718cfc283803fb75894d70de69f9fed5d66de43c: Bug 1197420 Part 3 Apply dynamic permission changes draft
Andrew Swan <aswan@mozilla.com> - Thu, 16 Mar 2017 15:20:18 -0700 - rev 500257
Push 49663 by aswan@mozilla.com at Thu, 16 Mar 2017 22:30:37 +0000
Bug 1197420 Part 3 Apply dynamic permission changes MozReview-Commit-ID: 6TdcUv1fHPh
afbe26590f553a2d45d72a67232e97ced9de634b: Bug 1197420 Part 2 Initial browser.permissions api support draft
Andrew Swan <aswan@mozilla.com> - Thu, 16 Mar 2017 15:20:05 -0700 - rev 500256
Push 49663 by aswan@mozilla.com at Thu, 16 Mar 2017 22:30:37 +0000
Bug 1197420 Part 2 Initial browser.permissions api support With this patch, permissions are not actually applied, but the permissions api is in place. MozReview-Commit-ID: CTaXz5sa1xy
02342616c6458e08befe75994d6026c5a96be7bc: Bug 1197420 Part 1 Schema groundwork for optional permissions draft
Andrew Swan <aswan@mozilla.com> - Sat, 25 Feb 2017 09:25:00 -0800 - rev 500255
Push 49663 by aswan@mozilla.com at Thu, 16 Mar 2017 22:30:37 +0000
Bug 1197420 Part 1 Schema groundwork for optional permissions MozReview-Commit-ID: JIfPYjxjFfM
7771f0fc793b2493504ee1351c28f80de91250bf: Bug 1348122 - Share code between the customizable zoom control and the location bar's zoom indicator. r?gijs draft
Dão Gottwald <dao@mozilla.com> - Thu, 16 Mar 2017 23:03:59 +0100 - rev 500254
Push 49662 by dgottwald@mozilla.com at Thu, 16 Mar 2017 22:04:52 +0000
Bug 1348122 - Share code between the customizable zoom control and the location bar's zoom indicator. r?gijs MozReview-Commit-ID: 2xRqVzHAwEY
0b071514ceaef15e5d5b9fc70d8a1dadd76b15cd: Bug 1339440 use frameUrl to match context menus on frames draft
Shane Caraveo <scaraveo@mozilla.com> - Thu, 16 Mar 2017 15:02:28 -0700 - rev 500253
Push 49661 by mixedpuppy@gmail.com at Thu, 16 Mar 2017 22:03:20 +0000
Bug 1339440 use frameUrl to match context menus on frames MozReview-Commit-ID: LIlAZIP4HNx
dc95b7fdb2cceaf16dbce7e3fde36dc6c6f72d07: Bug 1347619 - Change boxmodel.css to use calc(50% - 1px) instead of 49.5%. r?gl draft
Stanford Lockhart <lockhart@cs.dal.ca> - Thu, 16 Mar 2017 18:39:53 -0300 - rev 500252
Push 49660 by bmo:lockhart@cs.dal.ca at Thu, 16 Mar 2017 21:42:02 +0000
Bug 1347619 - Change boxmodel.css to use calc(50% - 1px) instead of 49.5%. r?gl MozReview-Commit-ID: 1CA5WgfKNL7
7e5ab33694bac07a5fd665cdbb00162a0f74b2bf: Bug 1344844 - Enable flake8 rule E222: "multiple spaces after operator". r?dexter draft
Federico Padua <federico_padua@yahoo.it> - Tue, 14 Mar 2017 01:44:29 +0100 - rev 500251
Push 49659 by bmo:federico_padua@yahoo.it at Thu, 16 Mar 2017 21:37:03 +0000
Bug 1344844 - Enable flake8 rule E222: "multiple spaces after operator". r?dexter This patch enables flake8 rule E222: "multiple spaces after operator" in toolkit/components/telemetry by removing the relative E222 entry from toolkit/components/telemetry/.flake8 and fixing the files for which the E222 error was reported. Precisely 1 error violating E222 rule was found and solved. MozReview-Commit-ID: 3xx09jkVu3E
21e28e0f6f174c472a59ca60a4139efe98ad461c: Bug 1347700 - Fix async loading of content scripts draft
Tomislav Jovanovic <tomica@gmail.com> - Thu, 16 Mar 2017 15:40:37 +0100 - rev 500250
Push 49658 by bmo:tomica@gmail.com at Thu, 16 Mar 2017 21:36:35 +0000
Bug 1347700 - Fix async loading of content scripts MozReview-Commit-ID: LjuiizBh1OK
de60122b9d619ae4a777e70f0df491b8f63a6d1a: Bug 1339739 fix intermittent timeout by splitting browser_ext_sidebarAction.js, r?kmag draft
Shane Caraveo <scaraveo@mozilla.com> - Thu, 16 Mar 2017 14:27:05 -0700 - rev 500249
Push 49657 by mixedpuppy@gmail.com at Thu, 16 Mar 2017 21:34:58 +0000
Bug 1339739 fix intermittent timeout by splitting browser_ext_sidebarAction.js, r?kmag MozReview-Commit-ID: KeBJHSXo23a
7e6f90cb5bd487da27d8d782769002c4c4fbfa93: Bug 893505 - Simplify the application update UI draft
Doug Thayer <dougathayer@gmail.com> - Mon, 06 Mar 2017 21:34:36 -0800 - rev 500248
Push 49656 by bmo:dothayer@mozilla.com at Thu, 16 Mar 2017 21:30:42 +0000
Bug 893505 - Simplify the application update UI There's quite a few changes in here. At a high level, all we're trying to do is to replace the old update popup with a less intrusive and more modern doorhanger (set of doorhangers) for various update failure conditions. MozReview-Commit-ID: 24sESMTosNX
b20f4a6a4bc04654008428d1cb4c604a768c7785: Bug 1338347 - Record frame throughput ratio for APZ animations r?botond draft
Kartikaya Gupta <kgupta@mozilla.com> - Thu, 16 Mar 2017 17:22:20 -0400 - rev 500247
Push 49655 by kgupta@mozilla.com at Thu, 16 Mar 2017 21:22:54 +0000
Bug 1338347 - Record frame throughput ratio for APZ animations r?botond MozReview-Commit-ID: 4BiFjBvgvfP
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip