Changeset 53993 in webkit


Ignore:
Timestamp:
Jan 28, 2010 2:46:26 AM (14 years ago)
Author:
eric@webkit.org
Message:

2010-01-28 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Simplify isActivation method.

https://bugs.webkit.org/show_bug.cgi?id=34161

Test: WebCore/manual-tests/inspector/debugger-scopes-inspection.html

  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::JSInjectedScriptHost::isActivation):
Location:
trunk/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/WebCore/ChangeLog

    r53975 r53993  
     12010-01-28  Yury Semikhatsky  <yurys@chromium.org>
     2
     3        Reviewed by Pavel Feldman.
     4
     5        Simplify isActivation method.
     6
     7        https://bugs.webkit.org/show_bug.cgi?id=34161
     8
     9        Test: WebCore/manual-tests/inspector/debugger-scopes-inspection.html
     10
     11        * bindings/js/JSInjectedScriptHostCustom.cpp:
     12        (WebCore::JSInjectedScriptHost::isActivation):
     13
    1142010-01-27  Kent Tamura  <tkent@chromium.org>
    215
  • trunk/WebCore/bindings/js/JSInjectedScriptHostCustom.cpp

    r53945 r53993  
    127127}
    128128
    129 JSValue JSInjectedScriptHost::isActivation(ExecState* exec, const ArgList& args)
    130 {
    131     if (args.size() < 1)
    132         return jsUndefined();
    133 
    134     JSValue value = args.at(0);
    135     if (!value.isObject())
    136         return jsBoolean(false);
    137 
    138     JSObject* object = value.toObject(exec);
    139     return jsBoolean(object->isActivationObject());
     129JSValue JSInjectedScriptHost::isActivation(ExecState*, const ArgList& args)
     130{
     131    JSObject* object = args.at(0).getObject();
     132    return jsBoolean(object && object->isActivationObject());
    140133}
    141134
Note: See TracChangeset for help on using the changeset viewer.