Commit 98a7ac69 authored by darin@apple.com's avatar darin@apple.com

2009-01-05 Darin Adler <darin@apple.com>

        Reviewed by Alexey Proskuryakov.

        Bug 23102: turn on unused parameter warnings in WebCore
        https://bugs.webkit.org/show_bug.cgi?id=23102

        First step: Fix the simple cases where we can just remove an argument name.
        Also made a few more things protected and private.

        * many files



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@39601 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 5b68ea1d
2009-01-05 Darin Adler <darin@apple.com>
Reviewed by Alexey Proskuryakov.
Bug 23102: turn on unused parameter warnings in WebCore
https://bugs.webkit.org/show_bug.cgi?id=23102
First step: Fix the simple cases where we can just remove an argument name.
Also made a few more things protected and private.
* many files
2009-01-05 Alexey Proskuryakov <ap@webkit.org>
Reviewed by Darin Adler.
......@@ -8,6 +20,12 @@
* loader/appcache/ApplicationCacheStorage.cpp:
(WebCore::ApplicationCacheStorage::verifySchemaVersion): Use the new ASSERT_UNUSED macro.
2009-01-05 Alexey Proskuryakov <ap@webkit.org>
Non-Mac build fix.
* loader/appcache/ApplicationCacheStorage.cpp: Include wtf/StringExtras.h for snprintf.
2009-01-05 Adam Treat <treat@kde.org>
Reviewed by George Staikos.
......@@ -16,12 +34,6 @@
* loader/EmptyClients.h:
2009-01-05 Alexey Proskuryakov <ap@webkit.org>
Non-Mac build fix.
* loader/appcache/ApplicationCacheStorage.cpp: Include wtf/StringExtras.h for snprintf.
2009-01-05 Alexey Proskuryakov <ap@webkit.org>
Reviewed by Darin Adler.
......@@ -31,7 +31,7 @@ using namespace JSC;
namespace WebCore {
inline JSValue* JSCanvasPixelArray::getByIndex(ExecState* exec, unsigned index)
inline JSValue* JSCanvasPixelArray::getByIndex(ExecState*, unsigned index)
{
unsigned char result;
if (!impl()->get(index, result))
......
......@@ -49,7 +49,7 @@ static JSValue* toJS(ExecState* exec, CanvasStyle* style)
return jsString(exec, style->color());
}
static PassRefPtr<CanvasStyle> toHTMLCanvasStyle(ExecState* exec, JSValue* value)
static PassRefPtr<CanvasStyle> toHTMLCanvasStyle(ExecState*, JSValue* value)
{
if (value->isString())
return CanvasStyle::create(asString(value)->value());
......
......@@ -95,7 +95,7 @@ JSEventListener* JSDOMGlobalObject::findJSEventListener(JSValue* val, bool isInl
return listeners.get(object);
}
PassRefPtr<JSEventListener> JSDOMGlobalObject::findOrCreateJSEventListener(ExecState* exec, JSValue* val, bool isInline)
PassRefPtr<JSEventListener> JSDOMGlobalObject::findOrCreateJSEventListener(ExecState*, JSValue* val, bool isInline)
{
if (JSEventListener* listener = findJSEventListener(val, isInline))
return listener;
......@@ -107,7 +107,7 @@ PassRefPtr<JSEventListener> JSDOMGlobalObject::findOrCreateJSEventListener(ExecS
return JSEventListener::create(asObject(val), this, isInline).get();
}
JSUnprotectedEventListener* JSDOMGlobalObject::findJSUnprotectedEventListener(ExecState* exec, JSValue* val, bool isInline)
JSUnprotectedEventListener* JSDOMGlobalObject::findJSUnprotectedEventListener(ExecState*, JSValue* val, bool isInline)
{
if (!val->isObject())
return 0;
......
......@@ -302,22 +302,22 @@ DOMWindow* toDOMWindow(JSValue* value)
return 0;
}
JSValue* nonCachingStaticCloseFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* nonCachingStaticCloseFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot&)
{
return new (exec) PrototypeFunction(exec, 0, propertyName, jsDOMWindowPrototypeFunctionClose);
}
JSValue* nonCachingStaticBlurFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* nonCachingStaticBlurFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot&)
{
return new (exec) PrototypeFunction(exec, 0, propertyName, jsDOMWindowPrototypeFunctionBlur);
}
JSValue* nonCachingStaticFocusFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* nonCachingStaticFocusFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot&)
{
return new (exec) PrototypeFunction(exec, 0, propertyName, jsDOMWindowPrototypeFunctionFocus);
}
JSValue* nonCachingStaticPostMessageFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* nonCachingStaticPostMessageFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot&)
{
return new (exec) PrototypeFunction(exec, 2, propertyName, jsDOMWindowPrototypeFunctionPostMessage);
}
......
......@@ -34,7 +34,7 @@ using namespace JSC;
namespace WebCore {
bool JSHTMLFormElement::canGetItemsForName(ExecState* exec, HTMLFormElement* form, const Identifier& propertyName)
bool JSHTMLFormElement::canGetItemsForName(ExecState*, HTMLFormElement* form, const Identifier& propertyName)
{
Vector<RefPtr<Node> > namedItems;
form->getNamedElements(propertyName, namedItems);
......
......@@ -46,7 +46,7 @@ bool JSHTMLFrameSetElement::canGetItemsForName(ExecState*, HTMLFrameSetElement*
return frame && frame->hasTagName(frameTag);
}
JSValue* JSHTMLFrameSetElement::nameGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* JSHTMLFrameSetElement::nameGetter(ExecState*, const Identifier& propertyName, const PropertySlot& slot)
{
JSHTMLElement* thisObj = static_cast<JSHTMLElement*>(asObject(slot.slotBase()));
HTMLElement* element = static_cast<HTMLElement*>(thisObj->impl());
......
......@@ -37,17 +37,17 @@ using namespace JSC;
namespace WebCore {
JSValue* nonCachingStaticBackFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* nonCachingStaticBackFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot&)
{
return new (exec) PrototypeFunction(exec, 0, propertyName, jsHistoryPrototypeFunctionBack);
}
JSValue* nonCachingStaticForwardFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* nonCachingStaticForwardFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot&)
{
return new (exec) PrototypeFunction(exec, 0, propertyName, jsHistoryPrototypeFunctionForward);
}
JSValue* nonCachingStaticGoFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* nonCachingStaticGoFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot&)
{
return new (exec) PrototypeFunction(exec, 1, propertyName, jsHistoryPrototypeFunctionGo);
}
......@@ -92,7 +92,7 @@ bool JSHistory::customGetOwnPropertySlot(ExecState* exec, const Identifier& prop
return true;
}
bool JSHistory::customPut(ExecState* exec, const Identifier& propertyName, JSValue* value, PutPropertySlot&)
bool JSHistory::customPut(ExecState* exec, const Identifier&, JSValue*, PutPropertySlot&)
{
// Only allow putting by frames in the same origin.
if (!allowsAccessFromFrame(exec, impl()->frame()))
......
......@@ -44,7 +44,7 @@ JSValue* JSJavaScriptCallFrame::evaluate(ExecState* exec, const ArgList& args)
return result;
}
JSValue* JSJavaScriptCallFrame::thisObject(ExecState* exec) const
JSValue* JSJavaScriptCallFrame::thisObject(ExecState*) const
{
return impl()->thisObject() ? impl()->thisObject() : jsNull();
}
......
......@@ -37,17 +37,17 @@ using namespace JSC;
namespace WebCore {
JSValue* nonCachingStaticReplaceFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* nonCachingStaticReplaceFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot&)
{
return new (exec) PrototypeFunction(exec, 1, propertyName, jsLocationPrototypeFunctionReplace);
}
JSValue* nonCachingStaticReloadFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* nonCachingStaticReloadFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot&)
{
return new (exec) PrototypeFunction(exec, 0, propertyName, jsLocationPrototypeFunctionReload);
}
JSValue* nonCachingStaticAssignFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* nonCachingStaticAssignFunctionGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot&)
{
return new (exec) PrototypeFunction(exec, 1, propertyName, jsLocationPrototypeFunctionAssign);
}
......@@ -261,7 +261,7 @@ JSValue* JSLocation::replace(ExecState* exec, const ArgList& args)
return jsUndefined();
}
JSValue* JSLocation::reload(ExecState* exec, const ArgList& args)
JSValue* JSLocation::reload(ExecState* exec, const ArgList&)
{
Frame* frame = impl()->frame();
if (!frame)
......
......@@ -48,13 +48,13 @@ JSNamedNodesCollection::JSNamedNodesCollection(ExecState* exec, const Vector<Ref
{
}
JSValue* JSNamedNodesCollection::lengthGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* JSNamedNodesCollection::lengthGetter(ExecState* exec, const Identifier&, const PropertySlot& slot)
{
JSNamedNodesCollection* thisObj = static_cast<JSNamedNodesCollection*>(asObject(slot.slotBase()));
return jsNumber(exec, thisObj->m_nodes->size());
}
JSValue* JSNamedNodesCollection::indexGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* JSNamedNodesCollection::indexGetter(ExecState* exec, const Identifier&, const PropertySlot& slot)
{
JSNamedNodesCollection *thisObj = static_cast<JSNamedNodesCollection*>(asObject(slot.slotBase()));
return toJS(exec, (*thisObj->m_nodes)[slot.index()].get());
......
......@@ -37,7 +37,7 @@ void JSNodeIterator::mark()
DOMObject::mark();
}
JSValue* JSNodeIterator::nextNode(ExecState* exec, const ArgList& args)
JSValue* JSNodeIterator::nextNode(ExecState* exec, const ArgList&)
{
ExceptionCode ec = 0;
RefPtr<Node> node = impl()->nextNode(exec, ec);
......@@ -52,7 +52,7 @@ JSValue* JSNodeIterator::nextNode(ExecState* exec, const ArgList& args)
return toJS(exec, node.get());
}
JSValue* JSNodeIterator::previousNode(ExecState* exec, const ArgList& args)
JSValue* JSNodeIterator::previousNode(ExecState* exec, const ArgList&)
{
ExceptionCode ec = 0;
RefPtr<Node> node = impl()->previousNode(exec, ec);
......
......@@ -61,7 +61,7 @@ static RuntimeObjectImp* getRuntimeObject(ExecState* exec, Node* node)
return JSC::Bindings::Instance::createRuntimeObject(exec, instance);
}
JSValue* runtimeObjectGetter(ExecState* exec, const Identifier& propertyName, const PropertySlot& slot)
JSValue* runtimeObjectGetter(ExecState* exec, const Identifier&, const PropertySlot& slot)
{
JSHTMLElement* thisObj = static_cast<JSHTMLElement*>(asObject(slot.slotBase()));
HTMLElement* element = static_cast<HTMLElement*>(thisObj->impl());
......
......@@ -267,7 +267,7 @@ CallType JSQuarantinedObjectWrapper::getCallData(CallData& callData)
return CallTypeHost;
}
void JSQuarantinedObjectWrapper::getPropertyNames(ExecState* exec, PropertyNameArray& array)
void JSQuarantinedObjectWrapper::getPropertyNames(ExecState*, PropertyNameArray& array)
{
if (!allowsGetPropertyNames())
return;
......
......@@ -191,7 +191,7 @@ public:
return (m_parent.get()->*m_getter)();
}
virtual void commitChange(PODType type, SVGElement* context)
virtual void commitChange(PODType type, SVGElement*)
{
(m_parent.get()->*m_setter)(type);
}
......
......@@ -35,7 +35,7 @@ using namespace JSC;
namespace WebCore {
JSValue* JSSVGPathSegList::clear(ExecState* exec, const ArgList& args)
JSValue* JSSVGPathSegList::clear(ExecState* exec, const ArgList&)
{
ExceptionCode ec = 0;
......
......@@ -37,7 +37,7 @@ void JSTreeWalker::mark()
DOMObject::mark();
}
JSValue* JSTreeWalker::parentNode(ExecState* exec, const ArgList& args)
JSValue* JSTreeWalker::parentNode(ExecState* exec, const ArgList&)
{
Node* node = impl()->parentNode(exec);
if (exec->hadException())
......@@ -45,7 +45,7 @@ JSValue* JSTreeWalker::parentNode(ExecState* exec, const ArgList& args)
return toJS(exec, node);
}
JSValue* JSTreeWalker::firstChild(ExecState* exec, const ArgList& args)
JSValue* JSTreeWalker::firstChild(ExecState* exec, const ArgList&)
{
Node* node = impl()->firstChild(exec);
if (exec->hadException())
......@@ -53,7 +53,7 @@ JSValue* JSTreeWalker::firstChild(ExecState* exec, const ArgList& args)
return toJS(exec, node);
}
JSValue* JSTreeWalker::lastChild(ExecState* exec, const ArgList& args)
JSValue* JSTreeWalker::lastChild(ExecState* exec, const ArgList&)
{
Node* node = impl()->lastChild(exec);
if (exec->hadException())
......@@ -61,7 +61,7 @@ JSValue* JSTreeWalker::lastChild(ExecState* exec, const ArgList& args)
return toJS(exec, node);
}
JSValue* JSTreeWalker::nextSibling(ExecState* exec, const ArgList& args)
JSValue* JSTreeWalker::nextSibling(ExecState* exec, const ArgList&)
{
Node* node = impl()->nextSibling(exec);
if (exec->hadException())
......@@ -69,7 +69,7 @@ JSValue* JSTreeWalker::nextSibling(ExecState* exec, const ArgList& args)
return toJS(exec, node);
}
JSValue* JSTreeWalker::previousSibling(ExecState* exec, const ArgList& args)
JSValue* JSTreeWalker::previousSibling(ExecState* exec, const ArgList&)
{
Node* node = impl()->previousSibling(exec);
if (exec->hadException())
......@@ -77,7 +77,7 @@ JSValue* JSTreeWalker::previousSibling(ExecState* exec, const ArgList& args)
return toJS(exec, node);
}
JSValue* JSTreeWalker::previousNode(ExecState* exec, const ArgList& args)
JSValue* JSTreeWalker::previousNode(ExecState* exec, const ArgList&)
{
Node* node = impl()->previousNode(exec);
if (exec->hadException())
......@@ -85,7 +85,7 @@ JSValue* JSTreeWalker::previousNode(ExecState* exec, const ArgList& args)
return toJS(exec, node);
}
JSValue* JSTreeWalker::nextNode(ExecState* exec, const ArgList& args)
JSValue* JSTreeWalker::nextNode(ExecState* exec, const ArgList&)
{
Node* node = impl()->nextNode(exec);
if (exec->hadException())
......
......@@ -47,7 +47,7 @@ JSWorkerConstructor::JSWorkerConstructor(ExecState* exec)
putDirect(exec->propertyNames().length, jsNumber(exec, 1), ReadOnly|DontDelete|DontEnum);
}
static JSObject* constructWorker(ExecState* exec, JSObject* constructor, const ArgList& args)
static JSObject* constructWorker(ExecState* exec, JSObject*, const ArgList& args)
{
if (args.size() == 0)
return throwError(exec, SyntaxError, "Not enough arguments");
......
......@@ -65,7 +65,7 @@ void JSWorkerContext::mark()
}
}
JSValue* JSWorkerContext::self(ExecState* exec) const
JSValue* JSWorkerContext::self(ExecState*) const
{
return asValue();
}
......
......@@ -47,7 +47,7 @@ JSXSLTProcessorConstructor::JSXSLTProcessorConstructor(ExecState* exec)
putDirect(exec->propertyNames().prototype, JSXSLTProcessorPrototype::self(exec), None);
}
static JSObject* constructXSLTProcessor(ExecState* exec, JSObject*, const ArgList& args)
static JSObject* constructXSLTProcessor(ExecState* exec, JSObject*, const ArgList&)
{
return CREATE_DOM_OBJECT_WRAPPER(exec, XSLTProcessor, XSLTProcessor::create().get());
}
......
......@@ -134,9 +134,10 @@ void ScriptController::disconnectPlatformScriptObjects()
}
#if ENABLE(MAC_JAVA_BRIDGE)
static pthread_t mainThread;
static void updateRenderingForBindings(JSC::ExecState* exec, JSC::JSObject* rootObject)
static void updateRenderingForBindings(JSC::ExecState*, JSC::JSObject* rootObject)
{
if (pthread_self() != mainThread)
return;
......@@ -165,6 +166,7 @@ void ScriptController::initJavaJSBindings()
JSC::Bindings::JavaJSObject::initializeJNIThreading();
JSC::Bindings::Instance::setDidExecuteFunction(updateRenderingForBindings);
}
#endif
}
......@@ -416,7 +416,7 @@ bool _NPN_Enumerate(NPP, NPObject* o, NPIdentifier** identifier, uint32_t* count
return false;
}
bool _NPN_Construct(NPP npp, NPObject* o, const NPVariant* args, uint32_t argCount, NPVariant* result)
bool _NPN_Construct(NPP, NPObject* o, const NPVariant* args, uint32_t argCount, NPVariant* result)
{
if (o->_class == NPScriptObjectClass) {
JavaScriptObject* obj = reinterpret_cast<JavaScriptObject*>(o);
......
......@@ -47,7 +47,7 @@ RuntimeObjectImp::RuntimeObjectImp(ExecState* exec, PassRefPtr<Instance> i)
instance->rootObject()->addRuntimeObject(this);
}
RuntimeObjectImp::RuntimeObjectImp(ExecState* exec, PassRefPtr<Structure> structure, PassRefPtr<Instance> i)
RuntimeObjectImp::RuntimeObjectImp(ExecState*, PassRefPtr<Structure> structure, PassRefPtr<Instance> i)
: JSObject(structure)
, instance(i)
{
......@@ -172,7 +172,7 @@ bool RuntimeObjectImp::getOwnPropertySlot(ExecState *exec, const Identifier& pro
return false;
}
void RuntimeObjectImp::put(ExecState* exec, const Identifier& propertyName, JSValue* value, PutPropertySlot& slot)
void RuntimeObjectImp::put(ExecState* exec, const Identifier& propertyName, JSValue* value, PutPropertySlot&)
{
if (!instance) {
throwInvalidAccessError(exec);
......
......@@ -44,7 +44,7 @@ String CSSCanvasValue::cssText() const
return result;
}
void CSSCanvasValue::canvasChanged(HTMLCanvasElement* element, const FloatRect& changedRect)
void CSSCanvasValue::canvasChanged(HTMLCanvasElement*, const FloatRect& changedRect)
{
IntRect imageChangeRect = enclosingIntRect(changedRect);
HashMap<RenderObject*, IntSize>::const_iterator end = m_clients.end();
......@@ -52,7 +52,7 @@ void CSSCanvasValue::canvasChanged(HTMLCanvasElement* element, const FloatRect&
curr->first->imageChanged(static_cast<WrappedImagePtr>(this), &imageChangeRect);
}
void CSSCanvasValue::canvasResized(HTMLCanvasElement* element)
void CSSCanvasValue::canvasResized(HTMLCanvasElement*)
{
HashMap<RenderObject*, IntSize>::const_iterator end = m_clients.end();
for (HashMap<RenderObject*, IntSize>::const_iterator curr = m_clients.begin(); curr != end; ++curr)
......
......@@ -43,8 +43,8 @@ public:
virtual PassRefPtr<CSSValue> getPropertyCSSValue(int propertyID) const;
virtual String getPropertyValue(int propertyID) const;
virtual bool getPropertyPriority(int propertyID) const;
virtual int getPropertyShorthand(int propertyID) const { return -1; }
virtual bool isPropertyImplicit(int propertyID) const { return false; }
virtual int getPropertyShorthand(int /*propertyID*/) const { return -1; }
virtual bool isPropertyImplicit(int /*propertyID*/) const { return false; }
virtual PassRefPtr<CSSMutableStyleDeclaration> copy() const;
virtual PassRefPtr<CSSMutableStyleDeclaration> makeMutable();
......
......@@ -4566,23 +4566,37 @@ CSSRule* CSSParser::createFontFaceRule()
return result;
}
#if !ENABLE(CSS_VARIABLES)
CSSRule* CSSParser::createVariablesRule(MediaList*, bool)
{
return 0;
}
bool CSSParser::addVariable(const CSSParserString&, CSSParserValueList*)
{
return false;
}
bool CSSParser::addVariableDeclarationBlock(const CSSParserString&)
{
return false;
}
#else
CSSRule* CSSParser::createVariablesRule(MediaList* mediaList, bool variablesKeyword)
{
#if ENABLE(CSS_VARIABLES)
RefPtr<CSSVariablesRule> rule = CSSVariablesRule::create(m_styleSheet, mediaList, variablesKeyword);
rule->setDeclaration(CSSVariablesDeclaration::create(rule.get(), m_variableNames, m_variableValues));
clearVariables();
CSSRule* result = rule.get();
m_parsedStyleObjects.append(rule.release());
return result;
#else
return 0;
#endif
}
bool CSSParser::addVariable(const CSSParserString& name, CSSParserValueList* valueList)
{
#if ENABLE(CSS_VARIABLES)
if (checkForVariables(valueList)) {
delete valueList;
return false;
......@@ -4590,24 +4604,21 @@ bool CSSParser::addVariable(const CSSParserString& name, CSSParserValueList* val
m_variableNames.append(String(name));
m_variableValues.append(CSSValueList::createFromParserValueList(valueList));
return true;
#else
return false;
#endif
}
bool CSSParser::addVariableDeclarationBlock(const CSSParserString& name)
bool CSSParser::addVariableDeclarationBlock(const CSSParserString&)
{
// FIXME: Disabling declarations as variable values for now since they no longer have a common base class with CSSValues.
#if ENABLE(CSS_VARIABLES) && 0
#if 0
m_variableNames.append(String(name));
m_variableValues.append(CSSMutableStyleDeclaration::create(0, m_parsedProperties, m_numParsedProperties));
clearProperties();
return true;
#else
return false;
#endif
return true;
}
#endif
void CSSParser::clearVariables()
{
m_variableNames.clear();
......
......@@ -55,7 +55,7 @@ public:
virtual String cssText() const = 0;
void setCssText(const String&, ExceptionCode&);
virtual void addSubresourceStyleURLs(ListHashSet<KURL>& urls) { }
virtual void addSubresourceStyleURLs(ListHashSet<KURL>&) { }
protected:
CSSRule(CSSStyleSheet* parent)
......
......@@ -1606,7 +1606,7 @@ PassRefPtr<CSSRuleList> CSSStyleSelector::styleRulesForElement(Element* e, bool
return m_ruleList.release();
}
PassRefPtr<CSSRuleList> CSSStyleSelector::pseudoStyleRulesForElement(Element*, const String& pseudoStyle, bool authorOnly)
PassRefPtr<CSSRuleList> CSSStyleSelector::pseudoStyleRulesForElement(Element*, const String&, bool)
{
// FIXME: Implement this.
return 0;
......@@ -2424,7 +2424,7 @@ bool CSSStyleSelector::SelectorChecker::checkOneSelector(CSSSelector* sel, Eleme
return true;
}
bool CSSStyleSelector::SelectorChecker::checkScrollbarPseudoClass(CSSSelector* sel, RenderStyle::PseudoId& dynamicPseudo) const
bool CSSStyleSelector::SelectorChecker::checkScrollbarPseudoClass(CSSSelector* sel, RenderStyle::PseudoId&) const
{
RenderScrollbar* scrollbar = RenderScrollbar::scrollbarForStyleResolve();
ScrollbarPart part = RenderScrollbar::partForStyleResolve();
......@@ -2512,7 +2512,7 @@ void CSSStyleSelector::addVariables(CSSVariablesRule* variables)
}
}
CSSValue* CSSStyleSelector::resolveVariableDependentValue(CSSVariableDependentValue* val)
CSSValue* CSSStyleSelector::resolveVariableDependentValue(CSSVariableDependentValue*)
{
return 0;
}