Commit Graph

28341 Commits

Author SHA1 Message Date
Bill McCloskey
c98949c21f Bug 1221371 - Switch chromium IPC code to use mozilla::Tuple (r=jld,cpearce,kats) 2015-11-09 10:21:11 -08:00
L. David Baron
5aa1fea3ca Bug 1221436 patch 7 - Remove ReplaceStyleRule/ReplaceRuleInGroup mechanism. r=heycam
This is no longer used, thanks to patch 5.
2015-11-09 15:57:17 +08:00
L. David Baron
52f605130c Bug 1221436 patch 6 - Remove mechanism for replacing style rules. r=heycam
This is no longer used, thanks to patch 5.
2015-11-09 15:57:17 +08:00
L. David Baron
954591e81a Bug 1221436 patch 5 - Stop generating new style rules in DeclarationChanged (and rename it to SetDeclaration), since we no longer need a new style rule for nsIStyleRule identity rules. r=heycam
(I think the change DOMCSSDeclarationImpl::GetCSSDeclaration fixes a
purely theoretical bug that would happen if it were possible to get
ahold of a CSS rule without calling either
CSSStyleSheet::EnsureUniqueInner (which is called by
CSSRuleListImpl::IndexedGetter) or Declaration::SetImmutable (which is
called by rule matching, including that triggered by
inIDomUtils::GetCSSStyleRules).  If that were possible, then mutating
such a declaration would mutate the shared Declaration, and then call
WillDirty which would clone the now-mutated Declaration, leading to the
mutation applying to both sheets that shared it rather than just the one
that should have been modified.)

This is the simplification allowed by bug 978833 patch 11 (and part of
what we would otherwise have needed to duplicate in @page and keyframe
rules; we'd also have needed to duplicate the object split between the
internal object and the DOM-exposed object, but for style rules that's
probably worth keeping for the memory savings).
2015-11-09 15:57:17 +08:00
L. David Baron
d4d445f5d6 Bug 1221436 patch 4 - Use the same pointer to store the owning rule and the nsHTMLCSSStyleSheet on css::Declaration, since we never need both. r=heycam
The pointer to nsHTMLCSSStyleSheet was added in patch 1 (which was
needed before patch 2), but it was only patch 2 that created the
invariant that we'd only have one pointer or the other, and never both.
Thus this needs to be done separately, after patch 1.
2015-11-09 15:57:17 +08:00
L. David Baron
abadd0da91 Bug 1221436 patch 3 - Remove Rule::SetHTMLCSSStyleSheet and related code, now unused. r=heycam
This was made unused by patch 2.
2015-11-09 15:57:16 +08:00
L. David Baron
1b093e6db8 Bug 1221436 patch 2 - For style attributes, only store a css::Declaration instead of a css::StyleRule. r=heycam
Now that Declaration implements nsIStyleRule, we don't need the memory
overhead of storing a StyleRule object for style attributes.

We also need to change this prior to patch 5, because the changes in
patch 5 that will allow rules to change (but declarations not) would
otherwise break due to style attribute object merging done by
nsAttrValue::ParseStyleAttribute.
2015-11-09 15:57:16 +08:00
L. David Baron
aaf6592743 Bug 1221436 patch 1 - Have a pointer back from the css::Declaration for style attributes to the nsHTMLCSSStyleSheet. r=heycam
This is needed for patch 2, since we need a replacement for the
mechanism to get from rules to the style attribute sheet
(nsHTMLCSSStyleSheet) that works instead for declarations rather than
rules.

