From 4b0c4e5ba20a3df76275ab8273aed56e24ca7fc0 Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Thu, 29 Jan 2015 17:11:04 +0100 Subject: [PATCH 1/8] Rename getPropertyDescriptor to get_property_descriptor. --- components/script/dom/bindings/codegen/CodegenRust.py | 4 ++-- components/script/dom/bindings/proxyhandler.rs | 9 +++++---- components/script/dom/browsercontext.rs | 5 +++-- 3 files changed, 10 insertions(+), 8 deletions(-) diff --git a/components/script/dom/bindings/codegen/CodegenRust.py b/components/script/dom/bindings/codegen/CodegenRust.py index 63e58f2c03c..680fba1471b 100644 --- a/components/script/dom/bindings/codegen/CodegenRust.py +++ b/components/script/dom/bindings/codegen/CodegenRust.py @@ -2098,7 +2098,7 @@ class CGDefineProxyHandler(CGAbstractMethod): body = """\ let traps = ProxyTraps { - getPropertyDescriptor: Some(getPropertyDescriptor as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, bool, *mut JSPropertyDescriptor) -> bool), + getPropertyDescriptor: Some(get_property_descriptor as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, bool, *mut JSPropertyDescriptor) -> bool), getOwnPropertyDescriptor: Some(getOwnPropertyDescriptor as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, bool, *mut JSPropertyDescriptor) -> bool), defineProperty: Some(%s as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, *mut JSPropertyDescriptor) -> bool), getOwnPropertyNames: Some(getOwnPropertyNames_ as unsafe extern "C" fn(*mut JSContext, *mut JSObject, *mut AutoIdVector) -> bool), @@ -4582,7 +4582,7 @@ class CGBindingRoot(CGThing): 'dom::bindings::proxyhandler', 'dom::bindings::proxyhandler::{_obj_toString, defineProperty_}', 'dom::bindings::proxyhandler::{FillPropertyDescriptor, GetExpandoObject}', - 'dom::bindings::proxyhandler::{delete_, getPropertyDescriptor}', + 'dom::bindings::proxyhandler::{delete_, get_property_descriptor}', 'dom::bindings::proxyhandler::{getOwnPropertyNames_, enumerate_}', 'dom::bindings::str::ByteString', 'page::JSPageInfo', diff --git a/components/script/dom/bindings/proxyhandler.rs b/components/script/dom/bindings/proxyhandler.rs index 53e69c48e84..fcf2de62aa4 100644 --- a/components/script/dom/bindings/proxyhandler.rs +++ b/components/script/dom/bindings/proxyhandler.rs @@ -31,10 +31,11 @@ static JSPROXYSLOT_EXPANDO: u32 = 0; /// with argument `id` and return the result, if it is not `undefined`. /// Otherwise, walk along the prototype chain to find a property with that /// name. -pub unsafe extern fn getPropertyDescriptor(cx: *mut JSContext, proxy: *mut JSObject, - id: jsid, set: bool, - desc: *mut JSPropertyDescriptor) - -> bool { +pub unsafe extern fn get_property_descriptor(cx: *mut JSContext, + proxy: *mut JSObject, + id: jsid, set: bool, + desc: *mut JSPropertyDescriptor) + -> bool { let handler = GetProxyHandler(proxy); if !InvokeGetOwnPropertyDescriptor(handler, cx, proxy, id, set, desc) { return false; diff --git a/components/script/dom/browsercontext.rs b/components/script/dom/browsercontext.rs index cbe3bcbe63d..16165f68780 100644 --- a/components/script/dom/browsercontext.rs +++ b/components/script/dom/browsercontext.rs @@ -7,7 +7,7 @@ use dom::bindings::conversions::{ToJSValConvertible}; use dom::bindings::js::{JS, JSRef, Temporary, Root}; use dom::bindings::js::{OptionalRootable, OptionalRootedRootable, ResultRootable}; use dom::bindings::js::{OptionalRootedReference, OptionalOptionalRootedRootable}; -use dom::bindings::proxyhandler::{getPropertyDescriptor, FillPropertyDescriptor}; +use dom::bindings::proxyhandler::{get_property_descriptor, FillPropertyDescriptor}; use dom::bindings::utils::{Reflectable, WindowProxyHandler}; use dom::bindings::utils::{GetArrayIndexFromId}; use dom::document::{Document, DocumentHelpers}; @@ -194,7 +194,8 @@ unsafe extern fn set(cx: *mut JSContext, proxy: *mut JSObject, _receiver: *mut J } static PROXY_HANDLER: ProxyTraps = ProxyTraps { - getPropertyDescriptor: Some(getPropertyDescriptor as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, bool, *mut JSPropertyDescriptor) -> bool), + getPropertyDescriptor: Some(get_property_descriptor + as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, bool, *mut JSPropertyDescriptor) -> bool), getOwnPropertyDescriptor: Some(getOwnPropertyDescriptor as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, bool, *mut JSPropertyDescriptor) From d7de6973c7ed07b6b163a7dab582bf49e09223b0 Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Thu, 29 Jan 2015 17:14:21 +0100 Subject: [PATCH 2/8] Rename defineProperty_ to define_property. --- components/script/dom/bindings/codegen/CodegenRust.py | 6 +++--- components/script/dom/bindings/proxyhandler.rs | 5 +++-- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/components/script/dom/bindings/codegen/CodegenRust.py b/components/script/dom/bindings/codegen/CodegenRust.py index 680fba1471b..db06ad603fb 100644 --- a/components/script/dom/bindings/codegen/CodegenRust.py +++ b/components/script/dom/bindings/codegen/CodegenRust.py @@ -2088,7 +2088,7 @@ class CGDefineProxyHandler(CGAbstractMethod): return CGAbstractMethod.define(self) def definition_body(self): - customDefineProperty = 'defineProperty_' + customDefineProperty = 'proxyhandler::define_property' if self.descriptor.operations['IndexedSetter'] or self.descriptor.operations['NamedSetter']: customDefineProperty = 'defineProperty' @@ -3764,7 +3764,7 @@ class CGDOMJSProxyHandler_defineProperty(CGAbstractExternMethod): " }\n" + " return true;\n" "}\n") % (self.descriptor.name) - return set + """return proxyhandler::defineProperty_(%s);""" % ", ".join(a.name for a in self.args) + return set + """return proxyhandler::define_property(%s);""" % ", ".join(a.name for a in self.args) def definition_body(self): return CGGeneric(self.getBody()) @@ -4580,7 +4580,7 @@ class CGBindingRoot(CGThing): 'dom::bindings::error::throw_dom_exception', 'dom::bindings::error::throw_type_error', 'dom::bindings::proxyhandler', - 'dom::bindings::proxyhandler::{_obj_toString, defineProperty_}', + 'dom::bindings::proxyhandler::{_obj_toString}', 'dom::bindings::proxyhandler::{FillPropertyDescriptor, GetExpandoObject}', 'dom::bindings::proxyhandler::{delete_, get_property_descriptor}', 'dom::bindings::proxyhandler::{getOwnPropertyNames_, enumerate_}', diff --git a/components/script/dom/bindings/proxyhandler.rs b/components/script/dom/bindings/proxyhandler.rs index fcf2de62aa4..186e2ef3591 100644 --- a/components/script/dom/bindings/proxyhandler.rs +++ b/components/script/dom/bindings/proxyhandler.rs @@ -55,8 +55,9 @@ pub unsafe extern fn get_property_descriptor(cx: *mut JSContext, } /// Defines an expando on the given `proxy`. -pub unsafe extern fn defineProperty_(cx: *mut JSContext, proxy: *mut JSObject, id: jsid, - desc: *mut JSPropertyDescriptor) -> bool { +pub unsafe extern fn define_property(cx: *mut JSContext, proxy: *mut JSObject, + id: jsid, desc: *mut JSPropertyDescriptor) + -> bool { static JSMSG_GETTER_ONLY: libc::c_uint = 160; //FIXME: Workaround for https://github.com/mozilla/rust/issues/13385 From 6920aa970d0ff85afd148902a93a9252e4099797 Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Thu, 29 Jan 2015 17:15:12 +0100 Subject: [PATCH 3/8] Rename delete_ to delete. --- components/script/dom/bindings/codegen/CodegenRust.py | 6 +++--- components/script/dom/bindings/proxyhandler.rs | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/components/script/dom/bindings/codegen/CodegenRust.py b/components/script/dom/bindings/codegen/CodegenRust.py index db06ad603fb..997ff7b7d50 100644 --- a/components/script/dom/bindings/codegen/CodegenRust.py +++ b/components/script/dom/bindings/codegen/CodegenRust.py @@ -2092,7 +2092,7 @@ class CGDefineProxyHandler(CGAbstractMethod): if self.descriptor.operations['IndexedSetter'] or self.descriptor.operations['NamedSetter']: customDefineProperty = 'defineProperty' - customDelete = 'delete_' + customDelete = 'proxyhandler::delete' if self.descriptor.operations['NamedDeleter']: customDelete = 'delete' @@ -3785,7 +3785,7 @@ class CGDOMJSProxyHandler_delete(CGAbstractExternMethod): "let this = JS::from_raw(this);\n" + "let this = this.root();\n" + "%s") % (CGProxyNamedDeleter(self.descriptor).define()) - set += "return proxyhandler::delete_(%s);" % ", ".join(a.name for a in self.args) + set += "return proxyhandler::delete(%s);" % ", ".join(a.name for a in self.args) return set def definition_body(self): @@ -4582,7 +4582,7 @@ class CGBindingRoot(CGThing): 'dom::bindings::proxyhandler', 'dom::bindings::proxyhandler::{_obj_toString}', 'dom::bindings::proxyhandler::{FillPropertyDescriptor, GetExpandoObject}', - 'dom::bindings::proxyhandler::{delete_, get_property_descriptor}', + 'dom::bindings::proxyhandler::{get_property_descriptor}', 'dom::bindings::proxyhandler::{getOwnPropertyNames_, enumerate_}', 'dom::bindings::str::ByteString', 'page::JSPageInfo', diff --git a/components/script/dom/bindings/proxyhandler.rs b/components/script/dom/bindings/proxyhandler.rs index 186e2ef3591..4f03b6127fe 100644 --- a/components/script/dom/bindings/proxyhandler.rs +++ b/components/script/dom/bindings/proxyhandler.rs @@ -77,8 +77,8 @@ pub unsafe extern fn define_property(cx: *mut JSContext, proxy: *mut JSObject, } /// Deletes an expando off the given `proxy`. -pub unsafe extern fn delete_(cx: *mut JSContext, proxy: *mut JSObject, id: jsid, - bp: *mut bool) -> bool { +pub unsafe extern fn delete(cx: *mut JSContext, proxy: *mut JSObject, id: jsid, + bp: *mut bool) -> bool { let expando = GetExpandoObject(proxy); if expando.is_null() { *bp = true; From f84cbd402565757f6b87adec7d66f2e387101467 Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Thu, 29 Jan 2015 17:16:55 +0100 Subject: [PATCH 4/8] Rename _obj_toString to object_to_string. --- components/script/dom/bindings/codegen/CodegenRust.py | 3 +-- components/script/dom/bindings/proxyhandler.rs | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/components/script/dom/bindings/codegen/CodegenRust.py b/components/script/dom/bindings/codegen/CodegenRust.py index 997ff7b7d50..bd046a78d55 100644 --- a/components/script/dom/bindings/codegen/CodegenRust.py +++ b/components/script/dom/bindings/codegen/CodegenRust.py @@ -3946,7 +3946,7 @@ class CGDOMJSProxyHandler_obj_toString(CGAbstractExternMethod): JSString* jsresult; return xpc_qsStringToJsstring(cx, result, &jsresult) ? jsresult : NULL;""" - return """_obj_toString(cx, "%s")""" % self.descriptor.name + return """proxyhandler::object_to_string(cx, "%s")""" % self.descriptor.name def definition_body(self): return CGGeneric(self.getBody()) @@ -4580,7 +4580,6 @@ class CGBindingRoot(CGThing): 'dom::bindings::error::throw_dom_exception', 'dom::bindings::error::throw_type_error', 'dom::bindings::proxyhandler', - 'dom::bindings::proxyhandler::{_obj_toString}', 'dom::bindings::proxyhandler::{FillPropertyDescriptor, GetExpandoObject}', 'dom::bindings::proxyhandler::{get_property_descriptor}', 'dom::bindings::proxyhandler::{getOwnPropertyNames_, enumerate_}', diff --git a/components/script/dom/bindings/proxyhandler.rs b/components/script/dom/bindings/proxyhandler.rs index 4f03b6127fe..b43c9bc5c16 100644 --- a/components/script/dom/bindings/proxyhandler.rs +++ b/components/script/dom/bindings/proxyhandler.rs @@ -89,7 +89,7 @@ pub unsafe extern fn delete(cx: *mut JSContext, proxy: *mut JSObject, id: jsid, } /// Returns the stringification of an object with class `name`. -pub fn _obj_toString(cx: *mut JSContext, name: &str) -> *mut JSString { +pub fn object_to_string(cx: *mut JSContext, name: &str) -> *mut JSString { unsafe { let result = format!("[object {}]", name); From 0b9549746bc3bcada536fd2b6f7b232eecc1460a Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Thu, 29 Jan 2015 17:19:45 +0100 Subject: [PATCH 5/8] Rename GetExpandoObject to get_expando_object. --- components/script/dom/bindings/codegen/CodegenRust.py | 8 ++++---- components/script/dom/bindings/proxyhandler.rs | 6 +++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/components/script/dom/bindings/codegen/CodegenRust.py b/components/script/dom/bindings/codegen/CodegenRust.py index bd046a78d55..514a194556f 100644 --- a/components/script/dom/bindings/codegen/CodegenRust.py +++ b/components/script/dom/bindings/codegen/CodegenRust.py @@ -3691,7 +3691,7 @@ class CGDOMJSProxyHandler_getOwnPropertyDescriptor(CGAbstractExternMethod): namedGet = "" return setOrIndexedGet + """\ -let expando: *mut JSObject = GetExpandoObject(proxy); +let expando: *mut JSObject = get_expando_object(proxy); //if (!xpc::WrapperFactory::IsXrayWrapper(proxy) && (expando = GetExpandoObject(proxy))) { if !expando.is_null() { let flags = if set { JSRESOLVE_ASSIGNING } else { 0 } | JSRESOLVE_QUALIFIED; @@ -3829,7 +3829,7 @@ class CGDOMJSProxyHandler_hasOwn(CGAbstractExternMethod): named = "" return indexed + """\ -let expando: *mut JSObject = GetExpandoObject(proxy); +let expando: *mut JSObject = get_expando_object(proxy); if !expando.is_null() { let mut b: JSBool = 1; let ok = JS_HasPropertyById(cx, expando, id, &mut b) != 0; @@ -3854,7 +3854,7 @@ class CGDOMJSProxyHandler_get(CGAbstractExternMethod): self.descriptor = descriptor def getBody(self): getFromExpando = """\ -let expando = GetExpandoObject(proxy); +let expando = get_expando_object(proxy); if !expando.is_null() { let mut hasProp = 0; if JS_HasPropertyById(cx, expando, id, &mut hasProp) == 0 { @@ -4580,7 +4580,7 @@ class CGBindingRoot(CGThing): 'dom::bindings::error::throw_dom_exception', 'dom::bindings::error::throw_type_error', 'dom::bindings::proxyhandler', - 'dom::bindings::proxyhandler::{FillPropertyDescriptor, GetExpandoObject}', + 'dom::bindings::proxyhandler::{FillPropertyDescriptor, get_expando_object}', 'dom::bindings::proxyhandler::{get_property_descriptor}', 'dom::bindings::proxyhandler::{getOwnPropertyNames_, enumerate_}', 'dom::bindings::str::ByteString', diff --git a/components/script/dom/bindings/proxyhandler.rs b/components/script/dom/bindings/proxyhandler.rs index b43c9bc5c16..7d055caed91 100644 --- a/components/script/dom/bindings/proxyhandler.rs +++ b/components/script/dom/bindings/proxyhandler.rs @@ -79,7 +79,7 @@ pub unsafe extern fn define_property(cx: *mut JSContext, proxy: *mut JSObject, /// Deletes an expando off the given `proxy`. pub unsafe extern fn delete(cx: *mut JSContext, proxy: *mut JSObject, id: jsid, bp: *mut bool) -> bool { - let expando = GetExpandoObject(proxy); + let expando = get_expando_object(proxy); if expando.is_null() { *bp = true; return true; @@ -103,7 +103,7 @@ pub fn object_to_string(cx: *mut JSContext, name: &str) -> *mut JSString { } /// Get the expando object, or null if there is none. -pub fn GetExpandoObject(obj: *mut JSObject) -> *mut JSObject { +pub fn get_expando_object(obj: *mut JSObject) -> *mut JSObject { unsafe { assert!(is_dom_proxy(obj)); let val = GetProxyExtra(obj, JSPROXYSLOT_EXPANDO); @@ -120,7 +120,7 @@ pub fn GetExpandoObject(obj: *mut JSObject) -> *mut JSObject { pub fn EnsureExpandoObject(cx: *mut JSContext, obj: *mut JSObject) -> *mut JSObject { unsafe { assert!(is_dom_proxy(obj)); - let mut expando = GetExpandoObject(obj); + let mut expando = get_expando_object(obj); if expando.is_null() { expando = JS_NewObjectWithGivenProto(cx, ptr::null_mut(), ptr::null_mut(), From 2592045ed489a958e7bc604448e75f221f969eed Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Thu, 29 Jan 2015 17:20:51 +0100 Subject: [PATCH 6/8] Rename EnsureExpandoObject to ensure_expando_object. --- components/script/dom/bindings/proxyhandler.rs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/components/script/dom/bindings/proxyhandler.rs b/components/script/dom/bindings/proxyhandler.rs index 7d055caed91..c3645322c44 100644 --- a/components/script/dom/bindings/proxyhandler.rs +++ b/components/script/dom/bindings/proxyhandler.rs @@ -71,7 +71,7 @@ pub unsafe extern fn define_property(cx: *mut JSContext, proxy: *mut JSObject, JSMSG_GETTER_ONLY) != 0; } - let expando = EnsureExpandoObject(cx, proxy); + let expando = ensure_expando_object(cx, proxy); return JS_DefinePropertyById(cx, expando, id, (*desc).value, (*desc).getter, (*desc).setter, (*desc).attrs) != 0; } @@ -117,7 +117,8 @@ pub fn get_expando_object(obj: *mut JSObject) -> *mut JSObject { /// Get the expando object, or create it if it doesn't exist yet. /// Fails on JSAPI failure. -pub fn EnsureExpandoObject(cx: *mut JSContext, obj: *mut JSObject) -> *mut JSObject { +pub fn ensure_expando_object(cx: *mut JSContext, obj: *mut JSObject) + -> *mut JSObject { unsafe { assert!(is_dom_proxy(obj)); let mut expando = get_expando_object(obj); From bcd9ca6081120f8018c01c1eb5421cf2bf28adaf Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Thu, 29 Jan 2015 17:23:46 +0100 Subject: [PATCH 7/8] Rename FillPropertyDescriptor to fill_property_descriptor. --- components/script/dom/bindings/codegen/CodegenRust.py | 10 +++++----- components/script/dom/bindings/proxyhandler.rs | 3 ++- components/script/dom/browsercontext.rs | 4 ++-- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/components/script/dom/bindings/codegen/CodegenRust.py b/components/script/dom/bindings/codegen/CodegenRust.py index 514a194556f..40c3d90babe 100644 --- a/components/script/dom/bindings/codegen/CodegenRust.py +++ b/components/script/dom/bindings/codegen/CodegenRust.py @@ -3631,7 +3631,7 @@ class CGDOMJSProxyHandler_getOwnPropertyDescriptor(CGAbstractExternMethod): if indexedGetter: readonly = toStringBool(self.descriptor.operations['IndexedSetter'] is None) - fillDescriptor = "FillPropertyDescriptor(&mut *desc, proxy, %s);\nreturn true;" % readonly + fillDescriptor = "fill_property_descriptor(&mut *desc, proxy, %s);\nreturn true;" % readonly templateValues = {'jsvalRef': '(*desc).value', 'successCode': fillDescriptor} get = ("if index.is_some() {\n" + " let index = index.unwrap();\n" + @@ -3649,7 +3649,7 @@ class CGDOMJSProxyHandler_getOwnPropertyDescriptor(CGAbstractExternMethod): if not 'IndexedCreator' in self.descriptor.operations: # FIXME need to check that this is a 'supported property index' assert False - setOrIndexedGet += (" FillPropertyDescriptor(&mut *desc, proxy, false);\n" + + setOrIndexedGet += (" fill_property_descriptor(&mut *desc, proxy, false);\n" + " return true;\n" + " }\n") if self.descriptor.operations['NamedSetter']: @@ -3657,7 +3657,7 @@ class CGDOMJSProxyHandler_getOwnPropertyDescriptor(CGAbstractExternMethod): if not 'NamedCreator' in self.descriptor.operations: # FIXME need to check that this is a 'supported property name' assert False - setOrIndexedGet += (" FillPropertyDescriptor(&mut *desc, proxy, false);\n" + + setOrIndexedGet += (" fill_property_descriptor(&mut *desc, proxy, false);\n" + " return true;\n" + " }\n") setOrIndexedGet += "}" @@ -3674,7 +3674,7 @@ class CGDOMJSProxyHandler_getOwnPropertyDescriptor(CGAbstractExternMethod): namedGetter = self.descriptor.operations['NamedGetter'] if namedGetter: readonly = toStringBool(self.descriptor.operations['NamedSetter'] is None) - fillDescriptor = "FillPropertyDescriptor(&mut *desc, proxy, %s);\nreturn true;" % readonly + fillDescriptor = "fill_property_descriptor(&mut *desc, proxy, %s);\nreturn true;" % readonly templateValues = {'jsvalRef': '(*desc).value', 'successCode': fillDescriptor} # Once we start supporting OverrideBuiltins we need to make # ResolveOwnProperty or EnumerateOwnProperties filter out named @@ -4580,7 +4580,7 @@ class CGBindingRoot(CGThing): 'dom::bindings::error::throw_dom_exception', 'dom::bindings::error::throw_type_error', 'dom::bindings::proxyhandler', - 'dom::bindings::proxyhandler::{FillPropertyDescriptor, get_expando_object}', + 'dom::bindings::proxyhandler::{fill_property_descriptor, get_expando_object}', 'dom::bindings::proxyhandler::{get_property_descriptor}', 'dom::bindings::proxyhandler::{getOwnPropertyNames_, enumerate_}', 'dom::bindings::str::ByteString', diff --git a/components/script/dom/bindings/proxyhandler.rs b/components/script/dom/bindings/proxyhandler.rs index c3645322c44..03f049dd8ff 100644 --- a/components/script/dom/bindings/proxyhandler.rs +++ b/components/script/dom/bindings/proxyhandler.rs @@ -136,7 +136,8 @@ pub fn ensure_expando_object(cx: *mut JSContext, obj: *mut JSObject) /// Set the property descriptor's object to `obj` and set it to enumerable, /// and writable if `readonly` is true. -pub fn FillPropertyDescriptor(desc: &mut JSPropertyDescriptor, obj: *mut JSObject, readonly: bool) { +pub fn fill_property_descriptor(desc: &mut JSPropertyDescriptor, + obj: *mut JSObject, readonly: bool) { desc.obj = obj; desc.attrs = if readonly { JSPROP_READONLY } else { 0 } | JSPROP_ENUMERATE; desc.getter = None; diff --git a/components/script/dom/browsercontext.rs b/components/script/dom/browsercontext.rs index 16165f68780..f3e52d0f46d 100644 --- a/components/script/dom/browsercontext.rs +++ b/components/script/dom/browsercontext.rs @@ -7,7 +7,7 @@ use dom::bindings::conversions::{ToJSValConvertible}; use dom::bindings::js::{JS, JSRef, Temporary, Root}; use dom::bindings::js::{OptionalRootable, OptionalRootedRootable, ResultRootable}; use dom::bindings::js::{OptionalRootedReference, OptionalOptionalRootedRootable}; -use dom::bindings::proxyhandler::{get_property_descriptor, FillPropertyDescriptor}; +use dom::bindings::proxyhandler::{get_property_descriptor, fill_property_descriptor}; use dom::bindings::utils::{Reflectable, WindowProxyHandler}; use dom::bindings::utils::{GetArrayIndexFromId}; use dom::document::{Document, DocumentHelpers}; @@ -116,7 +116,7 @@ unsafe extern fn getOwnPropertyDescriptor(cx: *mut JSContext, proxy: *mut JSObje if let Some(window) = window { let window = window.root(); (*desc).value = window.to_jsval(cx); - FillPropertyDescriptor(&mut *desc, proxy, true); + fill_property_descriptor(&mut *desc, proxy, true); return true; } From 009cc87b6a214655bf9634717beccf87d4e4f03b Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Thu, 29 Jan 2015 17:25:06 +0100 Subject: [PATCH 8/8] Rename getOwnPropertyNames_ and enumerate_ to get_own_property_names and enumerate. --- components/script/dom/bindings/codegen/CodegenRust.py | 5 ++--- components/script/dom/bindings/proxyhandler.rs | 10 +++++----- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/components/script/dom/bindings/codegen/CodegenRust.py b/components/script/dom/bindings/codegen/CodegenRust.py index 40c3d90babe..c9ea798b1d9 100644 --- a/components/script/dom/bindings/codegen/CodegenRust.py +++ b/components/script/dom/bindings/codegen/CodegenRust.py @@ -2101,9 +2101,9 @@ let traps = ProxyTraps { getPropertyDescriptor: Some(get_property_descriptor as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, bool, *mut JSPropertyDescriptor) -> bool), getOwnPropertyDescriptor: Some(getOwnPropertyDescriptor as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, bool, *mut JSPropertyDescriptor) -> bool), defineProperty: Some(%s as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, *mut JSPropertyDescriptor) -> bool), - getOwnPropertyNames: Some(getOwnPropertyNames_ as unsafe extern "C" fn(*mut JSContext, *mut JSObject, *mut AutoIdVector) -> bool), + getOwnPropertyNames: Some(proxyhandler::get_own_property_names as unsafe extern "C" fn(*mut JSContext, *mut JSObject, *mut AutoIdVector) -> bool), delete_: Some(%s as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, *mut bool) -> bool), - enumerate: Some(enumerate_ as unsafe extern "C" fn(*mut JSContext, *mut JSObject, *mut AutoIdVector) -> bool), + enumerate: Some(proxyhandler::enumerate as unsafe extern "C" fn(*mut JSContext, *mut JSObject, *mut AutoIdVector) -> bool), has: None, hasOwn: Some(hasOwn as unsafe extern "C" fn(*mut JSContext, *mut JSObject, jsid, *mut bool) -> bool), @@ -4582,7 +4582,6 @@ class CGBindingRoot(CGThing): 'dom::bindings::proxyhandler', 'dom::bindings::proxyhandler::{fill_property_descriptor, get_expando_object}', 'dom::bindings::proxyhandler::{get_property_descriptor}', - 'dom::bindings::proxyhandler::{getOwnPropertyNames_, enumerate_}', 'dom::bindings::str::ByteString', 'page::JSPageInfo', 'libc', diff --git a/components/script/dom/bindings/proxyhandler.rs b/components/script/dom/bindings/proxyhandler.rs index 03f049dd8ff..4441bdf2335 100644 --- a/components/script/dom/bindings/proxyhandler.rs +++ b/components/script/dom/bindings/proxyhandler.rs @@ -146,14 +146,14 @@ pub fn fill_property_descriptor(desc: &mut JSPropertyDescriptor, } /// No-op required hook. -pub unsafe extern fn getOwnPropertyNames_(_cx: *mut JSContext, - _obj: *mut JSObject, - _v: *mut AutoIdVector) -> bool { +pub unsafe extern fn get_own_property_names(_cx: *mut JSContext, + _obj: *mut JSObject, + _v: *mut AutoIdVector) -> bool { true } /// No-op required hook. -pub unsafe extern fn enumerate_(_cx: *mut JSContext, _obj: *mut JSObject, - _v: *mut AutoIdVector) -> bool { +pub unsafe extern fn enumerate(_cx: *mut JSContext, _obj: *mut JSObject, + _v: *mut AutoIdVector) -> bool { true }