37954 Commits

Author SHA1 Message Date
Kartikaya Gupta
370d1627f6 Bug 918682 - Fix the check for finding the root content document's root scroll frame. r=tn 2013-09-27 15:15:57 -04:00
Ryan VanderMeulen
1286b0b31d Merge fx-team to m-c. 2013-09-26 21:36:35 -04:00
Blake Kaplan
e2d7482db4 Bug 917515 - Check for null before checking the document. r=ehsan 2013-09-26 18:42:53 +02:00
Carsten "Tomcat" Book
d41c61b11c Merge m-c to fx-team 2013-09-26 15:57:51 +02:00
Nicholas Nethercote
228510e1ec Bug 414087 (part 2) - Remove insignificant wasted space in StackBlock. r=bz. 2013-09-25 21:28:55 -07:00
Nicholas Nethercote
929154dfe5 Bug 414087 (part 1) - Clean up the style of StackArena.{h,cpp}. r=bz. 2013-09-25 21:28:46 -07:00
Martijn Wargers
e49d1bf33b Bug 917061 - Fix the last 2 crashtests that use enablePrivilege. r=dholbert 2013-09-25 20:03:38 +02:00
L. David Baron
c5d18cf77f Followup to bug 828312 patch 9a: Mark test as fuzzy-if on Android. CLOSED TREE 2013-09-25 17:53:12 -07:00
L. David Baron
6404578a6b Followup to bug 828312 patch 9a: Mark test as fuzzy-if on Mac OS X 10.8, where the Ahem font is behaving in ways it shouldn't. CLOSED TREE 2013-09-25 15:45:24 -07:00
Ryan VanderMeulen
0eb7473289 Bug 907908 - Mark 776265-1a.html as random-if on Android 2.2 for too many intermittent failures.
DONTBUILD CLOSED TREE
2013-09-25 17:02:07 -04:00
Ryan VanderMeulen
aaa79d7829 Bug 907905 - Mark 647192-1.html as random-if on Android 2.2 for too many intermittent failures. 2013-09-25 17:01:53 -04:00
L. David Baron
d01753dad0 Bug 828312 patch 11: Don't generate change hints for restyling of later continuations, since the handling of the change hints from the first continuation is required to do the necessary work. r=bzbarsky
This depends on bug 898333 in order to avoid causing:
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/dom/tests/mochitest/chrome/test_focused_link_scroll.xul | Assertion count 1 is greater than expected range 0-0 assertions.
due to the assertion:
###!!! ASSERTION: Shouldn't be trying to restyle non-elements directly: '!aContent || aContent->IsElement()', file ../../../layout/base/nsStyleChangeList.cpp, line 62

The assertion count change in layout/generic/crashtests/571995.xhtml is
expected because it changes us from having 7 of:
###!!! ASSERTION: Shouldn't be trying to restyle non-elements directly: '!aContent || aContent->IsElement()', file ../../../layout/base/nsStyleChangeList.cpp, line 62
with the stack:
  mozilla::ElementRestyler::CaptureChange(nsStyleContext*, nsStyleContext*, nsChangeHint) [layout/base/nsChangeHint.h:191]
  mozilla::ElementRestyler::RestyleSelf(nsRestyleHint) [layout/base/RestyleManager.cpp:2304]
to only having one.  This is expected since this patch changes
RestyleSelf to only call CaptureChange for the first continuation or
block-in-inline sibling.
2013-09-25 12:28:08 -07:00
L. David Baron
36acc15f31 Bug 828312 patch 10: Add assertions to check that handling of nsChangeHint_ChildrenOnlyTransform doesn't need to check continuations. r=bzbarsky
This is part of the patch stack making change hints apply across all
continuations and block-in-inline siblings.
2013-09-25 12:28:08 -07:00
L. David Baron
4e703ff333 Bug 828312 patch 9b: Make handling of RecomputePosition hint check continuations. r=bzbarsky
This is part of the patch stack making change hints apply across all
continuations and block-in-inline siblings.
2013-09-25 12:28:08 -07:00
L. David Baron
d0255c650f Bug 828312 patch 9a: Make sticky positioning handle block-in-inline splits correctly. r=dholbert
This assumes that the specification for how position:sticky behaves for
block-in-inline splits matches the specification for position:relative,
in other words, matches
http://www.w3.org/TR/CSS21/visuren.html#anonymous-block-level .

