RESOLVED FIXED 53402
Add sanity check to help diagnose bug 52819 
https://bugs.webkit.org/show_bug.cgi?id=53402
Summary Add sanity check to help diagnose bug 52819
Charles Reis
Reported 2011-01-30 12:58:01 PST
We're seeing many crash reports for bug 52819 in HistoryController::itemsAreClones. The crashes happen when fromItem in recursiveSetProvisionalItem is null, which happens if the parent HistoryItem does not have a child item with the target frame name. The crash reports suggest that fromItem has 0 children, which should be impossible. (The previous call to itemsAreClones should verify that fromItem and item have the same number of children, and item clearly has children.) On the flip side, it's possible for the crash to happen if fromItem has ended up with two children of the same name, though the crash reports don't seen to be consistent with that. This bug is for adding a sanity check to help diagnose which (if either) of these is happening in practice, since we don't have any information about how to reproduce the crash yet.
Attachments
Patch (1.66 KB, patch)
2011-01-30 13:05 PST, Charles Reis 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Charles Reis
Comment 1 2011-01-30 13:05:12 PST
Created attachment 80599 [details] Patch
WebKit Commit Bot
Comment 2 2011-01-31 20:04:29 PST
Comment on attachment 80599 [details] Patch Clearing flags on attachment: 80599 Committed r77210: <http://trac.webkit.org/changeset/77210>
WebKit Commit Bot
Comment 3 2011-01-31 20:04:33 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.