Commit Graph

62 Commits

Author SHA1 Message Date
Jean-Yves Avenard
31ed9de297 Bug 1504643 - Enforce template-parameter classes restrictions in MozPromise. r=gerald
Make thorough use of move semantic for MozPromise using nsTArray

Differential Revision: https://phabricator.services.mozilla.com/D11346

--HG--
extra : moz-landing-system : lando
2018-11-09 15:45:31 +00:00
Dorel Luca
c34d2cd0fa Backed out changeset 157be32d85fe (bug 1504643) for conflicts while backing out bug 1471535 2018-11-09 12:13:54 +02:00
Jean-Yves Avenard
c30e8ca9f0 Bug 1504643 - Enforce template-parameter classes restrictions in MozPromise. r=gerald
Make thorough use of move semantic for MozPromise using nsTArray

Differential Revision: https://phabricator.services.mozilla.com/D11346

--HG--
extra : moz-landing-system : lando
2018-11-09 08:41:45 +00:00
Jean-Yves Avenard
2349b2eb93 Bug 1496529 - P7. Pass full range of CreateDecoderParam::Option to RemoteVideoDecoder r=bryce
Depends on D7882

Differential Revision: https://phabricator.services.mozilla.com/D7895

--HG--
extra : moz-landing-system : lando
2018-10-09 16:59:24 +00:00
Jean-Yves Avenard
27e67380d0 Bug 1322234 - P5. Pass the disallow HW decoder flag to remote video decoder. r=mattwoodrow
We'll need to handle EnumSet in ipdl at some stage.

Depends on D7648

Differential Revision: https://phabricator.services.mozilla.com/D7699

--HG--
extra : moz-landing-system : lando
2018-10-04 09:45:26 +00: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
Jean-Yves Avenard
f1c09e4a82 Bug 1397307 - P9. Pass video frame rate to RemoteVideoDecoder and GPU process. r=mattwoodrow
MozReview-Commit-ID: BDSO332f3B6

--HG--
extra : rebase_source : 1ef2b6d7b1c4e237fa90afa99a446678f0f16e25
2017-09-12 18:29:40 +02:00
Alastor Wu
686b38b20a Bug 1397141 - part5 : update error description from GPU process. r=jya,mattwoodrow
MozReview-Commit-ID: 9aKyYftBnUo

--HG--
extra : rebase_source : a497e12d98e3a2b0e0ec2afa60298cacbe1a5ea9
2017-09-13 15:37:24 +08:00
Sebastian Hengst
7d84900829 Backed out changeset 6dade48b3326 (bug 1397141) 2017-09-13 09:25:39 +02:00
Alastor Wu
45f6a63ccf Bug 1397141 - part5 : update error description from GPU process. r=jya,mattwoodrow
MozReview-Commit-ID: CuBZOtjh4g2

--HG--
extra : rebase_source : 82c68ea3fe54b148b0cf98fe37d954efd4acbcbe
2017-09-13 15:05:59 +08:00
Jean-Yves Avenard
1559f6a705 Bug 1394591 - P5. Remove need for atomic members. r=mattwoodrow
Instead set members after initialising the child decoder, and only ever access the child decoder on the same thread.

MozReview-Commit-ID: 4mfhVWbNLEu

--HG--
extra : rebase_source : 90a8e7bc8975fd08fc6b262c81cbf43a45751a06
2017-09-01 22:51:20 +02:00
Jean-Yves Avenard
19333d8da8 Bug 1394591 - P4. Pass decoder description to RemoteVideoDecoder. r=mattwoodrow
This makes it easier to determine the actual decoder in use within the GPU process.

MozReview-Commit-ID: 5TF6AsyXYWW

--HG--
extra : rebase_source : 0e73dc17206a83006040cf422182da560b3cf70a
2017-09-01 18:22:55 +02:00
Sebastian Hengst
c6ac74c855 Backed out changeset b741cbca23bd (bug 1394591) 2017-09-04 00:07:02 +02:00
Sebastian Hengst
239e4df31a Backed out changeset ba1a3d16916b (bug 1394591) 2017-09-04 00:06:56 +02:00
Jean-Yves Avenard
e23a2d9405 Bug 1394591 - P5. Remove need for atomic members. r=mattwoodrow
Instead set members after initialising the child decoder, and only ever access the child decoder on the same thread.

