2014-02-06 16:17:38 +00:00
|
|
|
/*
|
|
|
|
* This file is part of the MediaWiki extension MultimediaViewer.
|
|
|
|
*
|
|
|
|
* MultimediaViewer is free software: you can redistribute it and/or modify
|
|
|
|
* it under the terms of the GNU General Public License as published by
|
|
|
|
* the Free Software Foundation, either version 2 of the License, or
|
|
|
|
* (at your option) any later version.
|
|
|
|
*
|
|
|
|
* MultimediaViewer is distributed in the hope that it will be useful,
|
|
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
|
|
* GNU General Public License for more details.
|
|
|
|
*
|
|
|
|
* You should have received a copy of the GNU General Public License
|
|
|
|
* along with MultimediaViewer. If not, see <http://www.gnu.org/licenses/>.
|
|
|
|
*/
|
|
|
|
|
|
|
|
( function ( mw, $ ) {
|
2014-11-21 10:07:40 +00:00
|
|
|
QUnit.module( 'mmv.logging.PerformanceLogger', QUnit.newMwEnvironment() );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
Use cross-origin img attribute instead of data URI
After lots of experimenting with Wireshark and
current Chrome + Firefox on Ubuntu 13.10, this is my
current understanding of the caching when preloading images
with AJAX requests:
* on Chrome, the image request always comes from browser cache
* Firefox makes two separate requests by default
* Firefox with img.crossOrigin = 'anonymous' makes two separate
requests, but the second one is a 304 (does not load the
image twice)
* when the image has already been cached by the browser (but not in
this session), Chrome skips both requests; Firefox skips the AJAX
request, but sends the normal one, and it returns with 304.
"wish I knew this when I started" things:
* the Chrome DevTools has an option to disable cache. When this is
enabled, requests in the same document context still come from
cache (so if I load the page, fire an AJAX request, then without
reloading the page, fire an AJAX request to the same URL, then the
second request will be cached), but an AJAX request - image request
pair is an exception from this.
* when using Ctrl-F5 in Firefox, requests on that page will never hit
the cache (even AJAX request fired after user activity; even if
two identical requests follow each other). When using clear cache
+ normal reload, this is not the case.
* if the image does not have an Allow-Origin header and is loaded
with crossOrigin=true, Firefox will refuse to load it. Chrome will
log an error in the console saying it refused to load it, but will
actually load it.
* Wireshark rocks.
Pushed some tech debt (browser + domain whitelist) into other tickets:
https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/232
https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/233
Reverted commits:
8a8d74f01d3dbd6d0c43b7fadc5284d204091761.
63021d0b0e95442cce101f9f92de8f0ff97d5f49.
Change-Id: I84ab2f3ac0a9706926adf7fe8726ecd9e9f843e0
Bug: 61542
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/207
2014-02-23 21:46:18 +00:00
|
|
|
function createFakeXHR( response ) {
|
|
|
|
return {
|
|
|
|
readyState: 0,
|
|
|
|
open: $.noop,
|
|
|
|
send: function () {
|
|
|
|
var xhr = this;
|
|
|
|
|
|
|
|
setTimeout( function () {
|
|
|
|
xhr.readyState = 4;
|
|
|
|
xhr.response = response;
|
|
|
|
if ( $.isFunction( xhr.onreadystatechange ) ) {
|
|
|
|
xhr.onreadystatechange();
|
|
|
|
}
|
|
|
|
}, 0 );
|
|
|
|
}
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
QUnit.test( 'recordEntry: basic', 7, function ( assert ) {
|
2014-11-21 10:07:40 +00:00
|
|
|
var performance = new mw.mmv.logging.PerformanceLogger(),
|
2014-05-19 09:24:54 +00:00
|
|
|
fakeEventLog = { logEvent : this.sandbox.stub() },
|
2014-02-06 16:17:38 +00:00
|
|
|
type = 'gender',
|
2014-05-05 06:33:58 +00:00
|
|
|
total = 100;
|
2014-02-06 16:17:38 +00:00
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
this.sandbox.stub( performance, 'loadDependencies' ).returns( $.Deferred().resolve() );
|
|
|
|
this.sandbox.stub( performance, 'isInSample' );
|
|
|
|
performance.setEventLog( fakeEventLog );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
performance.isInSample.returns( false );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
|
|
|
performance.recordEntry( type, total );
|
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
assert.strictEqual( fakeEventLog.logEvent.callCount, 0, 'No stats should be logged if not in sample' );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
performance.isInSample.returns( true );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
|
|
|
performance.recordEntry( type, total );
|
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 0 ], 'MultimediaViewerNetworkPerformance', 'EventLogging schema is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].type, type, 'type is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].total, total, 'total is correct' );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
assert.strictEqual( fakeEventLog.logEvent.callCount, 1, 'Stats should be logged' );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
|
|
|
performance.recordEntry( type, total, 'URL' );
|
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
assert.strictEqual( fakeEventLog.logEvent.callCount, 2, 'Stats should be logged' );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
|
|
|
performance.recordEntry( type, total, 'URL' );
|
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
assert.strictEqual( fakeEventLog.logEvent.callCount, 2, 'Stats should not be logged a second time for the same URL' );
|
2014-02-06 16:17:38 +00:00
|
|
|
} );
|
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
QUnit.test( 'recordEntry: with Navigation Timing data', 29, function ( assert ) {
|
2014-02-06 16:17:38 +00:00
|
|
|
var fakeRequest,
|
|
|
|
varnish1 = 'cp1061',
|
|
|
|
varnish2 = 'cp3006',
|
|
|
|
varnish3 = 'cp3005',
|
|
|
|
varnish1hits = 0,
|
|
|
|
varnish2hits = 2,
|
|
|
|
varnish3hits = 1,
|
|
|
|
xvarnish = '1754811951 1283049064, 1511828531, 1511828573 1511828528',
|
|
|
|
xcache = varnish1 + ' miss (0), ' + varnish2 + ' miss (2), ' + varnish3 + ' frontend hit (1), malformed(5)',
|
|
|
|
age = '12345',
|
|
|
|
contentLength = '23456',
|
|
|
|
urlHost = 'fail',
|
|
|
|
date = 'Tue, 04 Feb 2014 11:11:50 GMT',
|
|
|
|
timestamp = 1391512310,
|
|
|
|
url = 'https://' + urlHost + '/balls.jpg',
|
|
|
|
redirect = 500,
|
|
|
|
dns = 2,
|
|
|
|
tcp = 10,
|
|
|
|
request = 25,
|
|
|
|
response = 50,
|
|
|
|
cache = 1,
|
|
|
|
perfData = {
|
|
|
|
initiatorType: 'xmlhttprequest',
|
|
|
|
name: url,
|
|
|
|
duration: 12345,
|
|
|
|
redirectStart: 1000,
|
|
|
|
redirectEnd: 1500,
|
|
|
|
domainLookupStart: 2,
|
|
|
|
domainLookupEnd: 4,
|
|
|
|
connectStart: 50,
|
|
|
|
connectEnd: 60,
|
|
|
|
requestStart: 125,
|
|
|
|
responseStart: 150,
|
|
|
|
responseEnd: 200,
|
|
|
|
fetchStart: 1
|
|
|
|
},
|
|
|
|
country = 'FR',
|
|
|
|
type = 'image',
|
2014-11-21 10:07:40 +00:00
|
|
|
performance = new mw.mmv.logging.PerformanceLogger(),
|
2014-02-06 16:17:38 +00:00
|
|
|
status = 200,
|
|
|
|
metered = true,
|
2014-05-19 09:24:54 +00:00
|
|
|
bandwidth = 45.67,
|
|
|
|
fakeEventLog = { logEvent : this.sandbox.stub() };
|
|
|
|
|
|
|
|
this.sandbox.stub( performance, 'loadDependencies' ).returns( $.Deferred().resolve() );
|
|
|
|
performance.setEventLog( fakeEventLog );
|
2014-09-26 21:42:30 +00:00
|
|
|
performance.schemaSupportsCountry = this.sandbox.stub().returns( true );
|
2014-05-19 09:24:54 +00:00
|
|
|
|
|
|
|
this.sandbox.stub( performance, 'getWindowPerformance' ).returns( {
|
|
|
|
getEntriesByName: function () {
|
|
|
|
return [perfData, {
|
|
|
|
initiatorType: 'bogus',
|
|
|
|
duration: 1234,
|
|
|
|
name: url
|
|
|
|
}];
|
|
|
|
}
|
|
|
|
} );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
this.sandbox.stub( performance, 'getNavigatorConnection' ).returns( { metered : metered, bandwidth : bandwidth } );
|
|
|
|
this.sandbox.stub( performance, 'isInSample' ).returns( true );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
|
|
|
fakeRequest = {
|
|
|
|
getResponseHeader: function ( header ) {
|
|
|
|
switch ( header ) {
|
|
|
|
case 'X-Cache':
|
|
|
|
return xcache;
|
|
|
|
case 'X-Varnish':
|
|
|
|
return xvarnish;
|
|
|
|
case 'Age':
|
|
|
|
return age;
|
|
|
|
case 'Content-Length':
|
|
|
|
return contentLength;
|
|
|
|
case 'Date':
|
|
|
|
return date;
|
|
|
|
}
|
|
|
|
},
|
|
|
|
status: status
|
|
|
|
};
|
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
performance.setGeo( { country : country } );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
2014-05-05 06:33:58 +00:00
|
|
|
performance.recordEntry( type, 100, url, fakeRequest );
|
2014-02-06 16:17:38 +00:00
|
|
|
|
2014-05-19 09:24:54 +00:00
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 0 ], 'MultimediaViewerNetworkPerformance', 'EventLogging schema is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].type, type, 'type is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].varnish1, varnish1, 'varnish1 is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].varnish2, varnish2, 'varnish2 is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].varnish3, varnish3, 'varnish3 is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].varnish4, undefined, 'varnish4 is undefined' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].varnish1hits, varnish1hits, 'varnish1hits is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].varnish2hits, varnish2hits, 'varnish2hits is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].varnish3hits, varnish3hits, 'varnish3hits is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].varnish4hits, undefined, 'varnish4hits is undefined' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].XVarnish, xvarnish, 'XVarnish is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].XCache, xcache, 'XCache is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].age, parseInt( age, 10 ), 'age is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].contentLength, parseInt( contentLength, 10 ), 'contentLength is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].contentHost, window.location.host, 'contentHost is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].urlHost, urlHost, 'urlHost is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].timestamp, timestamp, 'timestamp is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].total, perfData.duration, 'total is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].redirect, redirect, 'redirect is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].dns, dns, 'dns is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].tcp, tcp, 'tcp is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].request, request, 'request is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].response, response, 'response is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].cache, cache, 'cache is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].country, country, 'country is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].isHttps, true, 'isHttps is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].status, status, 'status is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].metered, metered, 'metered is correct' );
|
|
|
|
assert.strictEqual( fakeEventLog.logEvent.getCall( 0 ).args[ 1 ].bandwidth, Math.round( bandwidth ), 'bandwidth is correct' );
|
2014-02-06 16:17:38 +00:00
|
|
|
} );
|
|
|
|
|
|
|
|
QUnit.test( 'parseVarnishXCacheHeader', 15, function ( assert ) {
|
|
|
|
var varnish1 = 'cp1061',
|
|
|
|
varnish2 = 'cp3006',
|
|
|
|
varnish3 = 'cp3005',
|
|
|
|
testString = varnish1 + ' miss (0), ' + varnish2 + ' miss (0), ' + varnish3 + ' frontend hit (1)',
|
2014-11-21 10:07:40 +00:00
|
|
|
performance = new mw.mmv.logging.PerformanceLogger(),
|
2014-02-06 16:17:38 +00:00
|
|
|
varnishXCache = performance.parseVarnishXCacheHeader( testString );
|
|
|
|
|
|
|
|
assert.strictEqual( varnishXCache.varnish1, varnish1, 'First varnish server name extracted' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish2, varnish2, 'Second varnish server name extracted' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish3, varnish3, 'Third varnish server name extracted' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish4, undefined, 'Fourth varnish server is undefined' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish1hits, 0, 'First varnish hit count extracted' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish2hits, 0, 'Second varnish hit count extracted' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish3hits, 1, 'Third varnish hit count extracted' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish4hits, undefined, 'Fourth varnish hit count is undefined' );
|
|
|
|
|
|
|
|
testString = varnish1 + ' miss (36), ' + varnish2 + ' miss (2)';
|
|
|
|
varnishXCache = performance.parseVarnishXCacheHeader( testString );
|
|
|
|
|
|
|
|
assert.strictEqual( varnishXCache.varnish1, varnish1, 'First varnish server name extracted' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish2, varnish2, 'Second varnish server name extracted' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish3, undefined, 'Third varnish server is undefined' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish1hits, 36, 'First varnish hit count extracted' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish2hits, 2, 'Second varnish hit count extracted' );
|
|
|
|
assert.strictEqual( varnishXCache.varnish3hits, undefined, 'Third varnish hit count is undefined' );
|
|
|
|
|
|
|
|
varnishXCache = performance.parseVarnishXCacheHeader( 'garbage' );
|
|
|
|
assert.ok( $.isEmptyObject( varnishXCache ), 'Varnish cache results are empty' );
|
|
|
|
} );
|
2014-02-15 02:15:54 +00:00
|
|
|
|
Use cross-origin img attribute instead of data URI
After lots of experimenting with Wireshark and
current Chrome + Firefox on Ubuntu 13.10, this is my
current understanding of the caching when preloading images
with AJAX requests:
* on Chrome, the image request always comes from browser cache
* Firefox makes two separate requests by default
* Firefox with img.crossOrigin = 'anonymous' makes two separate
requests, but the second one is a 304 (does not load the
image twice)
* when the image has already been cached by the browser (but not in
this session), Chrome skips both requests; Firefox skips the AJAX
request, but sends the normal one, and it returns with 304.
"wish I knew this when I started" things:
* the Chrome DevTools has an option to disable cache. When this is
enabled, requests in the same document context still come from
cache (so if I load the page, fire an AJAX request, then without
reloading the page, fire an AJAX request to the same URL, then the
second request will be cached), but an AJAX request - image request
pair is an exception from this.
* when using Ctrl-F5 in Firefox, requests on that page will never hit
the cache (even AJAX request fired after user activity; even if
two identical requests follow each other). When using clear cache
+ normal reload, this is not the case.
* if the image does not have an Allow-Origin header and is loaded
with crossOrigin=true, Firefox will refuse to load it. Chrome will
log an error in the console saying it refused to load it, but will
actually load it.
* Wireshark rocks.
Pushed some tech debt (browser + domain whitelist) into other tickets:
https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/232
https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/233
Reverted commits:
8a8d74f01d3dbd6d0c43b7fadc5284d204091761.
63021d0b0e95442cce101f9f92de8f0ff97d5f49.
Change-Id: I84ab2f3ac0a9706926adf7fe8726ecd9e9f843e0
Bug: 61542
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/207
2014-02-23 21:46:18 +00:00
|
|
|
QUnit.test( 'record()', 4, function ( assert ) {
|
|
|
|
var type = 'foo',
|
|
|
|
url = 'http://example.com/',
|
|
|
|
response = {},
|
2014-11-21 10:07:40 +00:00
|
|
|
performance = new mw.mmv.logging.PerformanceLogger();
|
Use cross-origin img attribute instead of data URI
After lots of experimenting with Wireshark and
current Chrome + Firefox on Ubuntu 13.10, this is my
current understanding of the caching when preloading images
with AJAX requests:
* on Chrome, the image request always comes from browser cache
* Firefox makes two separate requests by default
* Firefox with img.crossOrigin = 'anonymous' makes two separate
requests, but the second one is a 304 (does not load the
image twice)
* when the image has already been cached by the browser (but not in
this session), Chrome skips both requests; Firefox skips the AJAX
request, but sends the normal one, and it returns with 304.
"wish I knew this when I started" things:
* the Chrome DevTools has an option to disable cache. When this is
enabled, requests in the same document context still come from
cache (so if I load the page, fire an AJAX request, then without
reloading the page, fire an AJAX request to the same URL, then the
second request will be cached), but an AJAX request - image request
pair is an exception from this.
* when using Ctrl-F5 in Firefox, requests on that page will never hit
the cache (even AJAX request fired after user activity; even if
two identical requests follow each other). When using clear cache
+ normal reload, this is not the case.
* if the image does not have an Allow-Origin header and is loaded
with crossOrigin=true, Firefox will refuse to load it. Chrome will
log an error in the console saying it refused to load it, but will
actually load it.
* Wireshark rocks.
Pushed some tech debt (browser + domain whitelist) into other tickets:
https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/232
https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/233
Reverted commits:
8a8d74f01d3dbd6d0c43b7fadc5284d204091761.
63021d0b0e95442cce101f9f92de8f0ff97d5f49.
Change-Id: I84ab2f3ac0a9706926adf7fe8726ecd9e9f843e0
Bug: 61542
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/207
2014-02-23 21:46:18 +00:00
|
|
|
|
|
|
|
performance.newXHR = function () { return createFakeXHR( response ); };
|
|
|
|
|
|
|
|
QUnit.stop();
|
|
|
|
performance.recordEntryDelayed = function ( recordType, _, recordUrl, recordRequest ) {
|
|
|
|
assert.strictEqual( recordType, type, 'type is recorded correctly' );
|
|
|
|
assert.strictEqual( recordUrl, url, 'url is recorded correctly' );
|
|
|
|
assert.strictEqual( recordRequest.response, response, 'response is recorded correctly' );
|
|
|
|
QUnit.start();
|
|
|
|
};
|
|
|
|
|
|
|
|
QUnit.stop();
|
|
|
|
performance.record( type, url ).done( function ( recordResponse ) {
|
|
|
|
assert.strictEqual( recordResponse, response, 'response is passed to callback' );
|
|
|
|
QUnit.start();
|
|
|
|
} );
|
|
|
|
} );
|
|
|
|
|
|
|
|
QUnit.asyncTest( 'record() with old browser', 1, function ( assert ) {
|
|
|
|
var type = 'foo',
|
|
|
|
url = 'http://example.com/',
|
2014-11-21 10:07:40 +00:00
|
|
|
performance = new mw.mmv.logging.PerformanceLogger();
|
Use cross-origin img attribute instead of data URI
After lots of experimenting with Wireshark and
current Chrome + Firefox on Ubuntu 13.10, this is my
current understanding of the caching when preloading images
with AJAX requests:
* on Chrome, the image request always comes from browser cache
* Firefox makes two separate requests by default
* Firefox with img.crossOrigin = 'anonymous' makes two separate
requests, but the second one is a 304 (does not load the
image twice)
* when the image has already been cached by the browser (but not in
this session), Chrome skips both requests; Firefox skips the AJAX
request, but sends the normal one, and it returns with 304.
"wish I knew this when I started" things:
* the Chrome DevTools has an option to disable cache. When this is
enabled, requests in the same document context still come from
cache (so if I load the page, fire an AJAX request, then without
reloading the page, fire an AJAX request to the same URL, then the
second request will be cached), but an AJAX request - image request
pair is an exception from this.
* when using Ctrl-F5 in Firefox, requests on that page will never hit
the cache (even AJAX request fired after user activity; even if
two identical requests follow each other). When using clear cache
+ normal reload, this is not the case.
* if the image does not have an Allow-Origin header and is loaded
with crossOrigin=true, Firefox will refuse to load it. Chrome will
log an error in the console saying it refused to load it, but will
actually load it.
* Wireshark rocks.
Pushed some tech debt (browser + domain whitelist) into other tickets:
https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/232
https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/233
Reverted commits:
8a8d74f01d3dbd6d0c43b7fadc5284d204091761.
63021d0b0e95442cce101f9f92de8f0ff97d5f49.
Change-Id: I84ab2f3ac0a9706926adf7fe8726ecd9e9f843e0
Bug: 61542
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/207
2014-02-23 21:46:18 +00:00
|
|
|
|
|
|
|
performance.newXHR = function () { throw 'XMLHttpRequest? What\'s that?'; };
|
|
|
|
|
|
|
|
performance.record( type, url ).fail( function () {
|
|
|
|
assert.ok( true, 'the promise is rejected when XMLHttpRequest is not supported' );
|
|
|
|
QUnit.start();
|
|
|
|
} );
|
|
|
|
} );
|
|
|
|
|
2014-09-04 23:00:55 +00:00
|
|
|
QUnit.test( 'mw.mmv.logging.Api', 3, function ( assert ) {
|
2014-02-15 02:15:54 +00:00
|
|
|
var api,
|
2014-11-21 10:07:40 +00:00
|
|
|
oldRecord = mw.mmv.logging.PerformanceLogger.prototype.recordJQueryEntryDelayed,
|
2014-02-15 02:15:54 +00:00
|
|
|
oldAjax = mw.Api.prototype.ajax,
|
|
|
|
ajaxCalled = false,
|
|
|
|
fakeJqxhr = {};
|
|
|
|
|
|
|
|
mw.Api.prototype.ajax = function() {
|
|
|
|
ajaxCalled = true;
|
|
|
|
return $.Deferred().resolve( {}, fakeJqxhr );
|
|
|
|
};
|
|
|
|
|
2014-11-21 10:07:40 +00:00
|
|
|
mw.mmv.logging.PerformanceLogger.prototype.recordJQueryEntryDelayed = function ( type, total, jqxhr ) {
|
2014-02-15 02:15:54 +00:00
|
|
|
assert.strictEqual( type, 'foo', 'type was passed correctly' );
|
|
|
|
assert.strictEqual( jqxhr, fakeJqxhr, 'jqXHR was passed correctly' );
|
|
|
|
};
|
|
|
|
|
2014-09-04 23:00:55 +00:00
|
|
|
api = new mw.mmv.logging.Api( 'foo' );
|
2014-02-15 02:15:54 +00:00
|
|
|
|
|
|
|
api.ajax();
|
|
|
|
|
|
|
|
assert.ok( ajaxCalled, 'parent ajax() function was called' );
|
|
|
|
|
2014-11-21 10:07:40 +00:00
|
|
|
mw.mmv.logging.PerformanceLogger.prototype.recordJQueryEntryDelayed = oldRecord;
|
2014-02-15 02:15:54 +00:00
|
|
|
mw.Api.prototype.ajax = oldAjax;
|
|
|
|
} );
|
2014-02-06 16:17:38 +00:00
|
|
|
}( mediaWiki, jQuery ) );
|