Changeset 75120 in webkit


Ignore:
Timestamp:
Jan 5, 2011 5:22:46 PM (13 years ago)
Author:
simonjam@chromium.org
Message:

2011-01-05 James Simonsen <​simonjam@chromium.org>

Reviewed by Darin Fisher.

[Web Timing] requestStart and responseStart should be available even if the document is still loading
​https://bugs.webkit.org/show_bug.cgi?id=51368

  • http/tests/misc/webtiming-slow-load-expected.txt: Added.
  • http/tests/misc/webtiming-slow-load.php: Added.

2011-01-05 James Simonsen <​simonjam@chromium.org>

Reviewed by Darin Fisher.

[Web Timing] requestStart and responseStart should be available even if the document is still loading
​https://bugs.webkit.org/show_bug.cgi?id=51368

Test: http/tests/misc/webtiming-slow-load.php

  • page/PerformanceTiming.cpp: (WebCore::getPossiblySkewedTimeInKnownRange): Handle case where response is not yet complete. (WebCore::PerformanceTiming::resourceLoadTimeRelativeToAbsolute): Use full upper bound.
Location:
trunk
Files:
2 added
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r75116 r75120  
     12011-01-05  James Simonsen  <simonjam@chromium.org>
     2
     3        Reviewed by Darin Fisher.
     4
     5        [Web Timing] requestStart and responseStart should be available even if the document is still loading
     6        https://bugs.webkit.org/show_bug.cgi?id=51368
     7
     8        * http/tests/misc/webtiming-slow-load-expected.txt: Added.
     9        * http/tests/misc/webtiming-slow-load.php: Added.
     10
    1112011-01-05  Kent Tamura  <tkent@chromium.org>
    212

  • trunk/LayoutTests/platform/gtk/Skipped

    r74796 r75120  
    53145314fast/dom/webtiming-document-open.html
    53155315http/tests/misc/webtiming-one-redirect.php
     5316http/tests/misc/webtiming-slow-load.php
    53165317http/tests/misc/webtiming-ssl.php
    53175318http/tests/misc/webtiming-two-redirects.php

  • trunk/LayoutTests/platform/mac/Skipped

    r74888 r75120  
    217217fast/dom/webtiming-navigate-within-document.html
    218218http/tests/misc/webtiming-one-redirect.php
     219http/tests/misc/webtiming-slow-load.php
    219220http/tests/misc/webtiming-ssl.php
    220221http/tests/misc/webtiming-two-redirects.php

  • trunk/LayoutTests/platform/win/Skipped

    r74840 r75120  
    10441044fast/dom/webtiming-navigate-within-document.html
    10451045http/tests/misc/webtiming-one-redirect.php
     1046http/tests/misc/webtiming-slow-load.php
    10461047http/tests/misc/webtiming-ssl.php
    10471048http/tests/misc/webtiming-two-redirects.php

  • trunk/WebCore/ChangeLog

    r75118 r75120  
     12011-01-05  James Simonsen  <simonjam@chromium.org>
     2
     3        Reviewed by Darin Fisher.
     4
     5        [Web Timing] requestStart and responseStart should be available even if the document is still loading
     6        https://bugs.webkit.org/show_bug.cgi?id=51368
     7
     8        Test: http/tests/misc/webtiming-slow-load.php
     9
     10        * page/PerformanceTiming.cpp:
     11        (WebCore::getPossiblySkewedTimeInKnownRange): Handle case where response is not yet complete.
     12        (WebCore::PerformanceTiming::resourceLoadTimeRelativeToAbsolute): Use full upper bound.
     13
    1142011-01-05  Chris Rogers  <crogers@google.com>
    215

  • trunk/WebCore/page/PerformanceTiming.cpp

    r74315 r75120  
    4040#include "ResourceLoadTiming.h"
    4141#include "ResourceResponse.h"
     42#include <wtf/CurrentTime.h>
    4243
    4344namespace WebCore {
    … …  
    6061        return lowerBound;
    6162
     63    if (upperBound <= 0.0)
     64        upperBound = currentTime();
     65
    6266    if (skewedTime >= upperBound)
    6367        return upperBound;
    … …  
    379383    // Since ResourceLoadTimings came from the network platform layer, we must
    380384    // check them for skew because they may be from another thread/process.
    381     double baseTime = getPossiblySkewedTimeInKnownRange(resourceTiming->requestTime, documentTiming->fetchStart, documentTiming->responseEnd - (resourceTiming->receiveHeadersEnd / 1000.0));
     385    double baseTime = getPossiblySkewedTimeInKnownRange(resourceTiming->requestTime, documentTiming->fetchStart, documentTiming->responseEnd);
    382386    return toIntegerMilliseconds(baseTime) + relativeSeconds;
    383387}
Note: See TracChangeset for help on using the changeset viewer.