MozReview-Commit-ID: 4mfhVWbNLEu

--HG--
extra : rebase_source : 90a8e7bc8975fd08fc6b262c81cbf43a45751a06
2017-09-01 22:51:20 +02:00
Jean-Yves Avenard
ec392e67b8 Bug 1394591 - P4. Pass decoder description to RemoteVideoDecoder. r=mattwoodrow
This makes it easier to determine the actual decoder in use within the GPU process.

MozReview-Commit-ID: 5TF6AsyXYWW

--HG--
extra : rebase_source : 0e73dc17206a83006040cf422182da560b3cf70a
2017-09-01 18:22:55 +02:00
Jean-Yves Avenard
eccbe26070 Bug 1396428 - Fix end of lines. r=jya
MozReview-Commit-ID: 3reYozTMKk0

--HG--
extra : rebase_source : 1b9d259eca5609ffa68cf6a0d269dfa44aace038
2017-09-03 21:28:52 +02:00
Kaku Kuo
15e39cee1b Bug 1393399 P2 - keep the GPU process crash time and send back to MFR; r=gerald
We keep the GPU crash time and send back to MFR through MediaResult.

We cannot save the information in VideoDecoderChild as a static member because we are going to read it in MFR's task queue and the data was written in VideoDecoderManager's thread. This is going to be racing.

MozReview-Commit-ID: FXqOgelWY6e

--HG--
extra : rebase_source : 5c0561e009ad16983e1ff910216f9cf7901b5542
2017-08-31 17:21:28 +08:00
Kaku Kuo
d7d6aaab9d Bug 1393392 P4 - report telemetry when we shutdown a remote HW decoder; r=gerald,mattwoodrow
MozReview-Commit-ID: HEgv9s3JsXB

--HG--
extra : rebase_source : 1b30e4908048f7aa59dd7c8364763520ba6c1c33
2017-08-29 15:51:22 +08:00
Kaku Kuo
9b6ce81f30 Bug 1393392 P3 - keep DXVA driver information in VideoDecoderChild; r=mattwoodrow
MozReview-Commit-ID: 3uFTC1U5g76

--HG--
extra : rebase_source : 1092f7ce1463e57eaa3345bf617cbe96814b746b
2017-08-29 15:19:05 +08:00
Kaku Kuo
d824321d11 Bug 1388309 - let VideoDecoderChild::InitIPDL() return false if we don't want to decode video on GPU process; r=mattwoodrow
MozReview-Commit-ID: DD7xVkXT6fk

--HG--
extra : rebase_source : 0db172786e24d358640112e6a3461e8308103665
extra : intermediate-source : ce7e905e0ef5fdbc584c1e63cc6b62df9d566ec4
extra : source : 546c8abe515b4fc887d355c165643862346aff78
2017-08-14 18:03:12 +08:00
Bill McCloskey
f115503a0b Bug 1372405 - Provide names for all runnables in the tree (r=froydnj)
MozReview-Commit-ID: DKR6ROiHRS7
2017-06-26 14:19:58 -07:00
Carsten "Tomcat" Book
8a1350b5a6 Backed out changeset 4f6302a98ae4 (bug 1372405)
--HG--
extra : rebase_source : 41632f3158e88e692809731394a683d065a73dfb
2017-06-21 13:59:26 +02:00
Bill McCloskey
6b3e84ed5f Bug 1372405 - Provide names for all runnables in the tree (r=froydnj)
MozReview-Commit-ID: DKR6ROiHRS7
2017-06-20 21:44:11 -07:00
Carsten "Tomcat" Book
ea1b86680c Backed out changeset 9846de3bd954 (bug 1372405)
--HG--
extra : rebase_source : 5d4a48e8ec394c329994689d938d2a6e9b2752b0
2017-06-20 08:27:02 +02:00
Bill McCloskey
4592152411 Bug 1372405 - Provide names for all runnables in the tree (r=froydnj)
MozReview-Commit-ID: DKR6ROiHRS7
2017-06-19 22:25:47 -07:00
JW Wang
80c9f230f3 Bug 1359715 - let functions in MediaData.h take TimeUnit instead of int64_t. r=kaku
We want to replace the use of int64_t for microseconds by TimeUnit
whenever possible since int64_t is ambiguous which could be microseconds
or milliseconds.

