querySelector on createDocumentFragment triggers "ASSERTION: Unexpected root node"

RESOLVED FIXED in mozilla10

Status

()

P1
normal
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: jruderman, Assigned: bzbarsky)

Tracking

(Blocks: 1 bug, {assertion, regression, testcase})

Trunk
mozilla10
x86_64
Mac OS X
assertion, regression, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

7 years ago
Created attachment 571223 [details]
testcase

> ###!!! ASSERTION: Unexpected root node: 'aRoot->IsElement() || aRoot->IsNodeOfType(nsINode::eDOCUMENT)', file content/base/src/nsGenericElement.cpp, line 5395

> nsGenericElement::doQuerySelector [content/base/src/nsGenericElement.cpp:5463]
> nsIDOMNodeSelector_QuerySelector [dom_quickstubs.cpp:5689]

I'm guessing this is a regression from:

changeset:   b068485e4cc2
user:        Boris Zbarsky
date:        Mon Oct 31 22:50:50 2011 -0400
summary:     Bug 696205 part 2.  Add a querySelector fast-path for selectors whose rightmost sequence of simple selectors contains an id.  r=sicking
(Assignee)

Comment 1

7 years ago
Bah, indeed.  Fixing.
Assignee: nobody → bzbarsky
Priority: -- → P1
(Assignee)

Comment 2

7 years ago
Created attachment 571236 [details] [diff] [review]
Fix assertion to not assert things that are just false.
Attachment #571236 - Flags: review?(jonas)
(Assignee)

Updated

7 years ago
Whiteboard: [need review]
(Reporter)

Comment 3

7 years ago
What's the loop index change for?
(Assignee)

Comment 4

7 years ago
Fixes a compiler warning...
(Assignee)

Comment 5

7 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/5883190e98aa
Flags: in-testsuite+
Whiteboard: [need review]
Target Milestone: --- → mozilla10

Comment 6

7 years ago
https://hg.mozilla.org/mozilla-central/rev/5883190e98aa
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.