Open Bug 1376497 Opened 7 years ago Updated 2 years ago

Set mDocGroup for HTML docs being loaded via XHR

Categories

(Core :: DOM: Core & HTML, enhancement, P3)

Product:
Core
Component:
DOM: Core & HTML
Type:
enhancement

Tracking

()

People

(Reporter: hsivonen, Assigned: hsivonen)

Details

Attachments

(1 file)

Bug 1376497 - Put XHR docs in a DocGroup.
(deleted), text/x-review-board-request
Details
When loading an HTML doc via XHR, we sohuld set mDocGroup appropriately (before OnStartRequest fires to the HTML parser).
Assignee: nobody → hsivonen
Priority: -- → P3
Component: DOM → DOM: Core & HTML

Do we still care about this? (The bug was filed is the context of Quantum DOM.)

Flags: needinfo?(afarre)

Hmm, has the situation changed with the DOM Arena stuff. Do we already effectively do this?

Flags: needinfo?(sefeng)

I don't think so?

Because DOM Arena helps if there is a node that is allocated/adopted this document before OnStartRequest, but I don't think that is always the case for XHR document?

Looks like XMLHttpRequestMainThread::OnDataAvailable would parse the XML and the create XML element, if this always get executed before OnStartRequest, then DOM Arena helps here.

Flags: needinfo?(sefeng)

There is a point in keeping DocGroups around if you're dispatching to/with them with a clear distinction that the work that is done is tied to that DocGroup, because then we want that work to show up in about:performance.

Flags: needinfo?(afarre)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: