Chromium Code Reviews| Index: Source/bindings/templates/interface.cpp |
| diff --git a/Source/bindings/templates/interface.cpp b/Source/bindings/templates/interface.cpp |
| index cfa97977b98aa13218b17a3569e4ba3ffe96c952..7c3e2e49c895730c0662e07f88eb6a06f5ae93b0 100644 |
| --- a/Source/bindings/templates/interface.cpp |
| +++ b/Source/bindings/templates/interface.cpp |
| @@ -242,7 +242,7 @@ static v8::Handle<v8::FunctionTemplate> Configure{{v8_class}}Template(v8::Handle |
| { |
| functionTemplate->ReadOnlyPrototype(); |
| - v8::Local<v8::Signature> defaultSignature; |
| + v8::Local<v8::Signature> ALLOW_UNUSED defaultSignature; |
| {% if runtime_enabled_function %} |
| if (!{{runtime_enabled_function}}()) |
| {# FIXME: support inheritance #} |
| @@ -271,16 +271,13 @@ static v8::Handle<v8::FunctionTemplate> Configure{{v8_class}}Template(v8::Handle |
| isolate, currentWorldType); |
| {% endfilter %} |
| - UNUSED_PARAM(defaultSignature); |
| {% if has_constructor %} |
| functionTemplate->SetCallHandler({{v8_class}}::constructorCallback); |
| {# FIXME: compute length #} |
| functionTemplate->SetLength(0); |
| {% endif %} |
| - v8::Local<v8::ObjectTemplate> instanceTemplate = functionTemplate->InstanceTemplate(); |
| - v8::Local<v8::ObjectTemplate> prototypeTemplate = functionTemplate->PrototypeTemplate(); |
| - UNUSED_PARAM(instanceTemplate); |
| - UNUSED_PARAM(prototypeTemplate); |
| + v8::Local<v8::ObjectTemplate> ALLOW_UNUSED instanceTemplate = functionTemplate->InstanceTemplate(); |
| + v8::Local<v8::ObjectTemplate> ALLOW_UNUSED prototypeTemplate = functionTemplate->PrototypeTemplate(); |
| {% if is_check_security and interface_name != 'Window' %} |
| instanceTemplate->SetAccessCheckCallbacks({{cpp_class}}V8Internal::namedSecurityCheck, {{cpp_class}}V8Internal::indexedSecurityCheck, v8::External::New(isolate, const_cast<WrapperTypeInfo*>(&{{v8_class}}::wrapperTypeInfo))); |
| {% endif %} |
| @@ -480,10 +477,8 @@ void {{v8_class}}::installPerContextEnabledProperties(v8::Handle<v8::Object> ins |
| {% if has_per_context_enabled_methods %} |
| void {{v8_class}}::installPerContextEnabledMethods(v8::Handle<v8::Object> prototypeTemplate, v8::Isolate* isolate) |
| { |
| - UNUSED_PARAM(prototypeTemplate); |
| {# Define per-context enabled operations #} |
| - v8::Local<v8::Signature> defaultSignature = v8::Signature::New(isolate, GetTemplate(isolate, worldType(isolate))); |
| - UNUSED_PARAM(defaultSignature); |
| + v8::Local<v8::Signature> ALLOW_UNUSED defaultSignature = v8::Signature::New(isolate, GetTemplate(isolate, worldType(isolate))); |
|
Inactive
2013/12/09 15:24:47
Nit: We probably don't need ALLOW_UNUSED here beca
|
| ExecutionContext* context = toExecutionContext(prototypeTemplate->CreationContext()); |
| {% for method in methods if method.per_context_enabled_function %} |