1287308 - remove old NS_STYLE_HINT_NONE constant

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Jeremy Chen ([e-mail])]

This is part of the work in Bug 1077851, eliminating usage of NS_STYLE_HINT_* in favor of nsChangeHint_*.

Compared with other NS_STYLE_HINT_*, NS_STYLE_HINT_NONE is quite independent. So, I'm planning to fix it in this bug.

Comment 1

[Jeremy Chen ([e-mail])]

Attachment: Bug 1287308 - part1: replace uses of NS_STYLE_HINT_NONE with nsChangeHint(0).

Replace all the usages of NS_STYLE_HINT_NONE with nsChangeHint(0).

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

Comment 2

[Jeremy Chen ([e-mail])]

Attachment: Bug 1287308 - part2: remove old NS_STYLE_HINT_NONE constant.

Remove NS_STYLE_HINT_NONE definition in nsChangeHint.
Fix related debug log in RestyleManager.

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

Comment 3

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Comment on attachment 8771730 [details]
Bug 1287308 - part1: replace uses of NS_STYLE_HINT_NONE with nsChangeHint(0).

https://reviewboard.mozilla.org/r/64766/#review61790

Your commit message is currently:
>Bug 1287308 - part1: replace NS_STYLE_HINT_NONE with nsChangeHint(0).
>
>Replace all the usages of NS_STYLE_HINT_NONE with nsChangeHint(0).

"usages" should be "uses".

But you really don't need the bug's summary separately from which part of the bug.  All you need for the commit message is:

Bug 1287308 part 1 - replace uses of NS_STYLE_HINT_NONE with nsChangeHint(0)

r=dbaron with that or similar

Comment 4

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Comment on attachment 8771731 [details]
Bug 1287308 - part2: remove old NS_STYLE_HINT_NONE constant.

https://reviewboard.mozilla.org/r/64768/#review61792

Comment 5

[Jeremy Chen ([e-mail])]

Comment on attachment 8771730 [details]
Bug 1287308 - part1: replace uses of NS_STYLE_HINT_NONE with nsChangeHint(0).

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

Comment 6

[Jeremy Chen ([e-mail])]

Comment on attachment 8771731 [details]
Bug 1287308 - part2: remove old NS_STYLE_HINT_NONE constant.

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

Comment 7

[Jeremy Chen ([e-mail])]

(In reply to David Baron :dbaron: ⌚️UTC+8 (review requests must explain patch) from comment #3)
> Comment on attachment 8771730 [details]
> Bug 1287308 - part1: replace uses of NS_STYLE_HINT_NONE with nsChangeHint(0).
> 
> https://reviewboard.mozilla.org/r/64766/#review61790
> 
> Your commit message is currently:
> >Bug 1287308 - part1: replace NS_STYLE_HINT_NONE with nsChangeHint(0).
> >
> >Replace all the usages of NS_STYLE_HINT_NONE with nsChangeHint(0).
> 
> "usages" should be "uses".
> 
> But you really don't need the bug's summary separately from which part of
> the bug.  All you need for the commit message is:
> 
> Bug 1287308 part 1 - replace uses of NS_STYLE_HINT_NONE with nsChangeHint(0)
> 
> r=dbaron with that or similar

Will do. Thank you for the review.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/78bff0b2ded4
part1: replace uses of NS_STYLE_HINT_NONE with nsChangeHint(0). r=dbaron
https://hg.mozilla.org/integration/autoland/rev/0d1e96f2ac72
part2: remove old NS_STYLE_HINT_NONE constant. r=dbaron

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/78bff0b2ded4
https://hg.mozilla.org/mozilla-central/rev/0d1e96f2ac72