[Webkit-unassigned] [Bug 105330] REGRESSION(r137607): resource load client callbacks are not called for the main resource when loading HTML string

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 19 11:30:43 PST 2012


https://bugs.webkit.org/show_bug.cgi?id=105330





--- Comment #7 from Brady Eidson <beidson at apple.com>  2012-12-19 11:32:58 PST ---
(In reply to comment #6)
> (From update of attachment 180188 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=180188&action=review
> 
> > Source/WebCore/loader/MainResourceLoader.cpp:697
> >  unsigned long MainResourceLoader::identifier() const
> >  {
> > +    if (m_identifier)
> > +        return m_identifier;
> >      if (ResourceLoader* resourceLoader = loader())
> >          return resourceLoader->identifier();
> 
> This breaks the "setIdentifier" API on ResourceLoader that WebKit2 relies on.

Now that we've made MainResourceLoader no longer be a ResourceLoader, we're going to start getting in to trouble as we diverge the two.  Having two copies of the identifier seems like a great first step towards breaking things.

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


More information about the webkit-unassigned mailing list