[Webkit-unassigned] [Bug 52988] REGRESSION (r75555): Safari RSS sidebar jiggles when scrolling

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Apr 14 00:50:57 PDT 2011


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





--- Comment #27 from Maciej Stachowiak <mjs at apple.com>  2011-04-14 00:50:57 PST ---
(From update of attachment 89544)
View in context: https://bugs.webkit.org/attachment.cgi?id=89544&action=review

> Source/WebCore/dom/EventQueue.cpp:43
> +    return applicationIsSafari() && (document->url().protocolIs("feed") || document->url().protocolIs("feeds"));

Maybe this would make more sense as a preference setting, then it could be customized per view. That would be a cleaner way to do the check, I think, and it would avoid the problem in applicationIsSafari cited below.

> Source/WebCore/platform/RuntimeApplicationChecks.cpp:58
> +    static bool isSafari = mainBundleIsEqualTo("com.apple.Safari") || mainBundleIsEqualTo("com.apple.WebProcess");

This should have a FIXME, because long-term, WebProcess will not be used only by Safari. Or perhaps there is a better way to check specifically for Safari's WebProcess, in which case maybe it would be wise to future-proof this now.

-- 
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