607724 - TM: Create internal GetGlobalForScopeChain

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Chris Leary [:cdleary] (not checking bugmail)]

Followup to bug 601299 -- some simple cleanup to put our GetGlobal family of functions into the js namespace / context methods.

Comment 1

[Chris Leary [:cdleary] (not checking bugmail)]

Attachment: Cleanup.

Also adds a CHECK_REQUEST to JS_GetGlobalForScopeChain

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 486426 [details] [diff] [review]
Cleanup.

>+inline js::RegExpStatics *
>+JSContext::regExpStatics()
>+{
>+    JSObject *global = js::GetGlobalForScopeChain(this);
>+    js::RegExpStatics *res = js::RegExpStatics::extractFrom(global);
>+    return res;

Nit: res isn't needed and I think this still fits on one line without global either.

r=me with that

Comment 3

[Chris Leary [:cdleary] (not checking bugmail)]

http://hg.mozilla.org/tracemonkey/rev/d973801a27af

Comment 4

[Robert Sayre]

http://hg.mozilla.org/mozilla-central/rev/d973801a27af