Bug 1277978 - Part 4 - Actually cache the recent tabs count when it updates. r=liuche MozReview-Commit-ID: JUsM3qdxhIr

Bug 1277978 - Part 3 - Allow the RecentTabsAdapter to indicate whether the tab count is reliable. r=liuche After we've set the cached tabs count to display within the history adapter, we don't want to revise that number downwards as long as the RecentTabsAdapter hasn't yet checked all of its data sources. MozReview-Commit-ID: BMpiaEb3kGQ

Bug 1277978 - Part 2 - Allow saving a cached recent tabs count in BrowserApp. r=liuche Getting the total number of recently closed tabs involves waiting for Gecko to actually send the closed tabs to the Java UI. This means that (unless there are some "Tabs from last time" present) when showing the history panel we always start out with the Recently closed folder hidden and then unhide (and animate) it once we've finally received the closed tabs. Because this is visually distracting, we should cache the closed tabs count somewhere, so we can decide on the smart folder visibility as soon as the CombinedHistoryAdapter initialises. MozReview-Commit-ID: 8uYCbM7eiSt

Bug 1277978 - Part 1 - Hide Recent Tabs smart folder if there aren't any closed tabs to be shown. r=liuche This involves making the number of visible smart folders dynamic, so the history adapter can properly display its contents. MozReview-Commit-ID: 6b4V6IHB7BE

Bug 1271765 - Desktop video control visual refresh. r?jaws MozReview-Commit-ID: 1GfyGmrhgCs

Bug 1299070 - Merge all startup strings in startup.properties;r=bgrins MozReview-Commit-ID: 99zWNTHid21

Bug 1299070 - Move l10n.ELLIPSIS behind getter;r=bgrins This avoids loading the ellipsis properties file on browser startup. MozReview-Commit-ID: 8lfAeltfn10

Bug 1300337 - Replace None_ variants from nsStyleConsts.h with None; r?heycam MozReview-Commit-ID: CxHzbEzjLxT

Bug 1300296: P2. Don't rely on MP4 container to properly report if a frame is a keyframe. r?kentuckyfriedtakahe There are too many cases where the MP4 is improperly muxed and frames are incorrectly reported as keyframe. Instead we now look inside the H264 stream and check for IDR frames. We also ensure that the first frame returned after a seek or reset is always a keyframe. For plain MP4, seeking in those broken files will lead to broken A/V sync. The only way to fix this would be to check for the frame type when reading the samples table. However, this would require to read the entire stream which isn't viable. MozReview-Commit-ID: Cpv5y7HVD0N

Bug 1300296: P1. Add method to determine if an H264 frame is an I-Frame. r?jesup We do so by checking the frame data for NAL of type 5 as per ISO IEC 14496-2. MozReview-Commit-ID: JFeLysrZ6aG

Bug 961335 - ICODecoder Robocop tests; r?ahunt MozReview-Commit-ID: 9jtcfZCjAbd

Bug 1300385 - Fix flaws in SimpleTest.isDeeply - When one of the parameters to isDeeply is an object/function and the other is not, isDeeply returned false. Well, isDeeply is supposed to report an error instead of returning a value... - Change the implementation of isDeeply to have SameValue semantics instead of weak equality. - Change the representation of arrays to look like an array, instead of its default toString() value which is indistinguishable from a string due to the lack of brackets and quotes. - Account for holey arrays and missing object properties; Distinguish them from "undefined" with the special DNE tag. MozReview-Commit-ID: F1OJhbXcptl

Bug 1298710 - Remove ByteReader::DiscardRemaining and AutoByteReader - r?jya DiscardRemaning was needed to prevent debug-time assertion that the buffer was read completely or explicitly discarded. However this required extra work in cases where buffer didn't need to be read to the end. And also it could cause crashes (in debug versions) if a buffer was not fully read, be it because the parser was incorrect or because the media file itself was wrong (though possibly still readable despite that). Finding parser issues is still possible by manually instrumenting ByteReader during development. And reading media file with small recoverable errors is a bonus. MozReview-Commit-ID: 2RUYzaYAeRW

Bug 1300385 - Fix flaws in SimpleTest.isDeeply - When one of the parameters to isDeeply is an object/function and the other is not, isDeeply returned false. Well, isDeeply is supposed to report an error instead of returning a value... - Change the implementation of isDeeply to have SameValue semantics instead of weak equality. - Change the representation of arrays to look like an array, instead of its default toString() value which is indistinguishable from a string due to the lack of brackets and quotes. - Account for holey arrays and missing object properties; Distinguish them from "undefined" with the special DNE tag. MozReview-Commit-ID: F1OJhbXcptl

Bug 1300045 part 2 - Split KeyframeEffect.cpp into KeyframeEffect{ReadOnly}.cpp MozReview-Commit-ID: DdBEicunApv

Bug 1300045 part 1 - Tidy up some missing includes I came across these while working on a different bug that changed the grouping in the unified build. Adding these as a separate step will hopefully remove a little noise from the next patch in the series. MozReview-Commit-ID: Km7wTcVWG9n

Bug 1297306 - part7:replace StyleClear related NS_ASSERTION with MOZ_ASSERT. MozReview-Commit-ID: Dzd3uOoQCI

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

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

Bug 1297306 - part4:move IsEnumFittingWithin to mfbt/EnumTypeTraits.h. With this change, we could share this EnumTypeTraits between files easily. MozReview-Commit-ID: 9Q2augati7l