[FIX]nsCSSFrameConstructor::ContentInserted uses wrong container node in some places

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
11 years ago
11 months ago

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Tracking

Trunk
x86
macOS
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

In particular, it's using aContainer for things like :before/:after checks first-line stuff, etc.  But it should be using |container|, since that's the content of the XBL insertion point we're working with.

Patch with tests coming up.
The tests for this will depend on the patch in bug 482585.
Depends on: 482585
Posted patch FixSplinter Review
Attachment #366732 - Flags: superreview?(roc)
Attachment #366732 - Flags: review?(roc)
Blocks: 482398
Attachment #366732 - Flags: superreview?(roc)
Attachment #366732 - Flags: superreview+
Attachment #366732 - Flags: review?(roc)
Attachment #366732 - Flags: review+
Pushed http://hg.mozilla.org/mozilla-central/rev/501d8a053dc9.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Product: Core → Core Graveyard
Component: Layout: Misc Code → Layout
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.