1285365 - Remove trailing whitespace from nsCSSParser & add missing includes and namespaces relying on unification

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Jonathan Chan [:jyc]]

In the process of working on Bug 1273706, I found some trailing whitespace in nsCSSParser that I removed while removing trailing whitespace from my code.

In addition, because the patches add some new files, the groupings from unification changed, causing build errors due to missing includes or namespace qualifiers.

I will attach two trivial patches that remove whitespace and fix these build errors.

Comment 1

[Jonathan Chan [:jyc]]

Attachment: Bug 1285365 - Part 1: Remove trailing whitespace from nsCSSParser.

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

Comment 2

[Jonathan Chan [:jyc]]

Attachment: Bug 1285365 - Part 2: Add missing includes and namespace qualifiers.

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

Comment 3

[Daniel Holbert [:dholbert]]

Comment on attachment 8768938 [details]
Bug 1285365 - Part 1: Remove trailing whitespace from nsCSSParser.

https://reviewboard.mozilla.org/r/62934/#review59836

Part 1 looks good - r=me. Thanks for cleaning this up!

Comment 4

[Daniel Holbert [:dholbert]]

Comment on attachment 8768939 [details]
Bug 1285365 - Part 2: Add missing includes and namespace qualifiers.

https://reviewboard.mozilla.org/r/62936/#review59840

part 2 looks good too. r=me

Thanks for cleaning this up!

Comment 5

[Daniel Holbert [:dholbert]]

You should trigger a try build for the changes here (either via mozreview or via ./mach try), just to be sure that "part 2" doesn't accidentally cause some bustage (as include/namespacing changes occasionally can).  And then I'll be happy to autoland this for you.

For the Try run, we probably just need to verify that this builds correctly, on windows/mac/linux. No need to do any unit test runs.

Comment 6

[Jonathan Chan [:jyc]]

Here is the try run:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=fcc544635d7cbf6440560ca0e675878c5d29cafb

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cb183e7c648b
Part 1: Remove trailing whitespace from nsCSSParser. r=dholbert
https://hg.mozilla.org/integration/autoland/rev/dc8af4e3e454
Part 2: Add missing includes and namespace qualifiers. r=dholbert

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/cb183e7c648b
https://hg.mozilla.org/mozilla-central/rev/dc8af4e3e454