Bug 124639

Summary: Remove bogus assertions in updateNameForTreeScope and updateNameForDocument
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: DOMAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, benjamin, cmarcelo, commit-queue, darin, esprehn+autocc, kangil.han, koivisto, zalan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Removes the assertions none

Ryosuke Niwa
Reported 2013-11-19 23:20:12 PST
updateNameForTreeScope and updateNameForDocument are called in insertedInto and removedFrom. It doesn't make any sense to assert that the removed node is in a document or tree scope when they're called in removedFrom.
Attachments
Removes the assertions (1.54 KB, patch)
2013-11-19 23:21 PST, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2013-11-19 23:21:53 PST
Created attachment 217390 [details] Removes the assertions
WebKit Commit Bot
Comment 2 2013-11-20 09:55:22 PST
Comment on attachment 217390 [details] Removes the assertions Clearing flags on attachment: 217390 Committed r159566: <http://trac.webkit.org/changeset/159566>
WebKit Commit Bot
Comment 3 2013-11-20 09:55:24 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.