MozReview-Commit-ID: LRz9d4yKBYJ

--HG--
extra : rebase_source : 1f73f1f338142b3183491d04726821a881ccabbe
extra : intermediate-source : 88e167b7b06303d10d92cd5317502f405d1c553e
extra : source : 98deb30ec93d395f9951f5fc488170ae35e29675
2017-04-24 17:33:05 +08:00
JW Wang
302d82c85a Bug 1356530 - Change the type of MediaData::mTime to TimeUnit since int64_t is ambiguous. r=kaku
MozReview-Commit-ID: 4bVeqIuWO2O

--HG--
extra : rebase_source : d504ac15a6dc59ad42f3ab80faf23f629d74315f
extra : intermediate-source : 6e52995b6c8146451d98dffc62f6907755dc856e
extra : source : 82d2649cdafb5a6389f6858c23578811933580c9
2017-04-14 17:13:36 +08:00
JW Wang
235f1f55ed Bug 1356506 - change the type of MediaData::mTimecode to TimeUnit since int64_t is ambiguous. r=gerald
MozReview-Commit-ID: 7dO5OOUuORz

--HG--
extra : rebase_source : 1c1c020700d9180ef9f85d7e2e679f498ab71fce
extra : intermediate-source : d173c820bef7a113e28e9732c42e8c1af36160ab
extra : source : 04c08a780526ca3a5ac4d1a6f9b7ae30d9957e92
2017-04-14 14:52:14 +08:00
JW Wang
6359124d70 Bug 1355756. P3 - let CreateAndCopyData() and its friends take TimeUnit for duration. r=gerald
MozReview-Commit-ID: ES0on9VCuu3

--HG--
extra : rebase_source : 8d3e80ec2e1923587b5865516a16bfff9009397d
extra : intermediate-source : 3e59c61b1ccef78e3e8fe52791d7104aade7930c
extra : source : 46fd639ea6a2219bbed70f6555a2acf03ec01a7a
2017-04-12 17:46:09 +08:00
JW Wang
464497b945 Bug 1355756. P1 - change the type of MediaData::mDuration to TimeUnit. r=gerald
MozReview-Commit-ID: 3d4bUYtSuMI

--HG--
extra : rebase_source : 94c821b6d381421035e6a12cbe038436055c5822
extra : intermediate-source : 9a06beffc736486f47b9cf05e7f482e726d53068
extra : source : fdbdcd5c1474f04dc1dbde66fcf3a9ecec953053
2017-04-12 17:27:34 +08:00
John Lin
a5d3bc8a18 Bug 1344649 - part 2: let VideoData::CreateFromImage() accept only neccessary parameters. r=jya
VideoData doesn't care what's in aInfo but display size and aPicture are unused.

MozReview-Commit-ID: IBqq8Rm8dM4

--HG--
extra : rebase_source : 10e2390f87925ef9179d28d86240f68a35c6c6d4
2017-03-09 12:06:24 +08:00
Matt Woodrow
9fcf5f626a Bug 1329369 - Initialize the decoded video frame to the pixel size of the buffer. r=jya
MozReview-Commit-ID: ChfGNFxjJpc

--HG--
extra : rebase_source : 06c162f43a1ef2740bdc0cc2b4873ed4b377f54f
2017-02-17 15:01:25 +13:00
Jean-Yves Avenard
572d9a0a80 Bug 1339748: P2. Let the MediaDataDecoder tells the format it wants. r=cpearce
MozReview-Commit-ID: 22GEUnwEacd

--HG--
extra : rebase_source : 019627d3ff0ef0430bcaf83a4098ffd62fc53b05
2017-02-17 22:19:55 +01:00
Sebastian Hengst
955ce77a9f Backed out changeset f508a675e8dc (bug 1339748) 2017-02-18 17:36:21 +01:00
Jean-Yves Avenard
1aee5fb0ac Bug 1339748: P2. Let the MediaDataDecoder tells the format it wants. r=cpearce
MozReview-Commit-ID: 22GEUnwEacd

