Commit 49d4609c authored by darin@apple.com's avatar darin@apple.com

2008-07-08 Darin Adler <darin@apple.com>

        Reviewed by Mitz.

        - fix <rdar://problem/6043731> REGRESSION (3-4): RedEnvelope.com
          looks wrong due to OpenCube QuickMenu appVersion check

        * page/Navigator.cpp:
        (WebCore::shouldHideFourDot): Add dqm_loader.js as another filename.



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@35062 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 9cf92a4f
2008-07-08 Darin Adler <darin@apple.com>
Reviewed by Mitz.
- fix <rdar://problem/6043731> REGRESSION (3-4): RedEnvelope.com
looks wrong due to OpenCube QuickMenu appVersion check
* page/Navigator.cpp:
(WebCore::shouldHideFourDot): Add dqm_loader.js as another filename.
2008-07-08 Dan Bernstein <mitz@apple.com>
Reviewed by John Sullivan.
......@@ -102,17 +102,19 @@ String Navigator::appName() const
// If this function returns true, we need to hide the substring "4." that would otherwise
// appear in the appVersion string. This is to avoid problems with old versions of a
// library called OpenCube QuickMenus, which as of this writing is still being used on
// library called OpenCube QuickMenu, which as of this writing is still being used on
// sites such as nwa.com -- the library thinks Safari is Netscape 4 if we don't do this!
static bool shouldHideFourDot(Frame* frame)
{
const String* sourceURL = frame->script()->sourceURL();
if (!sourceURL)
return false;
if (!(sourceURL->endsWith("/dqm_script.js") || sourceURL->endsWith("/dqm_loader.js")))
return false;
Settings* settings = frame->settings();
if (!settings)
return false;
return sourceURL->endsWith("/dqm_script.js") && settings->needsSiteSpecificQuirks();
return settings->needsSiteSpecificQuirks();
}
String Navigator::appVersion() const
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment