[MediaWiki-commits] [Gerrit] mediawiki/core[wmf/1.28.0-wmf.18]: mediawiki.requestIdleCallback: Avoid native window.requestId...
jenkins-bot has submitted this change and it was merged. Change subject: mediawiki.requestIdleCallback: Avoid native window.requestIdleCallback .. mediawiki.requestIdleCallback: Avoid native window.requestIdleCallback Follows-up 482ad8d9fb. This currently has bugs in Chrome that can cause it to not get called for up to 30 seconds. https://bugs.chromium.org/p/chromium/issues/detail?id=647870 Change-Id: I221cd6c2bcdbee89bc7401c071f2ab1865a738ed --- M resources/src/mediawiki/mediawiki.requestIdleCallback.js M tests/qunit/suites/resources/mediawiki/mediawiki.requestIdleCallback.test.js 2 files changed, 6 insertions(+), 1 deletion(-) Approvals: Krinkle: Looks good to me, approved jenkins-bot: Verified diff --git a/resources/src/mediawiki/mediawiki.requestIdleCallback.js b/resources/src/mediawiki/mediawiki.requestIdleCallback.js index dfc98ad..b58cb69 100644 --- a/resources/src/mediawiki/mediawiki.requestIdleCallback.js +++ b/resources/src/mediawiki/mediawiki.requestIdleCallback.js @@ -24,8 +24,12 @@ * @member mw * @param {Function} callback */ + mw.requestIdleCallback = mw.requestIdleCallbackInternal; + /* + // XXX: Polyfill disabled due to https://bugs.chromium.org/p/chromium/issues/detail?id=647870 mw.requestIdleCallback = window.requestIdleCallback // Bind because it throws TypeError if context is not window ? window.requestIdleCallback.bind( window ) : mw.requestIdleCallbackInternal; + */ }( mediaWiki ) ); diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.requestIdleCallback.test.js b/tests/qunit/suites/resources/mediawiki/mediawiki.requestIdleCallback.test.js index 7a09964..df02693 100644 --- a/tests/qunit/suites/resources/mediawiki/mediawiki.requestIdleCallback.test.js +++ b/tests/qunit/suites/resources/mediawiki/mediawiki.requestIdleCallback.test.js @@ -95,8 +95,9 @@ if ( window.requestIdleCallback ) { QUnit.test( 'native', function ( assert ) { var done = assert.async(); - // Remove polyfill + // Remove polyfill and clock stub mw.requestIdleCallback.restore(); + this.clock.restore(); mw.requestIdleCallback( function () { assert.expect( 0 ); done(); -- To view, visit https://gerrit.wikimedia.org/r/311904 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I221cd6c2bcdbee89bc7401c071f2ab1865a738ed Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/core Gerrit-Branch: wmf/1.28.0-wmf.18 Gerrit-Owner: Krinkle Gerrit-Reviewer: Jack Phoenix Gerrit-Reviewer: Krinkle Gerrit-Reviewer: jenkins-bot <> ___ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
[MediaWiki-commits] [Gerrit] mediawiki/core[wmf/1.28.0-wmf.18]: mediawiki.requestIdleCallback: Avoid native window.requestId...
Krinkle has uploaded a new change for review. https://gerrit.wikimedia.org/r/311904 Change subject: mediawiki.requestIdleCallback: Avoid native window.requestIdleCallback .. mediawiki.requestIdleCallback: Avoid native window.requestIdleCallback Follows-up 482ad8d9fb. This currently has bugs in Chrome that can cause it to not get called for up to 30 seconds. https://bugs.chromium.org/p/chromium/issues/detail?id=647870 Change-Id: I221cd6c2bcdbee89bc7401c071f2ab1865a738ed --- M resources/src/mediawiki/mediawiki.requestIdleCallback.js M tests/qunit/suites/resources/mediawiki/mediawiki.requestIdleCallback.test.js 2 files changed, 6 insertions(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core refs/changes/04/311904/1 diff --git a/resources/src/mediawiki/mediawiki.requestIdleCallback.js b/resources/src/mediawiki/mediawiki.requestIdleCallback.js index dfc98ad..b58cb69 100644 --- a/resources/src/mediawiki/mediawiki.requestIdleCallback.js +++ b/resources/src/mediawiki/mediawiki.requestIdleCallback.js @@ -24,8 +24,12 @@ * @member mw * @param {Function} callback */ + mw.requestIdleCallback = mw.requestIdleCallbackInternal; + /* + // XXX: Polyfill disabled due to https://bugs.chromium.org/p/chromium/issues/detail?id=647870 mw.requestIdleCallback = window.requestIdleCallback // Bind because it throws TypeError if context is not window ? window.requestIdleCallback.bind( window ) : mw.requestIdleCallbackInternal; + */ }( mediaWiki ) ); diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.requestIdleCallback.test.js b/tests/qunit/suites/resources/mediawiki/mediawiki.requestIdleCallback.test.js index 7a09964..df02693 100644 --- a/tests/qunit/suites/resources/mediawiki/mediawiki.requestIdleCallback.test.js +++ b/tests/qunit/suites/resources/mediawiki/mediawiki.requestIdleCallback.test.js @@ -95,8 +95,9 @@ if ( window.requestIdleCallback ) { QUnit.test( 'native', function ( assert ) { var done = assert.async(); - // Remove polyfill + // Remove polyfill and clock stub mw.requestIdleCallback.restore(); + this.clock.restore(); mw.requestIdleCallback( function () { assert.expect( 0 ); done(); -- To view, visit https://gerrit.wikimedia.org/r/311904 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I221cd6c2bcdbee89bc7401c071f2ab1865a738ed Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: wmf/1.28.0-wmf.18 Gerrit-Owner: Krinkle ___ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits