Closed Bug 878443 Opened 11 years ago Closed 11 years ago

Let the inspector actor expire actors for nodes that are no longer connected to the tree

Categories

(DevTools :: Inspector, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 24

People

(Reporter: dcamp, Assigned: dcamp)

References

Details

Attachments

(1 file)

      No description provided.
Depends on: 878614
Attached patch v1Splinter Review
That ended up being surprisingly easy with all the rest of the groundwork done.
Attachment #757210 - Flags: review?(jwalker)
Blocks: 881120
Comment on attachment 757210 [details] [diff] [review]
v1

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

::: toolkit/devtools/server/actors/inspector.js
@@ +1231,5 @@
>      impl: "_querySelector"
>    }),
>  
> +  getMutations: protocol.custom(function(options={}) {
> +    dump("OPTIONS: " + JSON.stringify(options) + "\n");

dump should be removed at some stage. Happy for it not to be in this bug though.
Attachment #757210 - Flags: review?(jwalker) → review+
https://hg.mozilla.org/integration/fx-team/rev/88cd07a112ae
Assignee: nobody → dcamp
Whiteboard: [fixed-in-fx-team]
No longer depends on: 878442
https://hg.mozilla.org/mozilla-central/rev/88cd07a112ae
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 24
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: