gecko-dev/layout
Ting-Yu Lin 570d1f0e83 Bug 1304441 Part 3 - Insert main summary's frame construction item at front of the list. r=bz
Change the logic that moves the main summary to the front from operating
on generated frames in DetailsFrame::SetInitialChildList() to operating
on frame construction item list in AddFrameConstructionItemsInternal()
so that it will be correct when cooperating with ::first-line.

The root cause of the bug reported is because when specifying
::first-line on details element, the first frame of summary element,
which is generated due to ib-split, will be wrapped in nsFirstLineFrame.
The original code fails to find the summary frame in the wrapper frame
and triggers assertion because of the second ib-split summary frame. To
fix that, we need to descend into the child list of wrapper frames when
checking the main summary.

Add original test case as a crashtest as well as reftests to clearly
reproduce the issue.

Note that in the reftest, the blue color in ::first-line is applied
incorrectly to the second line in the summary due to bug 520605.

MozReview-Commit-ID: Bv4Vcvxp6pY
2016-10-05 14:43:32 +08:00
..
base Bug 1304441 Part 3 - Insert main summary's frame construction item at front of the list. r=bz 2016-10-05 14:43:32 +08:00
build Bug 209637 - Add support for application/vnd.wap.xhtml+xml. r=bz 2016-09-28 09:48:17 -05:00
doc Bug 906116 - part3.3: Replace NS_STYLE_HINT_FRAMECHANGE with nsChangeHint_ReconstructFrame. r=dbaron 2016-07-06 13:06:14 +08:00
forms Bug 1305282 - Test using the SPACE key in incremental search of options. 2016-10-04 21:13:53 +02:00
generic Bug 1304441 Part 3 - Insert main summary's frame construction item at front of the list. r=bz 2016-10-05 14:43:32 +08:00
inspector Bug 1306214 part 2 - Remove -moz-use-text-color from devtools. r=tromey 2016-09-29 17:45:02 +10:00
ipc Bug 1306644 - Have TabParent call TakeFocusForClickFromTap rather than doing it over a sync IPC roundtrip. r=dvander 2016-09-30 15:03:42 -04:00
mathml Bug 1299741 part 11 - Change text-{emphasis,fill,stroke}-color to using StyleComplexColor. r=dbaron 2016-09-16 14:40:45 +10:00
media Bug 1286877 - do not set c-basic-offset for python-mode; r=gps 2016-07-14 10:16:42 -06:00
printing Bug 1305309, backout Bug 1214805, r=backout 2016-09-27 01:03:24 +03:00
reftests Bug 1304441 Part 3 - Insert main summary's frame construction item at front of the list. r=bz 2016-10-05 14:43:32 +08:00
style Bug 1302949 - Drop unused aTargetElement parameter from ComputeValuesFromStyleRule; r=heycam 2016-10-03 15:41:55 +09:00
svg Merge mozilla-central to autoland 2016-09-30 12:11:06 +02:00
tables Bug 1288350 - Temporary reftest list for Stylo vs Gecko test r=heycam,manishearth 2016-09-05 11:45:39 +08:00
tools Bug 1269807 - Remove support for all NPAPI plugins except for Flash, behind a pref. Tests that use the testplugin for now set the pref to keep it working. This will be disabled for ESR 52, but enabled for release 52. In the next cycle, the pref will be removed and this will be hardcoded. r=jimm 2016-09-29 09:16:33 -04:00
xul merge mozilla-inbound to mozilla-central a=merge 2016-09-30 12:02:16 +02:00
moz.build Bug 1286877 - do not set c-basic-offset for python-mode; r=gps 2016-07-14 10:16:42 -06:00