Last Comment Bug 653356 - fix comment describing nsHTMLReflowState::availableWidth
: fix comment describing nsHTMLReflowState::availableWidth
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Layout (show other bugs)
: Trunk
: All All
: P4 normal (vote)
: mozilla6
Assigned To: David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-04-27 23:24 PDT by David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
Modified: 2011-04-28 10:27 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (1.52 KB, patch)
2011-04-27 23:24 PDT, David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
bzbarsky: review+
Details | Diff | Review

Description David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2011-04-27 23:24:55 PDT
Created attachment 528803 [details] [diff] [review]
patch

While reviewing bug 567872 and bug 633209 I came to the conclusion that this comment is wrong.  I'd like you to check that you agree with me, since it's sort of important...
Comment 1 Boris Zbarsky [:bz] (Out June 25-July 6) 2011-04-28 06:48:48 PDT
Comment on attachment 528803 [details] [diff] [review]
patch

Yeah, agreed.

The comments for availableHeight could use some clarification too, though margin collapsing makes that situation more complicated.
Comment 2 David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2011-04-28 10:27:29 PDT
https://hg.mozilla.org/mozilla-central/rev/2f41abc5a89d

Note You need to log in before you can comment on or make changes to this bug.