Fix some content includes

RESOLVED FIXED in mozilla22

Status

()

defect
RESOLVED FIXED
7 years ago
4 months ago

People

(Reporter: dzbarsky, Assigned: dzbarsky)

Tracking

unspecified
mozilla22
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 2 obsolete attachments)

Posted patch Part 1: nsIDocument (obsolete) — Splinter Review
No description provided.
Attachment #720246 - Flags: review?(Ms2ger)
Posted patch Part 2: nsIContent (obsolete) — Splinter Review
Attachment #720253 - Flags: review?(Ms2ger)
You're conflicting with bug 833009 already :/
That bug is marked fixed.  What is the conflict?
Comment on attachment 720246 [details] [diff] [review]
Part 1: nsIDocument

I promise speedy reviews if you get these updated :)
Attachment #720246 - Flags: review?(Ms2ger)
Attachment #720253 - Flags: review?(Ms2ger)
Attachment #720246 - Attachment is obsolete: true
Attachment #727850 - Flags: review?(Ms2ger)
Attachment #720253 - Attachment is obsolete: true
Attachment #727851 - Flags: review?(Ms2ger)
Comment on attachment 727850 [details] [diff] [review]
Part 1: nsIDocument

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

r=me

::: content/events/src/nsEventStateManager.h
@@ -21,5 @@
>  #include "nsCycleCollectionParticipant.h"
>  #include "nsIMarkupDocumentViewer.h"
>  #include "nsIScrollableFrame.h"
>  #include "nsFocusManager.h"
> -#include "nsIDocument.h"

Someone wanted to be really sure, it seems.
Attachment #727850 - Flags: review?(Ms2ger) → review+
Attachment #727851 - Flags: review?(Ms2ger) → review+
https://hg.mozilla.org/mozilla-central/rev/55ddaf3f474f
https://hg.mozilla.org/mozilla-central/rev/c1e0d7e7b164
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Depends on: 854115
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.