From 2826b54e7c10178a503fad44e8d0990e2c997fd4 Mon Sep 17 00:00:00 2001
From: Ehsan Akhgari <ehsan.akhgari@gmail.com>
Date: Fri, 30 Jan 2009 11:43:17 +0330
Subject: [PATCH] Bug 437844 - [RTL] about:plugins is hardcoded to LTR; r=bz

---
 toolkit/content/directionDetector.html        |   9 +
 toolkit/content/jar.mn                        |   1 +
 toolkit/content/plugins.css                   |   8 +-
 toolkit/content/plugins.html                  |  24 +++
 toolkit/content/tests/chrome/Makefile.in      |   3 +
 toolkit/content/tests/chrome/bug437844.css    |   2 +
 .../content/tests/chrome/bug437844.manifest   |   2 +
 .../content/tests/chrome/test_bug437844.xul   | 160 ++++++++++++++++++
 8 files changed, 205 insertions(+), 4 deletions(-)
 create mode 100644 toolkit/content/directionDetector.html
 create mode 100644 toolkit/content/tests/chrome/bug437844.css
 create mode 100644 toolkit/content/tests/chrome/bug437844.manifest
 create mode 100644 toolkit/content/tests/chrome/test_bug437844.xul
diff --git a/toolkit/content/directionDetector.html b/toolkit/content/directionDetector.html
new file mode 100644
index 00000000000..4c5faf01fd1
--- /dev/null
+++ b/toolkit/content/directionDetector.html
@@ -0,0 +1,9 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <link rel="stylesheet" type="text/css" href="chrome://global/locale/intl.css">
+  </head>
+  <body>
+    <window id="target" style="display: none;"></window>
+  </body>
+</html>
diff --git a/toolkit/content/jar.mn b/toolkit/content/jar.mn
index 8e40179a144..f088158e579 100644
--- a/toolkit/content/jar.mn
+++ b/toolkit/content/jar.mn
@@ -9,6 +9,7 @@ toolkit.jar:
    content/global/XPCNativeWrapper.js         (XPCNativeWrapper.js)
 *  content/global/xul.css                     (xul.css)
 *  content/global/about.xhtml                 (about.xhtml)
+   content/global/directionDetector.html
    content/global/plugins.html
    content/global/plugins.css
 *+  content/global/buildconfig.html            (buildconfig.html)
diff --git a/toolkit/content/plugins.css b/toolkit/content/plugins.css
index 2935a4b674b..edfd0a61330 100644
--- a/toolkit/content/plugins.css
+++ b/toolkit/content/plugins.css
@@ -71,7 +71,7 @@ div#findmore {
   margin-top: 2em;
   margin-bottom: 1em;
   font-size: large;
-  text-align: left;
+  text-align: start;
   font-weight: bold;
 }
 
@@ -83,7 +83,7 @@ table {
   background-color: -moz-Dialog;
   color: -moz-DialogText;
   font: message-box;
-  text-align: left;
+  text-align: start;
   width: 100%;
   border: 1px solid ThreeDShadow;
   border-spacing: 0px;
@@ -102,11 +102,11 @@ th {
 
 th + th,
 td + td {
-  border-left: 1px dotted ThreeDShadow; 
+  -moz-border-start: 1px dotted ThreeDShadow; 
 }
 
 td {
-  text-align: left;
+  text-align: start;
   border-top: 1px dotted ThreeDShadow;
 }
 
diff --git a/toolkit/content/plugins.html b/toolkit/content/plugins.html
index 168a7bca603..853b2ceac96 100644
--- a/toolkit/content/plugins.html
+++ b/toolkit/content/plugins.html
@@ -26,6 +26,7 @@
    -   Frank Tang <ftang@netscape.com>
    -   Mike Connelly <mozilla@shepherdstown.com>
    -   Robert Kaiser <kairo@kairo.at>
+   -   Ehsan Akhgari <ehsan.akhgari@gmail.com>
    -
    - Alternatively, the contents of this file may be used under the terms of
    - either the GNU General Public License Version 2 or later (the "GPL"), or
@@ -57,6 +58,29 @@
 <div id="outside">
 <script type="application/javascript">
 
+  function setDirection() {
+    var frame = document.getElementById("directionDetector");
+    var direction = frame.contentDocument
+                         .defaultView
+                         .window
+                         .getComputedStyle(frame.contentDocument.getElementById("target"), "")
+                         .getPropertyValue("direction");
+    document.body.removeChild(frame);
+    document.dir = direction;
+  }
+
+  function setupDirection() {
+    var frame = document.createElement("iframe");
+    frame.setAttribute("id", "directionDetector");
+    frame.setAttribute("src", "chrome://global/content/directionDetector.html");
+    frame.setAttribute("width", "0");
+    frame.setAttribute("height", "0");
+    frame.setAttribute("style", "visibility: hidden;");
+    frame.setAttribute("onload", "setDirection();");
+    document.body.appendChild(frame);
+  }
+  setupDirection();
+
   /* JavaScript to enumerate and display all installed plug-ins
 
    * First, refresh plugins in case anything has been changed recently in
diff --git a/toolkit/content/tests/chrome/Makefile.in b/toolkit/content/tests/chrome/Makefile.in
index eee012d26e9..3a7af0876e6 100644
--- a/toolkit/content/tests/chrome/Makefile.in
+++ b/toolkit/content/tests/chrome/Makefile.in
@@ -60,6 +60,9 @@ _TEST_FILES = 	findbar_window.xul \
 		test_bug418874.xul \
 		bug429723_window.xul \
 		test_bug429723.xul \
+		bug437844.css \
+		bug437844.manifest \
+		test_bug437844.xul \
 		bug451286_window.xul \
 		test_bug451286.xul \
 		test_popup_preventdefault_chrome.xul \
diff --git a/toolkit/content/tests/chrome/bug437844.css b/toolkit/content/tests/chrome/bug437844.css
new file mode 100644
index 00000000000..0fea0100196
--- /dev/null
+++ b/toolkit/content/tests/chrome/bug437844.css
@@ -0,0 +1,2 @@
+/* Imitate RTL UI */
+window { direction: rtl; }
diff --git a/toolkit/content/tests/chrome/bug437844.manifest b/toolkit/content/tests/chrome/bug437844.manifest
new file mode 100644
index 00000000000..630063b9c1b
--- /dev/null
+++ b/toolkit/content/tests/chrome/bug437844.manifest
@@ -0,0 +1,2 @@
+# Override intl.css with our own CSS file
+override chrome://global/locale/intl.css chrome://mochikit/content/chrome/toolkit/content/tests/chrome/bug437844.css
diff --git a/toolkit/content/tests/chrome/test_bug437844.xul b/toolkit/content/tests/chrome/test_bug437844.xul
new file mode 100644
index 00000000000..6c0595d56c5
--- /dev/null
+++ b/toolkit/content/tests/chrome/test_bug437844.xul
@@ -0,0 +1,160 @@
+<?xml version="1.0"?>
+<?xml-stylesheet href="chrome://global/skin" type="text/css"?>
+<?xml-stylesheet 
+  href="chrome://mochikit/content/tests/SimpleTest/test.css"
+  type="text/css"?>
+<!--
+https://bugzilla.mozilla.org/show_bug.cgi?id=437844
+-->
+<window title="Mozilla Bug 437844"
+  xmlns:html="http://www.w3.org/1999/xhtml"
+  xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
+
+  <title>Test for Bug 437844</title>
+  <script type="application/javascript" 
+          src="chrome://mochikit/content/MochiKit/packed.js"></script>
+  <script type="application/javascript" 
+          src="chrome://mochikit/content/tests/SimpleTest/SimpleTest.js"></script>
+
+  <body  xmlns="http://www.w3.org/1999/xhtml">
+    <a target="_blank" 
+       href="https://bugzilla.mozilla.org/show_bug.cgi?id=437844">
+      Mozilla Bug 437844
+    </a>
+
+    <p id="display"></p>
+    <div id="content" style="display: none">
+    </div>
+    <pre id="test">
+    </pre>
+  </body>
+
+  <script type="application/javascript">
+    <![CDATA[
+
+      /* This code is mostly copied from chrome/test/unit/head_crtestutils.js */
+      const NS_CHROME_MANIFESTS_FILE_LIST = "ChromeML";
+
+      const Cc = Components.classes;
+      const Ci = Components.interfaces;
+      const Cr = Components.results;
+
+      function ArrayEnumerator(array)
+      {
+        this.array = array;
+      }
+
+      ArrayEnumerator.prototype = {
+        pos: 0,
+        
+        hasMoreElements: function() {
+          return this.pos < this.array.length;
+        },
+        
+        getNext: function() {
+          if (this.pos < this.array.length)
+            return this.array[this.pos++];
+          throw Cr.NS_ERROR_FAILURE;
+        },
+
+        QueryInterface: function(iid) {
+          if (iid.equals(Ci.nsISimpleEnumerator)
+           || iid.equals(Ci.nsISupports))
+            return this;
+
+          throw Cr.NS_ERROR_NO_INTERFACE;
+        }
+      };
+
+      function ChromeProvider(manifests)
+      {
+        this._manifests = manifests;
+      }
+
+      ChromeProvider.prototype = {
+        getFile: function(prop, persistent) {
+          throw Cr.NS_ERROR_FAILURE;
+        },
+
+        getFiles: function(prop) {
+          if (prop == NS_CHROME_MANIFESTS_FILE_LIST) {
+            return new ArrayEnumerator(this._manifests);
+          }
+          throw Cr.NS_ERROR_FAILURE;
+        },
+
+        QueryInterface: function(iid) {
+          if (iid.equals(Ci.nsIDirectoryServiceProvider)
+           || iid.equals(Ci.nsIDirectoryServiceProvider2)
+           || iid.equals(Ci.nsISupports))
+            return this;
+
+          throw Cr.NS_ERROR_NO_INTERFACE;
+        }
+      };
+
+      let dirSvc = Cc["@mozilla.org/file/directory_service;1"].
+                   getService(Ci.nsIDirectoryService);
+      function registerManifests(manifests)
+      {
+        let provider = new ChromeProvider(manifests);
+        dirSvc.registerProvider(provider);
+        return provider;
+      }
+
+
+    ]]>
+  </script>
+
+  <script class="testbody" type="application/javascript">
+    <![CDATA[
+
+      /** Test for Bug 437844 **/
+      SimpleTest.waitForExplicitFinish();
+
+      // Disable XUL cache temporarily
+      let prefs = Cc["@mozilla.org/preferences-service;1"].
+                  getService(Ci.nsIPrefBranch);
+      const XUL_CACHE_PREF = "nglayout.debug.disable_xul_cache";
+      prefs.setBoolPref(XUL_CACHE_PREF, true);
+
+      // Register our manifest
+      let cr = Cc["@mozilla.org/chrome/chrome-registry;1"].
+               getService(Ci.nsIXULChromeRegistry);
+      let manifests = [];
+      let currentManifests = dirSvc.QueryInterface(Ci.nsIProperties)
+                                   .get(NS_CHROME_MANIFESTS_FILE_LIST,
+                                        Ci.nsISimpleEnumerator);
+      while (currentManifests.hasMoreElements())
+        manifests.push(currentManifests.getNext());
+      let uri = Cc["@mozilla.org/network/io-service;1"].
+                getService(Ci.nsIIOService).
+                newURI("chrome://mochikit/content/chrome/toolkit/content/tests/chrome/", null, null);
+      uri = cr.convertChromeURL(uri);
+      let newChromePath = uri.QueryInterface(Ci.nsIFileURL).file;
+      manifests.push(newChromePath);
+      let provider = registerManifests(manifests);
+      cr.checkForNewChrome();
+      cr.refreshSkins();
+
+      // Load about:plugins in an iframe
+      let frame = document.createElement("iframe");
+      frame.setAttribute("src", "about:plugins");
+      frame.addEventListener("load", function () {
+          frame.removeEventListener("load", arguments.callee, false);
+
+          is(frame.contentDocument.dir, "rtl", "about:plugins should be RTL in RTL locales");
+
+          // cleanup
+          dirSvc.unregisterProvider(provider);
+          cr.checkForNewChrome();
+          cr.refreshSkins();
+          prefs.clearUserPref(XUL_CACHE_PREF);
+          SimpleTest.finish();
+        }, false);
+      document.documentElement.appendChild(frame);
+
+    ]]>
+  </script>
+
+</window>
