From a6a5ddf955cc6485c7e7224d4f9f095a7f61d309 Mon Sep 17 00:00:00 2001 From: Marco Castelluccio Date: Thu, 3 Oct 2019 09:05:02 +0000 Subject: [PATCH] Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/wrappers. r=bholley Depends on D5655 Differential Revision: https://phabricator.services.mozilla.com/D5656 UltraBlame original commit: fa26c3594f36307988f575ce5520843427940733 --- js/xpconnect/wrappers/AccessCheck.cpp | 28 ++ js/xpconnect/wrappers/AccessCheck.h | 4 + js/xpconnect/wrappers/ChromeObjectWrapper.cpp | 4 + js/xpconnect/wrappers/FilteringWrapper.cpp | 30 +++ js/xpconnect/wrappers/WaiveXrayWrapper.cpp | 8 + js/xpconnect/wrappers/WrapperFactory.cpp | 38 +++ js/xpconnect/wrappers/XrayWrapper.cpp | 254 ++++++++++++++++++ js/xpconnect/wrappers/XrayWrapper.h | 8 + 8 files changed, 374 insertions(+) diff --git a/js/xpconnect/wrappers/AccessCheck.cpp b/js/xpconnect/wrappers/AccessCheck.cpp index 25011fb41b93..6106f4c51673 100644 --- a/js/xpconnect/wrappers/AccessCheck.cpp +++ b/js/xpconnect/wrappers/AccessCheck.cpp @@ -569,9 +569,11 @@ if ! propLength ) +{ return false ; +} char16_t propChar0 = @@ -588,6 +590,7 @@ type = CrossOriginLocation ) +{ return dom : @@ -602,6 +605,7 @@ propChar0 set ) ; +} if ( type @@ -609,6 +613,7 @@ type = CrossOriginWindow ) +{ return dom : @@ -623,6 +628,7 @@ propChar0 set ) ; +} return false ; @@ -840,9 +846,11 @@ Location " ) ) +{ return CrossOriginLocation ; +} if ( clasp @@ -871,9 +879,11 @@ Window " ) ) +{ return CrossOriginWindow ; +} return CrossOriginOpaque ; @@ -908,9 +918,11 @@ Wrapper : CALL ) +{ return false ; +} if ( act @@ -921,9 +933,11 @@ Wrapper : ENUMERATE ) +{ return true ; +} if ( act @@ -1008,10 +1022,12 @@ Wrapper SET ) ) +{ return true ; } +} if ( type @@ -1041,9 +1057,11 @@ Wrapper : GET ) +{ return false ; +} if ( type @@ -1131,9 +1149,11 @@ isObject ( ) ) +{ return true ; +} RootedObject obj ( @@ -1157,9 +1177,11 @@ IsWrapper obj ) ) +{ return true ; +} if ( mozilla @@ -1216,9 +1238,11 @@ wrapperSubsumes obj ) ) +{ return true ; +} JS_ReportErrorASCII ( cx @@ -1269,9 +1293,11 @@ thisv ) ) ) +{ return false ; +} for ( size_t @@ -1305,10 +1331,12 @@ i ] ) ) +{ return false ; } +} return true ; diff --git a/js/xpconnect/wrappers/AccessCheck.h b/js/xpconnect/wrappers/AccessCheck.h index 25253729a6af..58a1dadfda63 100644 --- a/js/xpconnect/wrappers/AccessCheck.h +++ b/js/xpconnect/wrappers/AccessCheck.h @@ -576,13 +576,16 @@ Wrapper : ENUMERATE ) +{ return true ; +} if ( mayThrow ) +{ AccessCheck : : @@ -598,6 +601,7 @@ access ) ) ; +} return false ; diff --git a/js/xpconnect/wrappers/ChromeObjectWrapper.cpp b/js/xpconnect/wrappers/ChromeObjectWrapper.cpp index 0f610f6bc973..167b77d0d2a9 100644 --- a/js/xpconnect/wrappers/ChromeObjectWrapper.cpp +++ b/js/xpconnect/wrappers/ChromeObjectWrapper.cpp @@ -104,9 +104,11 @@ value ) ) ) +{ return false ; +} return ChromeObjectWrapperBase : @@ -157,9 +159,11 @@ wrapper v ) ) +{ return false ; +} return ChromeObjectWrapperBase : diff --git a/js/xpconnect/wrappers/FilteringWrapper.cpp b/js/xpconnect/wrappers/FilteringWrapper.cpp index c56889e067ee..5643457b21ed 100644 --- a/js/xpconnect/wrappers/FilteringWrapper.cpp +++ b/js/xpconnect/wrappers/FilteringWrapper.cpp @@ -286,6 +286,7 @@ Wrapper SET ) ) +{ props [ w @@ -298,6 +299,7 @@ set id ) ; +} else if ( @@ -306,10 +308,12 @@ JS_IsExceptionPending cx ) ) +{ return false ; } +} if ( ! @@ -320,9 +324,11 @@ resize w ) ) +{ return false ; +} return true ; @@ -383,9 +389,11 @@ JS_IsExceptionPending cx ) ) +{ return false ; +} bool setAllowed = @@ -410,9 +418,11 @@ JS_IsExceptionPending cx ) ) +{ return false ; +} MOZ_ASSERT ( getAllowed @@ -447,6 +457,7 @@ if ! getAllowed ) +{ desc . value @@ -458,6 +469,7 @@ setUndefined ) ; } +} else { MOZ_ASSERT @@ -478,6 +490,7 @@ if ! getAllowed ) +{ desc . setGetter @@ -485,11 +498,13 @@ setGetter nullptr ) ; +} if ( ! setAllowed ) +{ desc . setSetter @@ -498,6 +513,7 @@ nullptr ) ; } +} return true ; @@ -569,9 +585,11 @@ id desc ) ) +{ return false ; +} return FilterPropertyDescriptor < @@ -652,9 +670,11 @@ id desc ) ) +{ return false ; +} return FilterPropertyDescriptor < @@ -895,9 +915,11 @@ wrapper args ) ) +{ return false ; +} return Base : @@ -962,9 +984,11 @@ wrapper args ) ) +{ return false ; +} return Base : @@ -1029,6 +1053,7 @@ test impl ) ) +{ return Base : @@ -1044,6 +1069,7 @@ impl args ) ; +} return Base : @@ -1241,9 +1267,11 @@ id desc ) ) +{ return false ; +} if ( desc @@ -1341,6 +1369,7 @@ setter ( ) ) +{ desc . attributesRef @@ -1351,6 +1380,7 @@ attributesRef JSPROP_READONLY ; } +} else if ( diff --git a/js/xpconnect/wrappers/WaiveXrayWrapper.cpp b/js/xpconnect/wrappers/WaiveXrayWrapper.cpp index 27d8feb01a95..4aa9ec389ae0 100644 --- a/js/xpconnect/wrappers/WaiveXrayWrapper.cpp +++ b/js/xpconnect/wrappers/WaiveXrayWrapper.cpp @@ -87,9 +87,11 @@ cx v ) ) +{ return false ; +} desc . setGetterObject @@ -150,9 +152,11 @@ cx v ) ) +{ return false ; +} desc . setSetterObject @@ -358,9 +362,11 @@ if ! obj ) +{ return nullptr ; +} if ( ! @@ -374,9 +380,11 @@ cx obj ) ) +{ return nullptr ; +} return obj ; diff --git a/js/xpconnect/wrappers/WrapperFactory.cpp b/js/xpconnect/wrappers/WrapperFactory.cpp index 281954b12a8a..3dbc4bf02140 100644 --- a/js/xpconnect/wrappers/WrapperFactory.cpp +++ b/js/xpconnect/wrappers/WrapperFactory.cpp @@ -249,9 +249,11 @@ scope > mWaiverWrapperMap ) +{ return nullptr ; +} return scope - @@ -324,9 +326,11 @@ if ! waiver ) +{ return nullptr ; +} if ( ! @@ -366,9 +370,11 @@ obj waiver ) ) +{ return nullptr ; +} return waiver ; @@ -585,9 +591,11 @@ WrapperFactory WAIVE_XRAY_WRAPPER_FLAG ) ) +{ return false ; +} if ( ! @@ -600,9 +608,11 @@ Wrapper CROSS_COMPARTMENT ) ) +{ return true ; +} JS : : @@ -1714,6 +1724,7 @@ if ! securityWrapper ) +{ return & CrossCompartmentWrapper @@ -1721,6 +1732,7 @@ CrossCompartmentWrapper : singleton ; +} return & FilteringWrapper @@ -1746,6 +1758,7 @@ xrayType = XrayForDOMObject ) +{ return & PermissiveXrayDOM @@ -1753,6 +1766,7 @@ PermissiveXrayDOM : singleton ; +} else if ( @@ -1761,6 +1775,7 @@ xrayType = XrayForJSObject ) +{ return & PermissiveXrayJS @@ -1768,6 +1783,7 @@ PermissiveXrayJS : singleton ; +} MOZ_ASSERT ( xrayType @@ -1800,6 +1816,7 @@ obj = CrossOriginOpaque ) +{ return & FilteringWrapper @@ -1811,6 +1828,7 @@ CrossOriginAccessiblePropertiesOnly : singleton ; +} if ( xrayType @@ -1824,6 +1842,7 @@ IsInContentXBLScope obj ) ) +{ return & FilteringWrapper @@ -1835,6 +1854,7 @@ OpaqueWithCall : singleton ; +} return & FilteringWrapper @@ -2365,6 +2385,7 @@ if ( existing ) +{ return Wrapper : @@ -2376,6 +2397,7 @@ obj wrapper ) ; +} return Wrapper : @@ -2409,6 +2431,7 @@ isPrimitive ( ) ) +{ return JS_WrapValue ( @@ -2416,6 +2439,7 @@ cx vp ) ; +} RootedObject obj ( @@ -2438,9 +2462,11 @@ cx obj ) ) +{ return false ; +} vp . setObject @@ -2570,9 +2596,11 @@ if ! obj ) +{ return false ; +} if ( ! @@ -2583,9 +2611,11 @@ cx obj ) ) +{ return false ; +} argObj . set @@ -2655,9 +2685,11 @@ if ! newWaiver ) +{ return false ; +} if ( ! @@ -2671,9 +2703,11 @@ oldWaiver newWaiver ) ) +{ return false ; +} XPCWrappedNativeScope * scope @@ -2777,9 +2811,11 @@ newIdentity ! oldWaiver ) +{ return newIdentity ; +} if ( ! @@ -2790,9 +2826,11 @@ oldWaiver newIdentity ) ) +{ return nullptr ; +} return newIdentity ; diff --git a/js/xpconnect/wrappers/XrayWrapper.cpp b/js/xpconnect/wrappers/XrayWrapper.cpp index ea5825436a1c..abe77c24b510 100644 --- a/js/xpconnect/wrappers/XrayWrapper.cpp +++ b/js/xpconnect/wrappers/XrayWrapper.cpp @@ -393,9 +393,11 @@ IsTypedArrayKey key ) ) +{ return true ; +} if ( IsErrorObjectKey @@ -403,9 +405,11 @@ IsErrorObjectKey key ) ) +{ return true ; +} switch ( key @@ -492,9 +496,11 @@ UseDOMXray obj ) ) +{ return XrayForDOMObject ; +} MOZ_ASSERT ( ! @@ -522,9 +528,11 @@ IsJSXraySupported standardProto ) ) +{ return XrayForJSObject ; +} if ( IsSandbox @@ -532,9 +540,11 @@ IsSandbox obj ) ) +{ return NotXray ; +} return XrayForOpaqueObject ; @@ -817,9 +827,11 @@ object ( ) ) +{ return ok ; +} return ReportWrapperDenial ( @@ -895,9 +907,11 @@ if ( alreadyWarnedOnce ) +{ return true ; +} # endif nsAutoJSString @@ -920,9 +934,11 @@ id idval ) ) +{ return false ; +} JSString * str @@ -938,9 +954,11 @@ if ! str ) +{ return false ; +} if ( ! @@ -952,9 +970,11 @@ cx str ) ) +{ return false ; +} AutoFilename filename ; @@ -1030,9 +1050,11 @@ if ( alreadyWarnedOnce ) +{ return true ; +} nsCOMPtr < nsIConsoleService @@ -1083,6 +1105,7 @@ CurrentWindowOrNull cx ) ) +{ windowId = win @@ -1092,6 +1115,7 @@ WindowID ( ) ; +} Maybe < nsPrintfCString @@ -1428,10 +1452,12 @@ id outDesc ) ) +{ return false ; } +} return JS_WrapPropertyDescriptor ( @@ -1548,9 +1574,11 @@ id desc ) ) +{ return false ; +} if ( ! @@ -1560,9 +1588,11 @@ object ( ) ) +{ return true ; +} if ( desc @@ -1833,6 +1863,7 @@ if ( foundOnProto ) +{ return ReportWrapperDenial ( @@ -1851,6 +1882,7 @@ prototype " ) ; +} outDesc . assign @@ -1960,9 +1992,11 @@ if ! fun ) +{ return false ; +} RootedObject funObj ( @@ -2108,9 +2142,11 @@ if ! getterFun ) +{ return false ; +} RootedObject getterObj ( @@ -2177,9 +2213,11 @@ if ! setterFun ) +{ return false ; +} setterObj = JS_GetFunctionObject @@ -2201,10 +2239,12 @@ setterObj flags ) ) +{ return false ; } +} else { if @@ -2266,9 +2306,11 @@ cx v ) ) +{ return false ; +} if ( ! @@ -2284,10 +2326,12 @@ flags JSPROP_INTERNAL_USE_BIT ) ) +{ return false ; } +} return JS_GetOwnPropertyDescriptorById ( @@ -2369,9 +2413,11 @@ object ( ) ) +{ return ok ; +} if ( ! @@ -2383,9 +2429,11 @@ id desc ) ) +{ return false ; +} if ( desc @@ -2523,10 +2571,12 @@ id innerDesc ) ) +{ return false ; } +} if ( innerDesc @@ -2678,10 +2728,12 @@ fun length ) ) +{ return false ; } +} FillPropertyDescriptor ( desc @@ -2731,6 +2783,7 @@ if ( fname ) +{ JS_MarkCrossZoneIdValue ( cx @@ -2740,6 +2793,7 @@ fname ) ) ; +} FillPropertyDescriptor ( desc @@ -2819,9 +2873,11 @@ standardConstructor standardProto ) ) +{ return false ; +} MOZ_ASSERT ( standardProto @@ -2838,9 +2894,11 @@ cx standardProto ) ) +{ return false ; +} FillPropertyDescriptor ( desc @@ -3042,9 +3100,11 @@ cx waiver ) ) +{ return false ; +} if ( ! @@ -3056,9 +3116,11 @@ id desc ) ) +{ return false ; +} bool valueMatchesType = @@ -3105,6 +3167,7 @@ hasGetterOrSetter ! valueMatchesType ) +{ FillPropertyDescriptor ( desc @@ -3115,6 +3178,7 @@ UndefinedValue ) ) ; +} return true ; @@ -3143,6 +3207,7 @@ XPCJSContext IDX_LASTINDEX ) ) +{ return getOwnPropertyFromWrapperIfSafe ( @@ -3153,6 +3218,7 @@ desc ) ; } +} return true ; @@ -3197,10 +3263,12 @@ key constructor ) ) +{ return false ; } +} if ( ! @@ -3211,9 +3279,11 @@ cx constructor ) ) +{ return false ; +} desc . object @@ -3387,9 +3457,11 @@ if ! holder ) +{ return false ; +} JSProtoKey key = @@ -3485,9 +3557,11 @@ id desc ) ) +{ return false ; +} if ( desc @@ -3496,6 +3570,7 @@ object ( ) ) +{ return JS_DeletePropertyById ( @@ -3506,6 +3581,7 @@ result ) ; } +} return result . @@ -3566,9 +3642,11 @@ if ! holder ) +{ return false ; +} JSProtoKey key = @@ -3972,9 +4050,11 @@ desc result ) ) +{ return false ; +} * defined = @@ -4026,9 +4106,11 @@ JSID_IS_SYMBOL id ) ) +{ return true ; +} return props . @@ -4093,9 +4175,11 @@ name id ) ) +{ return false ; +} if ( ! @@ -4106,10 +4190,12 @@ flags props ) ) +{ return false ; } +} for ( ; @@ -4143,9 +4229,11 @@ name id ) ) +{ return false ; +} if ( ! @@ -4156,10 +4244,12 @@ flags props ) ) +{ return false ; } +} return true ; @@ -4220,9 +4310,11 @@ if ! holder ) +{ return false ; +} JSProtoKey key = @@ -4307,9 +4399,11 @@ JSITER_OWNONLY targetProps ) ) +{ return false ; +} if ( ! @@ -4324,9 +4418,11 @@ length ) ) ) +{ return false ; +} for ( size_t @@ -4393,6 +4489,7 @@ object ( ) ) +{ props . infallibleAppend @@ -4402,6 +4499,7 @@ id ; } } +} for ( size_t @@ -4421,6 +4519,7 @@ length + i ) +{ JS_MarkCrossZoneId ( cx @@ -4430,6 +4529,7 @@ i ] ) ; +} return true ; @@ -4460,9 +4560,11 @@ reserve length ) ) +{ return false ; +} for ( int32_t @@ -4484,6 +4586,7 @@ length + i ) +{ props . infallibleAppend @@ -4495,6 +4598,7 @@ i ) ; } +} else if ( @@ -4521,9 +4625,11 @@ IDX_LENGTH ) ) ) +{ return false ; +} if ( ! @@ -4541,9 +4647,11 @@ IDX_NAME ) ) ) +{ return false ; +} JSProtoKey standardConstructor = @@ -4577,9 +4685,11 @@ IDX_PROTOTYPE ) ) ) +{ return false ; +} if ( ShouldResolveStaticProperties @@ -4766,10 +4876,12 @@ IDX_LASTINDEX ) ) ) +{ return false ; } +} return true ; @@ -4791,9 +4903,11 @@ IDX_CONSTRUCTOR ) ) ) +{ return false ; +} const js : @@ -4898,9 +5012,11 @@ if ! holder ) +{ return false ; +} if ( xpc @@ -4933,6 +5049,7 @@ standardConstructor = JSProto_Null ) +{ return baseInstance . @@ -4943,6 +5060,7 @@ wrapper args ) ; +} const js : @@ -4976,6 +5094,7 @@ flags JSCLASS_HAS_XRAYED_CONSTRUCTOR ) ) +{ return baseInstance . @@ -4986,6 +5105,7 @@ wrapper args ) ; +} RootedObject ctor ( @@ -5003,9 +5123,11 @@ standardConstructor ctor ) ) +{ return false ; +} RootedValue ctorVal ( @@ -5045,9 +5167,11 @@ vals result ) ) +{ return false ; +} AssertSameCompartment ( cx @@ -5143,9 +5267,11 @@ if ! holder ) +{ return nullptr ; +} bool isPrototype = @@ -5531,9 +5657,11 @@ Equals o ) ) +{ return false ; +} JSObject * owner @@ -5644,6 +5772,7 @@ head = expandoObject ) +{ head = JS_GetReservedSlot @@ -5656,6 +5785,7 @@ toObjectOrNull ( ) ; +} MOZ_ASSERT ( head @@ -5759,9 +5889,11 @@ if ! chain ) +{ return true ; +} bool isExclusive = @@ -5915,13 +6047,16 @@ origin existingExpandoObject ) ) +{ MOZ_ASSERT ( ! existingExpandoObject ) ; +} else +{ JS_ClearPendingException ( cx @@ -5929,6 +6064,7 @@ cx ; } } +} # endif const @@ -5974,9 +6110,11 @@ if ! expandoObject ) +{ return nullptr ; +} NS_ADDREF ( origin @@ -6028,9 +6166,11 @@ if ! wrapperHolder ) +{ return nullptr ; +} JS_SetReservedSlot ( wrapperHolder @@ -6053,9 +6193,11 @@ cx wrapperHolder ) ) +{ return nullptr ; +} JS_SetReservedSlot ( expandoObject @@ -6095,9 +6237,11 @@ cx cachedExpandoObject ) ) +{ return nullptr ; +} JSObject * holder @@ -6113,9 +6257,11 @@ if ! holder ) +{ return nullptr ; +} SetCachedXrayExpando ( holder @@ -6138,11 +6284,13 @@ if ! chain ) +{ preserveWrapper ( target ) ; +} JS_SetReservedSlot ( expandoObject @@ -6230,9 +6378,11 @@ wrapper expandoObject ) ) +{ return nullptr ; +} if ( ! @@ -6403,9 +6553,11 @@ cx exclusiveWrapper ) ) +{ return false ; +} exclusiveWrapperGlobal = JS @@ -6455,10 +6607,12 @@ dst oldHead ) ) +{ return false ; } +} else { RootedObject @@ -6488,10 +6642,12 @@ newHead oldHead ) ) +{ return false ; } +} oldHead = JS_GetReservedSlot @@ -6823,9 +6979,11 @@ if ( holder ) +{ return holder ; +} holder = createHolder @@ -6838,6 +6996,7 @@ if ( holder ) +{ js : : @@ -6852,6 +7011,7 @@ holder ) ) ; +} return holder ; @@ -6884,9 +7044,11 @@ if ! holder ) +{ return nullptr ; +} Value v = @@ -7209,9 +7371,11 @@ wrapper expando ) ) +{ return false ; +} bool found = @@ -7246,9 +7410,11 @@ id desc ) ) +{ return false ; +} found = ! @@ -7320,9 +7486,11 @@ key constructor ) ) +{ return false ; +} MOZ_ASSERT ( constructor @@ -7383,9 +7551,11 @@ cx eval ) ) +{ return false ; +} desc . value @@ -7421,9 +7591,11 @@ cx desc ) ) +{ return false ; +} desc . object @@ -7475,9 +7647,11 @@ id found ) ) +{ return false ; +} if ( ! @@ -7511,9 +7685,11 @@ id desc ) ) +{ return false ; +} desc . object @@ -7585,9 +7761,11 @@ object ( ) ) +{ return ok ; +} uint32_t index = @@ -7734,9 +7912,11 @@ id desc ) ) +{ return false ; +} if ( desc @@ -7777,9 +7957,11 @@ desc cacheOnHolder ) ) +{ return false ; +} MOZ_ASSERT ( ! @@ -7822,9 +8004,11 @@ object ! cacheOnHolder ) +{ return true ; +} return JS_DefinePropertyById ( @@ -8227,10 +8411,12 @@ base ) ) ) +{ return false ; } +} else { RootedValue @@ -8272,10 +8458,12 @@ wrapper args ) ) +{ return false ; } +} return JS_WrapValue ( @@ -8396,10 +8584,12 @@ base ) ) ) +{ return false ; } +} else { RootedValue @@ -8441,10 +8631,12 @@ wrapper args ) ) +{ return false ; } +} if ( ! @@ -8470,9 +8662,11 @@ rval ) ) ) +{ return false ; +} return true ; @@ -8549,8 +8743,10 @@ if ! identity ) +{ return ; +} nsWrapperCache * cache @@ -8568,6 +8764,7 @@ if ( cache ) +{ cache - > @@ -8577,6 +8774,7 @@ identity ) ; } +} JSObject * DOMXrayTraits @@ -8733,9 +8931,11 @@ id desc ) ) +{ return false ; +} if ( desc @@ -8935,9 +9135,11 @@ if ! holder ) +{ return false ; +} if ( ! @@ -8956,9 +9158,11 @@ id desc ) ) +{ return false ; +} if ( ! @@ -8978,9 +9182,11 @@ id desc ) ) +{ return false ; +} if ( desc @@ -9055,9 +9261,11 @@ id ) ) ) +{ return false ; +} if ( nsCOMPtr @@ -9106,6 +9314,7 @@ MOZ_UNLIKELY childObj ) ) +{ return xpc : @@ -9116,6 +9325,7 @@ cx NS_ERROR_FAILURE ) ; +} ExposeObjectToActiveJS ( childObj @@ -9246,9 +9456,11 @@ if ! holder ) +{ return false ; +} if ( ! @@ -9267,9 +9479,11 @@ id desc ) ) +{ return false ; +} if ( desc @@ -9278,6 +9492,7 @@ object ( ) ) +{ desc . object @@ -9289,6 +9504,7 @@ set wrapper ) ; +} return true ; @@ -9671,9 +9887,11 @@ id existing_desc ) ) +{ return false ; +} if ( existing_desc @@ -9809,16 +10027,20 @@ result defined ) ) +{ return false ; +} if ( defined ) +{ return true ; +} RootedObject target ( @@ -9854,9 +10076,11 @@ if ! expandoObject ) +{ return false ; +} JSAutoRealm ar ( @@ -9890,9 +10114,11 @@ cx wrappedDesc ) ) +{ return false ; +} if ( ! @@ -9908,9 +10134,11 @@ address wrappedDesc ) ) +{ return false ; +} return JS_DefinePropertyById ( @@ -10052,9 +10280,11 @@ wrapper expando ) ) +{ return false ; +} if ( expando @@ -10176,9 +10406,11 @@ id desc ) ) +{ return false ; +} desc . assertCompleteIfFound @@ -10381,9 +10613,11 @@ id desc ) ) +{ return false ; +} * bp = @@ -10781,6 +11015,7 @@ hasSecurityPolicy ( ) ) +{ return Base : @@ -10792,6 +11027,7 @@ wrapper protop ) ; +} RootedObject target ( @@ -10828,9 +11064,11 @@ wrapper expando ) ) +{ return false ; +} if ( expando @@ -10910,9 +11148,11 @@ if ! holder ) +{ return false ; +} Value cached = @@ -10953,9 +11193,11 @@ target protop ) ) +{ return false ; +} js : : @@ -11039,6 +11281,7 @@ hasSecurityPolicy ( ) ) +{ return Base : @@ -11051,6 +11294,7 @@ proto result ) ; +} RootedObject target ( @@ -11086,9 +11330,11 @@ if ! expando ) +{ return false ; +} JSAutoRealm ar ( @@ -11116,9 +11362,11 @@ cx v ) ) +{ return false ; +} JS_SetReservedSlot ( expando @@ -11296,9 +11544,11 @@ wrapper expando ) ) +{ return false ; +} if ( expando @@ -11326,10 +11576,12 @@ flags props ) ) +{ return false ; } +} for ( size_t @@ -11349,6 +11601,7 @@ length + i ) +{ JS_MarkCrossZoneId ( cx @@ -11358,6 +11611,7 @@ i ] ) ; +} return Traits : diff --git a/js/xpconnect/wrappers/XrayWrapper.h b/js/xpconnect/wrappers/XrayWrapper.h index 32a9d5718e75..edc3fb8cc0eb 100644 --- a/js/xpconnect/wrappers/XrayWrapper.h +++ b/js/xpconnect/wrappers/XrayWrapper.h @@ -149,6 +149,7 @@ if ( target ) +{ JS : : @@ -157,6 +158,7 @@ ExposeObjectToActiveJS target ) ; +} return target ; @@ -1088,6 +1090,7 @@ holder = JSProto_Function ) +{ return baseInstance . @@ -1098,6 +1101,7 @@ wrapper args ) ; +} JS : : @@ -1262,10 +1266,12 @@ key protop ) ) +{ return false ; } +} return JS_WrapObject ( @@ -1781,10 +1787,12 @@ JSProto_Object protop ) ) +{ return false ; } +} return JS_WrapObject (