1540990 - Get rid of unnecessary nsIPresShell.h inclusions

Status: RESOLVED FIXED

(Core :: General, defect)

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(PTO: 5/2-5/10)]

Attachment: Bug 1540990 - Get rid of unnecessary nsIPresShell.h inclusions

A lot of files include nsIPresShell.h even though currently they don't
need it. This patch removes the unnecessary inclusions.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3b94c20ba873
Get rid of unnecessary nsIPresShell.h inclusions r=emilio

Comment 3

[Razvan Maries]

Backed out for build bustages.

Push with failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&revision=3b94c20ba87374c8df1af15e903c38b0945975a0&selectedJob=237985844

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=237985844&repo=autoland&lineNumber=19821

Backout: https://hg.mozilla.org/integration/autoland/rev/d63733e7f42fc5da3b8896099b69bef27456ddbe

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(PTO: 5/2-5/10)]

Looks like that some cpp files don't include necessary nsIPresShellInlines.h directly. So, something was changed by other changes which made such files include it indirectly.

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(PTO: 5/2-5/10)]

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

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e875e0c671bf
Get rid of unnecessary nsIPresShell.h inclusions r=emilio

Comment 7

[Arthur Iakab [arthur_iakab]]

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