212145 - [FIXr]Generated content with overflow:auto; declared fails

Status: VERIFIED FIXED

(Core :: CSS Parsing and Computation, defect, P2)

Description

[Anne (:annevk)]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5a) Gecko/20030707
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5a) Gecko/20030707

Testcase
http://www.annevankesteren.nl/css/testcase/generated_content.html

Mozilla doens't show the generated content and applies the text-transform on the
wrong letter which is a direct result of the first.

Reproducible: Always

Steps to Reproduce:

Actual Results:  
Mozilla renders the code with most of the style applied. The first letter of the
code is transferred to uppercase. There isn't any generated content.

Expected Results:  
There should be a block-level element with in it: "The value of the class from
the element code", followed by a "space" and then "code:". The first letter of
the classname should be in uppercase. Then there should be a linebreak and then
comes the content of the element code.

Comment 1

[Boris Zbarsky [:bzbarsky]]

So the problem seems to be that in ConstructFrameByDisplayType the
IsScrollable(aPresContext, aDisplay) case passes PR_FALSE for the
aCanHaveGeneratedContent param of ProcessChildren when calling ProcessChildren
with the scrolled frame as a parent.

Changing that to PR_TRUE makes the testcase render correctly, modulo the
::before/::first-letter problem...

Is there a good reason that param is PR_FALSE?  The cvs blame for this line is
something back in 1999, and drilling back in cvs revision history did not give
me any insight into why it was written this way.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I doubt it.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: proposed patch

Comment 4

[Boris Zbarsky [:bzbarsky]]

.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Fixed for 1.5b.

Comment 6

[Anne (:annevk)]

The content is now generated in my testcase, however the first-letter is not
turned into uppercase. That's why I reopend the bug.

UA: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5b) Gecko/20030726

Comment 7

[Boris Zbarsky [:bzbarsky]]

One issue per bug, please.  Please file a separate bug and cc me on it, ok?

Comment 8

[mmc]

The other bug is bug 214004.

Comment 9

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

*** Bug 217868 has been marked as a duplicate of this bug. ***

Comment 10

[Anne (:annevk)]

v