Closed
Bug 1092262
Opened 11 years ago
Closed 11 years ago
Doxify the descriptions of APZ prefs in AsyncPanZoomController.cpp
Categories
(Core :: Panning and Zooming, defect)
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: botond, Assigned: milan)
References
Details
Attachments
(1 file, 1 obsolete file)
|
16.24 KB,
patch
|
milan
:
review+
|
Details | Diff | Splinter Review |
APZ prefs are documented in AsyncPanZoomController.cpp [1]. It would be nice if this documentation showed up in the doxygen documentation [2].
[1] http://mxr.mozilla.org/mozilla-central/source/gfx/layers/apz/src/AsyncPanZoomController.cpp?rev=ea9a3cbcdd0b#140
[2] http://people.mozilla.org/~bgirard/doxygen/gfx/
|
Assignee | |
Updated•11 years ago
|
Blocks: gfx-target-2.2
|
|
Assignee | |
Comment 1•11 years ago
|
||
It may be also worth referencing this page from another spot in the APZ docs...
Attachment #8517020 -
Flags: review?(botond)
|
||
Comment 2•11 years ago
|
||
Comment on attachment 8517020 [details] [diff] [review]
Have the APZ prefs comments show up in the doxygen docs...
Review of attachment 8517020 [details] [diff] [review]:
-----------------------------------------------------------------
Stealing review
Attachment #8517020 -
Flags: review?(botond) → review+
|
|
Assignee | |
Comment 3•11 years ago
|
||
Add a reference to this page from the APZ overview docs, not bothering with a new review.
Attachment #8517020 -
Attachment is obsolete: true
Attachment #8517556 -
Flags: review+
|
|
Assignee | |
Updated•11 years ago
|
Assignee: nobody → milan
|
|
Assignee | |
Updated•11 years ago
|
Keywords: checkin-needed
|
Reporter | |
Comment 4•11 years ago
|
||
Thanks Milan and Kats!
|
||
Comment 5•11 years ago
|
||
Keywords: checkin-needed
|
|
||
Comment 6•11 years ago
|
||
|
|
||
Comment 7•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•