[Webkit-unassigned] [Bug 12251] REGRESSION (r18822-r18823): Assertion failure opening document with non-existent resources (dom/xhtml/level2/html/HTMLIFrameElement11.xhtml)

Sun Jan 14 06:03:11 PST 2007

http://bugs.webkit.org/show_bug.cgi?id=12251

darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #12424|review?                     |review-
               Flag|                            |

------- Comment #10 from darin at apple.com  2007-01-14 06:03 PDT -------
(From update of attachment 12424)
We definitely need to fix this! Can't have layout tests failing.

But HashMap remove already has the behavior of removing it only if it's there,
so no need for the contains call.

And if the object that comes back from _objectForIdentifier is nil, then the
caller needs to not send the notification.

I think we should land a fixed-up version of this now. The real problem is that
the loader is sending multiple "done loading" calls to the client for the same
resource -- it can be fixed entirely on the WebCore side. But it's a complex
job that we should do later. This approach is correct for now.

-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.