1a5b3ef58354d2199cb018fbc6046e50a77c332a: Bug 1297306 - part2:make the implementation of nsLineBox::LastChild() be behind DEBUG_FRAME_DUMP flag. draft
Jeremy Chen <jeremychen@mozilla.com> - Mon, 29 Aug 2016 16:46:42 +0800 - rev 406655
Push 27787 by jichen@mozilla.com at Mon, 29 Aug 2016 08:47:21 +0000
Bug 1297306 - part2:make the implementation of nsLineBox::LastChild() be behind DEBUG_FRAME_DUMP flag. This should've been covered in Bug 956447. The declaration is already behind DEBUG_FRAME_DUMP in nsLineBox.h. Just move the implementation to agreee with that. MozReview-Commit-ID: 9N0WxKkajF1
45b4d4cddc648af2c7b77439aaddec16b2464ec0: Bug 1297306 - part1:remove unused NS_STYLE_CLEAR_* condition. draft
Jeremy Chen <jeremychen@mozilla.com> - Mon, 29 Aug 2016 16:46:42 +0800 - rev 406654
Push 27787 by jichen@mozilla.com at Mon, 29 Aug 2016 08:47:21 +0000
Bug 1297306 - part1:remove unused NS_STYLE_CLEAR_* condition. NS_STYLE_CLEAR_NONE has been defined to 0 for like forever, so this code should never be run. Remove it. MozReview-Commit-ID: IQ73H6QGsPX
bcff709130397ebad1b9d97d8d9bc0a96119184e: Bug 1297306 - part7:replace StyleClear related NS_ASSERTION with MOZ_ASSERT. draft
Jeremy Chen <jeremychen@mozilla.com> - Mon, 29 Aug 2016 16:38:57 +0800 - rev 406653
Push 27786 by jichen@mozilla.com at Mon, 29 Aug 2016 08:39:41 +0000
Bug 1297306 - part7:replace StyleClear related NS_ASSERTION with MOZ_ASSERT. MozReview-Commit-ID: Dzd3uOoQCI
0635d9d2857277284e5532d8dfac874330197e1f: Bug 1297306 - part6:replace NS_STYLE_CLEAR_* with StyleClear enum class. draft
Jeremy Chen <jeremychen@mozilla.com> - Mon, 29 Aug 2016 16:38:57 +0800 - rev 406652
Push 27786 by jichen@mozilla.com at Mon, 29 Aug 2016 08:39:41 +0000
Bug 1297306 - part6:replace NS_STYLE_CLEAR_* with StyleClear enum class. After using enum class, a switch-case warning in CombineBreakType is caught. This is one of such kind safty checks that we would like to gain. Fix it by adding default case for switch-case in CombineBreakType. MozReview-Commit-ID: BdS3LPN6qzX
1a91917d0f8ef98195868935b912ce77233cb55d: Bug 1297306 - part5:create enum constructors for EnumTable. draft
Jeremy Chen <jeremychen@mozilla.com> - Mon, 29 Aug 2016 16:38:57 +0800 - rev 406651
Push 27786 by jichen@mozilla.com at Mon, 29 Aug 2016 08:39:41 +0000
Bug 1297306 - part5:create enum constructors for EnumTable. Enable nsAttrValue::EnumTable to be initialized with enum. So, we could get rid of the castings in EnumTable. Fix EnumTable initialization comment. For those untyped enumerations, declare them with uint8_t, as to other typed enumerations with type size larger than int16_t, force casting to int16_t. Use {nullptr,0} instead of {0} to represent the last entry. MozReview-Commit-ID: 7Dma3Apkmxj
23ee557bded81661355097b2bb2718abc0d76758: Bug 1297306 - part4:move IsEnumFittingWithin to mfbt/EnumTypeTraits.h. draft
Jeremy Chen <jeremychen@mozilla.com> - Mon, 29 Aug 2016 16:38:57 +0800 - rev 406650
Push 27786 by jichen@mozilla.com at Mon, 29 Aug 2016 08:39:41 +0000
Bug 1297306 - part4:move IsEnumFittingWithin to mfbt/EnumTypeTraits.h. With this change, we could share this EnumTypeTraits between files easily. MozReview-Commit-ID: 9Q2augati7l
c98ae9401b2c97b11948bb80e152d9fc74ebb108: Bug 1297306 - part3:make BreakTypeToString() to be a member of nsLineBox. draft
Jeremy Chen <jeremychen@mozilla.com> - Mon, 29 Aug 2016 16:38:56 +0800 - rev 406649
Push 27786 by jichen@mozilla.com at Mon, 29 Aug 2016 08:39:41 +0000
Bug 1297306 - part3:make BreakTypeToString() to be a member of nsLineBox. With this change, we could export BreakTypeToString() to other files that desire to print break type for debugging. MozReview-Commit-ID: 34m1BWAmZTB
01f65e1feb9b734ec7af8c233da142b82377cc9a: Bug 1297306 - part2:make the implementation of nsLineBox::LastChild() be behind DEBUG_FRAME_DUMP flag. draft
Jeremy Chen <jeremychen@mozilla.com> - Mon, 29 Aug 2016 16:38:56 +0800 - rev 406648
Push 27786 by jichen@mozilla.com at Mon, 29 Aug 2016 08:39:41 +0000
Bug 1297306 - part2:make the implementation of nsLineBox::LastChild() be behind DEBUG_FRAME_DUMP flag. This should've been covered in Bug 956447. The declaration is already behind DEBUG_FRAME_DUMP in nsLineBox.h. Just move the implementation to agreee with that. MozReview-Commit-ID: 9N0WxKkajF1
a9c03fcdadacf7fae9c373b21c41f7cbee7a163d: Bug 1297306 - part1:remove unused NS_STYLE_CLEAR_* condition. draft
Jeremy Chen <jeremychen@mozilla.com> - Mon, 29 Aug 2016 16:38:56 +0800 - rev 406647
Push 27786 by jichen@mozilla.com at Mon, 29 Aug 2016 08:39:41 +0000
Bug 1297306 - part1:remove unused NS_STYLE_CLEAR_* condition. NS_STYLE_CLEAR_NONE has been defined to 0 for like forever, so this code should never be run. Remove it. MozReview-Commit-ID: IQ73H6QGsPX
29db5fe94296e0636f0ac61c95b50601e8f848d0: Bug 1298704 - Use flag to avoid calling AccessibleCaretEventHub::Reflow() recursively. r?mats draft
Ting-Yu Lin <tlin@mozilla.com> - Mon, 29 Aug 2016 15:36:45 +0800 - rev 406646
Push 27785 by bmo:tlin@mozilla.com at Mon, 29 Aug 2016 08:02:29 +0000
Bug 1298704 - Use flag to avoid calling AccessibleCaretEventHub::Reflow() recursively. r?mats MozReview-Commit-ID: Jkf7sSqxG2d
88cec41626bf83be5d54ce13e1566846e547ba6d: Bug 1298704 - Use flag to avoid calling AccessibleCaretEventHub::Reflow() recursively. r?mats draft
Ting-Yu Lin <tlin@mozilla.com> - Mon, 29 Aug 2016 15:36:45 +0800 - rev 406645
Push 27784 by bmo:tlin@mozilla.com at Mon, 29 Aug 2016 07:56:52 +0000
Bug 1298704 - Use flag to avoid calling AccessibleCaretEventHub::Reflow() recursively. r?mats MozReview-Commit-ID: Jkf7sSqxG2d
def5b2ffbff3fa45258585a8942f74f6597949b3: Bug 1298594: P5. Fix mochitest. r?gerald draft
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 29 Aug 2016 16:07:38 +1000 - rev 406644
Push 27783 by bmo:jyavenard@mozilla.com at Mon, 29 Aug 2016 07:55:20 +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
e54896c8837ab050f359cd35afc8fe99340c2893: Bug 1298594: P4. When calculating the next frame status, only check that we have at least one. r?jwwang draft
Jean-Yves Avenard <jyavenard@mozilla.com> - Mon, 29 Aug 2016 15:46:56 +1000 - rev 406643
Push 27783 by bmo:jyavenard@mozilla.com at Mon, 29 Aug 2016 07:55:20 +0000
Bug 1298594: P4. When calculating the next frame status, only check that we have at least one. r?jwwang Otherwise, we will set readyState to HAVE_CURRENT_DATA one frame too early. This is a partial reversal of commit 3577cfb4353 (part of bug 1143575), MDSM::HaveNextFrameData is only used for readyState calculation. MozReview-Commit-ID: H92Bmiki49Q
f0c5b9fa7d50107668cc5665292c845074f33141: Bug 1298594: P3. Ensure currentTime is updated prior changing readyState. r?jwwang draft
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 28 Aug 2016 21:20:52 +1000 - rev 406642
Push 27783 by bmo:jyavenard@mozilla.com at Mon, 29 Aug 2016 07:55:20 +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
01f3436435ccb1fb189ec7fb884cdf4405716c84: Bug 1298594: P2. Fire waiting event when readyState move back to HAVE_CURRENT_DATA. r?jwwang draft
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 27 Aug 2016 22:23:52 +1000 - rev 406641
Push 27783 by bmo:jyavenard@mozilla.com at Mon, 29 Aug 2016 07:55:20 +0000
Bug 1298594: P2. Fire waiting event when readyState move back to HAVE_CURRENT_DATA. r?jwwang MozReview-Commit-ID: BpwYY6njXGC
67a1ceb1b3a5f58add879065d5ef5d3f1b16ca0e: Bug 1298594: [MSE] P1. Add mochitest to verify correct behavior. r?gerald draft
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 27 Aug 2016 00:08:38 +1200 - rev 406640
Push 27783 by bmo:jyavenard@mozilla.com at Mon, 29 Aug 2016 07:55:20 +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
58e91ed9f1b5cc62f6540f6daabf7a2e73fa8fc9: Bug 1297036: [MSE] P6. Fix invalid mochitest. r?gerald draft
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 28 Aug 2016 12:26:40 +1000 - rev 406639
Push 27783 by bmo:jyavenard@mozilla.com at Mon, 29 Aug 2016 07:55:20 +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
b96f8193fe83c2520d6b9ce6181f0a389213a64e: Bug 1297036: [MSE] P5. Make fuzz research consistent. r?gerald draft
Jean-Yves Avenard <jyavenard@mozilla.com> - Sat, 27 Aug 2016 23:51:40 +1000 - rev 406638
Push 27783 by bmo:jyavenard@mozilla.com at Mon, 29 Aug 2016 07:55:20 +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
ab629be2f396296c8295935964e4a84fe941d30c: Bug 1297036: [MSE] P4. Only report end of stream when reaching the end. r?gerald draft
Jean-Yves Avenard <jyavenard@mozilla.com> - Sun, 28 Aug 2016 01:00:39 +1200 - rev 406637
Push 27783 by bmo:jyavenard@mozilla.com at Mon, 29 Aug 2016 07:55:20 +0000
Bug 1297036: [MSE] P4. Only report end of stream when reaching the end. r?gerald MozReview-Commit-ID: 5EWhBVnscXY
20162730737c9918b6595403be9231057a4cc865: Bug 1297036: P3. Revert "Bug 1293646: [MSE] P2. Only reject a seek request with EOS if it's passed the explicit duration." r?gerald draft
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 26 Aug 2016 19:30:50 +1200 - rev 406636
Push 27783 by bmo:jyavenard@mozilla.com at Mon, 29 Aug 2016 07:55:20 +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
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip