Closed Bug 1054563 Opened 6 years ago Closed 6 years ago

Clean up nsCSSFrameConstructor::BuildScrollFrame

Categories

(Core :: Layout, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla34

People

(Reporter: dholbert, Assigned: dholbert)

Details

Attachments

(2 files)

nsCSSFrameConstructor::BuildScrollFrame has an always-NS_OK return value (which that nobody checks anyway), so it should just return void.

It's also indented 4 spaces whereas the code around it is indented 2.

Filing this bug on cleaning it up.
Comment on attachment 8473989 [details] [diff] [review]
fix v1

(sorry, bugzilla autocompleted a patch name w/ some extra junk on the end)
Attachment #8473989 - Attachment description: fix v1a (with reviewer adjusted) → fix v1
Comment on attachment 8473989 [details] [diff] [review]
fix v1

Thanks for the whitespace ignoring diff
Attachment #8473989 - Flags: review?(tnikkel) → review+
https://hg.mozilla.org/mozilla-central/rev/b8041e7ee525
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.