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