Bug 55769 - Web Inspector: [Text editor] DOMNodeRemoved events are missing
Summary: Web Inspector: [Text editor] DOMNodeRemoved events are missing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 53588
  Show dependency tree / graph
 
Reported: 2011-03-04 05:39 PST by Andrey Adaikin
Modified: 2011-03-04 06:20 PST (History)
11 users (show)

See Also:

Attachments
Patch (7.30 KB, patch)
2011-03-04 05:42 PST, Andrey Adaikin 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Adaikin 2011-03-04 05:39:40 PST 
In WebKit the DOMNodeRemoved event is fired AFTER the node is removed, thus it should be attached to all DOM nodes that we want to track. Instead, we attach the DOMNodeRemoved listeners only on the line rows, and use DOMSubtreeModified to track node removals inside the line rows. 

For more info see: https://bugs.webkit.org/show_bug.cgi?id=55666
Comment 1 Andrey Adaikin 2011-03-04 05:42:21 PST 
Created attachment 84732 [details]
Patch
Comment 2 Pavel Podivilov 2011-03-04 06:20:13 PST 
Comment on attachment 84732 [details]
Patch

Clearing flags on attachment: 84732

Committed r80346: <http://trac.webkit.org/changeset/80346>
Comment 3 Pavel Podivilov 2011-03-04 06:20:25 PST 
All reviewed patches have been landed.  Closing bug.