ca64ab2e20a77a828dd8da126ff2bfeb13032d88: Bug 1356873 - Insert inline text nodes between open and close tags; r=gl draft
Patrick Brosset <pbrosset@mozilla.com> - Sun, 16 Apr 2017 18:11:14 +0200 - rev 563368
Push 54268 by bmo:pbrosset@mozilla.com at Sun, 16 Apr 2017 16:13:27 +0000
Bug 1356873 - Insert inline text nodes between open and close tags; r=gl MozReview-Commit-ID: 9bWm0EP7Wrp
c841dac1bc5565fb44e5a9f70d9be8c6bead36e8: Bug 1356655 - Let adjustTabstrip return early if the tab strip is overflowing. r?florian draft
Dão Gottwald <dao@mozilla.com> - Sun, 16 Apr 2017 17:08:42 +0200 - rev 563367
Push 54267 by dgottwald@mozilla.com at Sun, 16 Apr 2017 15:13:08 +0000
Bug 1356655 - Let adjustTabstrip return early if the tab strip is overflowing. r?florian MozReview-Commit-ID: LKYuFidCHWa
24e89ea8bcbfd171797ff0a5c008bb593a869970: Bug 1348062 - Test that channels are properly marked for downloads. r=mak draft
Paolo Amadini <paolo.mozmail@amadzone.org> - Sun, 16 Apr 2017 15:59:27 +0100 - rev 563366
Push 54266 by paolo.mozmail@amadzone.org at Sun, 16 Apr 2017 15:00:06 +0000
Bug 1348062 - Test that channels are properly marked for downloads. r=mak Downloads handled by nsIExternalHelperAppService pass a null request to the final onStateChange notification, thus we need to hold a reference to the request earlier. This also allows unit tests to access the request while the download is running. MozReview-Commit-ID: 21rapDmMIZw
744c15cf2fcb20a98eff7e9532cc595413189cc9: Bug 1348062 - Mark channels used for downloads as throttable, r=nick+paolo draft
Honza Bambas <honzab.moz@firemni.cz> - Sun, 16 Apr 2017 14:53:30 +0100 - rev 563365
Push 54266 by paolo.mozmail@amadzone.org at Sun, 16 Apr 2017 15:00:06 +0000
Bug 1348062 - Mark channels used for downloads as throttable, r=nick+paolo MozReview-Commit-ID: 9cg4PwrXtNa
47adb98a6d41f06882dd578a2f7ab08198fef051: Bug 1356663 - Calculate the status panel's mouse target rectangle lazily in getMouseTargetRect. r?florian draft
Dão Gottwald <dao@mozilla.com> - Sun, 16 Apr 2017 16:43:53 +0200 - rev 563364
Push 54265 by dgottwald@mozilla.com at Sun, 16 Apr 2017 14:48:04 +0000
Bug 1356663 - Calculate the status panel's mouse target rectangle lazily in getMouseTargetRect. r?florian MozReview-Commit-ID: 1g07rylDWkx
e4eb9ccd98c513f06438df01c58e164f2ec539aa: Bug 1351667 - UI test that traverses settings screens; r?sebastian draft
brainbreaker <gautamprajapati06@gmail.com> - Sun, 16 Apr 2017 18:10:48 +0530 - rev 563363
Push 54264 by bmo:gautamprajapati06@gmail.com at Sun, 16 Apr 2017 12:48:22 +0000
Bug 1351667 - UI test that traverses settings screens; r?sebastian This commit adds a robocop test that traverses all the settings screens except the 'Make default browser' one. MozReview-Commit-ID: LqWrFL9v877
f521cc2130f4510bdc3f5fa3c6750668233d6297: Bug 1356510 - stylo: Update test expectations. r?Manishearth draft
Cameron McCormack <cam@mcc.id.au> - Sun, 16 Apr 2017 22:11:18 +1000 - rev 563362
Push 54263 by bmo:cam@mcc.id.au at Sun, 16 Apr 2017 12:18:03 +0000
Bug 1356510 - stylo: Update test expectations. r?Manishearth
84f52a983b3806eacc71745b87a55c3c201a4b94: style: Make text-orientation:sideways-right an alias. draft
Cameron McCormack <cam@mcc.id.au> - Sun, 16 Apr 2017 18:26:27 +1000 - rev 563361
Push 54263 by bmo:cam@mcc.id.au at Sun, 16 Apr 2017 12:18:03 +0000
style: Make text-orientation:sideways-right an alias. Gecko does not preserve this specified value; it converts it to sideways at parse time.
aa68bf9a47751f15b5bc5ba2cffab8059e91cf6d: style: Add Gecko-only support for the deprecated SVG 1.1 writing-mode aliases. draft
Cameron McCormack <cam@mcc.id.au> - Sun, 16 Apr 2017 22:06:32 +1000 - rev 563360
Push 54263 by bmo:cam@mcc.id.au at Sun, 16 Apr 2017 12:18:03 +0000
style: Add Gecko-only support for the deprecated SVG 1.1 writing-mode aliases.
2cfe01cac4ba295085ebfaaeb714026834b829ac: style: Add Gecko-only support for writing-mode:sideways-{lr,rl}. draft
Cameron McCormack <cam@mcc.id.au> - Sat, 15 Apr 2017 20:10:24 +1000 - rev 563359
Push 54263 by bmo:cam@mcc.id.au at Sun, 16 Apr 2017 12:18:03 +0000
style: Add Gecko-only support for writing-mode:sideways-{lr,rl}. Fixes #15213. MozReview-Commit-ID: 9KLM3BqbMIo
d775c1002b055d53147eb020eee4b9e872eed1ae: style: Add support for property value aliases. draft
Cameron McCormack <cam@mcc.id.au> - Sat, 15 Apr 2017 20:10:02 +1000 - rev 563358
Push 54263 by bmo:cam@mcc.id.au at Sun, 16 Apr 2017 12:18:03 +0000
style: Add support for property value aliases. For keyword-typed properties with aliases but no extra_specified values, the storage of the specified and computed values could be the same, since value aliases are resolved at parse time. But to prevent computed_value::T::parse from recognizing these aliases, we keep the specified and computed value types distinct. MozReview-Commit-ID: ECfTWMWhroQ
a1b313ae2de4425c050346505f83cfad0642acfe: Bug 1356718 - Query gtk settings schemas to avoid crashing when schema is missing. r?jfkthame draft
Zibi Braniecki <gandalf@mozilla.com> - Sun, 16 Apr 2017 00:13:07 -0700 - rev 563357
Push 54262 by zbraniecki@mozilla.com at Sun, 16 Apr 2017 07:13:20 +0000
Bug 1356718 - Query gtk settings schemas to avoid crashing when schema is missing. r?jfkthame MozReview-Commit-ID: 2rqO4XpLbfO
57ff1bb6c8ef8436d105c4b7fc91dccdd35e494b: Bug 1353057 - Split requests summary button into multiple labels. r?Honza draft
Vangelis Katsikaros <vkatsikaros@gmail.com> - Tue, 11 Apr 2017 22:41:26 +0300 - rev 563356
Push 54261 by vkatsikaros@gmail.com at Sun, 16 Apr 2017 06:43:09 +0000
Bug 1353057 - Split requests summary button into multiple labels. r?Honza MozReview-Commit-ID: 1lwmft6Nucy
16e1b51169e0246390e9951c648b78cd47cd0f2e: Bug 1346616 - Migrate callsites that are retrieving requested locale from pref, to use LocaleService::GetRequestedLocales. r?pike, r=pike draft
Zibi Braniecki <gandalf@mozilla.com> - Sat, 11 Mar 2017 18:43:11 -0800 - rev 563355
Push 54260 by zbraniecki@mozilla.com at Sun, 16 Apr 2017 06:38:28 +0000
Bug 1346616 - Migrate callsites that are retrieving requested locale from pref, to use LocaleService::GetRequestedLocales. r?pike, r=pike I'm adding a helper function mozILocaleService::GetRequestedLocale to simplify most of the callsites that are looking for the first of the requested locales. In most cases, I'm just matching the behavior of the code with reusing LocaleService API instead of direct manipulation on the prefs. That includes how I handle error case scenarios. In case of sdk/l10n/locale.js I am reusing LocaleService heuristics over the custom one from the file since the ones in LocaleService are just more correct and unified accross the whole platform. In case of FallbackEncoding I have to turn it into a nsIObserver to listen to intl:requested-locales-changed. MozReview-Commit-ID: 7rOr2CovLK
ea9b83be8450902c736931604aea3efc2608d658: Bug 1341372 - Part 10: Update css-transitions/reftest-stylo.list. draft
Boris Chiou <boris.chiou@gmail.com> - Fri, 07 Apr 2017 15:01:07 +0800 - rev 563354
Push 54259 by bmo:boris.chiou@gmail.com at Sun, 16 Apr 2017 06:29:51 +0000
Bug 1341372 - Part 10: Update css-transitions/reftest-stylo.list. All pass. MozReview-Commit-ID: 99o8zcSKh1f
c4de49e4fa010067a31d269f22a51a3dc68b8bbe: Bug 1341372 - Part 9: Update mochitest expectations. draft
Boris Chiou <boris.chiou@gmail.com> - Thu, 06 Apr 2017 13:23:42 +0800 - rev 563353
Push 54259 by bmo:boris.chiou@gmail.com at Sun, 16 Apr 2017 06:29:51 +0000
Bug 1341372 - Part 9: Update mochitest expectations. Also update test_transitions_dynamic_changes.html because we don't support "font" property for stylo now. According to its context and purpose, we only need a property other than "text-indent", so it is not necessary to use "font". Besides, we don't support shorthand properties for transitions, so test_transitions.html have two intermittents. Mark the expect number as '*' until we fix shorthand properties for transitions. MozReview-Commit-ID: HyuOV29AJ9a
95749c12bdd6da38df977e57d6adbc557dadf177: Bug 1341372 - Part 8: Rename needs_update_animations. draft
Boris Chiou <boris.chiou@gmail.com> - Fri, 14 Apr 2017 10:44:38 +0800 - rev 563352
Push 54259 by bmo:boris.chiou@gmail.com at Sun, 16 Apr 2017 06:29:51 +0000
Bug 1341372 - Part 8: Rename needs_update_animations. MozReview-Commit-ID: 2bJlBbdX543
9d92760cc6aa2465ced080040e445a71202823f4: Bug 1341372 - Part 7: Add a utility method to get AnimationCollection by Element and nsIAtom. draft
Boris Chiou <boris.chiou@gmail.com> - Fri, 14 Apr 2017 10:17:43 +0800 - rev 563351
Push 54259 by bmo:boris.chiou@gmail.com at Sun, 16 Apr 2017 06:29:51 +0000
Bug 1341372 - Part 7: Add a utility method to get AnimationCollection by Element and nsIAtom. We need a utility method to get the AnimationCollection by dom::Element and the pseudo element tag. MozReview-Commit-ID: KCOY6EKFFX5
c99869c661b25cd5da51d6b1f434dbaa77ae46e0: Bug 1341372 - Part 6: Trigger transitions. draft
Boris Chiou <boris.chiou@gmail.com> - Fri, 31 Mar 2017 15:43:37 +0800 - rev 563350
Push 54259 by bmo:boris.chiou@gmail.com at Sun, 16 Apr 2017 06:29:51 +0000
Bug 1341372 - Part 6: Trigger transitions. 1. We need to call get_after_change_style, which is the computed styles without transition rules, while process_animations. 2. If we have after-change style, we may replace the new computed values with after-change style, according to whether we really need to update transitions. 3. There are some cases we don't update transitions, so we need to early return. might_needs_transitions_update() will check it first and it will filter out most common cases. 4. needs_transitions_update() will check each property and existing running transitions to make sure we really don't need to update transitions. The logic of this function is similar with that of nsTransitionManager::DoUpdateTransitions(). MozReview-Commit-ID: 9eLd8fxmNd9
d99ad92ef75ae4bbf0b7bc945f2c15179c5b29b0: Bug 1356843 - Enable -Wcomma clang warnings. r?glandium draft
Chris Peterson <cpeterson@mozilla.com> - Sat, 08 Apr 2017 12:36:48 -0700 - rev 563349
Push 54258 by cpeterson@mozilla.com at Sun, 16 Apr 2017 05:52:14 +0000
Bug 1356843 - Enable -Wcomma clang warnings. r?glandium Warn about possible misuse of the comma operator such as between two statements or to call a function for side effects within an expression. Only enable these -Wcomma warnings for C++ code because there are almost two hundred -Wcomma warnings in third-party C libraries. I reviewed the C warnings and confirmed none of them were latent bugs. We won't fix these libraries' warnings so they are just noise. MozReview-Commit-ID: 1JXJumg6DsJ
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip