Simplify protocal name and port comparison

Source/platform/weborigin/KnownPorts.cpp

 /*
  * Copyright (C) 2004, 2007, 2008, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2012 Research In Motion Limited. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 22 matching lines @@
 #include "wtf/Threading.h"
 #include "wtf/text/StringHash.h"
 
 namespace blink {
 
 bool isDefaultPortForProtocol(unsigned short port, const String& protocol)
 {
     if (protocol.isEmpty())
         return false;
 
-    typedef HashMap<String, unsigned, CaseFoldingHash> DefaultPortsMap;
-    AtomicallyInitializedStaticReference(DefaultPortsMap, defaultPorts, new DefaultPortsMap());
-    if (defaultPorts.isEmpty()) {
-        defaultPorts.set("http", 80);
-        defaultPorts.set("https", 443);
-        defaultPorts.set("ftp", 21);
-        defaultPorts.set("ftps", 990);
-    }
-    return defaultPorts.get(protocol) == port;
+    if (protocol == "http")
+        return port == 80;
+    if (protocol == "https")
+        return port == 443;
+    if (protocol == "ftp")
+        return port == 21;
+    if (protocol == "ftps")
+        return port == 990;

[kinuko, 2015/02/03 03:52:32]

Switching on port number, and then comparing protocol might be slightly faster?

[Ian Wen, 2015/02/03 19:08:54]

Indeed. Done.

+
+    return false;
 }
 
 bool portAllowed(const KURL& url)
 {
     unsigned short port = url.port();
 
     // Since most URLs don't have a port, return early for the "no port" case.
     if (!port)
         return true;
 
@@ skipping 87 matching lines @@
         return true;
 
     // Allow any port number in a file URL, since the port number is ignored.
     if (url.protocolIs("file"))
         return true;
 
     return false;
 }
 
 }