Closed
Bug 1465866
Opened 5 years ago
Closed 5 years ago
Remove methods from ScrollBoxObject
Categories
(Core :: XUL, enhancement, P3)
Core
XUL
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: emalysz, Assigned: emalysz)
References
Details
Attachments
(1 file, 3 obsolete files)
No description provided.
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Attachment #8982287 -
Attachment is patch: true
Attachment #8982287 -
Attachment mime type: text/x-c++src → text/plain
Comment hidden (mozreview-request) |
Assignee | ||
Updated•5 years ago
|
Attachment #8982287 -
Attachment is obsolete: true
Comment hidden (mozreview-request) |
Assignee | ||
Updated•5 years ago
|
Attachment #8982296 -
Attachment is obsolete: true
Assignee | ||
Updated•5 years ago
|
Attachment #8982580 -
Flags: review?(enndeakin)
Comment 4•5 years ago
|
||
mozreview-review |
Comment on attachment 8982580 [details] Bug 1465866: removes methods from ScrollBoxObject and deletes unneccessary spacing +6102 https://reviewboard.mozilla.org/r/248558/#review254826 This looks good, but let's test it. Watch here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=371cb71e84d75358c6d51ae8eca274687f9ced18 ::: commit-message-763f3:3 (Diff revision 1) > +Bug #1465866: remove methods from ScrollBoxObject > + > +MozReview-Commit-ID: GrwTPJYai6O You should make sure that the patch has a commit message for it. Also, make sure your 'name' is configured in 'hg'. ::: layout/xul/ScrollBoxObject.cpp:290 (Diff revision 1) > nsIPresShell::ScrollAxis(), > nsIPresShell::ScrollAxis(), > nsIPresShell::SCROLL_FIRST_ANCESTOR_ONLY | > nsIPresShell::SCROLL_OVERFLOW_HIDDEN); > } > - > + These is some extra space here that should be removed.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•5 years ago
|
Attachment #8982637 -
Attachment is obsolete: true
Comment hidden (mozreview-request) |
Updated•5 years ago
|
Priority: -- → P3
![]() |
||
Comment 8•5 years ago
|
||
mozreview-review |
Comment on attachment 8982580 [details] Bug 1465866: removes methods from ScrollBoxObject and deletes unneccessary spacing +6102 https://reviewboard.mozilla.org/r/248558/#review255608 Thank you for fixing this! It's nice to see the cruft disappear. ;) ::: dom/webidl/ScrollBoxObject.webidl (Diff revision 3) > - * > - * Get the current scroll position in css pixels. > - * @see scrollTo for the definition of x and y. > - */ > - [Throws] > - void getPosition(object x, object y); This is used in a few places in comm-central (in calendar/ and im/). Please file a bug on them to update to positionX/Y? ::: dom/webidl/ScrollBoxObject.webidl (Diff revision 3) > - > - /** > - * DEPRECATED: Please use scrolledWidth and scrolledHeight > - */ > - [Throws] > - void getScrolledSize(object width, object height); This is also used in im/ in comm-central. Probably just mention it in the same bug?
Attachment #8982580 -
Flags: review?(bzbarsky) → review+
![]() |
||
Updated•5 years ago
|
Assignee | ||
Comment 10•5 years ago
|
||
Comment on attachment 8982580 [details] Bug 1465866: removes methods from ScrollBoxObject and deletes unneccessary spacing +6102 https://reviewboard.mozilla.org/r/248558/#review261314
Comment 11•5 years ago
|
||
hg error in cmd: hg push -r tip ssh://hg.mozilla.org/integration/autoland: pushing to ssh://hg.mozilla.org/integration/autoland searching for changes remote: adding changesets remote: adding manifests remote: adding file changes remote: added 1 changesets with 3 changes to 3 files remote: remote: remote: ************************** ERROR **************************** remote: Rev 79940d7a5e2d needs "Bug N" or "No bug" in the commit message. remote: Emma Malysz <emalysz@mozilla.com> remote: Bug #1465866: removes methods from ScrollBoxObject and deletes unneccessary spacing r=bz+6102 remote: remote: MozReview-Commit-ID: GrwTPJYai6O remote: ************************************************************* remote: remote: remote: transaction abort! remote: rollback completed remote: pretxnchangegroup.c_commitmessage hook failed abort: push failed on remote
Comment hidden (mozreview-request) |
Comment 13•5 years ago
|
||
Pushed by bgrinstead@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/058b07756df6 removes methods from ScrollBoxObject and deletes unneccessary spacing r=bz+6102
Comment 14•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/058b07756df6
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Comment 15•5 years ago
|
||
Any particular reason for these white-space changes? https://hg.mozilla.org/mozilla-central/rev/058b07756df6#l2.169
Updated•2 years ago
|
Flags: needinfo?(emalysz)
You need to log in
before you can comment on or make changes to this bug.
Description
•