Make some DOM members private.

I have not reviewed if this is the complete set of members that can be made
private.

src/components/script/dom/attr.rs

@@ -58,7 +58,7 @@ impl AttrValue {
 
 #[deriving(Encodable)]
 pub struct Attr {
-    pub reflector_: Reflector,
+    reflector_: Reflector,
     pub local_name: DOMString,
     value: Traceable<RefCell<AttrValue>>,
     pub name: DOMString,
@@ -66,7 +66,7 @@ pub struct Attr {
     pub prefix: Option<DOMString>,
 
     /// the element that owns this attribute.
-    pub owner: Cell<JS<Element>>,
+    owner: Cell<JS<Element>>,
 }
 
 impl Reflectable for Attr {

src/components/script/dom/attrlist.rs

@@ -11,9 +11,9 @@ use dom::window::Window;
 
 #[deriving(Encodable)]
 pub struct AttrList {
-    pub reflector_: Reflector,
+    reflector_: Reflector,
-    pub window: JS<Window>,
+    window: JS<Window>,
-    pub owner: JS<Element>,
+    owner: JS<Element>,
 }
 
 impl AttrList {

src/components/script/dom/bindings/callback.rs

@@ -108,8 +108,8 @@ pub fn WrapCallThisObject<T: Reflectable>(cx: *mut JSContext,
 }
 
 pub struct CallSetup {
-    pub cx: *mut JSContext,
+    cx: *mut JSContext,
-    pub handling: ExceptionHandling
+    handling: ExceptionHandling
 }
 
 impl CallSetup {

src/components/script/dom/blob.rs

@@ -17,9 +17,9 @@ pub enum BlobType {
 
 #[deriving(Encodable)]
 pub struct Blob {
-    pub reflector_: Reflector,
+    reflector_: Reflector,
-    pub window: JS<Window>,
+    window: JS<Window>,
-    pub type_: BlobType
+    type_: BlobType
 }
 
 impl Blob {
@@ -58,4 +58,4 @@ impl FileDerived for Blob {
             _ => false
         }
     }
-}
+}

src/components/script/dom/clientrect.rs

@@ -10,12 +10,12 @@ use servo_util::geometry::Au;
 
 #[deriving(Encodable)]
 pub struct ClientRect {
-    pub reflector_: Reflector,
+    reflector_: Reflector,
-    pub top: f32,
+    top: f32,
-    pub bottom: f32,
+    bottom: f32,
-    pub left: f32,
+    left: f32,
-    pub right: f32,
+    right: f32,
-    pub window: JS<Window>,
+    window: JS<Window>,
 }
 
 impl ClientRect {

src/components/script/dom/clientrectlist.rs

@@ -10,9 +10,9 @@ use dom::window::Window;
 
 #[deriving(Encodable)]
 pub struct ClientRectList {
-    pub reflector_: Reflector,
+    reflector_: Reflector,
-    pub rects: Vec<JS<ClientRect>>,
+    rects: Vec<JS<ClientRect>>,
-    pub window: JS<Window>,
+    window: JS<Window>,
 }
 
 impl ClientRectList {

src/components/script/dom/document.rs

@@ -61,15 +61,15 @@ pub enum IsHTMLDocument {
 #[deriving(Encodable)]
 pub struct Document {
     pub node: Node,
-    pub reflector_: Reflector,
+    reflector_: Reflector,
     pub window: JS<Window>,
     idmap: Traceable<RefCell<HashMap<DOMString, Vec<JS<Element>>>>>,
-    pub implementation: Cell<Option<JS<DOMImplementation>>>,
+    implementation: Cell<Option<JS<DOMImplementation>>>,
-    pub content_type: DOMString,
+    content_type: DOMString,
     pub encoding_name: Traceable<RefCell<DOMString>>,
     pub is_html_document: bool,
-    pub url: Untraceable<Url>,
+    url: Untraceable<Url>,
-    pub quirks_mode: Untraceable<Cell<QuirksMode>>,
+    quirks_mode: Untraceable<Cell<QuirksMode>>,
 }
 
 impl DocumentDerived for EventTarget {

src/components/script/dom/domimplementation.rs

@@ -21,8 +21,8 @@ use servo_util::str::DOMString;
 
 #[deriving(Encodable)]
 pub struct DOMImplementation {
-    pub owner: JS<Window>,
+    owner: JS<Window>,
-    pub reflector_: Reflector,
+    reflector_: Reflector,
 }
 
 impl DOMImplementation {

src/components/script/dom/event.rs

@@ -48,7 +48,7 @@ pub enum EventTypeId {
 #[deriving(Encodable)]
 pub struct Event {
     pub type_id: EventTypeId,
-    pub reflector_: Reflector,
+    reflector_: Reflector,
     pub current_target: Cell<Option<JS<EventTarget>>>,
     pub target: Cell<Option<JS<EventTarget>>>,
     type_: Traceable<RefCell<DOMString>>,
@@ -61,7 +61,7 @@ pub struct Event {
     pub trusted: Traceable<Cell<bool>>,
     pub dispatching: Traceable<Cell<bool>>,
     pub initialized: Traceable<Cell<bool>>,
-    pub timestamp: u64,
+    timestamp: u64,
 }
 
 impl Event {

src/components/script/dom/eventtarget.rs

@@ -60,7 +60,7 @@ pub struct EventListenerEntry {
 #[deriving(Encodable)]
 pub struct EventTarget {
     pub type_id: EventTargetTypeId,
-    pub reflector_: Reflector,
+    reflector_: Reflector,
     handlers: Traceable<RefCell<HashMap<DOMString, Vec<EventListenerEntry>>>>,
 }
 

src/components/script/dom/formdata.rs

@@ -25,10 +25,10 @@ pub enum FormDatum {
 
 #[deriving(Encodable)]
 pub struct FormData {
-    pub data: Traceable<RefCell<HashMap<DOMString, Vec<FormDatum>>>>,
+    data: Traceable<RefCell<HashMap<DOMString, Vec<FormDatum>>>>,
-    pub reflector_: Reflector,
+    reflector_: Reflector,
-    pub window: JS<Window>,
+    window: JS<Window>,
-    pub form: Option<JS<HTMLFormElement>>
+    form: Option<JS<HTMLFormElement>>
 }
 
 impl FormData {

src/components/script/dom/htmlcollection.rs

@@ -32,9 +32,9 @@ pub enum CollectionTypeId {
 
 #[deriving(Encodable)]
 pub struct HTMLCollection {
-    pub collection: CollectionTypeId,
+    collection: CollectionTypeId,
-    pub reflector_: Reflector,
+    reflector_: Reflector,
-    pub window: JS<Window>,
+    window: JS<Window>,
 }
 
 impl HTMLCollection {

src/components/script/dom/location.rs

@@ -16,8 +16,8 @@ use serialize::{Encoder, Encodable};
 
 #[deriving(Encodable)]
 pub struct Location {
-    pub reflector_: Reflector, //XXXjdm cycle: window->Location->window
+    reflector_: Reflector, //XXXjdm cycle: window->Location->window
-    pub page: Rc<Page>,
+    page: Rc<Page>,
 }
 
 impl Location {

src/components/script/dom/nodelist.rs

@@ -16,9 +16,9 @@ pub enum NodeListType {
 
 #[deriving(Encodable)]
 pub struct NodeList {
-    pub list_type: NodeListType,
+    list_type: NodeListType,
-    pub reflector_: Reflector,
+    reflector_: Reflector,
-    pub window: JS<Window>
+    window: JS<Window>
 }
 
 impl NodeList {

src/components/script/dom/performance.rs

@@ -13,8 +13,8 @@ pub type DOMHighResTimeStamp = f64;
 
 #[deriving(Encodable)]
 pub struct Performance {
-    pub reflector_: Reflector,
+    reflector_: Reflector,
-    pub timing: JS<PerformanceTiming>,
+    timing: JS<PerformanceTiming>,
 }
 
 impl Performance {

src/components/script/dom/performancetiming.rs

@@ -9,9 +9,9 @@ use dom::window::Window;
 
 #[deriving(Encodable)]
 pub struct PerformanceTiming {
-    pub reflector_: Reflector,
+    reflector_: Reflector,
-    pub navigationStart: u64,
+    navigationStart: u64,
-    pub navigationStartPrecise: f64,
+    navigationStartPrecise: f64,
 }
 
 impl PerformanceTiming {

src/components/script/dom/uievent.rs

@@ -18,8 +18,8 @@ use std::cell::Cell;
 #[deriving(Encodable)]
 pub struct UIEvent {
     pub event: Event,
-    pub view: Cell<Option<JS<Window>>>,
+    view: Cell<Option<JS<Window>>>,
-    pub detail: Traceable<Cell<i32>>
+    detail: Traceable<Cell<i32>>
 }
 
 impl UIEventDerived for Event {

src/components/script/dom/validitystate.rs

@@ -10,9 +10,9 @@ use std::cell::Cell;
 
 #[deriving(Encodable)]
 pub struct ValidityState {
-    pub reflector_: Reflector,
+    reflector_: Reflector,
-    pub window: Cell<JS<Window>>,
+    window: Cell<JS<Window>>,
-    pub state: u8,
+    state: u8,
 }
 
 impl ValidityState {

src/components/script/dom/window.rs

@@ -48,9 +48,9 @@ pub struct TimerId(i32);
 
 #[deriving(Encodable)]
 pub struct TimerHandle {
-    pub handle: TimerId,
+    handle: TimerId,
     pub data: TimerData,
-    pub cancel_chan: Untraceable<Option<Sender<()>>>,
+    cancel_chan: Untraceable<Option<Sender<()>>>,
 }
 
 impl Hash for TimerId {
@@ -68,18 +68,18 @@ impl TimerHandle {
 
 #[deriving(Encodable)]
 pub struct Window {
-    pub eventtarget: EventTarget,
+    eventtarget: EventTarget,
     pub script_chan: ScriptChan,
-    pub console: Cell<Option<JS<Console>>>,
+    console: Cell<Option<JS<Console>>>,
-    pub location: Cell<Option<JS<Location>>>,
+    location: Cell<Option<JS<Location>>>,
-    pub navigator: Cell<Option<JS<Navigator>>>,
+    navigator: Cell<Option<JS<Navigator>>>,
     pub image_cache_task: ImageCacheTask,
     pub active_timers: Traceable<RefCell<HashMap<TimerId, TimerHandle>>>,
-    pub next_timer_handle: Traceable<Cell<i32>>,
+    next_timer_handle: Traceable<Cell<i32>>,
-    pub compositor: Untraceable<Box<ScriptListener>>,
+    compositor: Untraceable<Box<ScriptListener>>,
     pub browser_context: Traceable<RefCell<Option<BrowserContext>>>,
     pub page: Rc<Page>,
-    pub performance: Cell<Option<JS<Performance>>>,
+    performance: Cell<Option<JS<Performance>>>,
     pub navigationStart: u64,
     pub navigationStartPrecise: f64,
 }