Don't change dir pseudo-selector target

CSSJanus is taking :dir(ltr) as an URL, this fixes it by first turning
them into two temporarily tokens, then proceeding with URL change
logic, then putting back the original values, a similar pattern done
to swap ltr and rtl in the same place.

This is porting back Ifb38c598c3147689d084ec2535849a8224b794a6 to PHP.

Bug: T371466
Change-Id: I9198294c6bd98c9c064f461e251778185161abcc

Author: ebraminio

src/CSSJanus.php

@@ -28,6 +28,8 @@
  */
 class CSSJanus {
 	private const TOKEN_TMP = '`TMP`';
+	private const TOKEN_LTR_TMP = '`TMPLTR`';
+	private const TOKEN_RTL_TMP = '`TMPRTL`';
 	private const TOKEN_COMMENT = '`COMMENT`';
 
 	private static $patterns = null;
@@ -38,7 +40,6 @@ private static function buildPatterns() {
 		}
 		// Patterns defined as null are built dynamically
 		$patterns = [
-			'tmpToken' => '`TMP`',
 			'nonAscii' => '[\200-\377]',
 			'unicode' => '(?:(?:\\\\[0-9a-f]{1,6})(?:\r\n|\s)?)',
 			'num' => '(?:[0-9]*\.[0-9]+|[0-9]+)',
@@ -71,6 +72,8 @@ private static function buildPatterns() {
 			'right' => null,
 			'left_in_url' => null,
 			'right_in_url' => null,
+			'ltr_dir_selector' => '/(:dir\( *)ltr( *\))/',
+			'rtl_dir_selector' => '/(:dir\( *)rtl( *\))/',
 			'ltr_in_url' => null,
 			'rtl_in_url' => null,
 			'cursor_east' => null,
@@ -238,9 +241,13 @@ private static function fixDirection( $css ) {
 	 * @return string
 	 */
 	private static function fixLtrRtlInURL( $css ) {
+		$css = preg_replace( self::$patterns['ltr_dir_selector'], '$1' . self::TOKEN_LTR_TMP . '$2', $css );
+		$css = preg_replace( self::$patterns['rtl_dir_selector'], '$1' . self::TOKEN_RTL_TMP . '$2', $css );
 		$css = preg_replace( self::$patterns['ltr_in_url'], self::TOKEN_TMP, $css );
 		$css = preg_replace( self::$patterns['rtl_in_url'], 'ltr', $css );
 		$css = str_replace( self::TOKEN_TMP, 'rtl', $css );
+		$css = str_replace( self::TOKEN_LTR_TMP, 'ltr', $css );
+		$css = str_replace( self::TOKEN_RTL_TMP, 'rtl', $css );
 
 		return $css;
 	}

test/suites/CSSJanusTest.php

@@ -62,7 +62,7 @@ public function testTransform( $input, $args, $output, $name ) {
 	protected static function getSpec() {
 		static $json;
 		if ( $json == null ) {
-			$version = '2.2.0';
+			$version = '2.3.0';
 			$dir = dirname( __DIR__ );
 			$file = "$dir/data-v$version.json";
 			if ( !is_readable( $file ) ) {