Closed
Bug 1015644
Opened 10 years ago
Closed 10 years ago
Clean up ActiveElementManager
Categories
(Core :: Graphics, defect)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla33
People
(Reporter: dzbarsky, Assigned: dzbarsky)
Details
Attachments
(1 file)
5.71 KB,
patch
|
botond
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
Assignee: nobody → dzbarsky
Attachment #8428301 -
Flags: review?(botond)
Comment 2•10 years ago
|
||
Is there a reason for these changes?
Comment 3•10 years ago
|
||
Comment on attachment 8428301 [details] [diff] [review] Patch Review of attachment 8428301 [details] [diff] [review]: ----------------------------------------------------------------- ::: gfx/layers/apz/util/ActiveElementManager.h @@ +13,5 @@ > class CancelableTask; > > namespace mozilla { > +namespace dom { > +class Element; Don't you need to forward-declare EventTarget as well?
Comment 4•10 years ago
|
||
Comment on attachment 8428301 [details] [diff] [review] Patch Review of attachment 8428301 [details] [diff] [review]: ----------------------------------------------------------------- r=me with the missing include and forward-declaration fixed. (In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #2) > Is there a reason for these changes? Looks like part of a general transitioning from an old set of interfaces (nsIXXX) to a new set (dom::XXX). Fine with me if it doesn't regress anything :) ::: gfx/layers/apz/util/ActiveElementManager.cpp @@ +11,3 @@ > #include "base/message_loop.h" > #include "base/task.h" > +#include "mozilla/dom/Element.h" Likewise, shouldn't you also include "mozilla/dom/EventTarget.h"?
Attachment #8428301 -
Flags: review?(botond) → review+
Comment 5•10 years ago
|
||
(In reply to Botond Ballo [:botond] from comment #4) > Looks like part of a general transitioning from an old set of interfaces > (nsIXXX) to a new set (dom::XXX). Fine with me if it doesn't regress > anything :) In that case this bug should block a meta-bug tracking the overall work so that people can find any relevant context.
Assignee | ||
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8222bfcd04fb
Comment 7•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8222bfcd04fb
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Comment 8•10 years ago
|
||
Follow-up to fix bustage when AEM_LOG is enabled: https://hg.mozilla.org/integration/mozilla-inbound/rev/52f0f39c8889
Comment 9•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/52f0f39c8889
You need to log in
before you can comment on or make changes to this bug.
Description
•