Implement the LayoutData getters on LayoutJS<Node> rather than Node itself.

2 files changed, 8 insertions, 12 deletions

diff --git a/components/layout/data.rs b/components/layout/data.rs
index 9146e0110c4..f679435a91f 100644
--- a/components/layout/data.rs
+++ b/components/layout/data.rs
@@ -8,7 +8,7 @@ use construct::{ConstructionItem, ConstructionResult};
 use incremental::RestyleDamage;
 use msg::constellation_msg::ConstellationChan;
 use parallel::DomParallelInfo;
-use script::dom::node::SharedLayoutData;
+use script::dom::node::{LayoutNodeHelpers, SharedLayoutData};
 use script::layout_interface::LayoutChan;
 use std::cell::{Ref, RefMut};
 use std::mem;
@@ -116,20 +116,20 @@ pub trait LayoutDataAccess {
 impl<'ln> LayoutDataAccess for LayoutNode<'ln> {
     #[inline(always)]
     unsafe fn borrow_layout_data_unchecked(&self) -> *const Option<LayoutDataWrapper> {
-        mem::transmute(self.get().layout_data_unchecked())
+        mem::transmute(self.get_jsmanaged().layout_data_unchecked())
     }
 
     #[inline(always)]
     fn borrow_layout_data<'a>(&'a self) -> Ref<'a,Option<LayoutDataWrapper>> {
         unsafe {
-            mem::transmute(self.get().layout_data())
+            mem::transmute(self.get_jsmanaged().layout_data())
         }
     }
 
     #[inline(always)]
     fn mutate_layout_data<'a>(&'a self) -> RefMut<'a,Option<LayoutDataWrapper>> {
         unsafe {
-            mem::transmute(self.get().layout_data_mut())
+            mem::transmute(self.get_jsmanaged().layout_data_mut())
         }
     }
 }
diff --git a/components/layout/wrapper.rs b/components/layout/wrapper.rs
index b45162772d3..6db0552d2df 100644
--- a/components/layout/wrapper.rs
+++ b/components/layout/wrapper.rs
@@ -889,11 +889,11 @@ impl<'ln> ThreadSafeLayoutNode<'ln> {
         layout_data_wrapper_ref.data.after_style.is_some()
     }
 
-    /// Borrows the layout data without checking. Fails on a conflicting borrow.
+    /// Borrows the layout data without checking.
     #[inline(always)]
     fn borrow_layout_data_unchecked<'a>(&'a self) -> *const Option<LayoutDataWrapper> {
         unsafe {
-            mem::transmute(self.get().layout_data_unchecked())
+            self.node.borrow_layout_data_unchecked()
         }
     }
 
@@ -902,9 +902,7 @@ impl<'ln> ThreadSafeLayoutNode<'ln> {
     /// TODO(pcwalton): Make this private. It will let us avoid borrow flag checks in some cases.
     #[inline(always)]
     pub fn borrow_layout_data<'a>(&'a self) -> Ref<'a,Option<LayoutDataWrapper>> {
-        unsafe {
-            mem::transmute(self.get().layout_data())
-        }
+        self.node.borrow_layout_data()
     }
 
     /// Borrows the layout data mutably. Fails on a conflicting borrow.
@@ -912,9 +910,7 @@ impl<'ln> ThreadSafeLayoutNode<'ln> {
     /// TODO(pcwalton): Make this private. It will let us avoid borrow flag checks in some cases.
     #[inline(always)]
     pub fn mutate_layout_data<'a>(&'a self) -> RefMut<'a,Option<LayoutDataWrapper>> {
-        unsafe {
-            mem::transmute(self.get().layout_data_mut())
-        }
+        self.node.mutate_layout_data()
     }
 
     /// Traverses the tree in postorder.