0d7beb4dc862e03cc2d5410cd789eef0b22698c4: Bug 1291796 - Minor fixes to harness unit tests; r=maja_zf
Anjana Vakil <anjanavakil@gmail.com> - Fri, 26 Aug 2016 15:52:29 +0200 - rev 311696
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1291796 - Minor fixes to harness unit tests; r=maja_zf In test_marionette_runner.py, fix pytest warning raised when importing TestManifest class directly in global namespace. In test_marionette_arguments.py, improve readability by shortening/changing some names and removing unnecessary comments (not needed as code is self-explanatory). MozReview-Commit-ID: GDzxlEqb7MB
6a81df9a2129e940f62961e80203554c41c3109a: Bug 1291796 - Split Marionette harness tests by class; r=maja_zf
Anjana Vakil <anjanavakil@gmail.com> - Fri, 26 Aug 2016 15:47:09 +0200 - rev 311695
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1291796 - Split Marionette harness tests by class; r=maja_zf Split the Marionette harness unit tests, all of which were previously located in the single module test_marionette_runner.py, into modules that each test a specific class or group of similar classes. This will make it easier to add and locate new tests in the future, based on the class they are testing. The new module structure within tests/harness_unit/ is: * test_marionette_harness.py - tests for MarionetteHarness and the command-line interface * test_marionette_arguments.py - tests for MarionetteArguments (future: BaseMarionetteArguments and RemoteMarionetteArguments) * test_marionette_runner.py - tests for MarionetteTestRunner and BaseMarionetteTestRunner (future: MarionetteTextTestRunner) * test_marionette_test_result.py - tests for MarionetteTestResult (future: MarionetteTest) * conftest.py - pytest fixtures used in multiple modules (fixtures specific to a single module are defined in that module) MozReview-Commit-ID: CGh6Aa07lfV
6437da20140a6a9f10196dcb95e8aad48590930a: Bug 1298617: [MSE] P3. Optimize sample search by breaking loop early. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 28 Aug 2016 02:43:36 +1200 - rev 311694
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298617: [MSE] P3. Optimize sample search by breaking loop early. r=gerald MozReview-Commit-ID: 48YcQiy0p8S
cd44b608b9f8095d3e13bc9560971bbd2c6a76e9: Bug 1298617: [MSE] P2. Attempt to search the exactly matching sample first. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 28 Aug 2016 02:38:59 +1200 - rev 311693
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298617: [MSE] P2. Attempt to search the exactly matching sample first. r=gerald MozReview-Commit-ID: AW1T51n6WMl
f57482625bf51a39227869fe2d74cef12d771783: Bug 1298617: [MSE] P1. Don't attempt to estimate next sample time if exact value known. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 28 Aug 2016 02:34:44 +1200 - rev 311692
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298617: [MSE] P1. Don't attempt to estimate next sample time if exact value known. r=gerald MozReview-Commit-ID: 8DE9WHFsePt
b1da49e041bbbb508ccfeac3b0a1eb4bf91b55af: Bug 1298594: P5. Fix mochitest. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 29 Aug 2016 16:07:38 +1000 - rev 311691
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298594: P5. Fix mochitest. r=gerald The assumption was that the waiting event would be fired once the last frame prior the gap had been played. This is however incorrect, as per spec, the waiting event is to be fired once readyState is <= HAVE_CURRENT_DATA. So the waiting event is actually fired anytime between the start of the last frame and its end. MozReview-Commit-ID: AA4Qhn7okhB
2cdde44ebcc788af1a875217dc3022b1812d6538: Bug 1298594: P4. Pop the frame when current time is past the end of the current frame. r=jwwang
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 29 Aug 2016 15:46:56 +1000 - rev 311690
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298594: P4. Pop the frame when current time is past the end of the current frame. r=jwwang Most frames start when the previous one stop. MozReview-Commit-ID: H92Bmiki49Q
ded582db991fcbedb51f42980f9bc1aca93d8a5f: Bug 1298594: P3. Ensure currentTime is updated prior changing readyState. r=jwwang
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 28 Aug 2016 21:20:52 +1000 - rev 311689
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298594: P3. Ensure currentTime is updated prior changing readyState. r=jwwang Otherwise we get intermittent in mochitests checking the value of currenTime when events are fired MozReview-Commit-ID: AVktWrXochp
ce7dae2a0f061a4605bfc5102189fa6823ce9cb0: Bug 1298594: P2. Fire waiting event when readyState move back to HAVE_CURRENT_DATA. r=jwwang
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 27 Aug 2016 22:23:52 +1000 - rev 311688
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298594: P2. Fire waiting event when readyState move back to HAVE_CURRENT_DATA. r=jwwang MozReview-Commit-ID: BpwYY6njXGC
fb8f6e366481d3be76e79c8c737d91af2c4defeb: Bug 1298594: [MSE] P1. Add mochitest to verify correct behavior. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 27 Aug 2016 00:08:38 +1200 - rev 311687
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298594: [MSE] P1. Add mochitest to verify correct behavior. r=gerald 1- We shouldn't reach ended if we have a gap in the buffered range prior the end of the file (see bug 1297036) 2- Waiting should be fired when readyState goes below HAVE_FUTURE_DATA MozReview-Commit-ID: 18bEnkNpYvO
4c9dcf590dd3bb544e78cc15f096de7a552011ec: Bug 1297036: [MSE] P6. Fix invalid mochitest. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 28 Aug 2016 12:26:40 +1000 - rev 311686
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1297036: [MSE] P6. Fix invalid mochitest. r=gerald The test accidentally worked because any demuxing failures in ended mode would be treated as EOS. There's no audio between [0-3), so playback couldn't start MozReview-Commit-ID: 4B90CrVUTy4
1c3a3adf025651811c05f12980ca8a14bfc0dc59: Bug 1297036: [MSE] P5. Make fuzz research consistent. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 27 Aug 2016 23:51:40 +1000 - rev 311685
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1297036: [MSE] P5. Make fuzz research consistent. r=gerald The aim is to only allow skipping gaps of fuzz=500ms. MozReview-Commit-ID: 8uHxni2nPHI
2e03f016cece06b408c15817990eee9dc9986bec: Bug 1297036: [MSE] P4. Only report end of stream when reaching the end. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 28 Aug 2016 01:00:39 +1200 - rev 311684
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1297036: [MSE] P4. Only report end of stream when reaching the end. r=gerald MozReview-Commit-ID: 5EWhBVnscXY
1d7058e01654d6156e35aba3fb0554d5093110da: Bug 1297036: P3. Revert "Bug 1293646: [MSE] P2. Only reject a seek request with EOS if it's passed the explicit duration." r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 26 Aug 2016 19:30:50 +1200 - rev 311683
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1297036: P3. Revert "Bug 1293646: [MSE] P2. Only reject a seek request with EOS if it's passed the explicit duration." r=gerald This reverts commit 5a949eb358e27 Another more complete solution will follow. MozReview-Commit-ID: K3lTdrBxW7W
b70a6a4643884d96871d9bfb1f3d8b8ed12cb5b9: Bug 1297036: [MSE] P2. Make seek always succeed when attempting to seek past the end time. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 26 Aug 2016 19:23:29 +1200 - rev 311682
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1297036: [MSE] P2. Make seek always succeed when attempting to seek past the end time. r=gerald MozReview-Commit-ID: H2YJu7vY0aP
6fc901d7536105aff6cb37eb514048e8696ff4bd: Bug 1297036: [MSE] P1. Add mochitest to verify behavior. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 26 Aug 2016 23:10:00 +1200 - rev 311681
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1297036: [MSE] P1. Add mochitest to verify behavior. r=gerald Seeking in the buffered range should always succeed, even if we have no data in a given track MozReview-Commit-ID: FGjsDCNIdWC
7e473a486ce60a27fa806b2c5d2cb5324db886ce: Bug 1298606: P3. Fix coding style. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 29 Aug 2016 16:59:17 +1000 - rev 311680
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298606: P3. Fix coding style. r=gerald MozReview-Commit-ID: L5MHnYOkyDV
5a028fa1fda0c01a76031e11be1da696c3cab285: Bug 1298606: [MSE] P2. Properly determine next frame status in ended state. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 27 Aug 2016 22:27:22 +1200 - rev 311679
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298606: [MSE] P2. Properly determine next frame status in ended state. r=gerald MozReview-Commit-ID: 2m05GzauHes
2f3e74146cd13e1bc10ac9728c31dbc16ba6bbc8: Bug 1298606: P1. Add Intervals::ContainsStrict method. r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 27 Aug 2016 20:24:31 +1000 - rev 311678
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1298606: P1. Add Intervals::ContainsStrict method. r=gerald MozReview-Commit-ID: 7XA8xVUWSi7
59fed5da5f16aa1e87a420cc692cd814caeef251: Bug 1297218: [MSE] Remove unnecessary call to Find(). r=gerald
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 26 Aug 2016 16:51:18 +1200 - rev 311677
Push 81192 by kwierso@gmail.com at Tue, 30 Aug 2016 00:54:59 +0000
Bug 1297218: [MSE] Remove unnecessary call to Find(). r=gerald MozReview-Commit-ID: Af7mzXbLiNy
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip