searching for reviewer(gerald)
0d0995d0991141bacce45733ad194a122af1da82: Bug 1322587: [MSE] P2. Allow additional top-level boxes. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 09 Dec 2016 09:21:21 -1000 - rev 325552
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1322587: [MSE] P2. Allow additional top-level boxes. r=gerald MozReview-Commit-ID: AE6EaPnOiJu
9dfd492a1bf78aec2de87c7f2f5eba9d237c351f: Bug 1322587: [MSE] P1. Fix error message. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 09 Dec 2016 09:03:09 -1000 - rev 325551
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1322587: [MSE] P1. Fix error message. r=gerald MozReview-Commit-ID: E6IH5t2uBKg
99d76c84603d08dcccbd85630c5a62aebc909694: Bug 1309163: P2. Add AnnexB::HasPPS methods. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 24 Oct 2016 17:23:02 +1100 - rev 325345
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1309163: P2. Add AnnexB::HasPPS methods. r=gerald MozReview-Commit-ID: 7M58OXvCp5q
c58bfbe2800682c2c2c85716d3155e30a33997ad: Bug 1309163: Add BitReader::BitsLeft method. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 24 Oct 2016 17:22:05 +1100 - rev 325344
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1309163: Add BitReader::BitsLeft method. r=gerald MozReview-Commit-ID: 9HOunJRb1ht
52cc3c8b17c96e2a008fedaec7149427d1b31530: Bug 1320705: P8. Don't use stagefright to decode vorbis. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 05 Dec 2016 07:41:54 +1100 - rev 325262
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1320705: P8. Don't use stagefright to decode vorbis. r=gerald Fixing the codec mimetype in P6, had the side effect of Android PDM trying to use the stagefright decoder which crashes. MozReview-Commit-ID: 1OSGGCTdGFO
328475b7579ca8ef7b019703b9a10dfbc8a802dd: Bug 1320705: P7. Ensure audio decoder is recreated when chaining encountered. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 02 Dec 2016 12:28:28 +1100 - rev 325261
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1320705: P7. Ensure audio decoder is recreated when chaining encountered. r=gerald The Opus decoder will error if we feed it new data if it has seen the last packet of a stream. MozReview-Commit-ID: Kw3cZuEKxie
05a07920b83263f12235f283c5869be656dba405: Bug 1320705: P6. Fix codec mimetypes. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 02 Dec 2016 12:23:23 +1100 - rev 325260
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1320705: P6. Fix codec mimetypes. r=gerald MozReview-Commit-ID: FozWtbvDNnA
255480c6f63c3cc36b491f9abf1ff1bed456bd0f: Bug 1320705: P1. Fix function prototyping. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Wed, 30 Nov 2016 17:18:58 +1100 - rev 325255
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1320705: P1. Fix function prototyping. r=gerald Returning already_AddRefed is by convention preferred. MozReview-Commit-ID: 1UyIcyBm923
928ed9aab0b9a46c728758fd1f7959fa975f9350: Bug 1320895 - Test ADTS frame size calculation. r=gerald
Ralph Giles <giles@mozilla.com> - Wed, 30 Nov 2016 14:38:52 -0800 - rev 324881
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1320895 - Test ADTS frame size calculation. r=gerald We don't generate timestamps for ADTS but we can verify the Init Segment Range and Media Segment Range returned by the parser match the frame size declared in the Init Segment header. MozReview-Commit-ID: FCZfxn9b69R
ff6eea6062d469834591bc94a4c01ecdb76c0f8e: Bug 1320895 - Correct ADTS frame size calculation. r=gerald
Ralph Giles <giles@mozilla.com> - Wed, 30 Nov 2016 13:23:18 -0800 - rev 324880
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1320895 - Correct ADTS frame size calculation. r=gerald This should be bitwise OR rather than logical OR, which just returns 1. This code has clearly never worked, but it isn't currently called. MozReview-Commit-ID: 9Iuy7a7P85O
1e5621e43ac66ff8e06fe9a682780c3a187996d3: Bug 1320752 - remove mozilla/Function.h; r=gerald
Nathan Froyd <froydnj@mozilla.com> - Mon, 28 Nov 2016 11:03:53 -0500 - rev 324475
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1320752 - remove mozilla/Function.h; r=gerald We have std::function available now, which is likely to be somewhat more efficient.
f24049bdb0478ab51140e2718406cb02cd7bb33e: Bug 1320386 - Block D3D11 for the 64-bit version of the 9.17.10.2857 driver. r=gerald
Marco Castelluccio <mcastelluccio@mozilla.com> - Mon, 28 Nov 2016 11:59:51 +0000 - rev 324438
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1320386 - Block D3D11 for the 64-bit version of the 9.17.10.2857 driver. r=gerald
4694f43926bcbea37bf5656b51ce29217bfa5287: Bug 1320614 - Remove MediaKeys::mNodeId as it's only ever written and never read. r=gerald
Chris Pearce <cpearce@mozilla.com> - Mon, 28 Nov 2016 12:12:34 +1300 - rev 324424
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1320614 - Remove MediaKeys::mNodeId as it's only ever written and never read. r=gerald MozReview-Commit-ID: BWfrjzdfph
0390e208038169bbdfa832155bef6359af4775fa: Bug 1316215 - Merge SelectGMP and LaunchGMP into one synchronous IPC operation. r=gerald
Chris Pearce <cpearce@mozilla.com> - Tue, 22 Nov 2016 14:17:59 +1300 - rev 324049
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1316215 - Merge SelectGMP and LaunchGMP into one synchronous IPC operation. r=gerald We were seeing almost permaorange failures in the WebRTC H.264/GMP tests due to the GMP being shutdown in the parent process in between the content process performing an OOP select operation and then performing an OOP launch operation. That is, in GeckoMediaPluginServiceChild::GetContentParent() in between the SendSelectGMP completing and the SendLaunchGMP completing, the GMP would shutdown and so when the launch operation ran in the main process it would fail. The select and launch are seperate operations so that the crash handler can be reported to the content process and an association can be made in the content process between the plugin ID and the crash helper before we try to launch the GMP. This is so that if the GMP crashes on startup, we're ready to handle the crash. However it turns out that if the GMP crashes on startup, the crash report message comes in after another round of the event/IPC message loop. So we actually do have time in the content process to connect the crash helper after the launch fails. So in order to fix the problem of the GMP shutting down in between select and launch, we can partially revert the changes I made in Bug 1267918 to merge selecting and launching GMPs back into a single operation. MozReview-Commit-ID: 5n4T1Gqlvr3
efff1ff587e32ef436efe95ef3153b44ff0ab1a5: Bug 1316215 - Make GMPParent::IsUsed() take into account whether there are pending GetContentParent calls. r=gerald
Chris Pearce <cpearce@mozilla.com> - Fri, 18 Nov 2016 12:37:14 +1300 - rev 324048
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1316215 - Make GMPParent::IsUsed() take into account whether there are pending GetContentParent calls. r=gerald In a similar vein to the previous patch, while we're waiting on a GetContentParent promise to resolve, we don't want the GMPParent to shutdown. So make IsUsed() check whether we're waiting on a GetContentParent promise to resolve, so we don't pull the rug out from under any code waiting to get a content parent to bridge a GMP. MozReview-Commit-ID: 8cTCuXLXMsK
4be3169b9d0257f3e0182dd1ad5d60a9a98eef48: Bug 1316215 - Block GMPContentParent close while a GMPService::GetContentParent is being processed. r=gerald
Chris Pearce <cpearce@mozilla.com> - Wed, 16 Nov 2016 10:59:08 +1300 - rev 324047
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1316215 - Block GMPContentParent close while a GMPService::GetContentParent is being processed. r=gerald When GMPService::GetContentParent returns a MozPromise, we end up failing in test_peerConnection_scaleResolution.html with e10s enabled because we Close() the GMPContentParent twice. The test causes two GMPVideoEncoderParents to be created. When the number of IPDL actors on the GMPContentParent reach 0, we close the IPC connection. With GetContentParent() returning a MozPromise, it's more async, and so we can end up requesting the content parent in order to create the second GMPVideoEncoderParent, but while we're waiting for the promise to resolve the previous GMPVideoEncoderParent is destroyed and the GMPContentParent closes its IPC connection. Then the GetContentParent promise resolves, and that fails to operate correctly since it's closed its IPC connection. My solution here is to add a "blocker" that prevents the GMPContentParent from being shutdown while we're waiting for the GetContentParent promise to resolve. MozReview-Commit-ID: HxBkFkmv0tV
cb29571e849184f019cf7343afb0ac6d36fa0e11: Bug 1316215 - Promisify GMPService GetGMPContentParent and GetServiceChild. r=gerald
Chris Pearce <cpearce@mozilla.com> - Wed, 09 Nov 2016 15:52:22 +1300 - rev 324046
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1316215 - Promisify GMPService GetGMPContentParent and GetServiceChild. r=gerald MozReview-Commit-ID: 5ydGmZWGzWG
3e36c1367ba2453e8325fbd1f9487255f345bd1f: Bug 1319543: Prevent overflow if sample's time is negative. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Wed, 23 Nov 2016 11:21:42 +1100 - rev 323954
Push 24 by maklebus@msu.edu at Tue, 20 Dec 2016 03:11:33 +0000
Bug 1319543: Prevent overflow if sample's time is negative. r=gerald MozReview-Commit-ID: I94vvJccOqE
c7e0356aac9b09de5f2e402a000d6eed706b6842: Bug 1319277: P2. Bump VP9 benchmark version to force a new run. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 22 Nov 2016 17:26:57 +1100 - rev 323783
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1319277: P2. Bump VP9 benchmark version to force a new run. r=gerald ffvp9 3.2 added IDCT AVX2 optimisations. Perfs show up to 65% speed improvement over the AVX1/SSE3 versions. MozReview-Commit-ID: 4asJVb6FFYI
481e7fe6322632aacd36c6b4f2d5719173f9a0b4: Bug 1319277: P1. [ffmpeg] Resync ffvpx with 3.2 branch. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 22 Nov 2016 13:40:10 +1100 - rev 323782
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1319277: P1. [ffmpeg] Resync ffvpx with 3.2 branch. r=gerald Sync at version n3.2-65-gee56777 MozReview-Commit-ID: oFDGCF2dru
138abd270223714144ab8010e80499909980e406: Bug 1318833 - Part 3: Remove unused Silverlight checks from Decoder Doctor. r=gerald
Chris Peterson <cpeterson@mozilla.com> - Mon, 21 Nov 2016 21:11:04 -0800 - rev 323779
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1318833 - Part 3: Remove unused Silverlight checks from Decoder Doctor. r=gerald
3e47666dbac8edfd480d8b349d6e630d0c709fde: Bug 1317822 - Move GMPCrashHelper into its own file. r=gerald
Chris Pearce <cpearce@mozilla.com> - Wed, 16 Nov 2016 11:35:36 +1300 - rev 322663
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1317822 - Move GMPCrashHelper into its own file. r=gerald MozReview-Commit-ID: 7CinZ2Y2Fmz
b856b7987192e7e883b6e4d6de19de3e850c4cd9: Bug 1306314 - Allow Widevine CDM process to contain multiple CDM instances and to have persistent storage. r=gerald
Chris Pearce <cpearce@mozilla.com> - Mon, 14 Nov 2016 11:43:45 +1300 - rev 322421
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1306314 - Allow Widevine CDM process to contain multiple CDM instances and to have persistent storage. r=gerald This change ensures that we don't create a new random node Id for every MediaKeys object using Widevine - which has the effect of ensuring Widevine CDMs that are same origin get created in the same process, and that persistent storage can be used and retrieved. MozReview-Commit-ID: K55rkcu9jWo
6a563a098351b1c1b2e1e6c2f20ff732b1076b91: Bug 1306314 - Use decryptor ID in WidevineAdapter to link decryptors with decoders. r=gerald
Chris Pearce <cpearce@mozilla.com> - Fri, 11 Nov 2016 15:39:10 +1300 - rev 322420
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1306314 - Use decryptor ID in WidevineAdapter to link decryptors with decoders. r=gerald Store a mapping of decryptor ID to the CDM instance that the corresponding WidevineDecryptor is using. This allows us to link GMPDecryptor instances with the corresponding GMPVideoDecoder. The CDM is stored inside the CDMWrapper, so that we destroy the CDM instance when the last reference to the CDM is dropped. MozReview-Commit-ID: FQYzh77yjoC
e183a3804fbbb6f25228daacf9cf217548ff0129: Bug 1306314 - Pipe decryptor ID through to WidevineAdapter. r=gerald
Chris Pearce <cpearce@mozilla.com> - Fri, 11 Nov 2016 14:55:56 +1300 - rev 322419
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1306314 - Pipe decryptor ID through to WidevineAdapter. r=gerald MozReview-Commit-ID: HqRoImDhuFl
893b62158c45c65fdb5702866b2c0a6ff5a33cf7: Bug 1306314 - Pass decryptor ID to GMPVideoDecoder constructor. r=gerald
Chris Pearce <cpearce@mozilla.com> - Mon, 14 Nov 2016 11:07:02 +1300 - rev 322418
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1306314 - Pass decryptor ID to GMPVideoDecoder constructor. r=gerald Retrieve the ID of the GMPDecryptor from the GMPCDMProxy, and pass that through to the GMPVideoDecoder's constructor. MozReview-Commit-ID: IuNsSroZ9Zu
2b0c612e87232b099830ae3b918d2080cc73960c: Bug 1306314 - Add an ID to GMPDecryptor instances, reflect that on CDMProxy. r=gerald
Chris Pearce <cpearce@mozilla.com> - Fri, 11 Nov 2016 12:10:43 +1300 - rev 322417
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1306314 - Add an ID to GMPDecryptor instances, reflect that on CDMProxy. r=gerald This enables us to identify GMPDecryptor instances in the child process, so that in a later patch when we create a GMPVideoDecoder instance, we can associate it with a GMPDecryptor. Then the cdm::ContentDecryptionModule8 instance that these two actors are adapted to can know whom it's supposed to respond to. We use the IPDL PGMPDecryptorChild actor ID as the GMPDecryptor's ID. This is unique per GMP process, which is sufficient. MozReview-Commit-ID: 7NKND9VjPUW
5c5600e1f30fafc8065875d0fd6d25b9d238e334: Bug 1306314 - Allow Widevine CDM process to contain multiple CDM instances and to have persistent storage. r=gerald
Chris Pearce <cpearce@mozilla.com> - Mon, 14 Nov 2016 11:43:45 +1300 - rev 322274
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1306314 - Allow Widevine CDM process to contain multiple CDM instances and to have persistent storage. r=gerald This change ensures that we don't create a new random node Id for every MediaKeys object using Widevine - which has the effect of ensuring Widevine CDMs that are same origin get created in the same process, and that persistent storage can be used and retrieved. MozReview-Commit-ID: K55rkcu9jWo
d37e2830956092bbffd4b580d9b36d8b658cd41c: Bug 1306314 - Use decryptor ID in WidevineAdapter to link decryptors with decoders. r=gerald
Chris Pearce <cpearce@mozilla.com> - Fri, 11 Nov 2016 15:39:10 +1300 - rev 322273
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1306314 - Use decryptor ID in WidevineAdapter to link decryptors with decoders. r=gerald Store a mapping of decryptor ID to the CDM instance that the corresponding WidevineDecryptor is using. This allows us to link GMPDecryptor instances with the corresponding GMPVideoDecoder. The CDM is stored inside the CDMWrapper, so that we destroy the CDM instance when the last reference to the CDM is dropped. MozReview-Commit-ID: FQYzh77yjoC
e2a5702d96b0a81a2f47f919b843b4b155a02a42: Bug 1306314 - Pipe decryptor ID through to WidevineAdapter. r=gerald
Chris Pearce <cpearce@mozilla.com> - Fri, 11 Nov 2016 14:55:56 +1300 - rev 322272
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1306314 - Pipe decryptor ID through to WidevineAdapter. r=gerald MozReview-Commit-ID: HqRoImDhuFl
9343cb73c218292bbbd72ce499c125b525759aa0: Bug 1306314 - Pass decryptor ID to GMPVideoDecoder constructor. r=gerald
Chris Pearce <cpearce@mozilla.com> - Mon, 14 Nov 2016 11:07:02 +1300 - rev 322271
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1306314 - Pass decryptor ID to GMPVideoDecoder constructor. r=gerald Retrieve the ID of the GMPDecryptor from the GMPCDMProxy, and pass that through to the GMPVideoDecoder's constructor. MozReview-Commit-ID: IuNsSroZ9Zu
9e9727bdc48497378e2dfb846a7f9aca2d25ef5c: Bug 1306314 - Add an ID to GMPDecryptor instances, reflect that on CDMProxy. r=gerald
Chris Pearce <cpearce@mozilla.com> - Fri, 11 Nov 2016 12:10:43 +1300 - rev 322270
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1306314 - Add an ID to GMPDecryptor instances, reflect that on CDMProxy. r=gerald This enables us to identify GMPDecryptor instances in the child process, so that in a later patch when we create a GMPVideoDecoder instance, we can associate it with a GMPDecryptor. Then the cdm::ContentDecryptionModule8 instance that these two actors are adapted to can know whom it's supposed to respond to. We use the IPDL PGMPDecryptorChild actor ID as the GMPDecryptor's ID. This is unique per GMP process, which is sufficient. MozReview-Commit-ID: 7NKND9VjPUW
60bf1293eabf295bef0dd400ebde9668af3bd9ae: Bug 1316205: P3. Remove WaveDecoder::IsEnable(). r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 10 Nov 2016 11:53:07 +1100 - rev 321885
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1316205: P3. Remove WaveDecoder::IsEnable(). r=gerald It would always return true, and there's already a MediaDecoder::IsWaveEnabled() MozReview-Commit-ID: 6FXgMRMnVb9
eb1b930cef78eac6198213ccc013fd5c40519f51: Bug 1315567: P4. Add mochitest. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 07 Nov 2016 11:43:47 +1100 - rev 321380
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1315567: P4. Add mochitest. r=gerald MozReview-Commit-ID: JtDJhQasdu7
bfaa975490cfbe35626cb9a3b783844266d97769: Bug 1315567: P3. Expose flac codec as being available in mp4. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 07 Nov 2016 10:57:28 +1100 - rev 321379
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1315567: P3. Expose flac codec as being available in mp4. r=gerald MozReview-Commit-ID: 7C6W1KGQ1jE
d5ae76734cb6f12b2a0eba3a1d3b306a18af0adc: Bug 1315567: P2. Expose opus codec as being available in mp4. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 07 Nov 2016 11:08:32 +1100 - rev 321378
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1315567: P2. Expose opus codec as being available in mp4. r=gerald MozReview-Commit-ID: GU8me70Igsq
d7fec66162b778ce395f77045e0779a6c79ed6f8: Bug 1315567: P1. Remove invalid test. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 07 Nov 2016 12:56:55 +1100 - rev 321377
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1315567: P1. Remove invalid test. r=gerald audio/opus is a codec type, not a MP4 container one. aType can never be audio/opus MozReview-Commit-ID: E8tONlF4COA
f26d5e27e1b69638bb9c0e24a1936f40ac7b40dc: Bug 1314533: [MSE] P8. Update wpt expected results. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 06 Nov 2016 22:01:09 +1100 - rev 321374
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314533: [MSE] P8. Update wpt expected results. r=gerald We pass those tests now. MozReview-Commit-ID: 4UHkTr4uJEb
87882fa5ef49414b7dd978e35a01aa21a7166095: Bug 1314533: Do not expect init segment to be parsed when appending corrupted data. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 06 Nov 2016 21:57:06 +1100 - rev 321373
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314533: Do not expect init segment to be parsed when appending corrupted data. r=gerald From the Segment Parser Loop definition: https://w3c.github.io/media-source/index.html#sourcebuffer-segment-parser-loop "2. If the input buffer contains bytes that violate the SourceBuffer byte stream format specification, then run the append error algorithm and abort this algorithm." The test appends data with a valid init segment followed by corrupted content. The corrupted content satisfies the condition that the input buffer contains bytes that violate byte stream format specification. As such, the append error algorithm is to be run prior parsing the init segment. Add a new test verifying the defined behavior.
ca6e6783a37994cea645cb95308c52b0851becbe: Bug 1314533: [MSE] P6. Allow to detect error during preliminary container parsing. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 04 Nov 2016 23:09:47 +1100 - rev 321372
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314533: [MSE] P6. Allow to detect error during preliminary container parsing. r=gerald MozReview-Commit-ID: KZ858ISWDmu
7c84e5453411694e379a08bed40f2f0db46aa475: Bug 1314533: [MSE] P4. Reject invalid webm block. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 04 Nov 2016 02:16:53 +1100 - rev 321370
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314533: [MSE] P4. Reject invalid webm block. r=gerald An invalid webm block at this stage is either a non init segment or non media segment. MozReview-Commit-ID: 46NrhCwqas1
754c160ecbed3f2f34e05916e1c814b36403ac04: Bug 1314533: [MSE] P3. Reject invalid MP4 boxes. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 04 Nov 2016 01:49:19 +1100 - rev 321369
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314533: [MSE] P3. Reject invalid MP4 boxes. r=gerald MozReview-Commit-ID: F0FcQDqsLk7
33c8e33b0bb013da185f6bfaeeddf3f0074c067c: Bug 1314533: [MSE] P2. Reject appendBuffer if invalid data found. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Wed, 02 Nov 2016 21:39:14 +1100 - rev 321368
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314533: [MSE] P2. Reject appendBuffer if invalid data found. r=gerald This still requires all the ContainerParser to be updated in properly handling errors. MozReview-Commit-ID: A7gDmXSJXmc
2d85e1b19808d93ce58072b48a7fbd6bd2f525d9: Bug 1314533: [MSE] P1. Change member prototype. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Wed, 02 Nov 2016 21:29:13 +1100 - rev 321367
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314533: [MSE] P1. Change member prototype. r=gerald MozReview-Commit-ID: L8H4oDoTm2b
d51a0f8c1c7605e0df7b61fe4c31a3f5ec1fd58f: Bug 1302656: P2. Update expected wpt results. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Wed, 02 Nov 2016 17:43:20 +1100 - rev 321366
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1302656: P2. Update expected wpt results. r=gerald MozReview-Commit-ID: 4ebvj66fQng
e8ce7f5150920335be896dbc38dde25838bb54c9: Bug 1302656: P2. Update expected wpt results. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Wed, 02 Nov 2016 17:43:20 +1100 - rev 321099
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1302656: P2. Update expected wpt results. r=gerald MozReview-Commit-ID: 4ebvj66fQng
0e9928989e9c655af0cfef3dafbfa23e85cd3695: Bug 1314858 - Refactor multiple callers of HasPluginForAPI into a helper. r=gerald
Chris Pearce <cpearce@mozilla.com> - Thu, 03 Nov 2016 14:33:31 +1300 - rev 320988
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314858 - Refactor multiple callers of HasPluginForAPI into a helper. r=gerald I've repeated myself a few times, so make a helper to make determining which GMPs are available easier. MozReview-Commit-ID: 2fFLeaA5o8u
bc420ca09b6cf6cf3563d8ffb0401cde6c22fb42: Bug 1314858 - Remove PContent.NotifyGMPsChanged. r=gerald
Chris Pearce <cpearce@mozilla.com> - Thu, 03 Nov 2016 16:26:54 +1300 - rev 320987
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314858 - Remove PContent.NotifyGMPsChanged. r=gerald MozReview-Commit-ID: FMb3AbEEnl5
d0325b2bd63cdb685e1df4393eb03e6fc14c0400: Bug 1314858 - Remove GMPDecoderModule::UpdateUsableCodecs. r=gerald
Chris Pearce <cpearce@mozilla.com> - Wed, 02 Nov 2016 15:31:21 +1300 - rev 320986
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314858 - Remove GMPDecoderModule::UpdateUsableCodecs. r=gerald MozReview-Commit-ID: 7678JImLL9T
a37b70e85582ca5c46d5f9e7874c340472f27af1: Bug 1314858 - Make GMPDecoderModule::SupportsMimeType call GMPService::HasPluginForAPI() directly. r=gerald
Chris Pearce <cpearce@mozilla.com> - Wed, 02 Nov 2016 10:04:39 +1300 - rev 320985
Push 21 by maklebus@msu.edu at Thu, 01 Dec 2016 06:22:08 +0000
Bug 1314858 - Make GMPDecoderModule::SupportsMimeType call GMPService::HasPluginForAPI() directly. r=gerald MozReview-Commit-ID: 3VblaPbTQFJ