Closed
Bug 579258
Opened 14 years ago
Closed 14 years ago
Remove nsIWidget::Scroll and related code
Categories
(Core :: Widget, defect)
Core
Widget
Tracking
()
RESOLVED
FIXED
People
(Reporter: roc, Assigned: roc)
References
Details
Attachments
(3 files)
13.48 KB,
patch
|
tnikkel
:
review+
joe
:
approval2.0+
|
Details | Diff | Splinter Review |
43.15 KB,
patch
|
tnikkel
:
review+
joe
:
approval2.0+
|
Details | Diff | Splinter Review |
56.14 KB,
patch
|
karlt
:
review+
joe
:
approval2.0+
|
Details | Diff | Splinter Review |
Post 564991, it is no longer used. There is also scroll analysis code in nsDisplayList.cpp that is no longer used.
Assignee | ||
Comment 1•14 years ago
|
||
Change IsVaryingRelativeToMovingFrame to take the "moving frame" as a direct parameter instead of getting it from the builder. This simplifies things a bit and makes it easier to remove all the other code related to "moving frames".
Assignee: nobody → roc
Attachment #462324 -
Flags: review?(tnikkel)
Assignee | ||
Comment 2•14 years ago
|
||
Mostly removing the "before move" region parameter to ComputeVisibility, which is totally unused now.
Attachment #462326 -
Flags: review?(tnikkel)
Assignee | ||
Comment 3•14 years ago
|
||
This is purely removing dead code.
Attachment #462327 -
Flags: review?(karlt)
Comment 4•14 years ago
|
||
Comment on attachment 462327 [details] [diff] [review] Part 3: remove nsIWidget::Scroll nsSimpleRef<pixman_region32> and pixman.h can also be removed from gtk2/nsWindow.cpp. $(MOZ_CAIRO_LIBS) can be removed from gtk2/Makefile.in. Probably $(MOZ_CAIRO_CFLAGS) can also be removed, though I'm not sure how gfxASurface.h gets cairo.h.
Attachment #462327 -
Flags: review?(karlt) → review+
Updated•14 years ago
|
Attachment #462324 -
Flags: review?(tnikkel) → review+
Updated•14 years ago
|
Attachment #462326 -
Flags: review?(tnikkel) → review+
Assignee | ||
Comment 5•14 years ago
|
||
(In reply to comment #4) > Comment on attachment 462327 [details] [diff] [review] > Part 3: remove nsIWidget::Scroll > > nsSimpleRef<pixman_region32> and pixman.h can also be removed from > gtk2/nsWindow.cpp. Actually it can't, nsWindow::SetWindowClipRegion still uses it.
Assignee | ||
Comment 6•14 years ago
|
||
Comment on attachment 462324 [details] [diff] [review] Part 1: refactor IsVaryingRelativeToMovingFrame Removing dead code
Attachment #462324 -
Flags: approval2.0?
Assignee | ||
Comment 7•14 years ago
|
||
Comment on attachment 462326 [details] [diff] [review] Part 2: Remove remaining "moving frame" code remove dead code
Attachment #462326 -
Flags: approval2.0?
Assignee | ||
Comment 8•14 years ago
|
||
Comment on attachment 462327 [details] [diff] [review] Part 3: remove nsIWidget::Scroll remove dead code
Attachment #462327 -
Flags: approval2.0?
Updated•14 years ago
|
Attachment #462324 -
Flags: approval2.0? → approval2.0+
Updated•14 years ago
|
Attachment #462326 -
Flags: approval2.0? → approval2.0+
Updated•14 years ago
|
Attachment #462327 -
Flags: approval2.0? → approval2.0+
Assignee | ||
Updated•14 years ago
|
Whiteboard: [needs landing]
Assignee | ||
Comment 9•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/f62609dc783f http://hg.mozilla.org/mozilla-central/rev/4fbf9e1464c3 http://hg.mozilla.org/mozilla-central/rev/c6363df91109
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [needs landing]
You need to log in
before you can comment on or make changes to this bug.
Description
•