It's also necessary for patch 9b since the new rule for handling of
style change hints is that a style change hint applies to all
continuations and all block-in-inline siblings ("special siblings").

The change in StickyScrollContainer::GetScrollRanges is really the fix
for bug 918994, but adjusted for the change here to use block-in-inline
siblings ("special siblings") in addition to continuations.
2013-09-25 12:28:08 -07:00
L. David Baron
95e754510e Bug 828312 patch 8: Make handling of UpdateEffects hint check continuations. r=bzbarsky
This is part of the patch stack making change hints apply across all
continuations and block-in-inline siblings.
2013-09-25 12:28:07 -07:00
L. David Baron
82151d1958 Bug 828312 patch 7: Use more typical loop structure and don't mutate |frame| in UpdateOverflow hint handling. r=bzbarsky 2013-09-25 12:28:07 -07:00
L. David Baron
b741eb1e7b Bug 828312 patch 6: Make early transform handling check continuations. r=bzbarsky
This is part of the patch stack making change hints apply across all
continuations and block-in-inline siblings.
2013-09-25 12:28:07 -07:00
L. David Baron
ee558d6e79 Bug 828312 patch 5: Don't handle UpdateCursor more than once per round of style change processing, since it's global. r=bzbarsky
(This is part of the patch stack making change hints apply across all
continuations and block-in-inline siblings.  In this case, however, the
change hint only needs to apply once, globally.)
2013-09-25 12:28:07 -07:00
L. David Baron
e1a486d3f2 Bug 898209 patch 16: Move a comment that should have been moved in patch 15. No review. 2013-09-25 12:28:07 -07:00
L. David Baron
c18e1d2515 Bug 898333, patch 2: Change RestyleManager::Restyle's tree traversal to reach next-continuations (and same-display block-in-inline siblings) from their prev-continuation rather than from their parent. r=bzbarsky
This patch fundamentally makes three changes:

 (1) Change the way RestyleSelf is used so that it is called in a loop,
     over all of the same-style continuations and block-in-inline
     siblings.
     (I had a note here reminding myself:
       TODO: Don't set hints for descendants!
     but I no longer remember what it meant.)

 (2) Change the traversal of children to traverse all of the children of
     the items traversed in (1).

 (3) When traversing children, skip children that are continuations,
     since we already reached them in (1) and (2).

A later patch will change the RestyleSelf loop to copy for the later
continuations rather than repeating the work.  This will mean reverting
many of the RestyleSelf changes contained here.

