Commit Graph

13437 Commits

Author SHA1 Message Date
Andrea Marchesini
7ba8b77e07 Bug 1466023 - Separate FontTableURI and BlobURL, r=qdot
This patch splits FontTableURI and BlobURL in 2 classes:
FontTableURIProtocolHandler and BlobURLProtocolHandler
both under mozilla::dom.

It also removes a memory reporter because that report is already covered by the
BlobURL one.

--HG--
rename : dom/file/nsHostObjectProtocolHandler.cpp => dom/file/BlobURLProtocolHandler.cpp
rename : dom/file/nsHostObjectProtocolHandler.h => dom/file/BlobURLProtocolHandler.h
2018-06-02 15:51:42 +02:00
Andrea Marchesini
d64c4294eb Bug 1466023 - Get rid of NS_GetStreamForBlobURI, r=qdot 2018-06-02 15:51:41 +02:00
Emilio Cobos Álvarez
1e9c395548 Bug 1466168: Remove mozilla::Forward in favor of std::forward. r=froydnj
Same approach as the other bug, mostly replacing automatically by removing
'using mozilla::Forward;' and then:

  s/mozilla::Forward/std::forward/
  s/Forward</std::forward</

The only file that required manual fixup was TestTreeTraversal.cpp, which had
a class called TestNodeForward with template parameters :)

