Closed Bug 1183085 Opened 5 years ago Closed 5 years ago

Fix typos and update out-of-date documentation of Layer and DisplayItem

Categories

(Core :: Layout, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: TYLin, Assigned: TYLin)

Details

Attachments

(5 files)

I found some typos and out-of-date documentation when I read Layer and DisplayItem code. I have some patches at hand.
Bug 1183085 - Correct the name for nsDisplayItem; r=roc
Attachment #8632775 - Flags: review?(roc)
Bug 1183085 - Remove EndConstruction() from layer documentation; r=roc

EndConstruction() was removed in bug 564993.
Attachment #8632776 - Flags: review?(roc)
Bug 1183085 - Correct argument name for BuildContainerLayerFor; r=roc
Attachment #8632777 - Flags: review?(roc)
Bug 1183085 - Update description for nsDisplayList methods; r=roc

AppendNewToTop and AppendNewToBottom do not return
NS_ERROR_OUT_OF_MEMORY now.
Attachment #8632778 - Flags: review?(roc)
Bug 1183085 - Correct a typo in FrameLayerBuilder.h; r=roc
Attachment #8632779 - Flags: review?(roc)
Comment on attachment 8632775 [details]
MozReview Request: Bug 1183085 - Correct the name for nsDisplayItem; r=roc

https://reviewboard.mozilla.org/r/13125/#review11745

Ship It!
Attachment #8632775 - Flags: review?(roc) → review+
Comment on attachment 8632776 [details]
MozReview Request: Bug 1183085 - Remove EndConstruction() from layer documentation; r=roc

https://reviewboard.mozilla.org/r/13127/#review11747

Ship It!
Comment on attachment 8632777 [details]
MozReview Request: Bug 1183085 - Correct argument name for BuildContainerLayerFor; r=roc

https://reviewboard.mozilla.org/r/13129/#review11749

Ship It!
Attachment #8632777 - Flags: review?(roc) → review+
Comment on attachment 8632778 [details]
MozReview Request: Bug 1183085 - Update description for nsDisplayList methods; r=roc

https://reviewboard.mozilla.org/r/13131/#review11751

Ship It!
Comment on attachment 8632779 [details]
MozReview Request: Bug 1183085 - Correct a typo in FrameLayerBuilder.h; r=roc

https://reviewboard.mozilla.org/r/13133/#review11753

Ship It!
Attachment #8632779 - Flags: review?(roc) → review+
You need to log in before you can comment on or make changes to this bug.