Some of the pieces marked temporary will be removed in bug 828312 patch
11, and the rest should hopefully be removed in bug 918064.
2013-09-25 12:28:07 -07:00
L. David Baron
38a76e02ae Bug 898333, patch 1: Restyle the :after pseudo-element after the content children. r=bzbarsky 2013-09-25 12:28:07 -07:00
Robert Longson
4fc1c36524 Bug 919900 - Mark reftest fuzzy on windows 7 2013-09-26 15:29:09 +01:00
Robert Longson
6da1082e02 Bug 919900 - Mark reftest fuzzy on windows 7 2013-09-26 15:22:13 +01:00
Robert Longson
a9cb23d3ec Bug 919900 - Correct bounding box calculations for empty text elements with svg.text.css-frames.enabled. r=heycam 2013-09-26 13:42:15 +01:00
Frank Yan
483c701a7d Bug 880672: Add support for unprefixed cursor:grab and cursor:grabbing. r=bzbarsky sr=dbaron 2013-06-24 00:32:44 -07:00
Daniel Holbert
ffdecec1a8 Bug 919813 part 2: Upgrade NS_ERROR to MOZ_ASSERT in nsFrame's impl of SetPrevInFlow, SetNextInFlow, SetPrevContinuation, SetNextContinuation. rs=mats 2013-09-25 10:54:56 -07:00
Daniel Holbert
aa859ecb8b Bug 919813 part 1: Remove never-checked return value from frame methods SetPrevInFlow, SetNextInFlow, SetPrevContinuation, SetNextContinuation. r=mats 2013-09-25 10:54:55 -07:00
Birunthan Mohanathas
64f8d6e459 Bug 919729 - Remove unnecessary MOZILLA_INTERNAL_API checks from nsIFrame.h. r=bz 2013-09-25 09:48:20 -04:00
Birunthan Mohanathas
2f82a6a077 Bug 906852 - Make layout/style/nsFontFaceLoader.* pointer notation follow layout code style. r=mats 2013-09-25 09:48:20 -04:00
Mats Palmgren
eb2304a29c Bug 919318 - A few cosmetic changes after dropping the Get prefix on some methods. r=dholbert 2013-09-25 11:42:35 +00:00
Mats Palmgren
4974add798 Bug 919318 - Drop the Get prefix on the frame methods GetFirstContinuation, GetLastContinuation, GetFirstInFlow, GetLastInFlow and also on nsLayoutUtils::GetLastContinuationWithChild, because they never return null. r=dholbert 2013-09-25 11:42:34 +00:00
Masayuki Nakano
f90a5b1125 Bug 912956 part.18 Remove nsGUIEvent.h r=roc 2013-09-25 20:21:22 +09:00
Masayuki Nakano
80d3ef9ea0 Bug 912956 part.17 mozilla/ContentEvents.h should be included directly r=roc 2013-09-25 20:21:20 +09:00
Masayuki Nakano
e28d8401da Bug 912956 part.16 mozilla/MiscEvents.h should be included directly r=roc 2013-09-25 20:21:20 +09:00
Masayuki Nakano
015140fe0d Bug 912956 part.15 mozilla/TextEvents.h should be included directly r=roc 2013-09-25 20:21:19 +09:00
Masayuki Nakano
4983de6d4c Bug 912956 part.14 mozilla/MouseEvents.h should be included directly r=roc 2013-09-25 20:21:18 +09:00
Masayuki Nakano
743fbc6e59 Bug 912956 part.13 mozilla/TouchEvents.h should be included directly r=roc 2013-09-25 20:21:16 +09:00
Cameron McCormack
3e9878463d Bug 916704 - Always initialize SVGTextContentPaint::Paint and check SetupCairoState return value in nsSVGTextFrame2::PaintSVG. r=longsonr 2013-09-25 17:18:41 +10:00
Daniel Holbert
90f7eae182 Bug 919871: Make .cpp files in layout/generic #include their corresponding .h file first. r=mats 2013-09-24 18:43:43 -07:00
Phil Ringnalda
b121657ba8 Merge m-c to a CLOSED TREE m-i 2013-09-24 21:22:56 -07:00
Wes Kocher
4c8aedc06e Merge b2g-i to m-c 2013-09-24 19:58:09 -07:00
Michael Wu
5d2dd3df6a Bug 919845 - Really make Fira Sans the default font, r=jfkthame 2013-09-24 10:57:25 -07:00
Benoit Jacob
55d6cf1e20 Bug 913872 - Take nested enums out of gfxASurface - 1/3 : automatic changes - r=jrmuizel
Generated by these regexes:

find . -name '*.h' -o -name '*.cpp' -o -name '*.mm' | grep -v '\.hg' | grep -v '^\.\/obj' | xargs sed -i 's/gfx[A-Za-z0-9_]*Surface\:\:[a-z]*\(\(ImageFormat\|SurfaceType\|ContentType\|MemoryLocation\)[0-9A-Za-z_]*\)/gfx\1/g'

find . -name '*.h' -o -name '*.cpp' -o -name '*.mm' | grep -v '\.hg' | grep -v '^\.\/obj' | xargs sed -i 's/gfx[A-Za-z0-9_]*Surface\:\:[a-z]*\(\(CONTENT_\|MEMORY_\)[0-9A-Za-z_]*\)/GFX_\1/g'

