1466330 - Make nsITheme::GetWidgetBorder return the border directly instead of using an out-param

Status: RESOLVED FIXED

(Core :: Widget, enhancement, P4)

Description

[Mats Palmgren (inactive)]

It currently always returns NS_OK and no caller actually checks
what value it returned anyway...

Comment 1

[Mats Palmgren (inactive)]

Attachment: Make nsITheme::GetWidgetBorder return the border directly instead of using an out-param (idempotent patch)

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Comment on attachment 8982810 [details] [diff] [review]
Make nsITheme::GetWidgetBorder return the border directly instead of using an out-param (idempotent patch)

Review of attachment 8982810 [details] [diff] [review]:
-----------------------------------------------------------------

Looks great, thank you!

::: gfx/src/nsITheme.h
@@ +56,5 @@
>   * the constants in nsThemeConstants.h).
>   */
>  class nsITheme: public nsISupports {
>  public:
> +  using LayoutDeviceIntMargin = mozilla::LayoutDeviceIntMargin;

nit: Doesn't need to be public I think

::: widget/windows/nsNativeThemeWin.cpp
@@ +2844,4 @@
>    switch (aWidgetType) {
>      case NS_THEME_GROUPBOX:
>      case NS_THEME_BUTTON:
> +      result.top = result.left = result.bottom = result.right = 2; 

nit: maybe fix the trailing whitespace while touching this line?

@@ +2863,4 @@
>        break;
>      case NS_THEME_STATUSBARPANEL:
>      case NS_THEME_RESIZERPANEL: {
> +      result.top = 1;      

And this?

Comment 3

[Mats Palmgren (inactive)]

I fixed the nits as suggested, thanks.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1f62ecdf59b6
Make nsITheme::GetWidgetBorder return the border directly instead of using an out-param (idempotent patch).  r=emilio

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/1f62ecdf59b6