Remove OpenCube QuickMenu quirks from navigator.appVersion
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Mar 2019 06:07:14 +0000 (06:07 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Mar 2019 06:07:14 +0000 (06:07 +0000)
https://bugs.webkit.org/show_bug.cgi?id=195600

Reviewed by Simon Fraser.

Remove the site specific quirk added in r35050 for OpenCube QuickMenu library for nwa.com

Blink removed this code back in 2013. The fact Chrome has been shipping successfully without
this quirk for six years is a good evidence that it's no longer needed for the Web compatibility.

* page/Navigator.cpp:
(WebCore::Navigator::appVersion const):
(WebCore::shouldHideFourDot): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@242779 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/page/Navigator.cpp

index 127c605..ce9bc4f 100644 (file)
@@ -1,3 +1,19 @@
+2019-03-11  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Remove OpenCube QuickMenu quirks from navigator.appVersion
+        https://bugs.webkit.org/show_bug.cgi?id=195600
+
+        Reviewed by Simon Fraser.
+
+        Remove the site specific quirk added in r35050 for OpenCube QuickMenu library for nwa.com
+
+        Blink removed this code back in 2013. The fact Chrome has been shipping successfully without
+        this quirk for six years is a good evidence that it's no longer needed for the Web compatibility.
+
+        * page/Navigator.cpp:
+        (WebCore::Navigator::appVersion const):
+        (WebCore::shouldHideFourDot): Deleted.
+
 2019-03-11  Ryan Haddad  <ryanhaddad@apple.com>
 
         Unreviewed, rolling out r242763.
index a591ae9..f3d01d6 100644 (file)
@@ -56,20 +56,6 @@ Navigator::Navigator(ScriptExecutionContext* context, DOMWindow& window)
 
 Navigator::~Navigator() = default;
 
-// 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 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)
-{
-    auto* sourceURL = frame.script().sourceURL();
-    if (!sourceURL)
-        return false;
-    if (!(sourceURL->endsWith("/dqm_script.js") || sourceURL->endsWith("/dqm_loader.js") || sourceURL->endsWith("/tdqm_loader.js")))
-        return false;
-    return frame.settings().needsSiteSpecificQuirks();
-}
-
 String Navigator::appVersion() const
 {
     auto* frame = this->frame();
@@ -77,10 +63,7 @@ String Navigator::appVersion() const
         return String();
     if (RuntimeEnabledFeatures::sharedFeatures().webAPIStatisticsEnabled())
         ResourceLoadObserver::shared().logNavigatorAPIAccessed(*frame->document(), ResourceLoadStatistics::NavigatorAPI::AppVersion);
-    String appVersion = NavigatorBase::appVersion();
-    if (shouldHideFourDot(*frame))
-        appVersion.replace("4.", "4_");
-    return appVersion;
+    return NavigatorBase::appVersion();
 }
 
 const String& Navigator::userAgent() const