Changeset 35004 in webkit


Ignore:
Timestamp:
Jul 4, 2008 3:43:10 PM (16 years ago)
Author:
cwzwarich@webkit.org
Message:

2008-07-04 Cameron Zwarich <​cwzwarich@uwaterloo.ca>

Reviewed by Eric Seidel.

Bring our implementation of the Netscape plugin API closer to current
code style guidelines.

  • bridge/NP_jsobject.cpp: (getListFromVariantArgs): (jsAllocate): (jsDeallocate): (_NPN_CreateScriptObject): (_NPN_CreateNoScriptObject): (_NPN_InvokeDefault): (_NPN_Invoke): (_NPN_Evaluate): (_NPN_GetProperty): (_NPN_SetProperty): (_NPN_RemoveProperty): (_NPN_HasProperty): (_NPN_HasMethod): (_NPN_Enumerate):
  • bridge/npruntime_impl.h:
Location:
trunk/WebCore
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/WebCore/ChangeLog

    r35003 r35004  
     12008-07-04  Cameron Zwarich  <cwzwarich@uwaterloo.ca>
     2
     3        Reviewed by Eric Seidel.
     4
     5        Bring our implementation of the Netscape plugin API closer to current
     6        code style guidelines.
     7
     8        * bridge/NP_jsobject.cpp:
     9        (getListFromVariantArgs):
     10        (jsAllocate):
     11        (jsDeallocate):
     12        (_NPN_CreateScriptObject):
     13        (_NPN_CreateNoScriptObject):
     14        (_NPN_InvokeDefault):
     15        (_NPN_Invoke):
     16        (_NPN_Evaluate):
     17        (_NPN_GetProperty):
     18        (_NPN_SetProperty):
     19        (_NPN_RemoveProperty):
     20        (_NPN_HasProperty):
     21        (_NPN_HasMethod):
     22        (_NPN_Enumerate):
     23        * bridge/npruntime_impl.h:
     24
    1252008-07-04  Simon Hausmann  <hausmann@webkit.org>
    226

  • trunk/WebCore/bridge/NP_jsobject.cpp

    r34982 r35004  
    4747static void getListFromVariantArgs(ExecState* exec, const NPVariant* args, unsigned argCount, RootObject* rootObject, ArgList& aList)
    4848{
    49     for (unsigned i = 0; i < argCount; i++)
     49    for (unsigned i = 0; i < argCount; ++i)
    5050        aList.append(convertNPVariantToValue(exec, &args[i], rootObject));
    5151}
    … …  
    5353static NPObject* jsAllocate(NPP, NPClass*)
    5454{
    55     return (NPObject*)malloc(sizeof(JavaScriptObject));
     55    return static_cast<NPObject*>(malloc(sizeof(JavaScriptObject)));
    5656}
    5757
    5858static void jsDeallocate(NPObject* npObj)
    5959{
    60     JavaScriptObject* obj = (JavaScriptObject*)npObj;
     60    JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(npObj);
    6161
    6262    if (obj->rootObject && obj->rootObject->isValid())
    … …  
    7777NPObject* _NPN_CreateScriptObject(NPP npp, JSObject* imp, PassRefPtr<RootObject> rootObject)
    7878{
    79     JavaScriptObject* obj = (JavaScriptObject*)_NPN_CreateObject(npp, NPScriptObjectClass);
     79    JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(_NPN_CreateObject(npp, NPScriptObjectClass));
    8080
    8181    obj->rootObject = rootObject.releaseRef();
    … …  
    8585    obj->imp = imp;
    8686
    87     return (NPObject*)obj;
    88 }
    89 
    90 NPObject *_NPN_CreateNoScriptObject(void)
     87    return reinterpret_cast<NPObject*>(obj);
     88}
     89
     90NPObject* _NPN_CreateNoScriptObject(void)
    9191{
    9292    return _NPN_CreateObject(0, NPNoScriptObjectClass);
    … …  
    9696{
    9797    if (o->_class == NPScriptObjectClass) {
    98         JavaScriptObject* obj = (JavaScriptObject*)o;
     98        JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(o);
    9999       
    100100        VOID_TO_NPVARIANT(*result);
    … …  
    118118        getListFromVariantArgs(exec, args, argCount, rootObject, argList);
    119119        rootObject->globalObject()->startTimeoutCheck();
    120         JSValue *resultV = call(exec, function, callType, callData, function, argList);
     120        JSValue* resultV = call(exec, function, callType, callData, function, argList);
    121121        rootObject->globalObject()->stopTimeoutCheck();
    122122
    … …  
    135135{
    136136    if (o->_class == NPScriptObjectClass) {
    137         JavaScriptObject* obj = (JavaScriptObject*)o;
    138 
    139         PrivateIdentifier* i = (PrivateIdentifier*)methodName;
     137        JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(o);
     138
     139        PrivateIdentifier* i = static_cast<PrivateIdentifier*>(methodName);
    140140        if (!i->isString)
    141141            return false;
    … …  
    147147            if (args[0].type != NPVariantType_String)
    148148                return false;
    149             return _NPN_Evaluate(npp, o, (NPString *)&args[0].value.stringValue, result);
     149            return _NPN_Evaluate(npp, o, const_cast<NPString*>(&args[0].value.stringValue), result);
    150150        }
    151151
    … …  
    184184{
    185185    if (o->_class == NPScriptObjectClass) {
    186         JavaScriptObject* obj = (JavaScriptObject*)o;
     186        JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(o);
    187187
    188188        RootObject* rootObject = obj->rootObject;
    … …  
    219219{
    220220    if (o->_class == NPScriptObjectClass) {
    221         JavaScriptObject* obj = (JavaScriptObject*)o;
    222 
    223         RootObject* rootObject = obj->rootObject;
    224         if (!rootObject || !rootObject->isValid())
    225             return false;
    226 
    227         ExecState* exec = rootObject->globalObject()->globalExec();
    228         PrivateIdentifier* i = (PrivateIdentifier*)propertyName;
    229        
    230         JSLock lock(false);
    231         JSValue *result;
     221        JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(o);
     222
     223        RootObject* rootObject = obj->rootObject;
     224        if (!rootObject || !rootObject->isValid())
     225            return false;
     226
     227        ExecState* exec = rootObject->globalObject()->globalExec();
     228        PrivateIdentifier* i = static_cast<PrivateIdentifier*>(propertyName);
     229       
     230        JSLock lock(false);
     231        JSValue* result;
    232232        if (i->isString)
    233233            result = obj->imp->get(exec, identifierFromNPIdentifier(i->value.string));
    … …  
    252252{
    253253    if (o->_class == NPScriptObjectClass) {
    254         JavaScriptObject* obj = (JavaScriptObject*)o;
    255 
    256         RootObject* rootObject = obj->rootObject;
    257         if (!rootObject || !rootObject->isValid())
    258             return false;
    259 
    260         ExecState* exec = rootObject->globalObject()->globalExec();
    261         JSLock lock(false);
    262         PrivateIdentifier* i = (PrivateIdentifier*)propertyName;
     254        JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(o);
     255
     256        RootObject* rootObject = obj->rootObject;
     257        if (!rootObject || !rootObject->isValid())
     258            return false;
     259
     260        ExecState* exec = rootObject->globalObject()->globalExec();
     261        JSLock lock(false);
     262        PrivateIdentifier* i = static_cast<PrivateIdentifier*>(propertyName);
    263263        if (i->isString)
    264264            obj->imp->put(exec, identifierFromNPIdentifier(i->value.string), convertNPVariantToValue(exec, variant, rootObject));
    … …  
    277277{
    278278    if (o->_class == NPScriptObjectClass) {
    279         JavaScriptObject* obj = (JavaScriptObject*)o;
    280 
    281         RootObject* rootObject = obj->rootObject;
    282         if (!rootObject || !rootObject->isValid())
    283             return false;
    284 
    285         ExecState* exec = rootObject->globalObject()->globalExec();
    286         PrivateIdentifier* i = (PrivateIdentifier*)propertyName;
     279        JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(o);
     280
     281        RootObject* rootObject = obj->rootObject;
     282        if (!rootObject || !rootObject->isValid())
     283            return false;
     284
     285        ExecState* exec = rootObject->globalObject()->globalExec();
     286        PrivateIdentifier* i = static_cast<PrivateIdentifier*>(propertyName);
    287287        if (i->isString) {
    288288            if (!obj->imp->hasProperty(exec, identifierFromNPIdentifier(i->value.string)))
    … …  
    307307{
    308308    if (o->_class == NPScriptObjectClass) {
    309         JavaScriptObject* obj = (JavaScriptObject*)o;
    310 
    311         RootObject* rootObject = obj->rootObject;
    312         if (!rootObject || !rootObject->isValid())
    313             return false;
    314 
    315         ExecState* exec = rootObject->globalObject()->globalExec();
    316         PrivateIdentifier* i = (PrivateIdentifier*)propertyName;
     309        JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(o);
     310
     311        RootObject* rootObject = obj->rootObject;
     312        if (!rootObject || !rootObject->isValid())
     313            return false;
     314
     315        ExecState* exec = rootObject->globalObject()->globalExec();
     316        PrivateIdentifier* i = static_cast<PrivateIdentifier*>(propertyName);
    317317        JSLock lock(false);
    318318        if (i->isString)
    … …  
    330330{
    331331    if (o->_class == NPScriptObjectClass) {
    332         JavaScriptObject* obj = (JavaScriptObject*)o;
    333 
    334         PrivateIdentifier* i = (PrivateIdentifier*)methodName;
     332        JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(o);
     333
     334        PrivateIdentifier* i = static_cast<PrivateIdentifier*>(methodName);
    335335        if (!i->isString)
    336336            return false;
    … …  
    359359}
    360360
    361 bool _NPN_Enumerate(NPP, NPObject *o, NPIdentifier **identifier, uint32_t *count)
    362 {
    363     if (o->_class == NPScriptObjectClass) {
    364         JavaScriptObject* obj = (JavaScriptObject*)o;
     361bool _NPN_Enumerate(NPP, NPObject* o, NPIdentifier** identifier, uint32_t* count)
     362{
     363    if (o->_class == NPScriptObjectClass) {
     364        JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(o);
    365365       
    366366        RootObject* rootObject = obj->rootObject;
    … …  
    375375        unsigned size = static_cast<unsigned>(propertyNames.size());
    376376        // FIXME: This should really call NPN_MemAlloc but that's in WebKit
    377         NPIdentifier *identifiers = static_cast<NPIdentifier*>(malloc(sizeof(NPIdentifier) * size));
    378        
    379         for (unsigned i = 0; i < size; i++)
     377        NPIdentifier* identifiers = static_cast<NPIdentifier*>(malloc(sizeof(NPIdentifier) * size));
     378       
     379        for (unsigned i = 0; i < size; ++i)
    380380            identifiers[i] = _NPN_GetStringIdentifier(propertyNames[i].ustring().UTF8String().c_str());
    381381

  • trunk/WebCore/bridge/npruntime_impl.h

    r30897 r35004  
    3535#endif
    3636
    37 extern void _NPN_ReleaseVariantValue(NPVariant *variant);
    38 extern NPIdentifier _NPN_GetStringIdentifier(const NPUTF8 *name);
    39 extern void _NPN_GetStringIdentifiers(const NPUTF8 **names, int32_t nameCount, NPIdentifier *identifiers);
    40 extern NPIdentifier _NPN_GetIntIdentifier(int32_t intid);
    41 extern bool _NPN_IdentifierIsString(NPIdentifier identifier);
    42 extern NPUTF8 *_NPN_UTF8FromIdentifier(NPIdentifier identifier);
    43 extern int32_t _NPN_IntFromIdentifier(NPIdentifier identifier);   
    44 extern NPObject *_NPN_CreateObject(NPP npp, NPClass *aClass);
    45 extern NPObject *_NPN_RetainObject(NPObject *obj);
    46 extern void _NPN_ReleaseObject(NPObject *obj);
    47 extern void _NPN_DeallocateObject(NPObject *obj);
    48 extern bool _NPN_Invoke(NPP npp, NPObject *npobj, NPIdentifier methodName, const NPVariant *args, uint32_t argCount, NPVariant *result);
    49 extern bool _NPN_InvokeDefault(NPP npp, NPObject *npobj, const NPVariant *args, uint32_t argCount, NPVariant *result);
    50 extern bool _NPN_Evaluate(NPP npp, NPObject *npobj, NPString *script, NPVariant *result);
    51 extern bool _NPN_GetProperty(NPP npp, NPObject *npobj, NPIdentifier propertyName, NPVariant *result);
    52 extern bool _NPN_SetProperty(NPP npp, NPObject *npobj, NPIdentifier propertyName, const NPVariant *value);
    53 extern bool _NPN_RemoveProperty(NPP npp, NPObject *npobj, NPIdentifier propertyName);
    54 extern bool _NPN_HasProperty(NPP npp, NPObject *npobj, NPIdentifier propertyName);
    55 extern bool _NPN_HasMethod(NPP npp, NPObject *npobj, NPIdentifier methodName);
    56 extern void _NPN_SetException(NPObject *obj, const NPUTF8 *message);
    57 extern bool _NPN_Enumerate(NPP npp, NPObject *npobj, NPIdentifier **identifier, uint32_t *count);
     37extern void _NPN_ReleaseVariantValue(NPVariant*);
     38extern NPIdentifier _NPN_GetStringIdentifier(const NPUTF8*);
     39extern void _NPN_GetStringIdentifiers(const NPUTF8** names, int32_t nameCount, NPIdentifier* identifiers);
     40extern NPIdentifier _NPN_GetIntIdentifier(int32_t);
     41extern bool _NPN_IdentifierIsString(NPIdentifier);
     42extern NPUTF8* _NPN_UTF8FromIdentifier(NPIdentifier);
     43extern int32_t _NPN_IntFromIdentifier(NPIdentifier);   
     44extern NPObject* _NPN_CreateObject(NPP, NPClass*);
     45extern NPObject* _NPN_RetainObject(NPObject*);
     46extern void _NPN_ReleaseObject(NPObject*);
     47extern void _NPN_DeallocateObject(NPObject*);
     48extern bool _NPN_Invoke(NPP, NPObject*, NPIdentifier methodName, const NPVariant* args, uint32_t argCount, NPVariant* result);
     49extern bool _NPN_InvokeDefault(NPP, NPObject*, const NPVariant* args, uint32_t argCount, NPVariant* result);
     50extern bool _NPN_Evaluate(NPP, NPObject*, NPString*, NPVariant* result);
     51extern bool _NPN_GetProperty(NPP, NPObject*, NPIdentifier, NPVariant* result);
     52extern bool _NPN_SetProperty(NPP, NPObject*, NPIdentifier, const NPVariant*);
     53extern bool _NPN_RemoveProperty(NPP, NPObject*, NPIdentifier);
     54extern bool _NPN_HasProperty(NPP, NPObject*, NPIdentifier);
     55extern bool _NPN_HasMethod(NPP, NPObject*, NPIdentifier);
     56extern void _NPN_SetException(NPObject*, const NPUTF8*);
     57extern bool _NPN_Enumerate(NPP, NPObject*, NPIdentifier**, uint32_t* count);
    5858
    5959#ifdef __cplusplus
Note: See TracChangeset for help on using the changeset viewer.