1362806 - Optimize NS_GetDefaultPort() for HTTP(S)

Status: RESOLVED FIXED

(Core :: Networking, defect)

Description

[(no longer active)]

See this perf profile:

-    0.00%  Web Content  libxul.so  [.] imgLoader::LoadImage                                                  ▒
   - imgLoader::LoadImage                                                                                     ▒
      - 2.10% mozilla::net::HttpChannelChild::AsyncOpen2                                                      ▒
         + 1.75% mozilla::net::HttpChannelChild::AsyncOpen                                                    ▒
         - 0.34% nsContentSecurityManager::doContentSecurityCheck                                             ◆
            + 0.17% NS_CheckContentLoadPolicy                                                                 ▒
            - 0.13% nsContentSecurityManager::CheckChannel                                                    ▒
                 NS_HasBeenCrossOrigin                                                                        ▒
                 mozilla::BasePrincipal::CheckMayLoad                                                         ▒
               - ContentPrincipal::MayLoadInternal                                                            ▒
                  - 0.08% NS_SecurityCompareURIs                                                              ▒
                     - NS_GetRealPort                                                                         ▒
                        - 0.05% NS_GetDefaultPort                                                             ▒
                             net_EnsureIOService                                                              ▒
                             mozilla::services::GetIOService                                                  ▒
                        + 0.04% mozilla::net::nsStandardURL::GetScheme                                        ▒
                    0.05% nsACString::Finalize                                                                ▒
              0.05% mozilla::net::LoadInfo::GetExternalContentPolicyType                                      ▒

NS_SecurityCompareURIs()'s cost is totally from NS_GetDefaultPort, which is all XPCOM overhead.  :-(

I have a simple patch that optimizes the HTTP/HTTPS case away, I propose we hardcode these two schemes, and stick to XPCOM purity for the rest?

Comment 1

[(no longer active)]

Attachment: Optimize NS_GetDefaultPort() for HTTP(S)

Comment 2

[Patrick McManus [:mcmanus]]

Comment on attachment 8865156 [details] [diff] [review]
Optimize NS_GetDefaultPort() for HTTP(S)

Review of attachment 8865156 [details] [diff] [review]:
-----------------------------------------------------------------

thanks - we'll take small easy wins.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/98d1dd932bc6
Optimize NS_GetDefaultPort() for HTTP(S); r=mcmanus

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/98d1dd932bc6