This pointer will be merged with the owning rule pointer in patch 4, but
we can't do that until after patch 2.
2015-11-09 15:57:16 +08:00
sajitk
a4b97e7c7e Bug 1219470 - Replace PRLogModuleInfo with LazyLogModule. r=roc 2015-11-05 23:35:03 +01:00
Hiroyuki Ikezoe
d72a33c23e Bug 1216030 - Part 16: Move CanThrottleAnimation and CanThrottleTransformChanges from AnimationCollection into KeyframeEffectReadOnly::CanThrottle. r=bbirtles
The preference check has been removed from CanThrottleTransformChanges
because we already perform that check that when deciding if we should run
an animation on the compositor (in CanPerformOnCompositorThread, as called
by GetAnimationsForCompositor). Hence if the "is running on compositor" flag
is true, we can assume the preference is set (or was set when we decided to
put the animation on the compositor-- we don't worry about pulling the
animation off the compositor immediately if the preference changes while
it is running)
2015-11-06 02:53:00 +01:00
Hiroyuki Ikezoe
2baf369883 Bug 1216030 - Part 13: Remove existsProperty check from CanPerformOnCompositorThread. r=dbaron
That's because GetAnimationsForCompositor already checks that there is at least one
property (actually it's now transform or opacity).
2015-11-06 02:48:00 +01:00
Hiroyuki Ikezoe
984bb57762 Bug 1216030 - Part 12: Pass nsIFrame to CanPerformOnCompositorThread to avoid redundant process for getting nsIFrame. r=dbaron
That's because GetAnimationsForCompositor already has an approptiate nsIFrame there.
It's pseudo frame if the animation is on pseudo element.
2015-11-06 02:47:00 +01:00
Hiroyuki Ikezoe
cd0fb2668b Bug 1216030 - Part 11: Add KeyframeEffect::CanAnimatePropertyOnCompositor. r=bbirtles
Based on AnimationCollection::CanAnimatePropertyOnCompositor.
The first argument has been changed to nsIFrame* so that we don't need to
get style frame for CanAnimateTransformOnCompositor again.
2015-11-06 02:45:00 +01:00
Hiroyuki Ikezoe
27b79b5c7e Bug 1216030 - Part 10: Remove CanAnimate_AllowPartial flag. r=bbirtles 2015-11-06 02:43:00 +01:00
Hiroyuki Ikezoe
15970013a8 Bug 1216030 - Part 9: We don't need to call CanPerformOnCompositorThread in RequestRestyle.
That's because KeyFrameEffectReadOnly::mIsPropertyRunningOnCompositor indicates the
correct state that the animation property is now running on compositor or not.
So once the flag is set to true, we do not need to check animation properties can run on
compositor after Animation::CanThrottle again.
2015-10-28 02:55:00 +01:00
Hiroyuki Ikezoe
d6ec1beb20 Bug 1216030 - Part 7: Add nsIFrame::RefusedAsyncAnimation. r=dbaron
Original nsIFrame::RefusedAsyncAnimation was renamed to
nsIFrame::RefusedAsyncAnimationProperty.
2015-10-20 22:03:00 +02:00
Hiroyuki Ikezoe
baec64cf48 Bug 1216030 - Part 6: Add KeyframeEffectReadOnly::IsGeometricProperty. r=bbirtles
This method will be private soon.
2015-11-06 02:38:00 +01:00
Hiroyuki Ikezoe
7bad89ace5 Bug 1216030 - Part 5: Add KeyframeEffectReadOnly::CanAnimateTransformOnCompositor. r=bbirtles
This method will be private soon.
2015-11-06 02:38:00 +01:00
Hiroyuki Ikezoe
c547990fa6 Bug 1216030 - Part 4: Move IsCompositorAnimationDisabledForFrame outside animation properties loop. r=dbaron 2015-11-06 02:35:00 +01:00
Hiroyuki Ikezoe
c1f89b7942 Bug 1216030 - Part 3: Move AreAsyncAnimationsEnabled check outside animation properties loop. r=dbaron 2015-11-06 02:34:00 +01:00
Hiroyuki Ikezoe
7efe3424e8 Bug 1216030 - Part 2: Remove gfxPlatform::OffMainThreadCompositingEnabled from CanAnimatePropertyOnCompositor. r=dbaron
It is already checked in nsLayoutUtils::AreAsyncAnimationsEnabled.
And nsLayoutUtils::AreAsyncAnimationsEnabled check is moved at the first of CanAnimatePropertyOnCompositor.
2015-11-06 02:33:00 +01:00
Hiroyuki Ikezoe
1a51450427 Bug 1216030 - Part 1: Remove CanAnimate_HasGeometricProperty. r=dbaron
it's enough to check IsGeometricProperty() once within
AnimationCollection::CanPerformOnCompositorThread
2015-11-06 02:32:00 +01:00
Nigel Babu
ee1b130712 Backed out changeset f8fa4680e732 (bug 1038663) for OS X reftest bustage on a CLOSED TREE 2015-11-09 10:53:02 +05:30
Christoph Kerschbaumer
893ed45b8b Bug 1222297 - Use channel->Open2() in netwerk/base/nsNetUtil.cpp (r=sicking) 2015-11-08 18:54:38 -08:00
Nicholas Nethercote
0e3874f484 Bug 1038663 (part 7, attempt 2) - Add test for percentage values for 'word-spacing'. r=heycam. 2015-11-08 16:43:33 -08:00
Nicholas Nethercote
e1d2f44f23 Bug 1038663 (part 6, attempt 2) - Allow percentage values for 'word-spacing'. r=heycam. 2015-11-08 16:40:37 -08:00
Nicholas Nethercote
72d4c05707 Bug 1038663 (part 5, attempt 2) - Change GetSpacingFlags(). r=heycam. 2015-11-08 14:49:02 -08:00
Nicholas Nethercote
8fc4bb2082 Bug 1038663 (part 4, attempt 2) - Inline StyleToCoord(). r=heycam. 2015-11-08 14:49:02 -08:00
Nicholas Nethercote
04a6cdefa0 Bug 1038663 (part 3, attempt 2) - Factor out space width computation. r=heycam. 2015-11-08 14:48:58 -08:00
Nicholas Nethercote
7d210758ee Bug 1038663 (part 2, attempt 2) - Move GetFirstFontMetrics() up. r=heycam. 2015-11-08 13:52:58 -08:00
Sebastian Hengst
b86b5a59a1 Backed out 3 changesets (bug 1221368, bug 1221371) for build failures in B2G JB Emulator. r=backout
Backed out changeset a50c676caf7f (bug 1221371)
Backed out changeset bd99e5060e1e (bug 1221371)
Backed out changeset 3a22461c8ce8 (bug 1221368)

gecko/ipc/chromium/src/base/task.h:36:4: error: use of deleted function 'RefPtr<T>::operator T*() const && [with T = nsScreenGonk]'
make[6]: *** [nsScreenManagerGonk.o] Error 1
2015-11-08 21:25:22 +01:00
Bill McCloskey
46c0a80f3f Backout bug 967873 - Proxy nsDocumentViewer::PermitUnload to the child process 2015-11-08 12:24:42 -08:00
Bill McCloskey
3cada6954a Bug 967873 - Proxy nsDocumentViewer::PermitUnload to the child process (r=Gijs) 2015-11-08 10:27:49 -08:00
Bill McCloskey
55acd0e16b Bug 1221371 - Switch chromium IPC code to use mozilla::Tuple (r=jld,cpearce,kats) 2015-11-08 10:25:09 -08:00
Cameron McCormack
9847536a74 Bug 1222226 - Don't return eRestyleResult_StopWithStyleChange if the old style context is shared. r=dbaron 2015-11-08 11:06:14 +11:00
Timothy Nikkel
67b58b3a99 Bug 1220114. Part 2. If a scroll frame is forced to layerize because of an active descendant scroll frame then set a displayport on the ancestor scroll frame so that next paint we don't have to force layerization after the fact and we can do a fully proper paint. r=mstange 2015-11-06 17:32:03 -06:00
Timothy Nikkel
90124c654c Bug 1220114. Part 1. Change ScrollFrameHelper::DecideScrollableLayer to recompute the current animated geometry root any time mWillBuildScrollableLayer changes in addition to when usingDisplayPort changes. r=mstange
Changes in either one can result in AGRs being created or removed (although nothing removes them at this point in time).
2015-11-06 17:32:03 -06:00
Daniel Holbert
ff69daafb4 Bug 1219868: Don't warn for CreateAnonymousContent failure (due to e.g. detecting & breaking <use> reference loops). r=jwatt 2015-11-06 14:59:55 -08:00
Geoff Brown
b32950bbdb Bug 1197716 - On Android, include fonts in test profile; r=jmaher 2015-11-06 12:01:33 -07:00
Mason Chang
783b0333c2 Bug 1221674 - Add telemetry probe in the content process to measure the time between refresh driver ticks. r=kats 2015-11-06 08:20:58 -08:00
Hiroyuki Ikezoe
2fa7ebc224 Bug 1197620 - Part 3: Terminate *all* animations if corresponding element style is changed to display:none. r=bbirtles 2015-11-06 07:21:38 +09:00
Nigel Babu
575c935465 Backed out changeset b456daa0503f (bug 1221842) for B2G ICS Emulator Opt R7 bustage on a CLOSED TREE 2015-11-06 12:50:58 +05:30
Nigel Babu
b5f9a51ba9 Backed out 6 changesets (bug 1038663) for Android opt R2 bustage
Backed out changeset 97e3492d6080 (bug 1038663)
Backed out changeset d176322f2d36 (bug 1038663)
Backed out changeset f69af9161252 (bug 1038663)
Backed out changeset 8bb77e5fad8c (bug 1038663)
Backed out changeset 954e57438f51 (bug 1038663)
Backed out changeset 16c0919101cd (bug 1038663)
2015-11-06 12:49:36 +05:30
Nicholas Nethercote
3565700d66 Bug 1038663 (part 7) - Add test for percentage values for 'word-spacing'. r=heycam. 2015-11-05 17:43:28 -08:00
Nicholas Nethercote
ffd3c374b7 Bug 1038663 (part 6) - Allow percentage values for 'word-spacing'. r=heycam. 2015-11-05 17:25:46 -08:00
Nicholas Nethercote
dd68dc05ae Bug 1038663 (part 5) - Change GetSpacingFlags(). r=heycam.
Again, this doesn't make sense in isolation, but it will make it easier to
allow for percentage word-spacing in a subsequent patch.
2015-11-05 16:33:37 -08:00
Nicholas Nethercote
d74826c11a Bug 1038663 (part 4) - Inline StyleToCoord(). r=heycam.
This doesn't make much sense by itself, but one of the call sites will be
changed in a subsequent patch.
2015-11-04 21:10:36 -08:00
Nicholas Nethercote
894f3c51ba Bug 1038663 (part 3) - Factor out space width computation. r=heycam.
This will be reused in a subsequent patch.
2015-11-04 18:02:40 -08:00
Nicholas Nethercote
a64f843399 Bug 1038663 (part 2) - Move GetFirstFontMetrics() up. r=heycam.
This is necessary for a subsequent patch.
2015-11-04 18:02:20 -08:00
Nicholas Nethercote
d6ab162415 Bug 1187144 (part 1) - Replace nsBaseHashtable::Enumerate() calls in layout/ with iterators. r=heycam. 2015-11-03 15:51:03 -08:00