aboutsummaryrefslogtreecommitdiffstats
path: root/components/script/dom
diff options
context:
space:
mode:
Diffstat (limited to 'components/script/dom')
-rw-r--r--components/script/dom/browsingcontext.rs27
-rw-r--r--components/script/dom/document.rs19
-rw-r--r--components/script/dom/domimplementation.rs1
-rw-r--r--components/script/dom/domparser.rs2
-rw-r--r--components/script/dom/node.rs3
-rw-r--r--components/script/dom/window.rs9
-rw-r--r--components/script/dom/xmldocument.rs1
-rw-r--r--components/script/dom/xmlhttprequest.rs1
8 files changed, 45 insertions, 18 deletions
diff --git a/components/script/dom/browsingcontext.rs b/components/script/dom/browsingcontext.rs
index 7cd856cad7c..603b2c27f80 100644
--- a/components/script/dom/browsingcontext.rs
+++ b/components/script/dom/browsingcontext.rs
@@ -2,6 +2,7 @@
* 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/. */
+use dom::bindings::cell::DOMRefCell;
use dom::bindings::conversions::{ToJSValConvertible, root_from_handleobject};
use dom::bindings::js::{JS, Root, RootedReference};
use dom::bindings::proxyhandler::{fill_property_descriptor, get_property_descriptor};
@@ -25,26 +26,24 @@ use js::jsval::{ObjectValue, UndefinedValue, PrivateValue};
#[dom_struct]
pub struct BrowsingContext {
reflector: Reflector,
- history: Vec<SessionHistoryEntry>,
+ history: DOMRefCell<Vec<SessionHistoryEntry>>,
active_index: usize,
frame_element: Option<JS<Element>>,
}
impl BrowsingContext {
- pub fn new_inherited(document: &Document, frame_element: Option<&Element>) -> BrowsingContext {
+ pub fn new_inherited(frame_element: Option<&Element>) -> BrowsingContext {
BrowsingContext {
reflector: Reflector::new(),
- history: vec![SessionHistoryEntry::new(document)],
+ history: DOMRefCell::new(vec![]),
active_index: 0,
frame_element: frame_element.map(JS::from_ref),
}
}
#[allow(unsafe_code)]
- pub fn new(document: &Document, frame_element: Option<&Element>) -> Root<BrowsingContext> {
+ pub fn new(window: &Window, frame_element: Option<&Element>) -> Root<BrowsingContext> {
unsafe {
- let window = document.window();
-
let WindowProxyHandler(handler) = window.windowproxy_handler();
assert!(!handler.is_null());
@@ -58,7 +57,7 @@ impl BrowsingContext {
NewWindowProxy(cx, parent, handler));
assert!(!window_proxy.ptr.is_null());
- let object = box BrowsingContext::new_inherited(document, frame_element);
+ let object = box BrowsingContext::new_inherited(frame_element);
let raw = Box::into_raw(object);
SetProxyExtra(window_proxy.ptr, 0, PrivateValue(raw as *const _));
@@ -69,12 +68,18 @@ impl BrowsingContext {
}
}
- pub fn active_document(&self) -> &Document {
- &*self.history[self.active_index].document
+ pub fn init(&self, document: &Document) {
+ assert!(self.history.borrow().is_empty());
+ assert_eq!(self.active_index, 0);
+ self.history.borrow_mut().push(SessionHistoryEntry::new(document));
+ }
+
+ pub fn active_document(&self) -> Root<Document> {
+ Root::from_ref(&*self.history.borrow()[self.active_index].document)
}
- pub fn active_window(&self) -> &Window {
- self.active_document().window()
+ pub fn active_window(&self) -> Root<Window> {
+ Root::from_ref(self.active_document().window())
}
pub fn frame_element(&self) -> Option<&Element> {
diff --git a/components/script/dom/document.rs b/components/script/dom/document.rs
index c77eaf7c2c7..9fdff62aabb 100644
--- a/components/script/dom/document.rs
+++ b/components/script/dom/document.rs
@@ -30,6 +30,7 @@ use dom::bindings::reflector::{Reflectable, reflect_dom_object};
use dom::bindings::trace::RootedVec;
use dom::bindings::xmlname::XMLName::InvalidXMLName;
use dom::bindings::xmlname::{validate_and_extract, namespace_from_domstring, xml_name_type};
+use dom::browsingcontext::BrowsingContext;
use dom::comment::Comment;
use dom::customevent::CustomEvent;
use dom::documentfragment::DocumentFragment;
@@ -129,6 +130,8 @@ enum ParserBlockedByScript {
pub struct Document {
node: Node,
window: JS<Window>,
+ /// https://html.spec.whatwg.org/multipage/#concept-document-bc
+ browsing_context: Option<JS<BrowsingContext>>,
implementation: MutNullableHeap<JS<DOMImplementation>>,
location: MutNullableHeap<JS<Location>>,
content_type: DOMString,
@@ -279,6 +282,12 @@ impl Document {
self.loader.borrow_mut()
}
+ /// https://html.spec.whatwg.org/multipage/#concept-document-bc
+ #[inline]
+ pub fn browsing_context(&self) -> Option<&BrowsingContext> {
+ self.browsing_context.as_ref().map(|browsing_context| &**browsing_context)
+ }
+
#[inline]
pub fn window(&self) -> &Window {
&*self.window
@@ -309,7 +318,7 @@ impl Document {
let browsing_context = browsing_context.as_ref().unwrap();
let active_document = browsing_context.active_document();
- if self != active_document {
+ if self != &*active_document {
return false;
}
// FIXME: It should also check whether the browser context is top-level or not
@@ -1507,6 +1516,7 @@ impl LayoutDocumentHelpers for LayoutJS<Document> {
impl Document {
pub fn new_inherited(window: &Window,
+ browsing_context: Option<&BrowsingContext>,
url: Option<Url>,
is_html_document: IsHTMLDocument,
content_type: Option<DOMString>,
@@ -1525,6 +1535,7 @@ impl Document {
Document {
node: Node::new_document_node(),
window: JS::from_ref(window),
+ browsing_context: browsing_context.map(JS::from_ref),
implementation: Default::default(),
location: Default::default(),
content_type: match content_type {
@@ -1594,6 +1605,7 @@ impl Document {
let docloader = DocumentLoader::new(&*doc.loader());
Ok(Document::new(win,
None,
+ None,
IsHTMLDocument::NonHTMLDocument,
None,
None,
@@ -1602,6 +1614,7 @@ impl Document {
}
pub fn new(window: &Window,
+ browsing_context: Option<&BrowsingContext>,
url: Option<Url>,
doctype: IsHTMLDocument,
content_type: Option<DOMString>,
@@ -1610,6 +1623,7 @@ impl Document {
doc_loader: DocumentLoader)
-> Root<Document> {
let document = reflect_dom_object(box Document::new_inherited(window,
+ browsing_context,
url,
doctype,
content_type,
@@ -1673,6 +1687,7 @@ impl Document {
};
let new_doc = Document::new(self.window(),
None,
+ None,
doctype,
None,
None,
@@ -1761,7 +1776,7 @@ impl DocumentMethods for Document {
// Step 2.
let candidate = browsing_context.active_document();
// Step 3.
- if candidate == target {
+ if &*candidate == target {
true
} else {
false //TODO Step 4.
diff --git a/components/script/dom/domimplementation.rs b/components/script/dom/domimplementation.rs
index d994b111b69..01cd8acb7df 100644
--- a/components/script/dom/domimplementation.rs
+++ b/components/script/dom/domimplementation.rs
@@ -116,6 +116,7 @@ impl DOMImplementationMethods for DOMImplementation {
// Step 1-2.
let doc = Document::new(win,
None,
+ None,
IsHTMLDocument::HTMLDocument,
None,
None,
diff --git a/components/script/dom/domparser.rs b/components/script/dom/domparser.rs
index 21b0fcf0730..1a3fb7f3793 100644
--- a/components/script/dom/domparser.rs
+++ b/components/script/dom/domparser.rs
@@ -59,6 +59,7 @@ impl DOMParserMethods for DOMParser {
match ty {
Text_html => {
let document = Document::new(&self.window,
+ None,
Some(url.clone()),
IsHTMLDocument::HTMLDocument,
Some(content_type),
@@ -72,6 +73,7 @@ impl DOMParserMethods for DOMParser {
Text_xml => {
// FIXME: this should probably be FromParser when we actually parse the string (#3756).
let document = Document::new(&self.window,
+ None,
Some(url.clone()),
IsHTMLDocument::NonHTMLDocument,
Some(content_type),
diff --git a/components/script/dom/node.rs b/components/script/dom/node.rs
index ac96158f7e5..c2221408b13 100644
--- a/components/script/dom/node.rs
+++ b/components/script/dom/node.rs
@@ -1605,7 +1605,8 @@ impl Node {
};
let window = document.window();
let loader = DocumentLoader::new(&*document.loader());
- let document = Document::new(window, Some((*document.url()).clone()),
+ let document = Document::new(window, None,
+ Some((*document.url()).clone()),
is_html_doc, None,
None, DocumentSource::NotFromParser, loader);
Root::upcast::<Node>(document)
diff --git a/components/script/dom/window.rs b/components/script/dom/window.rs
index 4f55942111a..5e933ec6868 100644
--- a/components/script/dom/window.rs
+++ b/components/script/dom/window.rs
@@ -426,7 +426,7 @@ impl WindowMethods for Window {
// https://html.spec.whatwg.org/multipage/#dom-document-2
fn Document(&self) -> Root<Document> {
- Root::from_ref(self.browsing_context().as_ref().unwrap().active_document())
+ self.browsing_context().as_ref().unwrap().active_document()
}
// https://html.spec.whatwg.org/multipage/#dom-location
@@ -1095,8 +1095,9 @@ impl Window {
(element, response.rect)
}
- pub fn init_browsing_context(&self, doc: &Document, frame_element: Option<&Element>) {
- self.browsing_context.set(Some(&BrowsingContext::new(doc, frame_element)));
+ pub fn init_browsing_context(&self, browsing_context: &BrowsingContext) {
+ assert!(self.browsing_context.get().is_none());
+ self.browsing_context.set(Some(&browsing_context));
}
/// Commence a new URL load which will either replace this window or scroll to a fragment.
@@ -1283,7 +1284,7 @@ impl Window {
browsing_context.frame_element().map(|frame_element| {
let window = window_from_node(frame_element);
let context = window.browsing_context();
- Root::from_ref(context.unwrap().active_window())
+ context.unwrap().active_window()
})
}
}
diff --git a/components/script/dom/xmldocument.rs b/components/script/dom/xmldocument.rs
index 6f73ecbf8ea..200480d4fc0 100644
--- a/components/script/dom/xmldocument.rs
+++ b/components/script/dom/xmldocument.rs
@@ -35,6 +35,7 @@ impl XMLDocument {
doc_loader: DocumentLoader) -> XMLDocument {
XMLDocument {
document: Document::new_inherited(window,
+ None,
url,
is_html_document,
content_type,
diff --git a/components/script/dom/xmlhttprequest.rs b/components/script/dom/xmlhttprequest.rs
index eb565c8bf5c..140ad159792 100644
--- a/components/script/dom/xmlhttprequest.rs
+++ b/components/script/dom/xmlhttprequest.rs
@@ -1230,6 +1230,7 @@ impl XMLHttpRequest {
DOMString::from(format!("{}", mime))
});
Document::new(win,
+ None,
parsed_url,
is_html_document,
content_type,
generated by cgit v1.2.3 (git 2.25.1) at 2025-05-01 18:49:21 +0000