--HG--
extra : rebase_source : ce57fbd3028333485368267955b2645057f54805
2017-02-17 22:19:55 +01:00
Jean-Yves Avenard
73fd84bc5d Bug 1319987: P5. Promisify MediaDataDecoder. r=cpearce,gerald,mattwoodrow,snorp
This is a big change, and unfortunately impossible to break down with independently functional commits.

There are four main changes being applied here:

* Code cleanup, including making all MediaDataDecoder related code mozilla coding style compliant
* Make MediaDataDecoder use MozPromise
* Making Flush and Shutdown processes fully asynchronous
* Fixing few data races encountered across the code, in particular in the Android PDM

MozReview-Commit-ID: DpiZucGofJT

--HG--
extra : rebase_source : 80bd6c6f9726d536b6f306c40d9af6df27333be9
2017-01-26 13:56:46 +01:00
Matt Woodrow
61b46ea20c Bug 1319251 - Check Init promise in VideoDecoderChild before trying to resolve it. r=jya
--HG--
extra : rebase_source : b628ac151502806d7682294ce55949c0b87ccc5f
2016-11-22 21:47:00 +13:00
Matt Woodrow
f79cf83a6f Bug 1316506 - Initialize mFlushTask. r=bustage CLOSED TREE 2016-11-22 17:43:45 +13:00
Matt Woodrow
1c5f147c9f Bug 1318999 - Revoke MediaDataDecoder callback after RemoteVideoDecoder has been shut down. r=jya
--HG--
extra : rebase_source : 095eebbd7e2457a52aa7679d666ca10da3459259
2016-11-22 15:51:18 +13:00
Matt Woodrow
89ee34ca75 Bug 1316506 - Make RemoteVideoDecoder::Flush synchronous. r=billm
--HG--
extra : rebase_source : 1e0b6991a629de72248ac80dd64c30126d0c10fd
2016-11-22 15:50:46 +13:00
Matt Woodrow
50c5379ac8 Bug 1318083 - Make PVideoDecoder's ctor sync and fail if CreateVideoDecoder fails on the GPU process side. r=jya,dvander
--HG--
extra : rebase_source : 60a1e6686feaa1a841f5f822dfb0c91e220fe34f
2016-11-22 15:44:38 +13:00
Kan-Ru Chen
f8100451db Bug 1314254 - Add mozilla::ipc::IPCResult type and convert IPDL handlers to use new return type. r=billm
We will use the new type for the generated IPDL message handler
prototype to make sure correct error handling method is called.

MozReview-Commit-ID: AzVbApxFGZ0
2016-11-15 04:26:00 +01:00
Matt Woodrow
f5f8e37990 Bug 1316506 - Make Shutdown block on the manager thread so that we clear the callback pointer immediately. r=dvander 2016-11-13 16:09:35 +13:00
Matt Woodrow
a9d4c7433e Bug 1316771 - Don't hold a KnowsCompositor ref on the video manager thread since we don't want it to outlive the main thread ref. r=dvander 2016-11-13 14:19:02 +13:00
Matt Woodrow
815b1ded77 Bug 1315144 - Add new non-fatal media error so that we recreate decoders when the GPU process crashes. r=jya 2016-11-08 15:22:37 +13:00
Matt Woodrow
f880884b57 Bug 1315510 - Automatically recreate VideoDecoderManager if the GPU process crashes. r=dvander 2016-11-08 15:21:35 +13:00
Phil Ringnalda
27a5b8d2ae Backed out changeset d537051ade6a (bug 1315510) for nsTArray_base leaks and a fondess for crashing
CLOSED TREE

MozReview-Commit-ID: KD3jAkCg0O7
2016-11-07 22:18:21 -08:00
Phil Ringnalda
6c8fce2953 Backed out changeset 9ee070ee6dad (bug 1315144) to get at the patch below it
CLOSED TREE

MozReview-Commit-ID: 3d5vSfNXqO4
2016-11-07 22:17:28 -08:00
Matt Woodrow
f75e8a0868 Bug 1315144 - Add new non-fatal media error so that we recreate decoders when the GPU process crashes. r=jya 2016-11-08 15:22:37 +13:00