1465866 - Remove methods from ScrollBoxObject

Status: RESOLVED FIXED

(Core :: XUL, enhancement, P3)

Comment 1

[Emma Malysz]

Attachment: removes unused methods

Comment 2

[Emma Malysz]

Attachment: Bug 1465866, remove unused Scrollbox methods

Review commit: https://reviewboard.mozilla.org/r/248244/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/248244/

Comment 3

[Emma Malysz]

Attachment: Bug 1465866: removes methods from ScrollBoxObject and deletes unneccessary spacing +6102

Review commit: https://reviewboard.mozilla.org/r/248558/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/248558/

Comment 4

[Neil Deakin]

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 5

[Emma Malysz]

Attachment: Bug #1465866: removes methods from ScrollBoxObject and deletes unneccessary spacing

Review commit: https://reviewboard.mozilla.org/r/248604/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/248604/

Comment 6

[Emma Malysz]

Comment on attachment 8982580 [details]
Bug 1465866: removes methods from ScrollBoxObject and deletes unneccessary spacing +6102

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/248558/diff/1-2/

Comment 7

[Emma Malysz]

Comment on attachment 8982580 [details]
Bug 1465866: removes methods from ScrollBoxObject and deletes unneccessary spacing +6102

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/248558/diff/2-3/

Comment 8

[Boris Zbarsky [:bzbarsky]]

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?

Comment 9

[Neil Deakin]

Is this ready to be checked in?

Comment 10

[Emma Malysz]

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

[Mozilla Autoland]

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 <[e-mail]>
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 12

[Emma Malysz]

Comment on attachment 8982580 [details]
Bug 1465866: removes methods from ScrollBoxObject and deletes unneccessary spacing +6102

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/248558/diff/3-4/

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/058b07756df6
removes methods from ScrollBoxObject and deletes unneccessary spacing r=bz+6102

Comment 14

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/058b07756df6

Comment 15

[Jorg K (CEST = GMT+2)]

Any particular reason for these white-space changes?
https://hg.mozilla.org/mozilla-central/rev/058b07756df6#l2.169