find . -name '*.h' -o -name '*.cpp' -o -name '*.mm' | grep -v '\.hg' | grep -v '^\.\/obj' | xargs sed -i 's/\(^\|[^A-Za-z0-9_]\)\(CONTENT_COLOR\|CONTENT_ALPHA\|CONTENT_COLOR_ALPHA\|CONTENT_SENTINEL\|MEMORY_IN_PROCESS_HEAP\|MEMORY_IN_PROCESS_NONHEAP\|MEMORY_OUT_OF_PROCESS\)\($\|[^A-Za-z0-9_]\)/\1GFX_\2\3/g'

find . -name '*.h' -o -name '*.cpp' -o -name '*.mm' | grep -v '\.hg' | grep -v '^\.\/obj' | xargs sed -i 's/\(^\|[^A-Za-z0-9_]\)\(ImageFormatARGB32\|ImageFormatRGB24\|ImageFormatA8\|ImageFormatA1\|ImageFormatRGB16_565\|ImageFormatUnknown\|SurfaceTypeImage\|SurfaceTypePDF\|SurfaceTypePS\|SurfaceTypeXlib\|SurfaceTypeXcb\|SurfaceTypeGlitz\|SurfaceTypeQuartz\|SurfaceTypeWin32\|SurfaceTypeBeOS\|SurfaceTypeDirectFB\|SurfaceTypeSVG\|SurfaceTypeOS2\|SurfaceTypeWin32Printing\|SurfaceTypeQuartzImage\|SurfaceTypeScript\|SurfaceTypeQPainter\|SurfaceTypeRecording\|SurfaceTypeVG\|SurfaceTypeGL\|SurfaceTypeDRM\|SurfaceTypeTee\|SurfaceTypeXML\|SurfaceTypeSkia\|SurfaceTypeSubsurface\|SurfaceTypeD2D\|SurfaceTypeMax\)\($\|[^A-Za-z0-9_]\)/\1gfx\2\3/g'
2013-09-24 16:45:13 -04:00
Jonathan Watt
e6ad65635d Bug 919263 - Remove bogus assertion from nsRangeFrame::AttributeChanged. r=dholbert
--HG--
extra : rebase_source : a713be35993eb58e44e2bb4f506587831986c57e
2013-09-24 21:29:31 +01:00
Jonathan Watt
87073cc853 Bug 920158 - Make nsCSSFrameConstructor::ConstructFrameFromItemInternal handle frames that should suppress floating of descendants. r=bz
--HG--
extra : rebase_source : 3f0b806dad1c1381bd30bd25e943866415e226f2
2013-09-24 21:29:29 +01:00
Jonathan Watt
2266364c41 Bug 920154 - Rename nsIContent's SetNativeAnonymous method to SetIsNativeAnonymousRoot to be clear that it should only be called for the root native anonymous element. r=bz
--HG--
extra : rebase_source : 9be498a3ae51a4a4587c8035cb7bc58002b3a268
2013-09-24 21:29:27 +01:00
Jonathan Watt
06ac2cb0e3 Bug 920146 - Rename NODE_IS_ANONYMOUS to NODE_IS_ANONYMOUS_ROOT to avoid misinterpretation and for symmetry with NODE_IS_NATIVE_ANONYMOUS_ROOT. r=bz
--HG--
extra : rebase_source : ea8c2377f1f85248239a3f6b0f6f64bfe40e44ea
2013-09-24 20:28:32 +01:00
Ryan VanderMeulen
1d956c2ad1 Backed out changeset dec7ac293e74 (bug 782546) for breaking the test it was trying to annotate. 2013-09-24 13:45:59 -04:00
Randell Jesup
4127d9fdaa Bug 917491: Guarantee cleanup of AsyncLatencyLogger on xpcom-shutdown r=bsmedberg 2013-09-24 22:10:24 -04:00