Backing out bug 990685 as it changes some assumptions for XBL bindings.
authorDave Townsend <dtownsend@oxymoronical.com>
Thu, 24 Apr 2014 13:05:38 -0700
changeset 180534 6bd6ecc2185c177fdf5f1bb70a921448167366fe
parent 180533 a9cddc54b6f4a31ec9bad21125c3cd54247e68ef
child 180535 0e0d3955de465a48c58e123d084877cd6525497f
push id272
push userpvanderbeken@mozilla.com
push dateMon, 05 May 2014 16:31:18 +0000
bugs990685
milestone31.0a1
Backing out bug 990685 as it changes some assumptions for XBL bindings.
content/xul/document/src/XULDocument.cpp
content/xul/document/test/chrome.ini
content/xul/document/test/test_documentnotification.xul
content/xul/document/test/window_documentnotification.xul
--- a/content/xul/document/src/XULDocument.cpp
+++ b/content/xul/document/src/XULDocument.cpp
@@ -2480,18 +2480,16 @@ XULDocument::PrepareToWalk()
         if (NS_FAILED(rv)) return rv;
         
         rv = AddElementToRefMap(root);
         if (NS_FAILED(rv)) return rv;
 
         // Block onload until we've finished building the complete
         // document content model.
         BlockOnload();
-
-        nsContentSink::NotifyDocElementCreated(this);
     }
 
     // There'd better not be anything on the context stack at this
     // point! This is the basis case for our "induction" in
     // ResumeWalk(), below, which'll assume that there's always a
     // content element on the context stack if either 1) we're in the
     // "master" document, or 2) we're in an overlay, and we've got
     // more than one prototype element (the single, root "overlay"
--- a/content/xul/document/test/chrome.ini
+++ b/content/xul/document/test/chrome.ini
@@ -1,24 +1,22 @@
 [DEFAULT]
 support-files =
   bug497875-iframe.xul
   overlay1_bug335375.xul
   overlay2_bug335375.xul
   window_bug583948.xul
   window_bug757137.xul
-  window_documentnotification.xul
 
 [test_bug199692.xul]
 [test_bug311681.xul]
 [test_bug335375.xul]
 [test_bug391002.xul]
 [test_bug403868.xul]
 [test_bug414907.xul]
 [test_bug418216.xul]
 [test_bug445177.xul]
 [test_bug449457.xul]
 [test_bug468176.xul]
 [test_bug497875.xul]
 [test_bug583948.xul]
 [test_bug640158_overlay_persist.xul]
 [test_bug757137.xul]
-[test_documentnotification.xul]
deleted file mode 100644
--- a/content/xul/document/test/test_documentnotification.xul
+++ /dev/null
@@ -1,40 +0,0 @@
-<?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"?>
-
-<window xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
-
-<script type="application/javascript" src="chrome://mochikit/content/tests/SimpleTest/SimpleTest.js" />
-
-<body xmlns="http://www.w3.org/1999/xhtml">
-  <div id="content" style="display: none"/>
-</body>
-
-<script>
-SimpleTest.waitForExplicitFinish();
-
-var seenNotification = false;
-function notify(subject, topic, data) {
-  seenNotification = true;
-  is(topic, "document-element-inserted", "Should be the right notification");
-  is(subject, otherWindow.document, "Should have been notified about the right window");
-  ok(subject.documentElement, "documentElement should be defined");
-}
-
-var obs = Components.classes["@mozilla.org/observer-service;1"].
-          getService(Components.interfaces.nsIObserverService)
-obs.addObserver(notify, "document-element-inserted", false);
-
-var otherWindow = window.open("window_documentnotification.xul", "_new", "chrome");
-otherWindow.addEventListener("load", function() {
-  ok(seenNotification, "Should have seen the document-element-inserted")
-  obs.removeObserver(notify, "document-element-inserted");
-  window.close();
-  SimpleTest.waitForFocus(function() {
-    SimpleTest.finish();
-  });
-});
-</script>
-
-</window>
deleted file mode 100644
--- a/content/xul/document/test/window_documentnotification.xul
+++ /dev/null
@@ -1,6 +0,0 @@
-<?xml version="1.0"?>
-<?xml-stylesheet href="chrome://global/skin" type="text/css"?>
-
-<window xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
-  <label value="window_documentnotification.xul"/>
-</window>