MozReview-Commit-ID: A88qFG5AccP
2018-06-02 09:33:26 +02:00
Csoregi Natalia
e2f48890e1 Merge mozilla-central to inbound. a=merge CLOSED TREE 2018-06-02 01:05:17 +03:00
Csoregi Natalia
468205953d Merge inbound to mozilla-central. a=merge 2018-06-02 01:01:33 +03:00
Emilio Cobos Álvarez
fffb25b74f Bug 1465585: Switch from mozilla::Move to std::move. r=froydnj
This was done automatically replacing:

  s/mozilla::Move/std::move/
  s/ Move(/ std::move(/
  s/(Move(/(std::move(/

Removing the 'using mozilla::Move;' lines.

And then with a few manual fixups, see the bug for the split series..

MozReview-Commit-ID: Jxze3adipUh
2018-06-01 10:45:27 +02:00
Bryce Van Dyk
c9d6056854 Bug 1465409 - Change lint rules for mediasource tests to prefer no spaces for array notation. r=jya
Change rules so we enforce ["foo", "bar"] rather than [ "foo", "bar" ].

MozReview-Commit-ID: KX6wJd8jmeJ

--HG--
extra : rebase_source : 6800cf4e701a33f246418e0deb35a206af8104f4
2018-05-30 08:18:54 -04:00
Andreas Pehrson
265a0934ca Bug 1463581 - Signal enabled state changes also on UnregisterSink. r=jib
MozReview-Commit-ID: FDCO73SdBoL

--HG--
extra : rebase_source : 642b59d13fbec001faa7991075ab7809e9f6ce56
2018-05-30 16:48:22 +02:00
Paul Adenot
b0d0866f95 Bug 1456265 - Also update test_channelSplitterNodeWithVolume.html
--HG--
extra : rebase_source : 353b42cb78e73aa35caa4d6375f67f1769cf6add
extra : intermediate-source : eb7cba7e1120302f696c002d42cae86d3de1d4b5
extra : source : ee0c4548a78a794de83ce82d994bd5cfa6c84e23
2018-05-30 18:23:37 +02:00
Paul Adenot
f286db11ce Bug 1456946 - Properly reflect the channelCount when set in the ctor using the option object, on a ChannelSplitterNode. r=achronop
MozReview-Commit-ID: DrnadeBS8hi

--HG--
extra : rebase_source : ca28ff595fdefaf8327c0a6ef4826f3b2cd02368
extra : intermediate-source : 9af1ffb437be63b6956970b47eebc78e6e2310af
extra : source : 91a4b9b7cfa8444e760a99ae4518e678e1cc14ef
2018-05-29 18:16:24 +02:00
Paul Adenot
54199c6959 Bug 1456946 - Don't call AudioNode::Initialize on an ChannelSplitterNode during construction to avoid throwing. r=achronop
We initialize the channelCountMode and channelInterpretation (that cannot be
changed) appropriately in the C++ ctor, and the channelCount manually as well.

Then we check manually that the options for the channelCountMode, the
channelCount and channelInterpretation have not been passed, or are not
incorrect.

MozReview-Commit-ID: 4YdYfaGqKqn

--HG--
extra : rebase_source : 54000c4f79361945142bb413b7eeebb2967d4fbb
extra : intermediate-source : 9a085b0513f78344b07724c049860b089c313894
extra : source : 852233de6408712da07a2f09721c7e35d67d608f
2018-05-29 18:15:29 +02:00
Paul Adenot
a8662a7938 Bug 1456265 - Make ChannelSplitterNode have a channelCountMode of "explicit" and a channelIntepretation of "discrete" by default. r=achronop
MozReview-Commit-ID: 5pAdWn9zV9e

--HG--
extra : rebase_source : bcb5fcac629a403a12481b55e577facf57c2fca5
extra : intermediate-source : ebfc3c04016ab0cce5489973e63cdb6dfdee191c
extra : source : 42f845da213bcb73fb3400d6628e3ffc15b3992b
2018-05-29 17:18:39 +02:00
Arnaud Bienner
e1cac52f37 Bug 1460907 - Implement AudioParamMap definitions. r=karlt,qdot
MozReview-Commit-ID: AdsI9QU3VOX

--HG--
extra : rebase_source : b72d898550289f0637d7cb1c9796a12a7b4fc782
2018-05-13 16:16:23 +02:00
Andreas Pehrson
9e6a6da509 Bug 1282262 - Reduce number of (slow) gUM requests in test_gUM_mediaStreamClone. r=padenot
MozReview-Commit-ID: 1By46JEdXj6

--HG--
extra : rebase_source : 738ff1be92d618f5a6fe7dab8d3f63749d7b8cb6
2018-05-30 14:45:38 +02:00
Matthew Gregan
29d47f27a7 Bug 1465299 - Update AudioIPC pref defaults in CubebUtils.cpp to match actual defaults in all.js. r=padenot
MozReview-Commit-ID: 16fC71RT0AB

--HG--
extra : rebase_source : f85f04494d32afa24baeb7de89247b5360aea7bc
2018-05-30 14:02:13 +12:00
Dan Minor
a20dc982b3 Bug 1450658 - Add preference to disable focusing source; r=pehrsons
Focusing a source during a test run can cause Firefox to lose focus and
subsequent tests to timeout. This adds a preference which defaults to true
that allows the source to be focused. It is then set to false for tests that
involve screen sharing.

MozReview-Commit-ID: CNq2c87y8Ho

--HG--
extra : rebase_source : 805e2cbf9c09af419419a64d074e020784185b1d
2018-05-29 16:14:38 -04:00
Dan Minor
50a3ab6cd4 Bug 1450658 - Should bring window to front when screen-sharing a window; r=pehrsons
This adds a FocusOnSelectedSource method to PCameras and uses it to focus the
selected window while window sharing. We can't just focus the window as soon as
it is shared because we have a live preview in the getUserMedia permissions
prompt which would cause the prompt to lose focus. Instead, this only focuses the
window when the sharing is not done from a chrome context.


MozReview-Commit-ID: 5jre75E3JLi

--HG--
extra : rebase_source : 5f5154fc9fc7590cc02eb25146e5bc20b2243fa3
2018-05-08 15:55:36 -04:00
Jan-Ivar Bruaroey
4439872dc0 Bug 1462990 - Use async/await in mediasource mochitests r=bryce
MozReview-Commit-ID: AIr8CLyuGb7

--HG--
extra : rebase_source : 2136b7faccc24da6b163b0edf908a198136a7638
2018-05-21 16:40:13 -04:00
Jan-Ivar Bruaroey
5017b2c432 Bug 1462990 - Use async/await in mediasource/test/mediasource.js r=bryce,jya
MozReview-Commit-ID: BnQ30DnY4mB

--HG--
extra : rebase_source : fd89494f1f5946a9b8505456ab9761d3cc1914fa
2018-05-18 10:26:11 -04:00
Margareta Eliza Balazs
d11876a092 Backed out changeset 9e1a3230d085 (bug 1456265) for mda3 failures in dom/media/webaudio/test/test_channelSplitterNode.html on a CLOSED TREE 2018-05-29 19:32:26 +03:00
Paul Adenot
9f03d7a44c Bug 1456265 - Make ChannelSplitterNode have a channelCountMode of "explicit" and a channelIntepretation of "discrete" by default. r=achronop
MozReview-Commit-ID: 5pAdWn9zV9e

--HG--
extra : rebase_source : c001d5f5db0ff658027f18195ff5ff95abb314c9
2018-05-29 17:18:39 +02:00
Jean-Yves Avenard
34c40846b5 Bug 1457661 - P2. Ensure we call NotifyDataEnded for local resource once size is known. r=bryce
A call to NotifyDataEnded is required even if the size was known when the resource was created. This ensures that the readyState is properly updated and that playback can immediately as no more data can be added once first loaded.

MozReview-Commit-ID: FaJMBxJ9NkM

--HG--
extra : rebase_source : 448087a22635dac2aa31611c2b58a8e9c77121ec
2018-05-28 23:06:38 +02:00
Jean-Yves Avenard
cb6ce94cf2 Bug 1457661 - P1. Add mochitest. r=bryce
MozReview-Commit-ID: 9YC8MaXRF42

--HG--
extra : rebase_source : 5bb2dc4c1f2dbb68d3713d536ecbe7f35eabb544
2018-05-28 22:39:10 +02:00
Jean-Yves Avenard
535b2be403 Bug 1454630 - P4. Only wait for CDMProxy if actually needed for the given decoder. r=bryce
If the particular track isn't encrypted, there's no need to wait for a CDMProxy.

MozReview-Commit-ID: DPbvbwsO58N

--HG--
extra : rebase_source : 0e7fea134404c861268dc8759cd7c0ebdf83dca4
2018-05-28 15:10:19 +02:00
Jean-Yves Avenard
cc044be0ee Bug 1454630 - P3. Always recreate the PDMFactory when CDMProxy change. r=bryce
The code couldn't have worked and didn't do what the comment stated. When the CDMProxy changes, the current PDMFactory for encrypted content can no longer be used.

MozReview-Commit-ID: 7LpcQkK5gLL

--HG--
extra : rebase_source : e3926034069285be1559d0a1ea20d5f3c1561eb7
2018-05-28 14:46:35 +02:00
Jean-Yves Avenard
09c18364f7 Bug 1454630 - P2. Use new PDMFactory whenever encryption type change. r=bryce,cpearce
If the content being played was first non-encrypted, the PDMFactory would have been set without a CDMProxy. As such, it is necessary to use a new PDMFactory when the encryption type changes (from clear to encrypted).

Rather than attempting to detect if the encryption status has changed, simply use two PDMFactory, one with CDMProxy set and one without (for clear content)

Also, never attempt to recycle a decoder if the encryption type changed (used only on Android)

The TrackBuffersManager would have already handle the dispatching of the encrypted event when parsing the new init segment. As such, nothing more is necessary.

MozReview-Commit-ID: Jn14P2F6N5V

--HG--
extra : rebase_source : afe254fa8c4b835b15d9d48bb52d832f28196b7e
2018-05-27 18:42:25 +02:00
Jean-Yves Avenard
b0af7c4063 Bug 1454630 - P1. Simplify retrieval of current TrackInfo. r=bryce
Adding some documentation to clarify on the difference between mInfo and mOriginalInfo

MozReview-Commit-ID: DWBsoi16QKf

--HG--
extra : rebase_source : 719c17b9ce61efdb633db108230f1bf78773ee51
2018-05-26 21:21:45 +02:00
Jean-Yves Avenard
15d6ce1610 Bug 1450607 - P3. Remove unused argument. r=gerald
MozReview-Commit-ID: 6NU9BLJi6kl

--HG--
extra : rebase_source : a9fca396ca5c353be979196dde4b1b02137cc9ca
2018-05-24 17:38:02 +02:00
Jean-Yves Avenard
f1f3bc2758 Bug 1450607 - P2. Synchronously seek to prepare for resuming following stop request. r=gerald
Bug 1415090 attempted to remove the need to access from the MediaResource members of MediaCacheStream from the main thread.
However, by doing so the logic flow for resuming the channel changed from a synchronous access to an asynchronous one.
This changed some assumptions and allowed the ChannelMediaResource to be used before the Seek call completed.

For now, re-add a cross thread access to the MediaCacheStream. A more elegant fix will be worked on in bug 1464045

MozReview-Commit-ID: 2xBTjDEqrkI

--HG--
extra : rebase_source : 0aa3cfcb8371d5147cbed746d9200dd22df4821b
2018-05-24 14:36:58 +02:00
Jean-Yves Avenard
1d73f87b45 Bug 1450607 - P1. Fix constness. r=gerald
MozReview-Commit-ID: CeftB0qmzTf

--HG--
extra : rebase_source : 7d8a75754990fd6cab1ad8f620d5d5675c46ea88
2018-05-23 21:27:31 +02:00
Jean-Yves Avenard
aa3feebecc Bug 1451149 - P2. Don't fire the "stalled" event when using MSE. r=bryce
When using a media element with a Media Source, the resource fetching algorithm is to be called in "local" mode:
https://www.w3.org/TR/media-source/#mediasource-attach
"Continue the resource fetch algorithm by running the remaining "Otherwise (mode is local)" steps, with these clarifications"

https://html.spec.whatwg.org/multipage/media.html#concept-media-load-resource
Under the local mode, the steps that would cause the element to fire suspend, stalled or progress can never occur.
We only prevent the stalled event to be fired, many websites rely on the progress event to be fired (such as when the init segment has been parsed). The HTML5 media spec will be amended to clearly indicate that progress is to be fired even with mediasource

MozReview-Commit-ID: DkoQzoV0JzO

--HG--
extra : rebase_source : 1e916eee50c9935f168797bb5a92052191cda59d
2018-05-14 11:32:09 +02:00
Jean-Yves Avenard
8c2fb70982 Bug 1461268 - P2. Bump benchmark version ID. r=bryce
This will force the benchmark to be re-run, following de-activation of the AMD VP9 decoder.

MozReview-Commit-ID: KwwoQNfSJ4t

--HG--
extra : rebase_source : 4edc750d448d6f023a645457edcbd138974c7d9a
2018-05-24 15:21:15 +02:00
Natalia Csoregi
1ff5ce3bb8 Bug 1461963 - Removed OSX 10.6 reference. r=gbrown 2018-05-17 15:26:00 +03:00
Kris Maglione
8134b33e89 Bug 1462964: Remove obsolete nsIDOMBlob interface. r=bz
MozReview-Commit-ID: 2HIlaSrvfBe

--HG--
extra : rebase_source : 944a6244dbfe1f0a37dd34a041d1329624a86f93
extra : histedit_source : d7859be664fed00f8504e40969480d3af3d674e7%2C4456b502db338bd06823654306e47c2583101396
2018-05-21 17:32:44 -07:00
Kris Maglione
6b12d08f7d Bug 1462937: Update callers to use nsIFile::GetDirectoryEntries as a nsIDirectoryEnumerator. r=froydnj
MozReview-Commit-ID: Iv4T1MVAF5

--HG--
extra : rebase_source : 1c518883d082884db7f9323a5acc20361228c26b
extra : histedit_source : 70a73c23d1199d3bfbb5379c78930401166c094b
2018-05-19 20:17:45 -07:00
Chris Pearce
d08cfaf67f Bug 1461877 - Test that we don't dispatch 'playing' event for blocked playbacks. r=bryce
MozReview-Commit-ID: KXfviOtqGZI

--HG--
extra : rebase_source : 3ce026dc6fd6e05ce5f75d9860dc567e846a0cbf
2018-05-18 11:41:28 +12:00
Jim Chen
c68f5f3de7 Bug 1458020 - 5. Return default sample rate / frames per buffer; r=esawin
We apparently fail to get these properties when running in an emulator,
so we should just return default values.

MozReview-Commit-ID: IZVYIG9INaq
2018-05-18 10:16:04 -04:00
Jim Chen
4053e24872 Bug 1458020 - 3. Set JavaVM when actually using video capture; r=jib
We currently set the Android JavaVM pointer in MediaEngineWebRTC.
However, because of that, we end up setting the pointer in the child
process, even though we really want to set the pointer in the parent
process because that's where the camera will be accessed.

This patch makes us set JavaVM inside VideoEngine itself, where we
actually access the camera in the parent process.

MozReview-Commit-ID: 3TeLiiK2vyh
2018-05-18 10:16:04 -04:00
Chris Pearce
68af64e32b Bug 1461540 - Ensure errorMessage is initialized on all code paths in WebAudioDecodeJob::OnFailure(). r=jya
Doesn't look like it's strictly necessary, but suppresses the warning.

MozReview-Commit-ID: EQBar71yNVg

--HG--
extra : rebase_source : 063639000fd47b32343d47a8ca428c134fdce43b
2018-05-15 13:12:13 +12:00
Bryce Van Dyk
d74241d1f7 Bug 1461785 - Update dom/media/mediasource/test to abide eslint rules, add .eslintrc.js. r=jya,standard8
Add .eslintrc.js to configure globals from mediasource.js and to add extra
rules to encourage use of let and const over var.

Linting changes:
- Prefer const and let to var. This provides tighter scoping and avoids
  reassignment.
- Mozilla rules do not allow for shadowing of variables, so several tests have
  had promise lists renamed to avoid this.
- Numerous minor fixes to formatting including using double quotes, spacing,
  missing semicolons.
- Remove some unused variables.
- Arrays have spaces after opening braces and before closing braces, e.g.
  [ "foo" ], this is to be consistent with our clang-format rules.
- Fix naming of resourcePathSeen in test_MediaSource_memory_reporting.html.

MozReview-Commit-ID: 5q6oS7EWLTk

--HG--
extra : rebase_source : 9b67b294f338ca9205b52fded3af63e6c3ac9a5a
2018-05-14 10:08:59 -04:00
Jean-Yves Avenard
5c9362146d Bug 1457499 - P3. Query decoded frame size instead of calculating it. r=bryce
Under most cases, the frame decoded height is just the displayed height rounded to the next 16 row aligned value.
However, this doesn't appear to always be the case. So we query the WMF framework for the decoded frame size.
We continue to use the displayed sizes as found in the SPS to ensure proper display of non 1:1 aspect ratio.

Adding diagnostic assertion to find potential regressions, we will address those as they come.

MozReview-Commit-ID: L8VowEw6L9F

--HG--
extra : rebase_source : 49acd9fd36469ee0a4e1ed0fe5cd6f2211ba8117
2018-05-16 11:09:20 +02:00
Jean-Yves Avenard
9ad27a22c2 Bug 1457499 - P2. Fix compilation warning. r=bryce
TaskQueue::Dispatch returns a nsresult which must be checked.

MozReview-Commit-ID: 7Tl7O96rQNt

--HG--
extra : rebase_source : e898b776f765a5641a794a7242715728940075f6
2018-05-16 11:03:05 +02:00
Jean-Yves Avenard
4bbe4e2668 Bug 1457499 - P1. Remove unused variable. r=bryce
Prevent a compilation warning

MozReview-Commit-ID: HT1UzsFUNXj

--HG--
extra : rebase_source : c552745bad82ef3f6faf7fc29ad10682211fe6d2
2018-05-16 11:01:58 +02:00
shindli
55f2b0d77a Merge inbound to mozilla-central. a=merge 2018-05-16 21:31:38 +03:00
Andrea Marchesini
3ef72562fe Bug 1454889 - Remove createObjectURL()'s MediaStream overload, r=valentin 2018-04-24 16:19:51 +02:00
Jan de Mooij
80e44e8003 Bug 1461292 part 1 - Rename JSAutoCompartment to JSAutoRealm. r=bz,luke 2018-05-16 10:53:16 +02:00
Jean-Yves Avenard
559fa1a1f2 Bug 1280613 - P3. Mochitests. r=bryce
MozReview-Commit-ID: 5FIe9qfNZWd

--HG--
extra : rebase_source : b8f0a01abeba4dc4356f933a29ed0595fc9884c4
2018-05-11 09:01:25 +02:00
Jean-Yves Avenard
756284d1ab Bug 1280613 - P2. Add experimental SourceBuffer.removeAsync. r=bz
The aim of those changes is to be incubated in the WICG.

MozReview-Commit-ID: K93xiTod6tw

--HG--
extra : rebase_source : 788122a69b6cab96bab9c2ae6e8cdf7c61d4ed07
2018-05-11 04:07:24 +02:00
Jean-Yves Avenard
6039ae5766 Bug 1280613 - P1. Add experimental SourceBuffer.appendBufferAsync. r=bz
The aim of those changes is to be incubated in the WICG.

MozReview-Commit-ID: 5wEUnWz8i7kBug 1280613

--HG--
extra : rebase_source : 4360d2c34025bf3c87fff49f0e2aa01975fff15b
2018-05-11 03:56:50 +02:00
Kartikaya Gupta
2b15ef4e8e Bug 1405686 - Re-enable some mochitest that are now passing. r=jrmuizel
MozReview-Commit-ID: 5KqPkgRtH70

--HG--
extra : rebase_source : ff211244da8f7f65555a5958f4ca09f9afaed145
2018-05-14 15:30:45 -04:00