diff options
author | janekptacijarabaci <janekptacijarabaci@seznam.cz> | 2018-03-17 13:35:25 +0100 |
---|---|---|
committer | janekptacijarabaci <janekptacijarabaci@seznam.cz> | 2018-03-17 13:35:25 +0100 |
commit | a10b23932cd567324b7e1071f3ddc4c46a8270d8 (patch) | |
tree | 5c299765112a1c41800b08a93f9cc875764fdf4c /dom/performance/PerformanceTiming.cpp | |
parent | a24511ca8e19afe4d19008ba903e1edab0af3223 (diff) | |
download | UXP-a10b23932cd567324b7e1071f3ddc4c46a8270d8.tar UXP-a10b23932cd567324b7e1071f3ddc4c46a8270d8.tar.gz UXP-a10b23932cd567324b7e1071f3ddc4c46a8270d8.tar.lz UXP-a10b23932cd567324b7e1071f3ddc4c46a8270d8.tar.xz UXP-a10b23932cd567324b7e1071f3ddc4c46a8270d8.zip |
Set "secureConnectionStart" to 0 for pages with HTTP scheme
Issue #67
Diffstat (limited to 'dom/performance/PerformanceTiming.cpp')
-rwxr-xr-x | dom/performance/PerformanceTiming.cpp | 27 |
1 files changed, 24 insertions, 3 deletions
diff --git a/dom/performance/PerformanceTiming.cpp b/dom/performance/PerformanceTiming.cpp index 5f771f0aa..e2f76a21f 100755 --- a/dom/performance/PerformanceTiming.cpp +++ b/dom/performance/PerformanceTiming.cpp @@ -46,6 +46,23 @@ PerformanceTiming::PerformanceTiming(Performance* aPerformance, mReportCrossOriginRedirect = mTimingAllowed && redirectsPassCheck; } + mSecureConnection = false; + nsCOMPtr<nsIURI> uri; + if (aHttpChannel) { + aHttpChannel->GetURI(getter_AddRefs(uri)); + } else { + nsCOMPtr<nsIHttpChannel> httpChannel = do_QueryInterface(aChannel); + if (httpChannel) { + httpChannel->GetURI(getter_AddRefs(uri)); + } + } + + if (uri) { + nsresult rv = uri->SchemeIs("https", &mSecureConnection); + if (NS_FAILED(rv)) { + mSecureConnection = false; + } + } InitializeTimingInfo(aChannel); } @@ -89,7 +106,8 @@ PerformanceTiming::InitializeTimingInfo(nsITimedChannel* aChannel) mConnectStart = mAsyncOpen; } - if (!mSecureConnectionStart.IsNull() && mSecureConnectionStart < mAsyncOpen) { + if (mSecureConnection && !mSecureConnectionStart.IsNull() && + mSecureConnectionStart < mAsyncOpen) { mSecureConnectionStart = mAsyncOpen; } @@ -307,8 +325,11 @@ PerformanceTiming::SecureConnectionStartHighRes() if (!nsContentUtils::IsPerformanceTimingEnabled() || !IsInitialized()) { return mZeroTime; } - return mSecureConnectionStart.IsNull() ? mZeroTime - : TimerClamping::ReduceMsTimeValue(TimeStampToDOMHighRes(mSecureConnectionStart)); + return !mSecureConnection + ? 0 // We use 0 here, because mZeroTime is sometimes set to the navigation + // start time. + : (mSecureConnectionStart.IsNull() ? mZeroTime + : TimerClamping::ReduceMsTimeValue(TimeStampToDOMHighRes(mSecureConnectionStart))); } DOMTimeMilliSec |