Closed Bug 1560149 Opened 4 months ago Closed 4 months ago

write down the design of document-splitting

Categories

(Core :: Graphics: WebRender, task)

task
Not set

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: Gankra, Assigned: kats)

Details

Attachments

(1 file)

This needs to be written down, as the two people who know the most about this are going on leave (and one of them doesn't report to gfx, so we can't expect them to maintain it indefinitely).

Flags: needinfo?(kats)
Flags: needinfo?(dothayer)

I'm working on this right now as a matter of fact

Assignee: nobody → kats
Flags: needinfo?(kats)

WIP so far: https://github.com/staktrace/gecko/blob/fe76d58316c2/gfx/docs/DocumentSplitting.rst (EDIT: this is obsolete now, see updated link in comment 4)

Still need to fill in some stuff about the trickier bits of the implementation, but any feedback is welcome.

And also, the parts of document splitting that are internal to WR (i.e. in rust code) I'm not familiar with, so if there's anything there that needs documenting, Doug, please write up something.

Updated: https://github.com/staktrace/gecko/blob/docsplit_doc/gfx/docs/DocumentSplitting.rst

I think this is all that I can write for now, will put up the patch for review. Please provide feedback on what else you'd like to see documented.

Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/34b35632f194
Add some documentation for document splitting. r=dthayer
Backout by shindli@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ca0726ea50cb
Backed out changeset 34b35632f194 for linting failure in /builds/worker/checkouts/gecko/gfx/docs/DocumentSplitting.rst:162 CLOSED TREE
Flags: needinfo?(kats)
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/405f857977e5
Add some documentation for document splitting. r=dthayer
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
Flags: needinfo?(dothayer)
You need to log in before you can comment on or make changes to this bug.