Changeset 231169 in webkit


Ignore:
Timestamp:
Apr 30, 2018 2:06:05 PM (6 years ago)
Author:
Dewei Zhu
Message:

MeasurementSet._constructUrl should construct absolute url.
https://bugs.webkit.org/show_bug.cgi?id=185137

Reviewed by Ryosuke Niwa.

MeasurementSet._constructUrl should not construct an url with relative path.

  • public/v3/models/measurement-set.js: Should use '/api/measurement-set...' and '/data/measurement-set-...'

for uncached and cached fetching.

  • unit-tests/measurement-set-tests.js: Updated unit tests.

(const.set MeasurementSet):

Location:
trunk/Websites/perf.webkit.org
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/Websites/perf.webkit.org/ChangeLog

    r231087 r231169  
     12018-04-30  Dewei Zhu  <dewei_zhu@apple.com>
     2
     3        MeasurementSet._constructUrl should construct absolute url.
     4        https://bugs.webkit.org/show_bug.cgi?id=185137
     5
     6        Reviewed by Ryosuke Niwa.
     7
     8        MeasurementSet._constructUrl should not construct an url with relative path.
     9
     10        * public/v3/models/measurement-set.js: Should use '/api/measurement-set...' and '/data/measurement-set-...'
     11        for uncached and cached fetching.
     12        * unit-tests/measurement-set-tests.js: Updated unit tests.
     13        (const.set MeasurementSet):
     14
    1152018-04-26  Dewei Zhu  <dewei_zhu@apple.com>
    216
  • trunk/Websites/perf.webkit.org/public/v3/models/measurement-set.js

    r230295 r231169  
    9595    {
    9696        if (!useCache) {
    97             return `../api/measurement-set?platform=${this._platformId}&metric=${this._metricId}`;
     97            return `/api/measurement-set?platform=${this._platformId}&metric=${this._metricId}`;
    9898        }
    9999        var url;
    100         url = `../data/measurement-set-${this._platformId}-${this._metricId}`;
     100        url = `/data/measurement-set-${this._platformId}-${this._metricId}`;
    101101        if (clusterEndTime)
    102102            url += '-' + +clusterEndTime;
  • trunk/Websites/perf.webkit.org/unit-tests/measurement-set-tests.js

    r230719 r231169  
    4242            });
    4343            assert.equal(requests.length, 1);
    44             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     44            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    4545
    4646            requests[0].resolve({
     
    6868            set.fetchBetween(1000, 2000, () => callCount++);
    6969            assert.equal(requests.length, 1);
    70             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     70            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    7171            assert.equal(callCount, 0);
    7272        });
     
    7878            assert.equal(requests.length, 1);
    7979            assert.equal(callCount, 0);
    80             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     80            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    8181            set.fetchBetween(2000, 3000, () => callCount++);
    8282            assert.equal(requests.length, 1);
     
    8989            const promise = set.fetchBetween(2000, 3000, () => callCount++);
    9090            assert.equal(requests.length, 1);
    91             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     91            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    9292
    9393            requests[0].resolve({
     
    113113            const promise = set.fetchBetween(1000, 3000, () => callCount++);
    114114            assert.equal(requests.length, 1);
    115             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     115            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    116116
    117117            requests[0].resolve({
     
    129129                assert.equal(callCount, 1);
    130130                assert.equal(requests.length, 2);
    131                 assert.equal(requests[1].url, '../data/measurement-set-1-1-2000.json');
     131                assert.equal(requests[1].url, '/data/measurement-set-1-1-2000.json');
    132132            });
    133133        });
     
    138138            set.fetchBetween(2000, 3000, () => callCountForWaitingCallback++);
    139139            assert.equal(requests.length, 1);
    140             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     140            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    141141
    142142            requests[0].resolve({
     
    154154            return waitForMeasurementSet().then(() => {
    155155                assert.equal(requests.length, 2);
    156                 assert.equal(requests[1].url, '../data/measurement-set-1-1-3000.json');
     156                assert.equal(requests[1].url, '/data/measurement-set-1-1-3000.json');
    157157
    158158                set.fetchBetween(0, 7000, () => callCount++);
     
    163163                assert.equal(callCount, 1);
    164164                assert.equal(requests.length, 4);
    165                 assert.equal(requests[2].url, '../data/measurement-set-1-1-2000.json');
    166                 assert.equal(requests[3].url, '../data/measurement-set-1-1-4000.json');
     165                assert.equal(requests[2].url, '/data/measurement-set-1-1-2000.json');
     166                assert.equal(requests[3].url, '/data/measurement-set-1-1-4000.json');
    167167            });
    168168        });
     
    173173            const promise = set.fetchBetween(2707, 4207, () => callCount++);
    174174            assert.equal(requests.length, 1);
    175             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     175            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    176176
    177177            requests[0].resolve({
     
    188188            return waitForMeasurementSet().then(() => {
    189189                assert.equal(requests.length, 3);
    190                 assert.equal(requests[1].url, '../data/measurement-set-1-1-3000.json');
    191                 assert.equal(requests[2].url, '../data/measurement-set-1-1-4000.json');
     190                assert.equal(requests[1].url, '/data/measurement-set-1-1-3000.json');
     191                assert.equal(requests[2].url, '/data/measurement-set-1-1-4000.json');
    192192                assert.equal(callCount, 1); // 4000-4207
    193193            });
     
    199199            set.fetchBetween(3200, 3700, () => callCountForWaitingCallback++);
    200200            assert.equal(requests.length, 1);
    201             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     201            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    202202
    203203            requests[0].resolve({
     
    215215            return waitForMeasurementSet().then(() => {
    216216                assert.equal(requests.length, 2);
    217                 assert.equal(requests[1].url, '../data/measurement-set-1-1-4000.json');
     217                assert.equal(requests[1].url, '/data/measurement-set-1-1-4000.json');
    218218                set.fetchBetween(1207, 1293, () => callCount++);
    219219                return waitForMeasurementSet();
     
    222222                assert.equal(callCount, 0);
    223223                assert.equal(requests.length, 3);
    224                 assert.equal(requests[2].url, '../data/measurement-set-1-1-2000.json');
     224                assert.equal(requests[2].url, '/data/measurement-set-1-1-2000.json');
    225225                set.fetchBetween(1964, 3401, () => callCount++);
    226226                return waitForMeasurementSet();
     
    229229                assert.equal(callCount, 0);
    230230                assert.equal(requests.length, 4);
    231                 assert.equal(requests[3].url, '../data/measurement-set-1-1-3000.json');
     231                assert.equal(requests[3].url, '/data/measurement-set-1-1-3000.json');
    232232            });
    233233        });
     
    238238            set.fetchBetween(0, 3000, () => callCount++);
    239239            assert.equal(requests.length, 1);
    240             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     240            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    241241
    242242            requests[0].resolve({
     
    263263            set.fetchBetween(1000, 3000, () => callCount++).catch(() => rejected = true);
    264264            assert.equal(requests.length, 1);
    265             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     265            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    266266
    267267            requests[0].reject(500);
     
    279279            set.fetchBetween(1000, 2000, () => callCount++);
    280280            assert.equal(requests.length, 1);
    281             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     281            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    282282
    283283            requests[0].resolve({
     
    295295                assert.equal(callCount, 0);
    296296                assert.equal(requests.length, 2);
    297                 assert.equal(requests[1].url, '../api/measurement-set?platform=1&metric=1');
     297                assert.equal(requests[1].url, '/api/measurement-set?platform=1&metric=1');
    298298            });
    299299        });
     
    304304            set.fetchBetween(1000, 2000, () => callCount++);
    305305            assert.equal(requests.length, 1);
    306             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     306            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    307307
    308308            requests[0].reject(404);
     
    311311                assert.equal(callCount, 0);
    312312                assert.equal(requests.length, 2);
    313                 assert.equal(requests[1].url, '../api/measurement-set?platform=1&metric=1');
     313                assert.equal(requests[1].url, '/api/measurement-set?platform=1&metric=1');
    314314            });
    315315        });
     
    320320            set.fetchBetween(1000, 3000, () => callCount++);
    321321            assert.equal(requests.length, 1);
    322             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     322            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    323323
    324324            requests[0].resolve({
     
    339339                assert.equal(set._sortedClusters.length, 1);
    340340                assert.equal(requests.length, 2);
    341                 assert.equal(requests[1].url, '../data/measurement-set-1-1-2000.json');
     341                assert.equal(requests[1].url, '/data/measurement-set-1-1-2000.json');
    342342
    343343                requests[1].resolve({
     
    360360                assert.equal(set._sortedClusters.length, 2);
    361361                assert.equal(requests.length, 3);
    362                 assert.equal(requests[2].url, '../api/measurement-set?platform=1&metric=1');
     362                assert.equal(requests[2].url, '/api/measurement-set?platform=1&metric=1');
    363363
    364364                requests[2].resolve({
     
    379379                assert.equal(set._sortedClusters.length, 2);
    380380                assert.equal(requests.length, 4);
    381                 assert.equal(requests[3].url, '../data/measurement-set-1-1-2000.json');
     381                assert.equal(requests[3].url, '/data/measurement-set-1-1-2000.json');
    382382
    383383                requests[3].resolve({
     
    406406            set.fetchBetween(2000, 3000, () => callCount++);
    407407            assert.equal(requests.length, 1);
    408             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     408            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    409409
    410410            let alternativeCallCount = 0;
     
    434434            set.fetchBetween(3200, 3700, () => callCountFor4000++);
    435435            assert.equal(requests.length, 1);
    436             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     436            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    437437
    438438            requests[0].resolve({
     
    453453                assert.equal(callCountFor4000, 0);
    454454                assert.equal(requests.length, 2);
    455                 assert.equal(requests[1].url, '../data/measurement-set-1-1-4000.json');
     455                assert.equal(requests[1].url, '/data/measurement-set-1-1-4000.json');
    456456
    457457                set.fetchBetween(3708, 4800, () => callCountFor4000To5000++);
     
    466466                assert.equal(callCountFor2000, 0);
    467467                assert.equal(requests.length, 3);
    468                 assert.equal(requests[2].url, '../data/measurement-set-1-1-2000.json');
     468                assert.equal(requests[2].url, '/data/measurement-set-1-1-2000.json');
    469469
    470470                requests[2].resolve({
     
    487487                assert.equal(callCountFor2000To4000, 1);
    488488                assert.equal(requests.length, 4);
    489                 assert.equal(requests[3].url, '../data/measurement-set-1-1-3000.json');
     489                assert.equal(requests[3].url, '/data/measurement-set-1-1-3000.json');
    490490
    491491                requests[3].resolve({
     
    534534            const promise = set.fetchBetween(2000, 3000);
    535535            assert.equal(requests.length, 1);
    536             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     536            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    537537
    538538            requests[0].resolve({
     
    557557            const promise = set.fetchBetween(2000, 3000);
    558558            assert.equal(requests.length, 1);
    559             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     559            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    560560
    561561            requests[0].resolve({
     
    579579            const promise = set.fetchBetween(2000, 3000);
    580580            assert.equal(requests.length, 1);
    581             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     581            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    582582
    583583            requests[0].resolve({
     
    594594            return waitForMeasurementSet().then(() => {
    595595                assert.equal(requests.length, 2);
    596                 assert.equal(requests[1].url, '../data/measurement-set-1-1-3000.json');
     596                assert.equal(requests[1].url, '/data/measurement-set-1-1-3000.json');
    597597                requests[1].resolve({
    598598                    'clusterStart': 1000,
     
    614614            set.fetchBetween(2000, 3000);
    615615            assert.equal(requests.length, 1);
    616             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     616            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    617617
    618618            requests[0].resolve({
     
    629629            return waitForMeasurementSet().then(() => {
    630630                assert.equal(requests.length, 2);
    631                 assert.equal(requests[1].url, '../data/measurement-set-1-1-3000.json');
     631                assert.equal(requests[1].url, '/data/measurement-set-1-1-3000.json');
    632632                requests[1].resolve({
    633633                    'clusterStart': 1000,
     
    649649            set.fetchBetween(2000, 5000);
    650650            assert.equal(requests.length, 1);
    651             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     651            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    652652
    653653            requests[0].resolve({
     
    664664            return waitForMeasurementSet().then(() => {
    665665                assert.equal(requests.length, 3);
    666                 assert.equal(requests[1].url, '../data/measurement-set-1-1-3000.json');
    667                 assert.equal(requests[2].url, '../data/measurement-set-1-1-4000.json');
     666                assert.equal(requests[1].url, '/data/measurement-set-1-1-3000.json');
     667                assert.equal(requests[2].url, '/data/measurement-set-1-1-4000.json');
    668668                requests[1].resolve({
    669669                    'clusterStart': 1000,
     
    687687            const promise = set.fetchBetween(2000, 3000);
    688688            assert.equal(requests.length, 1);
    689             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     689            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    690690
    691691            requests[0].resolve({
     
    776776            const promise = set.fetchBetween(1476426488465, 1484203801573);
    777777            assert.equal(requests.length, 1);
    778             assert.equal(requests[0].url, '../data/measurement-set-15-769.json');
     778            assert.equal(requests[0].url, '/data/measurement-set-15-769.json');
    779779
    780780            requests[0].resolve(sampleCluster);
     
    976976            const promise = set.fetchBetween(4000, 5000);
    977977            assert.equal(requests.length, 1);
    978             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     978            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    979979
    980980            requests[0].resolve({
     
    10161016            const promise = set.fetchBetween(3000, 5000);
    10171017            assert.equal(requests.length, 1);
    1018             assert.equal(requests[0].url, '../data/measurement-set-1-1.json');
     1018            assert.equal(requests[0].url, '/data/measurement-set-1-1.json');
    10191019
    10201020            requests[0].resolve({
     
    10311031            return waitForMeasurementSet().then(() => {
    10321032                assert.equal(requests.length, 2);
    1033                 assert.equal(requests[1].url, '../data/measurement-set-1-1-4000.json');
     1033                assert.equal(requests[1].url, '/data/measurement-set-1-1-4000.json');
    10341034                return set.fetchSegmentation('segmentTimeSeriesByMaximizingSchwarzCriterion', [], 'current', false);
    10351035            }).then((segmentation) => {
Note: See TracChangeset for help on using the changeset viewer.