[Webkit-unassigned] [Bug 60527] Return empty Favicon URL instead of default one when the frame isn't top level one.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon May 9 21:26:09 PDT 2011
https://bugs.webkit.org/show_bug.cgi?id=60527
--- Comment #3 from michaelbai at chromium.org 2011-05-09 21:26:10 PST ---
If you checked patch https://bugs.webkit.org/show_bug.cgi?id=59143, you will find the 59143 patch changed the orignal logic. This patch will correct it.
(In reply to comment #2)
> (From update of attachment 92907 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=92907&action=review
>
> This patch is not well explained. Even if the code change is correct, there is no rationale in the change log for the code changes.
>
> > Source/WebCore/dom/Document.cpp:4437
> > - if (!iconURL(iconType).m_iconURL.isEmpty())
> > + if (iconURL(iconType).m_iconURL.isEmpty())
>
> What’s the reasoning behind reversing the logic here?
>
> > Source/WebCore/loader/FrameLoader.cpp:480
> > + // If this isn't a top level frame, return
> > + if (m_frame->tree() && m_frame->tree()->parent())
> > + return iconURLs;
>
> This is the only change that matches the change log comment. But even this is not explained.
>
> > Source/WebCore/loader/FrameLoader.cpp:-503
> > - // If this isn't a top level frame, return
> > - if (m_frame->tree() && m_frame->tree()->parent())
> > - return false;
>
> Why is this a good change?
--
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