Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(633)

Unified Diff: third_party/WebKit/Source/core/timing/PerformanceTiming.cpp

Issue 2393013002: reflow comments in core/{clipboard,streams,testing,timing} (Closed)
Patch Set: comments (heh!) Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/core/timing/PerformanceTest.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/timing/PerformanceTiming.cpp
diff --git a/third_party/WebKit/Source/core/timing/PerformanceTiming.cpp b/third_party/WebKit/Source/core/timing/PerformanceTiming.cpp
index 11ad4d7d930277440c06f0a2f833d653ad8e880f..49344bec52b08385f2c423d61a666d7c95d399c4 100644
--- a/third_party/WebKit/Source/core/timing/PerformanceTiming.cpp
+++ b/third_party/WebKit/Source/core/timing/PerformanceTiming.cpp
@@ -126,8 +126,9 @@ unsigned long long PerformanceTiming::domainLookupStart() const {
if (!timing)
return fetchStart();
- // This will be zero when a DNS request is not performed.
- // Rather than exposing a special value that indicates no DNS, we "backfill" with fetchStart.
+ // This will be zero when a DNS request is not performed. Rather than
+ // exposing a special value that indicates no DNS, we "backfill" with
+ // fetchStart.
double dnsStart = timing->dnsStart();
if (dnsStart == 0.0)
return fetchStart();
@@ -140,8 +141,9 @@ unsigned long long PerformanceTiming::domainLookupEnd() const {
if (!timing)
return domainLookupStart();
- // This will be zero when a DNS request is not performed.
- // Rather than exposing a special value that indicates no DNS, we "backfill" with domainLookupStart.
+ // This will be zero when a DNS request is not performed. Rather than
+ // exposing a special value that indicates no DNS, we "backfill" with
+ // domainLookupStart.
double dnsEnd = timing->dnsEnd();
if (dnsEnd == 0.0)
return domainLookupStart();
@@ -158,14 +160,16 @@ unsigned long long PerformanceTiming::connectStart() const {
if (!timing)
return domainLookupEnd();
- // connectStart will be zero when a network request is not made.
- // Rather than exposing a special value that indicates no new connection, we "backfill" with domainLookupEnd.
+ // connectStart will be zero when a network request is not made. Rather than
+ // exposing a special value that indicates no new connection, we "backfill"
+ // with domainLookupEnd.
double connectStart = timing->connectStart();
if (connectStart == 0.0 || loader->response().connectionReused())
return domainLookupEnd();
- // ResourceLoadTiming's connect phase includes DNS, however Navigation Timing's
- // connect phase should not. So if there is DNS time, trim it from the start.
+ // ResourceLoadTiming's connect phase includes DNS, however Navigation
+ // Timing's connect phase should not. So if there is DNS time, trim it from
+ // the start.
if (timing->dnsEnd() > 0.0 && timing->dnsEnd() > connectStart)
connectStart = timing->dnsEnd();
@@ -181,8 +185,9 @@ unsigned long long PerformanceTiming::connectEnd() const {
if (!timing)
return connectStart();
- // connectEnd will be zero when a network request is not made.
- // Rather than exposing a special value that indicates no new connection, we "backfill" with connectStart.
+ // connectEnd will be zero when a network request is not made. Rather than
+ // exposing a special value that indicates no new connection, we "backfill"
+ // with connectStart.
double connectEnd = timing->connectEnd();
if (connectEnd == 0.0 || loader->response().connectionReused())
return connectStart();
« no previous file with comments | « third_party/WebKit/Source/core/timing/PerformanceTest.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698