From f4ad20c1d4a99c4c471e9f841911be1780a6ddd9 Mon Sep 17 00:00:00 2001 From: Max Semenik Date: Fri, 4 Mar 2016 14:41:50 -0800 Subject: [PATCH] Instrument diff timing Bug: T128697 Change-Id: I748286abac025092abc33b3b7b8a0d3dabafdd25 --- includes/diff/DifferenceEngine.php | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/includes/diff/DifferenceEngine.php b/includes/diff/DifferenceEngine.php index 1fa19704db..1cf3918a45 100644 --- a/includes/diff/DifferenceEngine.php +++ b/includes/diff/DifferenceEngine.php @@ -842,6 +842,24 @@ class DifferenceEngine extends ContextSource { * @return bool|string */ public function generateTextDiffBody( $otext, $ntext ) { + $time = microtime( true ); + + $result = $this->textDiff( $otext, $ntext ); + + $time = microtime( true ) - $time; + $this->getStats()->timing( 'diff_time', $time ); + + return $result; + } + + /** + * Generates diff, to be wrapped internally in a logging/instrumentation + * + * @param string $otext Old text, must be already segmented + * @param string $ntext New text, must be already segmented + * @return bool|string + */ + protected function textDiff( $otext, $ntext ) { global $wgExternalDiffEngine, $wgContLang; $otext = str_replace( "\r\n", "\n", $otext ); -- 2.20.1