Gilles has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/186160

Change subject: Collect thumbnail width in the performance log
......................................................................

Collect thumbnail width in the performance log

Bug: T86609
Change-Id: Ia0934cf76c323b9bcb2ee7b0ee8ee6ca2aefc600
---
M MultimediaViewer.php
M resources/mmv/mmv.js
2 files changed, 7 insertions(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MultimediaViewer 
refs/changes/60/186160/1

diff --git a/MultimediaViewer.php b/MultimediaViewer.php
index d2eb903..2371ba4 100644
--- a/MultimediaViewer.php
+++ b/MultimediaViewer.php
@@ -1070,7 +1070,7 @@
 $wgHooks['EventLoggingRegisterSchemas'][] = function( array &$schemas ) {
        $schemas += array(
                'MediaViewer' => 10867062,
-               'MultimediaViewerNetworkPerformance' => 10774577,
+               'MultimediaViewerNetworkPerformance' => 11030254,
                'MultimediaViewerDuration' => 10427980,
                'MultimediaViewerAttribution' => 9758179,
                'MultimediaViewerDimensions' => 10014238,
diff --git a/resources/mmv/mmv.js b/resources/mmv/mmv.js
index 6368a95..1c06277 100644
--- a/resources/mmv/mmv.js
+++ b/resources/mmv/mmv.js
@@ -787,6 +787,12 @@
                        thumbnailPromise = this.thumbnailInfoProvider.get( 
fileTitle, width );
                }
 
+               // Add thumbnail width to the extra stats passed to the 
performance log
+               extraStatsDeferred = extraStatsDeferred.then( function ( 
extraStats ) {
+                       extraStats.imageWidth = width;
+                       return extraStats;
+               } );
+
                imagePromise = thumbnailPromise.then( function ( thumbnail ) {
                        return viewer.imageProvider.get( thumbnail.url, 
extraStatsDeferred );
                } );

-- 
To view, visit https://gerrit.wikimedia.org/r/186160
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia0934cf76c323b9bcb2ee7b0ee8ee6ca2aefc600
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MultimediaViewer
Gerrit-Branch: master
Gerrit-Owner: Gilles <gdu...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to