1448690 - Remove IsStyledByServo.

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement)

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1448690: Remove IsStyledByServo.

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

Comment 2

[Code Review Bot [:reviewbot]]

Comment on attachment 8962183 [details]
Bug 1448690: Remove IsStyledByServo.

https://reviewboard.mozilla.org/r/231026/#review236440


Code analysis found 4 defects in this patch:
 - 4 defects found by clang-tidy

You can run this analysis locally with:
 - `./mach static-analysis check path/to/file.cpp` (C/C++)


If you see a problem in this automated review, please report it here: http://bit.ly/2y9N9Vx


::: dom/base/DOMMatrix.h:31
(Diff revision 1)
>  struct DOMPointInit;
>  
>  class DOMMatrixReadOnly : public nsWrapperCache
>  {
>  public:
> -  DOMMatrixReadOnly(nsISupports* aParent, bool aIsServo)
> +  DOMMatrixReadOnly(nsISupports* aParent)

Error: Bad implicit conversion constructor for 'dommatrixreadonly' [clang-tidy: mozilla-implicit-constructor]

::: dom/base/DOMMatrix.h:31
(Diff revision 1)
>  struct DOMPointInit;
>  
>  class DOMMatrixReadOnly : public nsWrapperCache
>  {
>  public:
> -  DOMMatrixReadOnly(nsISupports* aParent, bool aIsServo)
> +  DOMMatrixReadOnly(nsISupports* aParent)

Error: Bad implicit conversion constructor for 'dommatrixreadonly' [clang-tidy: mozilla-implicit-constructor]

::: dom/base/DOMMatrix.h:158
(Diff revision 1)
>  };
>  
>  class DOMMatrix : public DOMMatrixReadOnly
>  {
>  public:
> -  DOMMatrix(nsISupports* aParent, bool aIsServo)
> +  DOMMatrix(nsISupports* aParent)

Error: Bad implicit conversion constructor for 'dommatrix' [clang-tidy: mozilla-implicit-constructor]

::: dom/base/DOMMatrix.h:158
(Diff revision 1)
>  };
>  
>  class DOMMatrix : public DOMMatrixReadOnly
>  {
>  public:
> -  DOMMatrix(nsISupports* aParent, bool aIsServo)
> +  DOMMatrix(nsISupports* aParent)

Error: Bad implicit conversion constructor for 'dommatrix' [clang-tidy: mozilla-implicit-constructor]

Comment 3

[Emilio Cobos Álvarez (:emilio)]

Comment on attachment 8962183 [details]
Bug 1448690: Remove IsStyledByServo.

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

Comment 4

[Xidorn Quan [:xidorn] UTC+10]

Comment on attachment 8962183 [details]
Bug 1448690: Remove IsStyledByServo.

https://reviewboard.mozilla.org/r/231026/#review236440

> Error: Bad implicit conversion constructor for 'dommatrixreadonly' [clang-tidy: mozilla-implicit-constructor]

As you are remiving the only other parameter here, you need to add `explicit` here to satisify the linter.

Comment 5

[Xidorn Quan [:xidorn] UTC+10]

Comment on attachment 8962183 [details]
Bug 1448690: Remove IsStyledByServo.

https://reviewboard.mozilla.org/r/231026/#review236454

::: dom/base/WebKitCSSMatrix.h:18
(Diff revision 2)
>  namespace dom {
>  
>  class WebKitCSSMatrix final : public DOMMatrix
>  {
>  public:
> -  WebKitCSSMatrix(nsISupports* aParent, bool aIsServo)
> +  WebKitCSSMatrix(nsISupports* aParent)

`explicit`

::: dom/base/nsINode.h
(Diff revision 2)
> -   * Returns true if this is a node belonging to a document that uses the Servo
> -   * style system.
> -   */
> -  bool IsStyledByServo() const;
> -
> -  inline void UnsetRestyleFlagsIfGecko();

It seems this removal should belong to bug 1448688 rather than here.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b84db00d7fd9
Remove IsStyledByServo. r=xidorn

Comment 7

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/b84db00d7fd9