[Webkit-unassigned] [Bug 96126] Appcache fallback URL match should use the longest candidate

[bugzilla-daemon at webkit.org]

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





--- Comment #2 from Yong Li <yoli at rim.com>  2012-09-07 10:29:36 PST ---
(From update of attachment 162801)
View in context: https://bugs.webkit.org/attachment.cgi?id=162801&action=review

basically lgtm. but we should get some AppCache experts involved.

> Source/WebCore/loader/appcache/ApplicationCache.cpp:42
> +    bool operator()(const std::pair<KURL, KURL>& lhs, const std::pair<KURL, KURL>& rhs)

static bool operator() is better? If not static, it should be const.

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