422dd7521fa3e2d1712b5b011fc978e7d9e3cddd: Bug 1294321 - IonMonkey: MIPS: Import Loongson optimizations to ma_ss and ma_sd baseindex. r=arai
Heiher <r@hev.cc> - Thu, 11 Aug 2016 15:50:03 +0800 - rev 309120
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1294321 - IonMonkey: MIPS: Import Loongson optimizations to ma_ss and ma_sd baseindex. r=arai --- .../jit/mips-shared/MacroAssembler-mips-shared.cpp | 38 ++++++++++++++++++++++ 1 file changed, 38 insertions(+)
0bd27824832d4213a90ec0ef08de422b33589358: Bug 1294133. Null check animated geometry root. r=mattwoodrow
Timothy Nikkel <tnikkel@gmail.com> - Thu, 11 Aug 2016 00:52:56 -0500 - rev 309119
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1294133. Null check animated geometry root. r=mattwoodrow
b8117f4a3659120291ecbfcc07650fed632dda02: Bug 1291970 (part 2) - Tweak nsCategoryManager memory reporter handling. r=erahm.
Nicholas Nethercote <nnethercote@mozilla.com> - Thu, 11 Aug 2016 09:29:44 +1000 - rev 309118
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1291970 (part 2) - Tweak nsCategoryManager memory reporter handling. r=erahm. This patch adds an assertion that makes sure that the nsCategoryManager is destroyed after the nsMemoryReporterManager, because bad things would happen otherwise. Also, nsCategoryManager uses manual memory management (it's AddRef/Release are hardwired to always return 2 and 1 respectively) so it doesn't matter if we register it as a strong or weak memory reporter. But it's more common to use RegisterWeakMemoryReporter when the argument is |this|, so this patch does that.
6612a73409a9029aff2461b9ab9f451b4677f51d: Bug 1291970 (part 1) - Use MOZ_MUST_USE in nsMemoryReporterManager. r=erahm.
Nicholas Nethercote <nnethercote@mozilla.com> - Thu, 04 Aug 2016 12:14:14 +1000 - rev 309117
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1291970 (part 1) - Use MOZ_MUST_USE in nsMemoryReporterManager. r=erahm.
f1f22c6fad8184a3b233846c22c979e84c903cea: Bug 1264530 - Hold on to Plugin Instance to survive frame poisoning. r=jimm
Benoit Girard <b56girard@gmail.com> - Wed, 10 Aug 2016 16:21:01 -0400 - rev 309116
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1264530 - Hold on to Plugin Instance to survive frame poisoning. r=jimm MozReview-Commit-ID: JHbce46rDBN
c45412486f686f78358df5572b28ad95ad6b563c: Bug 1292930: stylo: Make change hint processing more straight-forward. r=heycam
Emilio Cobos Álvarez <ecoal95@gmail.com> - Sat, 06 Aug 2016 14:18:12 -0700 - rev 309115
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1292930: stylo: Make change hint processing more straight-forward. r=heycam MozReview-Commit-ID: 4ZcCMKSc6Tv
9c18533ec3fac64f36959d88962b3a40de1318be: Bug 1292930: stylo: Remove expected warning. r=heycam
Emilio Cobos Álvarez <ecoal95@gmail.com> - Sat, 06 Aug 2016 13:39:04 -0700 - rev 309114
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1292930: stylo: Remove expected warning. r=heycam This should actually have been a followup for bug 1290335. MozReview-Commit-ID: GIl67fSMRQg
7be5dfc0e9c3218582418425b5de303e42dc2a91: Bug 1292930: stylo: Store the change hint generated by non-elements in their parent element. r=heycam
Emilio Cobos Álvarez <ecoal95@gmail.com> - Sat, 06 Aug 2016 13:34:53 -0700 - rev 309113
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1292930: stylo: Store the change hint generated by non-elements in their parent element. r=heycam Otherwise, the parent style context doesn't have some inherited structs in the cache, so it avoids comparing them entirely. For example, the following example: <p>Hey</p> p { color: blue } p:hover { color: red } Wouldn't work as intended, because when calculating the change hint the nsStyleColor struct in the element hasn't been accessed by layout (only the child text frame's has), so it will report a change hint of 0 when hovering over the paragraph, and we would ignore the nsChangeHint_ReconstructFrame generated by the text node. MozReview-Commit-ID: FW7Thhuh7LG
0890cda8c432c2e8d781fd4160f4c9eb4acc1b47: Bug 1292930: stylo: Fix handling of eRestyle_LaterSiblings when not combined with other change hints. r=heycam
Emilio Cobos Álvarez <ecoal95@gmail.com> - Fri, 05 Aug 2016 11:55:19 -0700 - rev 309112
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1292930: stylo: Fix handling of eRestyle_LaterSiblings when not combined with other change hints. r=heycam MozReview-Commit-ID: AeLLm64d5aP
294df63717f1e5b7665164bda1e080fd95914a38: Bug 1292930: Hoist ChangeHintToString to RestyleManagerBase. r=heycam
Emilio Cobos Álvarez <ecoal95@gmail.com> - Fri, 05 Aug 2016 11:54:22 -0700 - rev 309111
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1292930: Hoist ChangeHintToString to RestyleManagerBase. r=heycam MozReview-Commit-ID: BLW40PYhc0o
2d5a7f342256264b31c8378795c03b5844bfc53b: Bug 1292930: stylo: Reuse RestyleManager::PostRestyleEventInternal. r=heycam
Emilio Cobos Álvarez <ecoal95@gmail.com> - Wed, 03 Aug 2016 18:58:06 -0700 - rev 309110
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1292930: stylo: Reuse RestyleManager::PostRestyleEventInternal. r=heycam MozReview-Commit-ID: J6rw1feAYWh
32e07e32891d070cdba97e2872cd30265278c0bf: Bug 1293117 (part 7) - Remove two occurrences of |static NS_IMETHODIMP|. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Wed, 10 Aug 2016 10:20:41 +1000 - rev 309109
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1293117 (part 7) - Remove two occurrences of |static NS_IMETHODIMP|. r=froydnj. Because it's a combination that doesn't make sense. - ReadFuncBinaryString becomes NS_METHOD, because it's passed to ReadSegments, which requires a function matching NS_CALLBACK. - IccContactListToMozContactList just changes to a vanilla nsresult return type, because it doesn't need __stdcall on Windows.
05133363f4edf4633ce9ee0dff1ce31f56908c9d: Bug 1293117 (part 6) - Fix a comment and return type in XPCComponents.cpp. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 09 Aug 2016 14:03:12 +1000 - rev 309108
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1293117 (part 6) - Fix a comment and return type in XPCComponents.cpp. r=froydnj. The comment is wrong. NS_METHOD is the appropriate thing to use here.
2605dceca7b26e11f3a2c546cf5aaea7d0fb9385: Bug 1293117 (part 5) - Change many NS_METHOD occurrences to NS_IMETHOD. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Mon, 08 Aug 2016 10:54:50 +1000 - rev 309107
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1293117 (part 5) - Change many NS_METHOD occurrences to NS_IMETHOD. r=froydnj. This patch makes the following changes on many in-class methods. - NS_METHOD F() override; --> NS_IMETHOD F() override; - NS_METHOD F() override {...} --> NS_IMETHOD F() override {...} - NS_METHOD F() final; --> NS_IMETHOD F() final; - NS_METHOD F() final {...} --> NS_IMETHOD F() final {...} Using NS_IMETHOD is the preferred way of marking in-class virtual methods. Although these transformations add an explicit |virtual|, they are safe -- there's an implicit |virtual| anyway because |override| and |final| only work with virtual methods.
babe907f527681f351d62d689230d23e55d6a729: Bug 1293117 (part 4) - Change many NS_IMETHODIMP occurrences to NS_IMETHOD. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Mon, 08 Aug 2016 10:54:47 +1000 - rev 309106
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1293117 (part 4) - Change many NS_IMETHODIMP occurrences to NS_IMETHOD. r=froydnj. This patch makes the following changes on many in-class methods. - NS_IMETHODIMP F() override; --> NS_IMETHOD F() override; - NS_IMETHODIMP F() override {...} --> NS_IMETHOD F() override {...} - NS_IMETHODIMP F() final; --> NS_IMETHOD F() final; - NS_IMETHODIMP F() final {...} --> NS_IMETHOD F() final {...} Using NS_IMETHOD is the preferred way of marking in-class virtual methods. Although these transformations add an explicit |virtual|, they are safe -- there's an implicit |virtual| anyway because |override| and |final| only work with virtual methods.
4db4249540773fabd59acb0e3fe883c95d3213f4: Bug 1293117 (part 3) - Remove some unnecessary |virtual| annotations. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Mon, 08 Aug 2016 10:45:56 +1000 - rev 309105
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1293117 (part 3) - Remove some unnecessary |virtual| annotations. r=froydnj. This patch changes |virtual NS_IMETHODIMP| occurrences to |NS_IMETHOD|, which is equivalent and the more standard way of marking in-class virtual XPCOM methods.
2764e29be7dce8539f6194f11d479345577db2c5: Bug 1293117 (part 2) - Remove unnecessary comments on some NS_IMETHODIMP instances. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Mon, 08 Aug 2016 10:45:37 +1000 - rev 309104
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1293117 (part 2) - Remove unnecessary comments on some NS_IMETHODIMP instances. r=froydnj. This patch replaces numerous |/* virtual */ NS_IMETHODIMP| occurrences with |NS_IMETHODIMP|, because |NS_IMETHODIMP| always implies |virtual|.
581bf8b4f433e44150ae7d11fa033064608e443c: Bug 1293117 (part 1) - Add comment explaining how NS_IMETHOD, NS_IMETHODIMP and NS_METHOD should be used. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Mon, 08 Aug 2016 10:40:42 +1000 - rev 309103
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1293117 (part 1) - Add comment explaining how NS_IMETHOD, NS_IMETHODIMP and NS_METHOD should be used. r=froydnj.
ac0708fe8f742c67355d9578bb87954769a037ce: Bug 1274340 - Call SetReceiveCodec for RED and ULPFEC when FEC is enabled; r=jesup
Dan Minor <dminor@mozilla.com> - Thu, 30 Jun 2016 12:20:19 -0400 - rev 309102
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1274340 - Call SetReceiveCodec for RED and ULPFEC when FEC is enabled; r=jesup We need to call SetReceiveCodec for RED and ULPFEC so we know how to handle those packets when received. MozReview-Commit-ID: A9EluM7p2NH
c9b04662c81bcfdf8cf2d26c9afcabfdeb2ca87f: Bug 1274340 - Make RED and ULPFEC payload type match sdp values; r=jesup
Dan Minor <dminor@mozilla.com> - Thu, 30 Jun 2016 12:20:04 -0400 - rev 309101
Push 30557 by kwierso@gmail.com at Fri, 12 Aug 2016 20:45:15 +0000
Bug 1274340 - Make RED and ULPFEC payload type match sdp values; r=jesup To be able to send and receive video with FEC enabled it appears we need to have matching constant values here and in sdp/sipcc/ccsdp.h. MozReview-Commit-ID: LZzAyMW9eEu
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip