From b2a1045016f62063ccf8b119ec46bd4c24b058e3 Mon Sep 17 00:00:00 2001
From: Tim Taubert <ttaubert@mozilla.com>
Date: Fri, 6 Dec 2013 15:08:15 +0100
Subject: [PATCH 1/4] Bug 947216 - Remove special case for about:config's form
 data collection r=smacleod

From 1c6cfab02eab4fcb0fb3200a4c12c760ae411a77 Mon Sep 17 00:00:00 2001
---
 .../components/sessionstore/src/DocumentUtils.jsm   |  3 ++-
 .../sessionstore/src/TextAndScrollData.jsm          |  7 -------
 .../components/sessionstore/src/XPathGenerator.jsm  | 13 +++++++++++--
 3 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/browser/components/sessionstore/src/DocumentUtils.jsm b/browser/components/sessionstore/src/DocumentUtils.jsm
index 32202dab3c22..af4328144bb1 100644
--- a/browser/components/sessionstore/src/DocumentUtils.jsm
+++ b/browser/components/sessionstore/src/DocumentUtils.jsm
@@ -55,7 +55,8 @@ this.DocumentUtils = {
       }
 
       if (node instanceof Ci.nsIDOMHTMLInputElement ||
-          node instanceof Ci.nsIDOMHTMLTextAreaElement) {
+          node instanceof Ci.nsIDOMHTMLTextAreaElement ||
+          node instanceof Ci.nsIDOMXULTextBoxElement) {
         switch (node.type) {
           case "checkbox":
           case "radio":
diff --git a/browser/components/sessionstore/src/TextAndScrollData.jsm b/browser/components/sessionstore/src/TextAndScrollData.jsm
index 042bc2145056..b0c52dac800d 100644
--- a/browser/components/sessionstore/src/TextAndScrollData.jsm
+++ b/browser/components/sessionstore/src/TextAndScrollData.jsm
@@ -86,13 +86,6 @@ let TextAndScrollDataInternal = {
     let scrollX = {}, scrollY = {};
     domWindowUtils.getScrollXY(false, scrollX, scrollY);
     entry.scroll = scrollX.value + "," + scrollY.value;
-
-    if (topURL == "about:config") {
-      entry.formdata = {
-        id: { "textbox": content.top.document.getElementById("textbox").value },
-        xpath: {}
-      };
-    }
   },
 
   isAboutSessionRestore: function (url) {
diff --git a/browser/components/sessionstore/src/XPathGenerator.jsm b/browser/components/sessionstore/src/XPathGenerator.jsm
index 15b56fe1ce86..19222c126c54 100644
--- a/browser/components/sessionstore/src/XPathGenerator.jsm
+++ b/browser/components/sessionstore/src/XPathGenerator.jsm
@@ -8,8 +8,14 @@ this.EXPORTED_SYMBOLS = ["XPathGenerator"];
 
 this.XPathGenerator = {
   // these two hashes should be kept in sync
-  namespaceURIs:     { "xhtml": "http://www.w3.org/1999/xhtml" },
-  namespacePrefixes: { "http://www.w3.org/1999/xhtml": "xhtml" },
+  namespaceURIs:     {
+    "xhtml": "http://www.w3.org/1999/xhtml",
+    "xul": "http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
+  },
+  namespacePrefixes: {
+    "http://www.w3.org/1999/xhtml": "xhtml",
+    "http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul": "xul"
+  },
 
   /**
    * Generates an approximate XPath query to an (X)HTML node
@@ -93,6 +99,9 @@ this.XPathGenerator = {
     let formNodesXPath = "//textarea|//select|//xhtml:textarea|//xhtml:select|" +
       "//input[" + ignore + "]|//xhtml:input[" + ignore + "]";
 
+    // Special case for about:config's search field.
+    formNodesXPath += '|/xul:window[@id="config"]//xul:textbox[@id="textbox"]';
+
     delete this.restorableFormNodes;
     return (this.restorableFormNodes = formNodesXPath);
   }

From fd75b335c57261d6eb6d71d9f6aa19eacff762d1 Mon Sep 17 00:00:00 2001
From: Tim Taubert <ttaubert@mozilla.com>
Date: Fri, 6 Dec 2013 16:54:43 +0100
Subject: [PATCH 2/4] Bug 947262 - Don't dispatch a change event when trying to
 restore an invalid <select> value r=smacleod

From 25957e7878d88aab42b99a227ad6a0a5cc301ccc Mon Sep 17 00:00:00 2001
---
 browser/components/sessionstore/src/DocumentUtils.jsm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/browser/components/sessionstore/src/DocumentUtils.jsm b/browser/components/sessionstore/src/DocumentUtils.jsm
index af4328144bb1..310628e89c29 100644
--- a/browser/components/sessionstore/src/DocumentUtils.jsm
+++ b/browser/components/sessionstore/src/DocumentUtils.jsm
@@ -203,10 +203,10 @@ this.DocumentUtils = {
       for (let i = 0; i < aNode.options.length; i++) {
         if (aNode.options[i].value == aValue.value) {
           aNode.selectedIndex = i;
+          eventType = "change";
           break;
         }
       }
-      eventType = "change";
     } else if (aValue && aValue.fileList && aValue.type == "file" &&
       aNode.type == "file") {
       aNode.mozSetFileNameArray(aValue.fileList, aValue.fileList.length);

From 175692b3d170cce63bbd04e7dfef76c6bd5ac0f4 Mon Sep 17 00:00:00 2001
From: Mark Hammond <mhammond@skippinet.com.au>
Date: Tue, 10 Dec 2013 09:24:37 +1100
Subject: [PATCH 3/4] Bug 947770 - Move remaining tests in
 browser/devtools/webconsole/test/Makefile.in to manifest. r=ted

---
 browser/devtools/webconsole/test/Makefile.in | 16 ----------------
 browser/devtools/webconsole/test/browser.ini |  4 ++++
 2 files changed, 4 insertions(+), 16 deletions(-)
 delete mode 100644 browser/devtools/webconsole/test/Makefile.in

diff --git a/browser/devtools/webconsole/test/Makefile.in b/browser/devtools/webconsole/test/Makefile.in
deleted file mode 100644
index e762e23300e6..000000000000
--- a/browser/devtools/webconsole/test/Makefile.in
+++ /dev/null
@@ -1,16 +0,0 @@
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-ifeq ($(OS_ARCH), Darwin)
-MOCHITEST_BROWSER_FILES += \
-	browser_webconsole_bug_804845_ctrl_key_nav.js \
-        $(NULL)
-endif
-
-ifeq ($(OS_ARCH),WINNT)
-MOCHITEST_BROWSER_FILES += \
-	browser_webconsole_bug_623749_ctrl_a_select_all_winnt.js \
-	$(NULL)
-endif
-
diff --git a/browser/devtools/webconsole/test/browser.ini b/browser/devtools/webconsole/test/browser.ini
index 6374c111f012..56806b33c661 100644
--- a/browser/devtools/webconsole/test/browser.ini
+++ b/browser/devtools/webconsole/test/browser.ini
@@ -185,6 +185,8 @@ support-files =
 [browser_webconsole_bug_618311_close_panels.js]
 [browser_webconsole_bug_621644_jsterm_dollar.js]
 [browser_webconsole_bug_622303_persistent_filters.js]
+[browser_webconsole_bug_623749_ctrl_a_select_all_winnt.js]
+run-if = os == "win"
 [browser_webconsole_bug_630733_response_redirect_headers.js]
 [browser_webconsole_bug_632275_getters_document_width.js]
 [browser_webconsole_bug_632347_iterators_generators.js]
@@ -210,6 +212,8 @@ support-files =
 [browser_webconsole_bug_770099_bad_policyuri.js]
 [browser_webconsole_bug_770099_violation.js]
 [browser_webconsole_bug_782653_CSS_links_in_Style_Editor.js]
+[browser_webconsole_bug_804845_ctrl_key_nav.js]
+run-if = os == "mac"
 [browser_webconsole_bug_817834_add_edited_input_to_history.js]
 [browser_webconsole_bug_821877_csp_errors.js]
 [browser_webconsole_bug_837351_securityerrors.js]

From 9507c17040dfd57e49d9eead110f82f47fa1e866 Mon Sep 17 00:00:00 2001
From: Mark Hammond <mhammond@skippinet.com.au>
Date: Tue, 10 Dec 2013 09:27:06 +1100
Subject: [PATCH 4/4] Bug 947764 - correct support-files for devtools
 commandline tests.  r=ted

---
 browser/devtools/commandline/test/browser.ini | 43 +++++++++++--------
 1 file changed, 25 insertions(+), 18 deletions(-)

diff --git a/browser/devtools/commandline/test/browser.ini b/browser/devtools/commandline/test/browser.ini
index ec9cccd1e0ca..025fdb95d693 100644
--- a/browser/devtools/commandline/test/browser.ini
+++ b/browser/devtools/commandline/test/browser.ini
@@ -10,33 +10,40 @@ skip-if = true # Bug 845831
 [browser_cmd_calllog_chrome.js]
 skip-if = true # Bug 845831
 [browser_cmd_appcache_invalid.js]
-[browser_cmd_appcache_invalid_appcache.appcache]
-[browser_cmd_appcache_invalid_appcache.appcache^headers^]
-[browser_cmd_appcache_invalid_index.html]
-[browser_cmd_appcache_invalid_page1.html]
-[browser_cmd_appcache_invalid_page2.html]
-[browser_cmd_appcache_invalid_page3.html]
-[browser_cmd_appcache_invalid_page3.html^headers^]
+support-files =
+  browser_cmd_appcache_invalid_appcache.appcache
+  browser_cmd_appcache_invalid_appcache.appcache^headers^
+  browser_cmd_appcache_invalid_index.html
+  browser_cmd_appcache_invalid_page1.html
+  browser_cmd_appcache_invalid_page2.html
+  browser_cmd_appcache_invalid_page3.html
+  browser_cmd_appcache_invalid_page3.html^headers^
 [browser_cmd_appcache_valid.js]
-[browser_cmd_appcache_valid_appcache.appcache]
-[browser_cmd_appcache_valid_appcache.appcache^headers^]
-[browser_cmd_appcache_valid_index.html]
-[browser_cmd_appcache_valid_page1.html]
-[browser_cmd_appcache_valid_page2.html]
-[browser_cmd_appcache_valid_page3.html]
+support-files =
+  browser_cmd_appcache_valid_appcache.appcache
+  browser_cmd_appcache_valid_appcache.appcache^headers^
+  browser_cmd_appcache_valid_index.html
+  browser_cmd_appcache_valid_page1.html
+  browser_cmd_appcache_valid_page2.html
+  browser_cmd_appcache_valid_page3.html
 [browser_cmd_commands.js]
-[browser_cmd_cookie.html]
 [browser_cmd_cookie.js]
+support-files =
+ browser_cmd_cookie.html
 [browser_cmd_jsb.js]
-[browser_cmd_jsb_script.jsi]
-[browser_cmd_media.html]
+support-files =
+  browser_cmd_jsb_script.jsi
 [browser_cmd_media.js]
-[browser_cmd_pagemod_export.html]
+support-files =
+  browser_cmd_media.html
 [browser_cmd_pagemod_export.js]
+support-files =
+  browser_cmd_pagemod_export.html
 [browser_cmd_pref.js]
 [browser_cmd_restart.js]
-[browser_cmd_screenshot.html]
 [browser_cmd_screenshot.js]
+support-files =
+  browser_cmd_screenshot.html
 [browser_cmd_settings.js]
 [browser_gcli_async.js]
 [browser_gcli_canon.js]