2009-01-05 Darin Adler <darin@apple.com>
authordarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 5 Jan 2009 17:26:53 +0000 (17:26 +0000)
committerdarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 5 Jan 2009 17:26:53 +0000 (17:26 +0000)
        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: https://svn.webkit.org/repository/webkit/trunk@39601 268f45cc-cd09-0410-ab3c-d52691b4dbfc

192 files changed:
WebCore/ChangeLog
WebCore/bindings/js/JSCanvasPixelArrayCustom.h
WebCore/bindings/js/JSCanvasRenderingContext2DCustom.cpp
WebCore/bindings/js/JSDOMGlobalObject.cpp
WebCore/bindings/js/JSDOMWindowCustom.cpp
WebCore/bindings/js/JSHTMLFormElementCustom.cpp
WebCore/bindings/js/JSHTMLFrameSetElementCustom.cpp
WebCore/bindings/js/JSHistoryCustom.cpp
WebCore/bindings/js/JSJavaScriptCallFrameCustom.cpp
WebCore/bindings/js/JSLocationCustom.cpp
WebCore/bindings/js/JSNamedNodesCollection.cpp
WebCore/bindings/js/JSNodeIteratorCustom.cpp
WebCore/bindings/js/JSPluginElementFunctions.cpp
WebCore/bindings/js/JSQuarantinedObjectWrapper.cpp
WebCore/bindings/js/JSSVGPODTypeWrapper.h
WebCore/bindings/js/JSSVGPathSegListCustom.cpp
WebCore/bindings/js/JSTreeWalkerCustom.cpp
WebCore/bindings/js/JSWorkerConstructor.cpp
WebCore/bindings/js/JSWorkerContextCustom.cpp
WebCore/bindings/js/JSXSLTProcessorConstructor.cpp
WebCore/bindings/js/ScriptControllerMac.mm
WebCore/bridge/NP_jsobject.cpp
WebCore/bridge/runtime_object.cpp
WebCore/css/CSSCanvasValue.cpp
WebCore/css/CSSComputedStyleDeclaration.h
WebCore/css/CSSParser.cpp
WebCore/css/CSSRule.h
WebCore/css/CSSStyleSelector.cpp
WebCore/css/CSSVariablesDeclaration.cpp
WebCore/css/MediaQueryEvaluator.cpp
WebCore/dom/EditingText.cpp
WebCore/dom/Element.cpp
WebCore/dom/Element.h
WebCore/dom/EventTarget.h
WebCore/dom/MessagePort.cpp
WebCore/dom/Node.cpp
WebCore/dom/Node.h
WebCore/dom/Position.cpp
WebCore/dom/QualifiedName.cpp
WebCore/dom/StyleElement.cpp
WebCore/dom/Text.cpp
WebCore/dom/Tokenizer.h
WebCore/dom/Worker.cpp
WebCore/dom/WorkerContext.cpp
WebCore/dom/WorkerMessagingProxy.cpp
WebCore/dom/XMLTokenizerLibxml2.cpp
WebCore/editing/Editor.cpp
WebCore/editing/EditorCommand.cpp
WebCore/html/CanvasRenderingContext2D.cpp
WebCore/html/HTMLAppletElement.cpp
WebCore/html/HTMLButtonElement.cpp
WebCore/html/HTMLEmbedElement.cpp
WebCore/html/HTMLFieldSetElement.cpp
WebCore/html/HTMLFormControlElement.h
WebCore/html/HTMLFrameElement.cpp
WebCore/html/HTMLOptGroupElement.cpp
WebCore/html/HTMLOptionElement.cpp
WebCore/html/HTMLParser.cpp
WebCore/html/HTMLSelectElement.cpp
WebCore/inspector/InspectorController.cpp
WebCore/inspector/JavaScriptProfile.cpp
WebCore/inspector/JavaScriptProfileNode.cpp
WebCore/loader/CachedFont.cpp
WebCore/loader/EmptyClients.h
WebCore/loader/FTPDirectoryDocument.cpp
WebCore/loader/FrameLoader.cpp
WebCore/loader/ImageDocument.cpp
WebCore/loader/ImageLoader.cpp
WebCore/loader/MainResourceLoader.cpp
WebCore/loader/MediaDocument.cpp
WebCore/loader/PluginDocument.cpp
WebCore/loader/ResourceLoader.cpp
WebCore/loader/SubresourceLoaderClient.h
WebCore/loader/TextDocument.cpp
WebCore/loader/appcache/ApplicationCache.cpp
WebCore/loader/appcache/ApplicationCacheGroup.cpp
WebCore/loader/appcache/DOMApplicationCache.cpp
WebCore/loader/icon/IconDatabaseClient.h
WebCore/loader/icon/IconLoader.cpp
WebCore/loader/icon/IconRecord.cpp
WebCore/loader/mac/ResourceLoaderMac.mm
WebCore/page/AccessibilityObject.cpp
WebCore/page/Chrome.cpp
WebCore/page/DOMWindow.cpp
WebCore/page/EventHandler.cpp
WebCore/page/Frame.cpp
WebCore/page/animation/AnimationBase.cpp
WebCore/page/animation/AnimationBase.h
WebCore/page/animation/AnimationController.cpp
WebCore/page/animation/CompositeAnimation.cpp
WebCore/page/animation/ImplicitAnimation.cpp
WebCore/page/animation/KeyframeAnimation.cpp
WebCore/platform/Arena.cpp
WebCore/platform/ScrollbarTheme.h
WebCore/platform/Theme.cpp
WebCore/platform/Theme.h
WebCore/platform/graphics/BitmapImage.cpp
WebCore/platform/graphics/FontFastPath.cpp
WebCore/platform/graphics/GeneratedImage.h
WebCore/platform/graphics/Image.h
WebCore/platform/graphics/cg/ImageSourceCG.cpp
WebCore/platform/graphics/cg/PDFDocumentImage.h
WebCore/platform/graphics/mac/FontMacATSUI.mm
WebCore/platform/graphics/transforms/IdentityTransformOperation.h
WebCore/platform/graphics/transforms/MatrixTransformOperation.h
WebCore/platform/graphics/transforms/RotateTransformOperation.h
WebCore/platform/graphics/transforms/ScaleTransformOperation.h
WebCore/platform/graphics/transforms/SkewTransformOperation.h
WebCore/platform/graphics/transforms/TransformOperation.h
WebCore/platform/mac/DragDataMac.mm
WebCore/platform/network/AuthenticationChallengeBase.h
WebCore/platform/network/ResourceErrorBase.h
WebCore/platform/network/ResourceHandleClient.h
WebCore/platform/network/ResourceResponseBase.h
WebCore/platform/network/cf/DNSCFNet.cpp
WebCore/platform/network/mac/FormDataStreamMac.mm
WebCore/platform/network/mac/NetworkStateNotifierMac.cpp
WebCore/platform/text/TextCodecUTF16.cpp
WebCore/rendering/FixedTableLayout.cpp
WebCore/rendering/InlineBox.cpp
WebCore/rendering/InlineFlowBox.h
WebCore/rendering/InlineRunBox.h
WebCore/rendering/InlineTextBox.cpp
WebCore/rendering/RenderBR.cpp
WebCore/rendering/RenderBlock.cpp
WebCore/rendering/RenderBlock.h
WebCore/rendering/RenderBox.cpp
WebCore/rendering/RenderBox.h
WebCore/rendering/RenderContainer.cpp
WebCore/rendering/RenderFlow.cpp
WebCore/rendering/RenderFrameSet.cpp
WebCore/rendering/RenderImage.cpp
WebCore/rendering/RenderImage.h
WebCore/rendering/RenderLayer.cpp
WebCore/rendering/RenderListBox.cpp
WebCore/rendering/RenderObject.cpp
WebCore/rendering/RenderObject.h
WebCore/rendering/RenderPath.cpp
WebCore/rendering/RenderReplaced.cpp
WebCore/rendering/RenderReplaced.h
WebCore/rendering/RenderSVGContainer.cpp
WebCore/rendering/RenderSVGHiddenContainer.cpp
WebCore/rendering/RenderSVGImage.cpp
WebCore/rendering/RenderSVGInlineText.cpp
WebCore/rendering/RenderSVGRoot.cpp
WebCore/rendering/RenderSVGTSpan.cpp
WebCore/rendering/RenderSVGText.cpp
WebCore/rendering/RenderSVGTextPath.cpp
WebCore/rendering/RenderSlider.cpp
WebCore/rendering/RenderTableCol.cpp
WebCore/rendering/RenderTableRow.cpp
WebCore/rendering/RenderTableRow.h
WebCore/rendering/RenderTableSection.cpp
WebCore/rendering/RenderTableSection.h
WebCore/rendering/RenderText.cpp
WebCore/rendering/RenderText.h
WebCore/rendering/RenderTextControlSingleLine.cpp
WebCore/rendering/RenderTheme.cpp
WebCore/rendering/RenderThemeMac.mm
WebCore/rendering/RenderView.cpp
WebCore/rendering/SVGInlineFlowBox.cpp
WebCore/rendering/SVGInlineTextBox.cpp
WebCore/rendering/style/FillLayer.h
WebCore/rendering/style/StyleCachedImage.cpp
WebCore/rendering/style/StyleImage.h
WebCore/storage/DatabaseAuthorizer.cpp
WebCore/storage/DatabaseAuthorizer.h
WebCore/storage/StorageArea.cpp
WebCore/svg/SVGAElement.cpp
WebCore/svg/SVGAltGlyphElement.cpp
WebCore/svg/SVGAnimateMotionElement.cpp
WebCore/svg/SVGAnimationElement.cpp
WebCore/svg/SVGAnimationElement.h
WebCore/svg/SVGColor.cpp
WebCore/svg/SVGFont.cpp
WebCore/svg/SVGForeignObjectElement.cpp
WebCore/svg/SVGImageElement.cpp
WebCore/svg/SVGMarkerElement.cpp
WebCore/svg/SVGParserUtilities.cpp
WebCore/svg/SVGSVGElement.cpp
WebCore/svg/SVGTextContentElement.cpp
WebCore/svg/SVGTextElement.cpp
WebCore/svg/SVGTextPathElement.cpp
WebCore/svg/animation/SVGSMILElement.cpp
WebCore/svg/graphics/SVGImage.cpp
WebCore/svg/graphics/SVGImage.h
WebCore/svg/graphics/SVGPaintServerGradient.cpp
WebCore/svg/graphics/cg/SVGPaintServerCg.cpp
WebCore/xml/XMLHttpRequest.cpp
WebCore/xml/XPathGrammar.y
WebCore/xml/XSLStyleSheet.cpp
WebCore/xml/XSLTProcessor.cpp

index 48dc797..9f69070 100644 (file)
@@ -1,3 +1,15 @@
+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.
 
 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.
 
         https://bugs.webkit.org/show_bug.cgi?id=23077
index 9bdcccd..052727b 100644 (file)
@@ -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))
index a2f4f65..cb44024 100644 (file)
@@ -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());
index 5505cbf..0cc2a2f 100644 (file)
@@ -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;
index 4b5f386..2312b42 100644 (file)
@@ -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);
 }
index 196cb6d..df9e345 100644 (file)
@@ -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);
index 126ad8b..71a5a66 100644 (file)
@@ -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());
index 31699a0..aa940c2 100644 (file)
@@ -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()))
index ce16b31..36679bb 100644 (file)
@@ -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();
 }
index a5ad179..1094e0d 100644 (file)
@@ -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)
index ccf90f7..00727b7 100644 (file)
@@ -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());
index 445c8cb..7adbe78 100644 (file)
@@ -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);
index 7bd1a3d..6b0dd6b 100644 (file)
@@ -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());
index a82d872..ba593a2 100644 (file)
@@ -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;
index 1c1ba7c..ef81433 100644 (file)
@@ -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);
     }
index 95b60aa..69b424c 100644 (file)
@@ -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;
 
index 9822e1a..f9fb432 100644 (file)
@@ -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())
index 8f7bfa2..2c964c3 100644 (file)
@@ -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");
index 24303f6..10562f5 100644 (file)
@@ -65,7 +65,7 @@ void JSWorkerContext::mark()
     }
 }
 
-JSValue* JSWorkerContext::self(ExecState* exec) const
+JSValue* JSWorkerContext::self(ExecState*) const
 {
     return asValue();
 }
index 7486ecd..ed456e1 100644 (file)
@@ -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());
 }
index 91ef03c..39fcc67 100644 (file)
@@ -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
 
 }
index 11ecc3e..c687b08 100644 (file)
@@ -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);
index 3c00cac..e0e6606 100644 (file)
@@ -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);
index b3281ef..4fd5210 100644 (file)
@@ -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)
index a9fd67b..487c02b 100644 (file)
@@ -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();
index 1554b22..e8faca8 100644 (file)
@@ -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();
index 24d1554..fc48dd6 100644 (file)
@@ -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)
index f0159f3..03d66d0 100644 (file)
@@ -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;
 }
@@ -5601,14 +5601,14 @@ float CSSStyleSelector::fontSizeForKeyword(int keyword, bool quirksMode, bool fi
     return max(fontSizeFactors[keyword - CSSValueXxSmall]*mediumSize, minLogicalSize);
 }
 
-float CSSStyleSelector::largerFontSize(float size, bool quirksMode) const
+float CSSStyleSelector::largerFontSize(float size, bool) const
 {
     // FIXME: Figure out where we fall in the size ranges (xx-small to xxx-large) and scale up to
     // the next size level.  
     return size * 1.2f;
 }
 
-float CSSStyleSelector::smallerFontSize(float size, bool quirksMode) const
+float CSSStyleSelector::smallerFontSize(float size, bool) const
 {
     // FIXME: Figure out where we fall in the size ranges (xx-small to xxx-large) and scale down to
     // the next size level. 
index d1a0d78..5f8d601 100644 (file)
@@ -114,7 +114,7 @@ CSSValueList* CSSVariablesDeclaration::getParsedVariable(const String& variableN
     return 0;
 }
 
-CSSMutableStyleDeclaration* CSSVariablesDeclaration::getParsedVariableDeclarationBlock(const String& variableName)
+CSSMutableStyleDeclaration* CSSVariablesDeclaration::getParsedVariableDeclarationBlock(const String&)
 {
 // FIXME: Disabling declarations as variable values for now since they no longer have a common base class with CSSValues.
 #if 0
index 7e71969..c3888c2 100644 (file)
@@ -51,7 +51,7 @@ using namespace MediaFeatureNames;
 
 enum MediaFeaturePrefix { MinPrefix, MaxPrefix, NoPrefix };
 
-typedef bool (*EvalFunc)(CSSValue*, RenderStyle*, Frame*,  MediaFeaturePrefix);
+typedef bool (*EvalFunc)(CSSValue*, RenderStyle*, Frame*, MediaFeaturePrefix);
 typedef HashMap<AtomicStringImpl*, EvalFunc> FunctionMap;
 static FunctionMap* gFunctionMap;
 
@@ -63,7 +63,7 @@ static FunctionMap* gFunctionMap;
  * scan: The "scan" media feature describes the scanning process of
  * tv output devices. It's unknown how to retrieve this information from
  * the platform
- * resolution, min-resolution,  max-resolution: css parser doesn't seem to
+ * resolution, min-resolution, max-resolution: css parser doesn't seem to
  * support CSS_DIMENSION
  */
 
@@ -207,7 +207,7 @@ static bool numberValue(CSSValue* value, float& result)
     return false;
 }
 
-static bool colorMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix op)
+static bool colorMediaFeatureEval(CSSValue* value, RenderStyle*, Frame* frame, MediaFeaturePrefix op)
 {
     int bitsPerComponent = screenDepthPerComponent(frame->page()->mainFrame()->view());
     float number;
@@ -217,7 +217,7 @@ static bool colorMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* fr
     return bitsPerComponent != 0;
 }
 
-static bool monochromeMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix op)
+static bool monochromeMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix op)
 {
     if (!screenIsMonochrome(frame->page()->mainFrame()->view())) {
         if (value) {
@@ -230,7 +230,7 @@ static bool monochromeMediaFeatureEval(CSSValue* value, RenderStyle* style, Fram
     return colorMediaFeatureEval(value, style, frame, op);
 }
 
-static bool device_aspect_ratioMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix op)
+static bool device_aspect_ratioMediaFeatureEval(CSSValue* value, RenderStyle*, Frame* frame, MediaFeaturePrefix op)
 {
     if (value) {
         FloatRect sg = screenRect(frame->page()->mainFrame()->view());
@@ -246,7 +246,7 @@ static bool device_aspect_ratioMediaFeatureEval(CSSValue* value, RenderStyle* st
     return true;
 }
 
-static bool device_pixel_ratioMediaFeatureEval(CSSValue *value, RenderStyle* style, Frame* frame, MediaFeaturePrefix op)
+static bool device_pixel_ratioMediaFeatureEval(CSSValue *value, RenderStyle*, Frame* frame, MediaFeaturePrefix op)
 {
     if (value)
         return value->isPrimitiveValue() && compareValue(frame->page()->chrome()->scaleFactor(), static_cast<CSSPrimitiveValue*>(value)->getFloatValue(), op);
@@ -254,7 +254,7 @@ static bool device_pixel_ratioMediaFeatureEval(CSSValue *value, RenderStyle* sty
     return frame->page()->chrome()->scaleFactor() != 0;
 }
 
-static bool gridMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix op)
+static bool gridMediaFeatureEval(CSSValue* value, RenderStyle*, Frame*, MediaFeaturePrefix op)
 {
     // if output device is bitmap, grid: 0 == true
     // assume we have bitmap device
@@ -264,7 +264,7 @@ static bool gridMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* fra
     return false;
 }
 
-static bool device_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix op)
+static bool device_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix op)
 {
     if (value) {
         FloatRect sg = screenRect(frame->page()->mainFrame()->view());
@@ -275,7 +275,7 @@ static bool device_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, F
     return true;
 }
 
-static bool device_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix op)
+static bool device_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix op)
 {
     if (value) {
         FloatRect sg = screenRect(frame->page()->mainFrame()->view());
@@ -286,7 +286,7 @@ static bool device_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Fr
     return true;
 }
 
-static bool heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix op)
+static bool heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix op)
 {
     FrameView* view = frame->view();
     
@@ -296,7 +296,7 @@ static bool heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* f
     return view->layoutHeight() != 0;
 }
 
-static bool widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix op)
+static bool widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix op)
 {
     FrameView* view = frame->view();
     
@@ -308,82 +308,82 @@ static bool widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* fr
 
 // rest of the functions are trampolines which set the prefix according to the media feature expression used
 
-static bool min_colorMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool min_colorMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return colorMediaFeatureEval(value, style, frame, MinPrefix);
 }
 
-static bool max_colorMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool max_colorMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return colorMediaFeatureEval(value, style, frame, MaxPrefix);
 }
 
-static bool min_monochromeMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool min_monochromeMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return monochromeMediaFeatureEval(value, style, frame, MinPrefix);
 }
 
-static bool max_monochromeMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool max_monochromeMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return monochromeMediaFeatureEval(value, style, frame, MaxPrefix);
 }
 
-static bool min_device_aspect_ratioMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool min_device_aspect_ratioMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return device_aspect_ratioMediaFeatureEval(value, style, frame, MinPrefix);
 }
 
-static bool max_device_aspect_ratioMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool max_device_aspect_ratioMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return device_aspect_ratioMediaFeatureEval(value, style, frame, MaxPrefix);
 }
 
-static bool min_device_pixel_ratioMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool min_device_pixel_ratioMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return device_pixel_ratioMediaFeatureEval(value, style, frame, MinPrefix);
 }
 
-static bool max_device_pixel_ratioMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool max_device_pixel_ratioMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return device_pixel_ratioMediaFeatureEval(value, style, frame, MaxPrefix);
 }
 
-static bool min_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool min_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return heightMediaFeatureEval(value, style, frame, MinPrefix);
 }
 
-static bool max_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool max_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return heightMediaFeatureEval(value, style, frame, MaxPrefix);
 }
 
-static bool min_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool min_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return widthMediaFeatureEval(value, style, frame, MinPrefix);
 }
 
-static bool max_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool max_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return widthMediaFeatureEval(value, style, frame, MaxPrefix);
 }
 
-static bool min_device_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool min_device_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return device_heightMediaFeatureEval(value, style, frame, MinPrefix);
 }
 
-static bool max_device_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool max_device_heightMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return device_heightMediaFeatureEval(value, style, frame, MaxPrefix);
 }
 
-static bool min_device_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool min_device_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return device_widthMediaFeatureEval(value, style, frame, MinPrefix);
 }
 
-static bool max_device_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame,  MediaFeaturePrefix /*op*/)
+static bool max_device_widthMediaFeatureEval(CSSValue* value, RenderStyle* style, Frame* frame, MediaFeaturePrefix)
 {
     return device_widthMediaFeatureEval(value, style, frame, MaxPrefix);
 }
index 59661e9..40cf97d 100644 (file)
@@ -44,7 +44,7 @@ EditingText::~EditingText()
 {
 }
 
-bool EditingText::rendererIsNeeded(RenderStyle *style)
+bool EditingText::rendererIsNeeded(RenderStyle*)
 {
     return true;
 }
index 4163200..5faeea2 100644 (file)
@@ -494,7 +494,7 @@ PassRefPtr<Attribute> Element::createAttribute(const QualifiedName& name, const
     return Attribute::create(name, value);
 }
 
-void Element::attributeChanged(Attribute* attr, bool preserveDecls)
+void Element::attributeChanged(Attribute* attr, bool)
 {
     const QualifiedName& attrName = attr->name();
     if (attrName == aria_activedescendantAttr) {
@@ -590,7 +590,7 @@ void Element::createAttributeMap() const
     namedAttrMap = NamedAttrMap::create(const_cast<Element*>(this));
 }
 
-bool Element::isURLAttribute(Attribute *attr) const
+bool Element::isURLAttribute(Attribute*) const
 {
     return false;
 }
@@ -875,7 +875,7 @@ void Element::dispatchAttrRemovalEvent(Attribute*)
 #endif
 }
 
-void Element::dispatchAttrAdditionEvent(Attribute *attr)
+void Element::dispatchAttrAdditionEvent(Attribute*)
 {
     ASSERT(!eventDispatchForbidden());
 #if 0
@@ -1089,7 +1089,7 @@ void Element::focus(bool restorePreviousSelection)
     updateFocusAppearance(restorePreviousSelection);
 }
 
-void Element::updateFocusAppearance(bool restorePreviousSelection)
+void Element::updateFocusAppearance(bool /*restorePreviousSelection*/)
 {
     if (this == rootEditableElement()) { 
         Frame* frame = document()->frame();
index 86f45e5..b48424f 100644 (file)
@@ -138,7 +138,7 @@ public:
     // not part of the DOM
     void setAttributeMap(PassRefPtr<NamedAttrMap>);
 
-    virtual void copyNonAttributeProperties(const Element* source) {}
+    virtual void copyNonAttributeProperties(const Element* /*source*/) { }
 
     virtual void attach();
     virtual void detach();
@@ -154,7 +154,7 @@ public:
     void dispatchAttrRemovalEvent(Attribute*);
     void dispatchAttrAdditionEvent(Attribute*);
 
-    virtual void accessKeyAction(bool sendToAnyEvent) { }
+    virtual void accessKeyAction(bool /*sendToAnyEvent*/) { }
 
     virtual bool isURLAttribute(Attribute*) const;
     virtual const QualifiedName& imageSourceAttributeName() const;
index 1b78e3e..3a3ec84 100644 (file)
@@ -80,7 +80,7 @@ namespace WebCore {
         // Handlers to do/undo actions on the target node before an event is dispatched to it and after the event
         // has been dispatched.  The data pointer is handed back by the preDispatch and passed to postDispatch.
         virtual void* preDispatchEventHandler(Event*) { return 0; }
-        virtual void postDispatchEventHandler(Event*, void* dataFromPreDispatch) { }
+        virtual void postDispatchEventHandler(Event*, void* /*dataFromPreDispatch*/) { }
 
     protected:
         virtual ~EventTarget();
index 681939d..75f5a1e 100644 (file)
@@ -304,7 +304,7 @@ void MessagePort::addEventListener(const AtomicString& eventType, PassRefPtr<Eve
     }    
 }
 
-void MessagePort::removeEventListener(const AtomicString& eventType, EventListener* eventListener, bool useCapture)
+void MessagePort::removeEventListener(const AtomicString& eventType, EventListener* eventListener, bool)
 {
     EventListenersMap::iterator iter = m_eventListeners.find(eventType);
     if (iter == m_eventListeners.end())
index f45eb0d..a071b75 100644 (file)
@@ -837,7 +837,7 @@ Node* Node::traversePreviousSiblingPostOrder(const Node* stayWithin) const
     return 0;
 }
 
-void Node::checkSetPrefix(const AtomicString &_prefix, ExceptionCode& ec)
+void Node::checkSetPrefix(const AtomicString&, ExceptionCode& ec)
 {
     // Perform error checking as required by spec for setting Node.prefix. Used by
     // Element::setPrefix() and Attr::setPrefix()
@@ -867,19 +867,17 @@ void Node::checkSetPrefix(const AtomicString &_prefix, ExceptionCode& ec)
     }*/
 }
 
-bool Node::canReplaceChild(Node* newChild, Node* oldChild)
+bool Node::canReplaceChild(Node* newChild, Node*)
 {
     if (newChild->nodeType() != DOCUMENT_FRAGMENT_NODE) {
         if (!childTypeAllowed(newChild->nodeType()))
             return false;
-    }
-    else {
+    } else {
         for (Node *n = newChild->firstChild(); n; n = n->nextSibling()) {
             if (!childTypeAllowed(n->nodeType())) 
                 return false;
         }
     }
-    
     return true;
 }
 
@@ -1231,7 +1229,7 @@ bool Node::rendererIsNeeded(RenderStyle *style)
     return (document()->documentElement() == this) || (style->display() != NONE);
 }
 
-RenderObject *Node::createRenderer(RenderArena *arena, RenderStyle *style)
+RenderObject* Node::createRenderer(RenderArena*, RenderStyle*)
 {
     ASSERT(false);
     return 0;
index 6c42617..6ceb7b0 100644 (file)
@@ -456,14 +456,14 @@ public:
     // These functions are called whenever you are connected or disconnected from a tree.  That tree may be the main
     // document tree, or it could be another disconnected tree.  Override these functions to do any work that depends
     // on connectedness to some ancestor (e.g., an ancestor <form> for example).
-    virtual void insertedIntoTree(bool deep) { }
-    virtual void removedFromTree(bool deep) { }
+    virtual void insertedIntoTree(bool /*deep*/) { }
+    virtual void removedFromTree(bool /*deep*/) { }
 
     /**
      * Notifies the node that it's list of children have changed (either by adding or removing child nodes), or a child
      * node that is of the type CDATA_SECTION_NODE, TEXT_NODE or COMMENT_NODE has changed its value.
      */
-    virtual void childrenChanged(bool changedByParser = false, Node* beforeChange = 0, Node* afterChange = 0, int childCountDelta = 0) {};
+    virtual void childrenChanged(bool /*changedByParser*/ = false, Node* /*beforeChange*/ = 0, Node* /*afterChange*/ = 0, int /*childCountDelta*/ = 0) { }
 
 #ifndef NDEBUG
     virtual void formatForDebugger(char* buffer, unsigned length) const;
index 2652ca5..8613d55 100644 (file)
@@ -744,7 +744,7 @@ Position Position::leadingWhitespacePosition(EAffinity affinity, bool considerNo
 }
 
 // This assumes that it starts in editable content.
-Position Position::trailingWhitespacePosition(EAffinity affinity, bool considerNonCollapsibleWhitespace) const
+Position Position::trailingWhitespacePosition(EAffinity, bool considerNonCollapsibleWhitespace) const
 {
     ASSERT(isEditablePosition(*this));
     if (isNull())
index d38b56f..f40f398 100644 (file)
@@ -37,13 +37,16 @@ namespace WebCore {
 typedef HashSet<QualifiedName::QualifiedNameImpl*, QualifiedNameHash> QNameSet;
 
 struct QNameComponentsTranslator {
-    static unsigned hash(const QualifiedNameComponents& components) { 
+    static unsigned hash(const QualifiedNameComponents& components)
+    {
         return hashComponents(components); 
     }
-    static bool equal(QualifiedName::QualifiedNameImpl* name, const QualifiedNameComponents& c) {
+    static bool equal(QualifiedName::QualifiedNameImpl* name, const QualifiedNameComponents& c)
+    {
         return c.m_prefix == name->m_prefix.impl() && c.m_localName == name->m_localName.impl() && c.m_namespace == name->m_namespace.impl();
     }
-    static void translate(QualifiedName::QualifiedNameImpl*& location, const QualifiedNameComponents& components, unsigned hash) {
+    static void translate(QualifiedName::QualifiedNameImpl*& location, const QualifiedNameComponents& components, unsigned)
+    {
         location = QualifiedName::QualifiedNameImpl::create(components.m_prefix, components.m_localName, components.m_namespace).releaseRef();
     }
 };
index e8882e5..68c3ec7 100644 (file)
@@ -40,7 +40,7 @@ StyleSheet* StyleElement::sheet(Element* e)
     return m_sheet.get();
 }
 
-void StyleElement::insertedIntoDocument(Document* document, Element* element)
+void StyleElement::insertedIntoDocument(Document*, Element* element)
 {
     process(element);
 }
index f5bda40..187e39d 100644 (file)
@@ -231,12 +231,12 @@ bool Text::rendererIsNeeded(RenderStyle *style)
     return true;
 }
 
-RenderObject *Text::createRenderer(RenderArena *arena, RenderStyle *style)
+RenderObject *Text::createRenderer(RenderArena* arena, RenderStyle*)
 {
 #if ENABLE(SVG)
     if (parentNode()->isSVGElement())
         return new (arena) RenderSVGInlineText(this, m_data);
-#endif // ENABLE(SVG)
+#endif
     
     return new (arena) RenderText(this, m_data);
 }
index 1dfaca2..1ed9484 100644 (file)
@@ -31,12 +31,6 @@ namespace WebCore {
 
     class Tokenizer {
     public:
-        Tokenizer(bool viewSourceMode = false) 
-            : m_parserStopped(false)
-            , m_inViewSourceMode(viewSourceMode)
-        {
-        }
-
         virtual ~Tokenizer() { }
 
         // Script output must be prepended, while new data
@@ -51,7 +45,7 @@ namespace WebCore {
         virtual int executingScript() const { return 0; }
 
         virtual bool wantsRawData() const { return false; }
-        virtual bool writeRawData(const char* data, int len) { return false; }
+        virtual bool writeRawData(const char* /*data*/, int /*length*/) { return false; }
 
         bool inViewSourceMode() const { return m_inViewSourceMode; }
         void setInViewSourceMode(bool mode) { m_inViewSourceMode = mode; }
@@ -66,6 +60,12 @@ namespace WebCore {
         virtual bool isHTMLTokenizer() const { return false; }
 
     protected:
+        Tokenizer(bool viewSourceMode = false) 
+            : m_parserStopped(false)
+            , m_inViewSourceMode(viewSourceMode)
+        {
+        }
+
         // The tokenizer has buffers, so parsing may continue even after
         // it stops receiving data. We use m_parserStopped to stop the tokenizer
         // even when it has buffered data.
index 88a396f..bdd083f 100644 (file)
@@ -165,7 +165,7 @@ void Worker::addEventListener(const AtomicString& eventType, PassRefPtr<EventLis
     }    
 }
 
-void Worker::removeEventListener(const AtomicString& eventType, EventListener* eventListener, bool useCapture)
+void Worker::removeEventListener(const AtomicString& eventType, EventListener* eventListener, bool)
 {
     EventListenersMap::iterator iter = m_eventListeners.find(eventType);
     if (iter == m_eventListeners.end())
index ee0556c..8a9af07 100644 (file)
@@ -132,7 +132,7 @@ void WorkerContext::addEventListener(const AtomicString& eventType, PassRefPtr<E
     }    
 }
 
-void WorkerContext::removeEventListener(const AtomicString& eventType, EventListener* eventListener, bool useCapture)
+void WorkerContext::removeEventListener(const AtomicString& eventType, EventListener* eventListener, bool)
 {
     EventListenersMap::iterator iter = m_eventListeners.find(eventType);
     if (iter == m_eventListeners.end())
index e5647fb..b69b9c5 100644 (file)
@@ -134,7 +134,6 @@ private:
             context->reportException(m_errorMessage, m_lineNumber, m_sourceURL);
     }
 
-private:
     String m_errorMessage;
     int m_lineNumber;
     String m_sourceURL;
@@ -154,12 +153,11 @@ private:
     {
     }
 
-    virtual void performTask(ScriptExecutionContext* context)
+    virtual void performTask(ScriptExecutionContext*)
     {
         m_messagingProxy->workerContextDestroyedInternal();
     }
 
-private:
     WorkerMessagingProxy* m_messagingProxy;
 };
 
@@ -178,12 +176,11 @@ private:
     {
     }
 
-    virtual void performTask(ScriptExecutionContext* context)
+    virtual void performTask(ScriptExecutionContext*)
     {
         m_messagingProxy->reportWorkerThreadActivityInternal(m_confirmingMessage, m_hasPendingActivity);
     }
 
-private:
     WorkerMessagingProxy* m_messagingProxy;
     bool m_confirmingMessage;
     bool m_hasPendingActivity;
index 3598fbd..d0eacdc 100644 (file)
@@ -327,7 +327,7 @@ static int globalDescriptor = 0;
 static DocLoader* globalDocLoader = 0;
 static ThreadIdentifier libxmlLoaderThread = 0;
 
-static int matchFunc(const char* uri)
+static int matchFunc(const char*)
 {
     // Only match loads initiated due to uses of libxml2 from within XMLTokenizer to avoid
     // interfering with client applications that also use libxml2.  http://bugs.webkit.org/show_bug.cgi?id=17353
@@ -430,7 +430,7 @@ static int readFunc(void* context, char* buffer, int len)
     return data->readOutBytes(buffer, len);
 }
 
-static int writeFunc(void* context, const char* buffer, int len)
+static int writeFunc(void*, const char*, int)
 {
     // Always just do 0-byte writes
     return 0;
@@ -985,7 +985,7 @@ static void startElementNsHandler(void* closure, const xmlChar* localname, const
     getTokenizer(closure)->startElementNs(localname, prefix, uri, nb_namespaces, namespaces, nb_attributes, nb_defaulted, libxmlAttributes);
 }
 
-static void endElementNsHandler(void* closure, const xmlChar* localname, const xmlChar* prefix, const xmlChar* uri)
+static void endElementNsHandler(void* closure, const xmlChar*, const xmlChar*, const xmlChar*)
 {
     if (hackAroundLibXMLEntityBug(closure))
         return;
@@ -1123,7 +1123,7 @@ static void internalSubsetHandler(void* closure, const xmlChar* name, const xmlC
     xmlSAX2InternalSubset(closure, name, externalID, systemID);
 }
 
-static void externalSubsetHandler(void* closure, const xmlChar* name, const xmlChar* externalId, const xmlChar* systemId)
+static void externalSubsetHandler(void* closure, const xmlChar*, const xmlChar* externalId, const xmlChar*)
 {
     String extId = toString(externalId);
     if ((extId == "-//W3C//DTD XHTML 1.0 Transitional//EN")
@@ -1137,7 +1137,7 @@ static void externalSubsetHandler(void* closure, const xmlChar* name, const xmlC
         getTokenizer(closure)->setIsXHTMLDocument(true); // controls if we replace entities or not.
 }
 
-static void ignorableWhitespaceHandler(void* ctx, const xmlChar* ch, int len)
+static void ignorableWhitespaceHandler(void*, const xmlChar*, int)
 {
     // nothing to do, but we need this to work around a crasher
     // http://bugzilla.gnome.org/show_bug.cgi?id=172255
@@ -1303,9 +1303,9 @@ struct AttributeParseState {
     bool gotAttributes;
 };
 
-static void attributesStartElementNsHandler(void* closure, const xmlChar* xmlLocalName, const xmlChar* xmlPrefix,
-                                            const xmlChar* xmlURI, int nb_namespaces, const xmlChar** namespaces,
-                                            int nb_attributes, int nb_defaulted, const xmlChar** libxmlAttributes)
+static void attributesStartElementNsHandler(void* closure, const xmlChar* xmlLocalName, const xmlChar* /*xmlPrefix*/,
+                                            const xmlChar* /*xmlURI*/, int /*nb_namespaces*/, const xmlChar** /*namespaces*/,
+                                            int nb_attributes, int /*nb_defaulted*/, const xmlChar** libxmlAttributes)
 {
     if (strcmp(reinterpret_cast<const char*>(xmlLocalName), "attrs") != 0)
         return;
index c12ca40..ce0bcd2 100644 (file)
@@ -1440,7 +1440,7 @@ static PassRefPtr<Range> paragraphAlignedRangeForRange(Range* arbitraryRange, in
     return paragraphRange;
 }
 
-static int findFirstGrammarDetailInRange(const Vector<GrammarDetail>& grammarDetails, int badGrammarPhraseLocation, int badGrammarPhraseLength, Range *searchRange, int startOffset, int endOffset, bool markAll)
+static int findFirstGrammarDetailInRange(const Vector<GrammarDetail>& grammarDetails, int badGrammarPhraseLocation, int /*badGrammarPhraseLength*/, Range *searchRange, int startOffset, int endOffset, bool markAll)
 {
     // Found some bad grammar. Find the earliest detail range that starts in our search range (if any).
     // Optionally add a DocumentMarker for each detail in the range.
index c43f041..cc680a6 100644 (file)
@@ -306,7 +306,7 @@ static bool executeDeleteBackwardByDecomposingPreviousCharacter(Frame* frame, Ev
     return true;
 }
 
-static bool executeDeleteForward(Frame* frame, Event*, EditorCommandSource source, const String&)
+static bool executeDeleteForward(Frame* frame, Event*, EditorCommandSource, const String&)
 {
     frame->editor()->deleteWithDirection(SelectionController::FORWARD, CharacterGranularity, false, true);
     return true;
@@ -540,7 +540,7 @@ static bool executeJustifyRight(Frame* frame, Event*, EditorCommandSource source
     return executeApplyParagraphStyle(frame, source, EditActionAlignRight, CSSPropertyTextAlign, "right");
 }
 
-static bool executeMakeTextWritingDirectionLeftToRight(Frame* frame, Event*, EditorCommandSource source, const String&)
+static bool executeMakeTextWritingDirectionLeftToRight(Frame* frame, Event*, EditorCommandSource, const String&)
 {
     RefPtr<CSSMutableStyleDeclaration> style = CSSMutableStyleDeclaration::create();
     style->setProperty(CSSPropertyUnicodeBidi, CSSValueEmbed);
@@ -549,7 +549,7 @@ static bool executeMakeTextWritingDirectionLeftToRight(Frame* frame, Event*, Edi
     return true;
 }
 
-static bool executeMakeTextWritingDirectionNatural(Frame* frame, Event*, EditorCommandSource source, const String&)
+static bool executeMakeTextWritingDirectionNatural(Frame* frame, Event*, EditorCommandSource, const String&)
 {
     RefPtr<CSSMutableStyleDeclaration> style = CSSMutableStyleDeclaration::create();
     style->setProperty(CSSPropertyUnicodeBidi, CSSValueNormal);
@@ -557,7 +557,7 @@ static bool executeMakeTextWritingDirectionNatural(Frame* frame, Event*, EditorC
     return true;
 }
 
-static bool executeMakeTextWritingDirectionRightToLeft(Frame* frame, Event*, EditorCommandSource source, const String&)
+static bool executeMakeTextWritingDirectionRightToLeft(Frame* frame, Event*, EditorCommandSource, const String&)
 {
     RefPtr<CSSMutableStyleDeclaration> style = CSSMutableStyleDeclaration::create();
     style->setProperty(CSSPropertyUnicodeBidi, CSSValueEmbed);
index b0675b4..9246082 100644 (file)
@@ -1351,7 +1351,7 @@ PassRefPtr<TextMetrics> CanvasRenderingContext2D::measureText(const String& text
     return metrics;
 }
 
-void CanvasRenderingContext2D::drawTextInternal(const String& text, float x, float y, bool fill, float maxWidth, bool useMaxWidth)
+void CanvasRenderingContext2D::drawTextInternal(const String& text, float x, float y, bool fill, float /*maxWidth*/, bool /*useMaxWidth*/)
 {
     GraphicsContext* c = drawingContext();
     if (!c)
index f87e348..9d7ab6a 100644 (file)
@@ -99,12 +99,12 @@ void HTMLAppletElement::removedFromDocument()
     HTMLPlugInElement::removedFromDocument();
 }
 
-bool HTMLAppletElement::rendererIsNeeded(RenderStyle* style)
+bool HTMLAppletElement::rendererIsNeeded(RenderStyle*)
 {
     return !getAttribute(codeAttr).isNull();
 }
 
-RenderObject* HTMLAppletElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* HTMLAppletElement::createRenderer(RenderArena*, RenderStyle* style)
 {
     Settings* settings = document()->settings();
 
index 0aa6b18..571f30c 100644 (file)
@@ -50,7 +50,7 @@ HTMLButtonElement::~HTMLButtonElement()
 {
 }
 
-RenderObject* HTMLButtonElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* HTMLButtonElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     return new (arena) RenderButton(this);
 }
index 2c58504..1af7147 100644 (file)
@@ -140,7 +140,7 @@ bool HTMLEmbedElement::rendererIsNeeded(RenderStyle* style)
     return true;
 }
 
-RenderObject* HTMLEmbedElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* HTMLEmbedElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     if (isImageType())
         return new (arena) RenderImage(this);
index 5f52abb..d90550d 100644 (file)
@@ -59,7 +59,7 @@ const AtomicString& HTMLFieldSetElement::type() const
     return fieldset;
 }
 
-RenderObject* HTMLFieldSetElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* HTMLFieldSetElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     return new (arena) RenderFieldset(this);
 }
index 01eaa99..bdcc085 100644 (file)
@@ -90,7 +90,7 @@ public:
 
     virtual bool isSuccessfulSubmitButton() const { return false; }
     virtual bool isActivatedSubmit() const { return false; }
-    virtual void setActivatedSubmit(bool flag) { }
+    virtual void setActivatedSubmit(bool) { }
 
     virtual short tabIndex() const;
 
index a9b65fc..63ec265 100644 (file)
@@ -41,13 +41,13 @@ HTMLFrameElement::HTMLFrameElement(const QualifiedName& tagName, Document* doc,
     ASSERT(hasTagName(frameTag));
 }
 
-bool HTMLFrameElement::rendererIsNeeded(RenderStyle* style)
+bool HTMLFrameElement::rendererIsNeeded(RenderStyle*)
 {
     // For compatibility, frames render even when display: none is set.
     return isURLAllowed(m_URL);
 }
 
-RenderObject* HTMLFrameElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* HTMLFrameElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     return new (arena) RenderFrame(this);
 }
index d45eb92..2025f63 100644 (file)
@@ -180,7 +180,7 @@ HTMLSelectElement* HTMLOptGroupElement::ownerSelectElement() const
     return static_cast<HTMLSelectElement*>(select);
 }
 
-void HTMLOptGroupElement::accessKeyAction(bool sendToAnyElement)
+void HTMLOptGroupElement::accessKeyAction(bool)
 {
     HTMLSelectElement* select = ownerSelectElement();
     // send to the parent to bring focus to the list box
index e7feacf..999f4d3 100644 (file)
@@ -120,7 +120,7 @@ void HTMLOptionElement::setText(const String &text, ExceptionCode& ec)
     appendChild(new Text(document(), text), ec);
 }
 
-void HTMLOptionElement::accessKeyAction(bool sendToAnyElement)
+void HTMLOptionElement::accessKeyAction(bool)
 {
     HTMLSelectElement* select = ownerSelectElement();
     if (select)
index d804327..0403dad 100644 (file)
@@ -667,7 +667,7 @@ bool HTMLParser::commentCreateErrorCheck(Token* t, RefPtr<Node>& result)
     return false;
 }
 
-bool HTMLParser::headCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::headCreateErrorCheck(Token*, RefPtr<Node>& result)
 {
     if (!head || current->localName() == htmlTag) {
         head = new HTMLHeadElement(headTag, document);
@@ -677,7 +677,7 @@ bool HTMLParser::headCreateErrorCheck(Token* t, RefPtr<Node>& result)
     return false;
 }
 
-bool HTMLParser::bodyCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::bodyCreateErrorCheck(Token*, RefPtr<Node>&)
 {
     // body no longer allowed if we have a frameset
     if (haveFrameSet)
@@ -687,7 +687,7 @@ bool HTMLParser::bodyCreateErrorCheck(Token* t, RefPtr<Node>& result)
     return true;
 }
 
-bool HTMLParser::framesetCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::framesetCreateErrorCheck(Token*, RefPtr<Node>&)
 {
     popBlock(headTag);
     if (inBody && !haveFrameSet && !haveContent) {
@@ -731,7 +731,7 @@ bool HTMLParser::isindexCreateErrorCheck(Token* t, RefPtr<Node>& result)
     return false;
 }
 
-bool HTMLParser::selectCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::selectCreateErrorCheck(Token*, RefPtr<Node>&)
 {
     return true;
 }
@@ -752,7 +752,7 @@ bool HTMLParser::dtCreateErrorCheck(Token* t, RefPtr<Node>& result)
     return true;
 }
 
-bool HTMLParser::nestedCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::nestedCreateErrorCheck(Token* t, RefPtr<Node>&)
 {
     popBlock(t->tagName);
     return true;
@@ -765,19 +765,19 @@ bool HTMLParser::nestedPCloserCreateErrorCheck(Token* t, RefPtr<Node>& result)
     return true;
 }
 
-bool HTMLParser::nestedStyleCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::nestedStyleCreateErrorCheck(Token* t, RefPtr<Node>&)
 {
     return allowNestedRedundantTag(t->tagName);
 }
 
-bool HTMLParser::tableCellCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::tableCellCreateErrorCheck(Token*, RefPtr<Node>&)
 {
     popBlock(tdTag);
     popBlock(thTag);
     return true;
 }
 
-bool HTMLParser::tableSectionCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::tableSectionCreateErrorCheck(Token*, RefPtr<Node>&)
 {
     popBlock(theadTag);
     popBlock(tbodyTag);
@@ -785,19 +785,19 @@ bool HTMLParser::tableSectionCreateErrorCheck(Token* t, RefPtr<Node>& result)
     return true;
 }
 
-bool HTMLParser::noembedCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::noembedCreateErrorCheck(Token*, RefPtr<Node>&)
 {
     setSkipMode(noembedTag);
     return true;
 }
 
-bool HTMLParser::noframesCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::noframesCreateErrorCheck(Token*, RefPtr<Node>&)
 {
     setSkipMode(noframesTag);
     return true;
 }
 
-bool HTMLParser::noscriptCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::noscriptCreateErrorCheck(Token*, RefPtr<Node>&)
 {
     if (!m_isParsingFragment) {
         Settings* settings = document->settings();
@@ -807,14 +807,14 @@ bool HTMLParser::noscriptCreateErrorCheck(Token* t, RefPtr<Node>& result)
     return true;
 }
 
-bool HTMLParser::pCloserCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::pCloserCreateErrorCheck(Token*, RefPtr<Node>&)
 {
     if (hasPElementInScope())
         popBlock(pTag);
     return true;
 }
 
-bool HTMLParser::pCloserStrictCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::pCloserStrictCreateErrorCheck(Token*, RefPtr<Node>&)
 {
     if (document->inCompatMode())
         return true;
@@ -823,7 +823,7 @@ bool HTMLParser::pCloserStrictCreateErrorCheck(Token* t, RefPtr<Node>& result)
     return true;
 }
 
-bool HTMLParser::mapCreateErrorCheck(Token* t, RefPtr<Node>& result)
+bool HTMLParser::mapCreateErrorCheck(Token*, RefPtr<Node>& result)
 {
     m_currentMapElement = new HTMLMapElement(mapTag, document);
     result = m_currentMapElement;
index fd6b93a..4402384 100644 (file)
@@ -413,7 +413,7 @@ void HTMLSelectElement::selectAll()
     listBoxOnChange();
 }
 
-RenderObject *HTMLSelectElement::createRenderer(RenderArena *arena, RenderStyle *style)
+RenderObject* HTMLSelectElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     if (usesMenuList())
         return new (arena) RenderMenuList(this);
index 7df2330..5d32aaf 100644 (file)
@@ -895,7 +895,7 @@ static JSValueRef localizedStrings(JSContextRef ctx, JSObjectRef /*function*/, J
     return JSValueMakeString(ctx, jsStringRef(url).get());
 }
 
-static JSValueRef platform(JSContextRef ctx, JSObjectRef /*function*/, JSObjectRef thisObject, size_t /*argumentCount*/, const JSValueRef[] /*arguments[]*/, JSValueRef* /*exception*/)
+static JSValueRef platform(JSContextRef ctx, JSObjectRef /*function*/, JSObjectRef /*thisObject*/, size_t /*argumentCount*/, const JSValueRef[] /*arguments[]*/, JSValueRef* /*exception*/)
 {
 #if PLATFORM(MAC)
 #ifdef BUILDING_ON_TIGER
@@ -960,7 +960,7 @@ static JSValueRef setAttachedWindowHeight(JSContextRef ctx, JSObjectRef /*functi
     return JSValueMakeUndefined(ctx);
 }
 
-static JSValueRef wrapCallback(JSContextRef ctx, JSObjectRef /*function*/, JSObjectRef thisObject, size_t argumentCount, const JSValueRef arguments[], JSValueRef* exception)
+static JSValueRef wrapCallback(JSContextRef ctx, JSObjectRef /*function*/, JSObjectRef thisObject, size_t argumentCount, const JSValueRef arguments[], JSValueRef* /*exception*/)
 {
     InspectorController* controller = reinterpret_cast<InspectorController*>(JSObjectGetPrivate(thisObject));
     if (!controller)
@@ -1471,7 +1471,7 @@ void InspectorController::toggleSearchForNodeInPage()
         hideHighlight();
 }
 
-void InspectorController::mouseDidMoveOverElement(const HitTestResult& result, unsigned modifierFlags)
+void InspectorController::mouseDidMoveOverElement(const HitTestResult& result, unsigned)
 {
     if (!enabled() || !m_searchingForNode)
         return;
@@ -2431,7 +2431,7 @@ void InspectorController::identifierForInitialRequest(unsigned long identifier,
         addAndUpdateScriptResource(resource.get());
 }
 
-void InspectorController::willSendRequest(DocumentLoader* loader, unsigned long identifier, ResourceRequest& request, const ResourceResponse& redirectResponse)
+void InspectorController::willSendRequest(DocumentLoader*, unsigned long identifier, ResourceRequest& request, const ResourceResponse& redirectResponse)
 {
     if (!enabled())
         return;
@@ -2494,7 +2494,7 @@ void InspectorController::didReceiveContentLength(DocumentLoader*, unsigned long
         updateScriptResource(resource, resource->length);
 }
 
-void InspectorController::didFinishLoading(DocumentLoader* loader, unsigned long identifier)
+void InspectorController::didFinishLoading(DocumentLoader*, unsigned long identifier)
 {
     if (!enabled())
         return;
@@ -2516,7 +2516,7 @@ void InspectorController::didFinishLoading(DocumentLoader* loader, unsigned long
     }
 }
 
-void InspectorController::didFailLoading(DocumentLoader* loader, unsigned long identifier, const ResourceError& /*error*/)
+void InspectorController::didFailLoading(DocumentLoader*, unsigned long identifier, const ResourceError& /*error*/)
 {
     if (!enabled())
         return;
@@ -2889,9 +2889,10 @@ bool InspectorController::handleException(JSContextRef context, JSValueRef excep
 }
 
 #if ENABLE(JAVASCRIPT_DEBUGGER)
+
 // JavaScriptDebugListener functions
 
-void InspectorController::didParseSource(ExecState* exec, const SourceCode& source)
+void InspectorController::didParseSource(ExecState*, const SourceCode& source)
 {
     JSValueRef sourceIDValue = JSValueMakeNumber(m_scriptContext, source.provider()->asID());
     JSValueRef sourceURLValue = JSValueMakeString(m_scriptContext, jsStringRef(source.provider()->url()).get());
@@ -2903,7 +2904,7 @@ void InspectorController::didParseSource(ExecState* exec, const SourceCode& sour
     callFunction(m_scriptContext, m_scriptObject, "parsedScriptSource", 4, arguments, exception);
 }
 
-void InspectorController::failedToParseSource(ExecState* exec, const SourceCode& source, int errorLine, const UString& errorMessage)
+void InspectorController::failedToParseSource(ExecState*, const SourceCode& source, int errorLine, const UString& errorMessage)
 {
     JSValueRef sourceURLValue = JSValueMakeString(m_scriptContext, jsStringRef(source.provider()->url()).get());
     JSValueRef sourceValue = JSValueMakeString(m_scriptContext, jsStringRef(source.data()).get());
@@ -2921,6 +2922,7 @@ void InspectorController::didPause()
     JSValueRef exception = 0;
     callFunction(m_scriptContext, m_scriptObject, "pausedScript", 0, 0, exception);
 }
+
 #endif
 
 } // namespace WebCore
index 33028e7..96213d2 100644 (file)
@@ -54,7 +54,7 @@ static ProfileMap& profileCache()
 
 static JSClassRef ProfileClass();
 
-static JSValueRef getTitleCallback(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getTitleCallback(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     if (!JSValueIsObjectOfClass(ctx, thisObject, ProfileClass()))
         return JSValueMakeUndefined(ctx);
@@ -63,7 +63,7 @@ static JSValueRef getTitleCallback(JSContextRef ctx, JSObjectRef thisObject, JSS
     return JSValueMakeString(ctx, OpaqueJSString::create(profile->title()).get());
 }
 
-static JSValueRef getHeadCallback(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getHeadCallback(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     if (!JSValueIsObjectOfClass(ctx, thisObject, ProfileClass()))
         return JSValueMakeUndefined(ctx);
@@ -72,7 +72,7 @@ static JSValueRef getHeadCallback(JSContextRef ctx, JSObjectRef thisObject, JSSt
     return toRef(toJS(toJS(ctx), profile->head()));
 }
 
-static JSValueRef getHeavyProfileCallback(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getHeavyProfileCallback(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     if (!JSValueIsObjectOfClass(ctx, thisObject, ProfileClass()))
         return JSValueMakeUndefined(ctx);
@@ -81,7 +81,7 @@ static JSValueRef getHeavyProfileCallback(JSContextRef ctx, JSObjectRef thisObje
     return toRef(toJS(toJS(ctx), profile->heavyProfile()));
 }
 
-static JSValueRef getTreeProfileCallback(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getTreeProfileCallback(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     if (!JSValueIsObjectOfClass(ctx, thisObject, ProfileClass()))
         return JSValueMakeUndefined(ctx);
@@ -90,7 +90,7 @@ static JSValueRef getTreeProfileCallback(JSContextRef ctx, JSObjectRef thisObjec
     return toRef(toJS(toJS(ctx), profile->treeProfile()));
 }
 
-static JSValueRef getUniqueIdCallback(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getUniqueIdCallback(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     if (!JSValueIsObjectOfClass(ctx, thisObject, ProfileClass()))
         return JSValueMakeUndefined(ctx);
@@ -135,7 +135,7 @@ static JSValueRef exclude(JSContextRef ctx, JSObjectRef /*function*/, JSObjectRe
     return JSValueMakeUndefined(ctx);
 }
 
-static JSValueRef restoreAll(JSContextRef ctx, JSObjectRef /*function*/, JSObjectRef thisObject, size_t argumentCount, const JSValueRef arguments[], JSValueRef* /*exception*/)
+static JSValueRef restoreAll(JSContextRef ctx, JSObjectRef /*function*/, JSObjectRef thisObject, size_t /*argumentCount*/, const JSValueRef[] /*arguments*/, JSValueRef* /*exception*/)
 {
     if (!JSValueIsObjectOfClass(ctx, thisObject, ProfileClass()))
         return JSValueMakeUndefined(ctx);
index 7b8528e..60e4bc3 100644 (file)
@@ -51,7 +51,7 @@ static ProfileNodeMap& profileNodeCache()
     return staticProfileNodes;
 }
 
-static JSValueRef getFunctionName(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getFunctionName(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     if (!JSValueIsObjectOfClass(ctx, thisObject, ProfileNodeClass()))
         return JSValueMakeUndefined(ctx);
@@ -61,7 +61,7 @@ static JSValueRef getFunctionName(JSContextRef ctx, JSObjectRef thisObject, JSSt
     return JSValueMakeString(ctx, functionNameString.get());
 }
 
-static JSValueRef getURL(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getURL(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     if (!JSValueIsObjectOfClass(ctx, thisObject, ProfileNodeClass()))
         return JSValueMakeUndefined(ctx);
@@ -71,7 +71,7 @@ static JSValueRef getURL(JSContextRef ctx, JSObjectRef thisObject, JSStringRef p
     return JSValueMakeString(ctx, urlString.get());
 }
 
-static JSValueRef getLineNumber(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getLineNumber(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     if (!JSValueIsObjectOfClass(ctx, thisObject, ProfileNodeClass()))
         return JSValueMakeUndefined(ctx);
@@ -80,7 +80,7 @@ static JSValueRef getLineNumber(JSContextRef ctx, JSObjectRef thisObject, JSStri
     return JSValueMakeNumber(ctx, profileNode->lineNumber());
 }
 
-static JSValueRef getTotalTime(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getTotalTime(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     JSC::JSLock lock(false);
 
@@ -91,7 +91,7 @@ static JSValueRef getTotalTime(JSContextRef ctx, JSObjectRef thisObject, JSStrin
     return JSValueMakeNumber(ctx, profileNode->totalTime());
 }
 
-static JSValueRef getSelfTime(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getSelfTime(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     JSC::JSLock lock(false);
 
@@ -102,7 +102,7 @@ static JSValueRef getSelfTime(JSContextRef ctx, JSObjectRef thisObject, JSString
     return JSValueMakeNumber(ctx, profileNode->selfTime());
 }
 
-static JSValueRef getTotalPercent(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getTotalPercent(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     JSC::JSLock lock(false);
 
@@ -113,7 +113,7 @@ static JSValueRef getTotalPercent(JSContextRef ctx, JSObjectRef thisObject, JSSt
     return JSValueMakeNumber(ctx, profileNode->totalPercent());
 }
 
-static JSValueRef getSelfPercent(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getSelfPercent(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     JSC::JSLock lock(false);
 
@@ -124,7 +124,7 @@ static JSValueRef getSelfPercent(JSContextRef ctx, JSObjectRef thisObject, JSStr
     return JSValueMakeNumber(ctx, profileNode->selfPercent());
 }
 
-static JSValueRef getNumberOfCalls(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getNumberOfCalls(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     JSC::JSLock lock(false);
 
@@ -135,7 +135,7 @@ static JSValueRef getNumberOfCalls(JSContextRef ctx, JSObjectRef thisObject, JSS
     return JSValueMakeNumber(ctx, profileNode->numberOfCalls());
 }
 
-static JSValueRef getChildren(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getChildren(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef* exception)
 {
     JSC::JSLock lock(false);
 
@@ -181,7 +181,7 @@ static JSValueRef getChildren(JSContextRef ctx, JSObjectRef thisObject, JSString
     return result;
 }
 
-static JSValueRef getVisible(JSContextRef ctx, JSObjectRef thisObject, JSStringRef propertyName, JSValueRef* exception)
+static JSValueRef getVisible(JSContextRef ctx, JSObjectRef thisObject, JSStringRef, JSValueRef*)
 {
     JSC::JSLock lock(false);
 
index 705ad0c..5fd9a22 100644 (file)
@@ -65,7 +65,7 @@ CachedFont::~CachedFont()
 #endif
 }
 
-void CachedFont::load(DocLoader* docLoader)
+void CachedFont::load(DocLoader*)
 {
     // Don't load the file yet.  Wait for an access before triggering the load.
     m_loading = true;
index 6f811b1..75982b6 100644 (file)
@@ -92,16 +92,16 @@ public:
 
     virtual void setResizable(bool) { }
 
-    virtual void addMessageToConsole(const String& message, unsigned int lineNumber, const String& sourceID) { }
+    virtual void addMessageToConsole(const String&, unsigned, const String&) { }
 
     virtual bool canRunBeforeUnloadConfirmPanel() { return false; }
-    virtual bool runBeforeUnloadConfirmPanel(const String& message, Frame* frame) { return true; }
+    virtual bool runBeforeUnloadConfirmPanel(const String&, Frame*) { return true; }
 
     virtual void closeWindowSoon() { }
 
     virtual void runJavaScriptAlert(Frame*, const String&) { }
     virtual bool runJavaScriptConfirm(Frame*, const String&) { return false; }
-    virtual bool runJavaScriptPrompt(Frame*, const String& message, const String& defaultValue, String& result) { return false; }
+    virtual bool runJavaScriptPrompt(Frame*, const String&, const String&, String&) { return false; }
     virtual bool shouldInterruptJavaScript() { return false; }
 
     virtual void setStatusbarText(const String&) { }
@@ -110,16 +110,16 @@ public:
 
     virtual IntRect windowResizerRect() const { return IntRect(); }
     virtual void addToDirtyRegion(const IntRect&) { }
-    virtual void scrollBackingStore(int dx, int dy, const IntRect& scrollViewRect, const IntRect& clipRect) { }
+    virtual void scrollBackingStore(int, int, const IntRect&, const IntRect&) { }
     virtual void updateBackingStore() { }
 
-    virtual void repaint(const IntRect&, bool contentChanged, bool immediate = false, bool repaintContentOnly = false) { }
-    virtual void scroll(const IntSize& scrollDelta, const IntRect& rectToScroll, const IntRect& clipRect) { }
+    virtual void repaint(const IntRect&, bool, bool, bool) { }
+    virtual void scroll(const IntSize&, const IntRect&, const IntRect&) { }
     virtual IntPoint screenToWindow(const IntPoint& p) const { return p; }
     virtual IntRect windowToScreen(const IntRect& r) const { return r; }
     virtual PlatformWidget platformWindow() const { return 0; }
 
-    virtual void mouseDidMoveOverElement(const HitTestResult&, unsigned modifierFlags) { }
+    virtual void mouseDidMoveOverElement(const HitTestResult&, unsigned) { }
 
     virtual void setToolTip(const String&) { }
 
@@ -150,26 +150,26 @@ public:
 
     virtual void download(ResourceHandle*, const ResourceRequest&, const ResourceRequest&, const ResourceResponse&) { }
 
-    virtual void assignIdentifierToInitialRequest(unsigned long identifier, DocumentLoader*, const ResourceRequest&) { }
-    virtual bool shouldUseCredentialStorage(DocumentLoader*, unsigned long identifier) { return false; }
-    virtual void dispatchWillSendRequest(DocumentLoader*, unsigned long identifier, ResourceRequest&, const ResourceResponse& redirectResponse) { }
-    virtual void dispatchDidReceiveAuthenticationChallenge(DocumentLoader*, unsigned long identifier, const AuthenticationChallenge&) { }
-    virtual void dispatchDidCancelAuthenticationChallenge(DocumentLoader*, unsigned long identifier, const AuthenticationChallenge&) { }
-    virtual void dispatchDidReceiveResponse(DocumentLoader*, unsigned long identifier, const ResourceResponse&) { }
-    virtual void dispatchDidReceiveContentLength(DocumentLoader*, unsigned long identifier, int lengthReceived) { }
-    virtual void dispatchDidFinishLoading(DocumentLoader*, unsigned long identifier) { }
-    virtual void dispatchDidFailLoading(DocumentLoader*, unsigned long identifier, const ResourceError&) { }
-    virtual bool dispatchDidLoadResourceFromMemoryCache(DocumentLoader*, const ResourceRequest&, const ResourceResponse&, int length) { return false; }
+    virtual void assignIdentifierToInitialRequest(unsigned long, DocumentLoader*, const ResourceRequest&) { }
+    virtual bool shouldUseCredentialStorage(DocumentLoader*, unsigned long) { return false; }
+    virtual void dispatchWillSendRequest(DocumentLoader*, unsigned long, ResourceRequest&, const ResourceResponse&) { }
+    virtual void dispatchDidReceiveAuthenticationChallenge(DocumentLoader*, unsigned long, const AuthenticationChallenge&) { }
+    virtual void dispatchDidCancelAuthenticationChallenge(DocumentLoader*, unsigned long, const AuthenticationChallenge&) { }
+    virtual void dispatchDidReceiveResponse(DocumentLoader*, unsigned long, const ResourceResponse&) { }
+    virtual void dispatchDidReceiveContentLength(DocumentLoader*, unsigned long, int) { }
+    virtual void dispatchDidFinishLoading(DocumentLoader*, unsigned long) { }
+    virtual void dispatchDidFailLoading(DocumentLoader*, unsigned long, const ResourceError&) { }
+    virtual bool dispatchDidLoadResourceFromMemoryCache(DocumentLoader*, const ResourceRequest&, const ResourceResponse&, int) { return false; }
 
     virtual void dispatchDidHandleOnloadEvents() { }
     virtual void dispatchDidReceiveServerRedirectForProvisionalLoad() { }
     virtual void dispatchDidCancelClientRedirect() { }
-    virtual void dispatchWillPerformClientRedirect(const KURL&, double interval, double fireDate) { }
+    virtual void dispatchWillPerformClientRedirect(const KURL&, double, double) { }
     virtual void dispatchDidChangeLocationWithinPage() { }
     virtual void dispatchWillClose() { }
     virtual void dispatchDidReceiveIcon() { }
     virtual void dispatchDidStartProvisionalLoad() { }
-    virtual void dispatchDidReceiveTitle(const String& title) { }
+    virtual void dispatchDidReceiveTitle(const String&) { }
     virtual void dispatchDidCommitLoad() { }
     virtual void dispatchDidFailProvisionalLoad(const ResourceError&) { }
     virtual void dispatchDidFailLoad(const ResourceError&) { }
@@ -181,8 +181,8 @@ public:
     virtual Frame* dispatchCreatePage() { return 0; }
     virtual void dispatchShow() { }
 
-    virtual void dispatchDecidePolicyForMIMEType(FramePolicyFunction, const String& MIMEType, const ResourceRequest&) { }
-    virtual void dispatchDecidePolicyForNewWindowAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>, const String& frameName) { }
+    virtual void dispatchDecidePolicyForMIMEType(FramePolicyFunction, const String&, const ResourceRequest&) { }
+    virtual void dispatchDecidePolicyForNewWindowAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>, const String&) { }
     virtual void dispatchDecidePolicyForNavigationAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>) { }
     virtual void cancelPolicyCheck() { }
 
@@ -222,9 +222,9 @@ public:
     virtual bool shouldFallBack(const ResourceError&) { return false; }
 
     virtual bool canHandleRequest(const ResourceRequest&) const { return false; }
-    virtual bool canShowMIMEType(const String& MIMEType) const { return false; }
-    virtual bool representationExistsForURLScheme(const String& URLScheme) const { return false; }
-    virtual String generatedMIMETypeForURLScheme(const String& URLScheme) const { return ""; }
+    virtual bool canShowMIMEType(const String&) const { return false; }
+    virtual bool representationExistsForURLScheme(const String&) const { return false; }
+    virtual String generatedMIMETypeForURLScheme(const String&) const { return ""; }
 
     virtual void frameLoadCompleted() { }
     virtual void restoreViewState() { }
@@ -234,7 +234,7 @@ public:
     virtual void prepareForDataSourceReplacement() { }
 
     virtual PassRefPtr<DocumentLoader> createDocumentLoader(const ResourceRequest& request, const SubstituteData& substituteData) { return DocumentLoader::create(request, substituteData); }
-    virtual void setTitle(const String& title, const KURL&) { }
+    virtual void setTitle(const String&, const KURL&) { }
 
     virtual String userAgent(const KURL&) { return ""; }
 
@@ -247,22 +247,21 @@ public:
     virtual void saveViewStateToItem(HistoryItem*) { }
     virtual bool canCachePage() const { return false; }
 
-    virtual PassRefPtr<Frame> createFrame(const KURL& url, const String& name, HTMLFrameOwnerElement* ownerElement,
-                               const String& referrer, bool allowsScrolling, int marginWidth, int marginHeight) { return 0; }
-    virtual Widget* createPlugin(const IntSize&,Element*, const KURL&, const Vector<String>&, const Vector<String>&, const String&, bool) { return 0; }
+    virtual PassRefPtr<Frame> createFrame(const KURL&, const String&, HTMLFrameOwnerElement*, const String&, bool, int, int) { return 0; }
+    virtual Widget* createPlugin(const IntSize&, Element*, const KURL&, const Vector<String>&, const Vector<String>&, const String&, bool) { return 0; }
     virtual Widget* createJavaAppletWidget(const IntSize&, Element*, const KURL&, const Vector<String>&, const Vector<String>&) { return 0; }
 
-    virtual ObjectContentType objectContentType(const KURL& url, const String& mimeType) { return ObjectContentType(); }
+    virtual ObjectContentType objectContentType(const KURL&, const String&) { return ObjectContentType(); }
     virtual String overrideMediaType() const { return String(); }
 
-    virtual void redirectDataToPlugin(Widget*) {}
-    virtual void windowObjectCleared() {}
-    virtual void didPerformFirstNavigation() const {}
+    virtual void redirectDataToPlugin(Widget*) { }
+    virtual void windowObjectCleared() { }
+    virtual void didPerformFirstNavigation() const { }
 
-    virtual void registerForIconNotification(bool listen) {}
+    virtual void registerForIconNotification(bool) { }
 
 #if PLATFORM(MAC)
-    virtual NSCachedURLResponse* willCacheResponse(DocumentLoader*, unsigned long identifier, NSCachedURLResponse* response) const { return response; }
+    virtual NSCachedURLResponse* willCacheResponse(DocumentLoader*, unsigned long, NSCachedURLResponse* response) const { return response; }
 #endif
 
 };
@@ -290,7 +289,7 @@ public:
     virtual bool shouldInsertNode(Node*, Range*, EditorInsertAction) { return false; }
     //  virtual bool shouldInsertNode(Node*, Range* replacingRange, WebViewInsertAction) { return false; }
     virtual bool shouldInsertText(const String&, Range*, EditorInsertAction) { return false; }
-    virtual bool shouldChangeSelectedRange(Range* fromRange, Range* toRange, EAffinity, bool stillSelecting) { return false; }
+    virtual bool shouldChangeSelectedRange(Range*, Range*, EAffinity, bool) { return false; }
 
     virtual bool shouldApplyStyle(CSSStyleDeclaration*, Range*) { return false; }
     virtual bool shouldMoveRangeAfterDelete(Range*, Range*) { return false; }
@@ -337,14 +336,14 @@ public:
 #endif
     virtual void ignoreWordInSpellDocument(const String&) { }
     virtual void learnWord(const String&) { }
-    virtual void checkSpellingOfString(const UChar*, int length, int* misspellingLocation, int* misspellingLength) { }
-    virtual void checkGrammarOfString(const UChar*, int length, Vector<GrammarDetail>&, int* badGrammarLocation, int* badGrammarLength) { }
+    virtual void checkSpellingOfString(const UChar*, int, int*, int*) { }
+    virtual void checkGrammarOfString(const UChar*, int, Vector<GrammarDetail>&, int*, int*) { }
     virtual void updateSpellingUIWithGrammarString(const String&, const GrammarDetail&) { }
     virtual void updateSpellingUIWithMisspelledWord(const String&) { }
-    virtual void showSpellingUI(bool show) { }
+    virtual void showSpellingUI(bool) { }
     virtual bool spellingUIIsShowing() { return false; }
-    virtual void getGuessesForWord(const String&, Vector<String>& guesses) { }
-    virtual void setInputMethodState(bool enabled) { }
+    virtual void getGuessesForWord(const String&, Vector<String>&) { }
+    virtual void setInputMethodState(bool) { }
 
 
 };
@@ -357,7 +356,7 @@ public:
     virtual PlatformMenuDescription getCustomMenuFromDefaultItems(ContextMenu*) { return 0; }
     virtual void contextMenuItemSelected(ContextMenuItem*, const ContextMenu*) { }
 
-    virtual void downloadURL(const KURL& url) { }
+    virtual void downloadURL(const KURL&) { }
     virtual void copyImageToClipboard(const HitTestResult&) { }
     virtual void searchWithGoogle(const Frame*) { }
     virtual void lookUpInDictionary(Frame*) { }
@@ -377,7 +376,7 @@ public:
     virtual DragDestinationAction actionMaskForDrag(DragData*) { return DragDestinationActionNone; }
     virtual DragSourceAction dragSourceActionMaskForPoint(const IntPoint&) { return DragSourceActionNone; }
     virtual void startDrag(DragImageRef, const IntPoint&, const IntPoint&, Clipboard*, Frame*, bool) { }
-    virtual DragImageRef createDragImageForLink(KURL&, const String& label, Frame*) { return 0; }
+    virtual DragImageRef createDragImageForLink(KURL&, const String&, Frame*) { return 0; }
     virtual void dragControllerDestroyed() { }
 };
 
@@ -401,11 +400,11 @@ public:
 
     virtual void highlight(Node*) { }
     virtual void hideHighlight() { }
-    virtual void inspectedURLChanged(const String& newURL) { }
+    virtual void inspectedURLChanged(const String&) { }
 
-    virtual void populateSetting(const String& key, InspectorController::Setting&) { }
-    virtual void storeSetting(const String& key, const InspectorController::Setting&) { }
-    virtual void removeSetting(const String& key) { }
+    virtual void populateSetting(const String&, InspectorController::Setting&) { }
+    virtual void storeSetting(const String&, const InspectorController::Setting&) { }
+    virtual void removeSetting(const String&) { }
 };
 
 }
index 7acc81c..08ef896 100644 (file)
@@ -399,7 +399,7 @@ void FTPDirectoryTokenizer::createBasicDocument()
     bodyElement->appendChild(m_tableElement, ec);
 }
 
-bool FTPDirectoryTokenizer::write(const SegmentedString& s, bool appendData)
+bool FTPDirectoryTokenizer::write(const SegmentedString& s, bool /*appendData*/)
 {    
     // Make sure we have the table element to append to by loading the template set in the pref, or
     // creating a very basic document with the appropriate table
index 8d20edb..eff2cb1 100644 (file)
@@ -3887,7 +3887,7 @@ void FrameLoader::callContinueLoadAfterNavigationPolicy(void* argument,
     loader->continueLoadAfterNavigationPolicy(request, formState, shouldContinue);
 }
 
-void FrameLoader::continueLoadAfterNavigationPolicy(const ResourceRequest& request, PassRefPtr<FormState> formState, bool shouldContinue)
+void FrameLoader::continueLoadAfterNavigationPolicy(const ResourceRequest&, PassRefPtr<FormState> formState, bool shouldContinue)
 {
     // If we loaded an alternate page to replace an unreachableURL, we'll get in here with a
     // nil policyDataSource because loading the alternate page will have passed
@@ -4806,7 +4806,7 @@ void FrameLoader::setMainDocumentError(DocumentLoader* loader, const ResourceErr
     m_client->setMainDocumentError(loader, error);
 }
 
-void FrameLoader::mainReceivedCompleteError(DocumentLoader* loader, const ResourceError& error)
+void FrameLoader::mainReceivedCompleteError(DocumentLoader* loader, const ResourceError&)
 {
     loader->setPrimaryLoadComplete(true);
     m_client->dispatchDidLoadMainResource(activeDocumentLoader());
index 701d279..8f58179 100644 (file)
@@ -91,13 +91,13 @@ private:
 
 // --------
 
-bool ImageTokenizer::write(const SegmentedString& s, bool appendData)
+bool ImageTokenizer::write(const SegmentedString&, bool)
 {
     ASSERT_NOT_REACHED();
     return false;
 }
 
-bool ImageTokenizer::writeRawData(const char* data, int len)
+bool ImageTokenizer::writeRawData(const char*, int)
 {
     CachedImage* cachedImage = m_doc->cachedImage();
     cachedImage->data(m_doc->frame()->loader()->documentLoader()->mainResourceData(), false);
@@ -341,7 +341,7 @@ bool ImageDocument::shouldShrinkToFit() const
 
 // --------
 
-void ImageEventListener::handleEvent(Event* event, bool isWindowEvent)
+void ImageEventListener::handleEvent(Event* event, bool)
 {
     if (event->type() == eventNames().resizeEvent)
         m_doc->windowSizeChanged();
index 23b622b..2d3eb69 100644 (file)
@@ -134,7 +134,7 @@ void ImageLoader::updateFromElementIgnoringPreviousError()
     updateFromElement();
 }
 
-void ImageLoader::notifyFinished(CachedResource *image)
+void ImageLoader::notifyFinished(CachedResource*)
 {
     ASSERT(m_failedLoadURL.isEmpty());
     m_imageComplete = true;
index adbdcd6..7aa1b5d 100644 (file)
@@ -120,7 +120,7 @@ void MainResourceLoader::callContinueAfterNavigationPolicy(void* argument, const
     static_cast<MainResourceLoader*>(argument)->continueAfterNavigationPolicy(request, shouldContinue);
 }
 
-void MainResourceLoader::continueAfterNavigationPolicy(const ResourceRequest& request, bool shouldContinue)
+void MainResourceLoader::continueAfterNavigationPolicy(const ResourceRequest&, bool shouldContinue)
 {
     if (!shouldContinue)
         stopLoadingForPolicyChange();
index 5689457..8ed5b45 100644 (file)
@@ -53,6 +53,7 @@ class MediaTokenizer : public Tokenizer {
 public:
     MediaTokenizer(Document* doc) : m_doc(doc), m_mediaElement(0) {}
         
+private:
     virtual bool write(const SegmentedString&, bool appendData);
     virtual void stopParsing();
     virtual void finish();
@@ -62,12 +63,12 @@ public:
     virtual bool writeRawData(const char* data, int len);
         
     void createDocumentStructure();
-private:
+
     Document* m_doc;
     HTMLMediaElement* m_mediaElement;
 };
 
-bool MediaTokenizer::write(const SegmentedString& s, bool appendData)
+bool MediaTokenizer::write(const SegmentedString&, bool)
 {
     ASSERT_NOT_REACHED();
     return false;
@@ -103,7 +104,7 @@ void MediaTokenizer::createDocumentStructure()
     frame->loader()->activeDocumentLoader()->mainResourceLoader()->setShouldBufferData(false);
 }
     
-bool MediaTokenizer::writeRawData(const char* data, int len)
+bool MediaTokenizer::writeRawData(const char*, int)
 {
     ASSERT(!m_mediaElement);
     if (m_mediaElement)
index 8be6ae2..42c801c 100644 (file)
@@ -48,6 +48,7 @@ class PluginTokenizer : public Tokenizer {
 public:
     PluginTokenizer(Document* doc) : m_doc(doc), m_embedElement(0) {}
         
+private:
     virtual bool write(const SegmentedString&, bool appendData);
     virtual void stopParsing();
     virtual void finish();
@@ -57,12 +58,12 @@ public:
     virtual bool writeRawData(const char* data, int len);
         
     void createDocumentStructure();
-private:
+
     Document* m_doc;
     HTMLEmbedElement* m_embedElement;
 };
     
-bool PluginTokenizer::write(const SegmentedString& s, bool appendData)
+bool PluginTokenizer::write(const SegmentedString&, bool)
 {
     ASSERT_NOT_REACHED();
     return false;
@@ -94,7 +95,7 @@ void PluginTokenizer::createDocumentStructure()
     body->appendChild(embedElement, ec);    
 }
     
-bool PluginTokenizer::writeRawData(const char* data, int len)
+bool PluginTokenizer::writeRawData(const char*, int)
 {
     ASSERT(!m_embedElement);
     if (m_embedElement)
index 3ee60e0..3bac2a7 100644 (file)
@@ -227,7 +227,7 @@ void ResourceLoader::willSendRequest(ResourceRequest& request, const ResourceRes
     m_request = request;
 }
 
-void ResourceLoader::didSendData(unsigned long long bytesSent, unsigned long long totalBytesToBeSent)
+void ResourceLoader::didSendData(unsigned long long, unsigned long long)
 {
 }
 
index 5fda572..4e14dea 100644 (file)
@@ -42,15 +42,15 @@ public:
     virtual ~SubresourceLoaderClient() { } 
 
     // request may be modified
-    virtual void willSendRequest(SubresourceLoader*, ResourceRequest&, const ResourceResponse& redirectResponse) { }
-    virtual void didSendData(SubresourceLoader*, unsigned long long bytesSent, unsigned long long totalBytesToBeSent) { }
+    virtual void willSendRequest(SubresourceLoader*, ResourceRequest&, const ResourceResponse& /*redirectResponse*/) { }
+    virtual void didSendData(SubresourceLoader*, unsigned long long /*bytesSent*/, unsigned long long /*totalBytesToBeSent*/) { }
 
     virtual void didReceiveResponse(SubresourceLoader*, const ResourceResponse&) { }
     virtual void didReceiveData(SubresourceLoader*, const char*, int) { }
     virtual void didFinishLoading(SubresourceLoader*) { }
     virtual void didFail(SubresourceLoader*, const ResourceError&) { }
     
-    virtual bool getShouldUseCredentialStorage(SubresourceLoader*, bool& shouldUseCredentialStorage) { return false; }
+    virtual bool getShouldUseCredentialStorage(SubresourceLoader*, bool& /*shouldUseCredentialStorage*/) { return false; }
     virtual void didReceiveAuthenticationChallenge(SubresourceLoader*, const AuthenticationChallenge&) { }
     virtual void receivedCancellation(SubresourceLoader*, const AuthenticationChallenge&) { }
 
index 6b06ede..bd2c446 100644 (file)
@@ -93,7 +93,7 @@ TextTokenizer::TextTokenizer(HTMLViewSourceDocument* doc)
     m_dest = m_buffer;
 }    
 
-bool TextTokenizer::write(const SegmentedString& s, bool appendData)
+bool TextTokenizer::write(const SegmentedString& s, bool)
 {
     ExceptionCode ec;
 
index 693732b..d738799 100644 (file)
@@ -129,7 +129,7 @@ unsigned ApplicationCache::numDynamicEntries() const
     return 0;
 }
     
-String ApplicationCache::dynamicEntry(unsigned index) const
+String ApplicationCache::dynamicEntry(unsigned) const
 {
     // FIXME: Implement
     return String();
@@ -137,15 +137,14 @@ String ApplicationCache::dynamicEntry(unsigned index) const
     
 bool ApplicationCache::addDynamicEntry(const String& url)
 {
-    if (!equalIgnoringCase(m_group->manifestURL().protocol(),
-                           KURL(url).protocol()))
+    if (!equalIgnoringCase(m_group->manifestURL().protocol(), KURL(url).protocol()))
         return false;
 
     // FIXME: Implement
     return true;
 }
     
-void ApplicationCache::removeDynamicEntry(const String& url)
+void ApplicationCache::removeDynamicEntry(const String&)
 {
     // FIXME: Implement
 }
index f6f63e0..7fd49b3 100644 (file)
@@ -76,7 +76,7 @@ ApplicationCacheGroup::~ApplicationCacheGroup()
     cacheStorage().cacheGroupDestroyed(this);
 }
     
-ApplicationCache* ApplicationCacheGroup::cacheForMainRequest(const ResourceRequest& request, DocumentLoader* loader)
+ApplicationCache* ApplicationCacheGroup::cacheForMainRequest(const ResourceRequest& request, DocumentLoader*)
 {
     if (!ApplicationCache::requestIsHTTPOrHTTPSGet(request))
         return 0;
@@ -90,7 +90,7 @@ ApplicationCache* ApplicationCacheGroup::cacheForMainRequest(const ResourceReque
     return 0;
 }
     
-ApplicationCache* ApplicationCacheGroup::fallbackCacheForMainRequest(const ResourceRequest& request, DocumentLoader* loader)
+ApplicationCache* ApplicationCacheGroup::fallbackCacheForMainRequest(const ResourceRequest& request, DocumentLoader*)
 {
     if (!ApplicationCache::requestIsHTTPOrHTTPSGet(request))
         return 0;
@@ -376,7 +376,7 @@ void ApplicationCacheGroup::didReceiveResponse(ResourceHandle* handle, const Res
     m_currentResource = ApplicationCacheResource::create(url, response, type);
 }
 
-void ApplicationCacheGroup::didReceiveData(ResourceHandle* handle, const char* data, int length, int lengthReceived)
+void ApplicationCacheGroup::didReceiveData(ResourceHandle* handle, const char* data, int length, int)
 {
     if (handle == m_manifestHandle) {
         didReceiveManifestData(data, length);
index 8e07db3..e208d70 100644 (file)
@@ -198,7 +198,7 @@ void DOMApplicationCache::addEventListener(const AtomicString& eventType, PassRe
     }    
 }
 
-void DOMApplicationCache::removeEventListener(const AtomicString& eventType, EventListener* eventListener, bool useCapture)
+void DOMApplicationCache::removeEventListener(const AtomicString& eventType, EventListener* eventListener, bool)
 {
     EventListenersMap::iterator iter = m_eventListeners.find(eventType);
     if (iter == m_eventListeners.end())
index e642895..8806406 100644 (file)
@@ -41,8 +41,9 @@ public:
     virtual ~IconDatabaseClient() { }
     virtual bool performImport() { return true; }
     virtual void dispatchDidRemoveAllIcons() { }
-    virtual void dispatchDidAddIconForPageURL(const String& pageURL) { }
+    virtual void dispatchDidAddIconForPageURL(const String& /*pageURL*/) { }
 };
  
 } // namespace WebCore 
+
 #endif
index 4337f51..933a6ab 100644 (file)
@@ -121,7 +121,7 @@ void IconLoader::didFail(SubresourceLoader* resourceLoader, const ResourceError&
     }
 }
 
-void IconLoader::didReceiveAuthenticationChallenge(SubresourceLoader* resourceLoader, const AuthenticationChallenge& challenge)
+void IconLoader::didReceiveAuthenticationChallenge(SubresourceLoader*, const AuthenticationChallenge&)
 {
     // We don't ever want to prompt for authentication just for a site icon, so
     // implement this method to cancel the resource load
index f070cc9..599aad3 100644 (file)
@@ -53,7 +53,7 @@ IconRecord::~IconRecord()
     LOG(IconDatabase, "Destroying IconRecord for icon url %s", m_iconURL.ascii().data());
 }
 
-Image* IconRecord::image(const IntSize& size)
+Image* IconRecord::image(const IntSize&)
 {
     // FIXME rdar://4680377 - For size right now, we are returning our one and only image and the Bridge
     // is resizing it in place.  We need to actually store all the original representations here and return a native
index 9769ac9..d6ee923 100644 (file)
@@ -35,7 +35,7 @@
 
 namespace WebCore {
 
-NSCachedURLResponse* ResourceLoader::willCacheResponse(ResourceHandle* handle, NSCachedURLResponse* response)
+NSCachedURLResponse* ResourceLoader::willCacheResponse(ResourceHandle*, NSCachedURLResponse* response)
 {
     return frameLoader()->client()->willCacheResponse(documentLoader(), identifier(), response);
 }
index e83cf9b..0b072cf 100644 (file)
@@ -257,7 +257,7 @@ void AccessibilityObject::setSelectedText(const String&)
     notImplemented();
 }
 
-void AccessibilityObject::setSelectedTextRange(const PlainTextRange& range)
+void AccessibilityObject::setSelectedTextRange(const PlainTextRange&)
 {
 }
 
@@ -272,11 +272,11 @@ KURL AccessibilityObject::url() const
     return KURL();
 }
 
-void AccessibilityObject::setFocused(bool on)
+void AccessibilityObject::setFocused(bool)
 {
 }
 
-void AccessibilityObject::setValue(const String& string)
+void AccessibilityObject::setValue(const String&)
 {
 }
 
@@ -327,17 +327,17 @@ VisiblePositionRange AccessibilityObject::visiblePositionRange() const
     return VisiblePositionRange();
 }
 
-VisiblePositionRange AccessibilityObject::visiblePositionRangeForLine(unsigned lineCount) const
+VisiblePositionRange AccessibilityObject::visiblePositionRangeForLine(unsigned) const
 {
     return VisiblePositionRange();
 }
 
-VisiblePosition AccessibilityObject::visiblePositionForIndex(int index) const
+VisiblePosition AccessibilityObject::visiblePositionForIndex(int) const
 {
     return VisiblePosition();
 }
     
-int AccessibilityObject::indexForVisiblePosition(const VisiblePosition& pos) const
+int AccessibilityObject::indexForVisiblePosition(const VisiblePosition&) const
 {
     return 0;
 }
@@ -600,7 +600,7 @@ String AccessibilityObject::stringForVisiblePositionRange(const VisiblePositionR
     return String::adopt(resultVector);
 }
 
-IntRect AccessibilityObject::boundsForVisiblePositionRange(const VisiblePositionRange& visiblePositionRange) const
+IntRect AccessibilityObject::boundsForVisiblePositionRange(const VisiblePositionRange&) const
 {
     return IntRect();
 }
@@ -636,7 +636,7 @@ void AccessibilityObject::setSelectedVisiblePositionRange(const VisiblePositionR
 {
 }
 
-VisiblePosition AccessibilityObject::visiblePositionForPoint(const IntPoint& point) const
+VisiblePosition AccessibilityObject::visiblePositionForPoint(const IntPoint&) const
 {
     return VisiblePosition();
 }
@@ -798,7 +798,7 @@ VisiblePosition AccessibilityObject::previousParagraphStartPosition(const Visibl
 }
 
 // NOTE: Consider providing this utility method as AX API
-VisiblePosition AccessibilityObject::visiblePositionForIndex(unsigned indexValue, bool lastIndexOK) const
+VisiblePosition AccessibilityObject::visiblePositionForIndex(unsigned, bool) const
 {
     return VisiblePosition();
 }
@@ -849,14 +849,14 @@ PlainTextRange AccessibilityObject::plainTextRangeForVisiblePositionRange(const
 }
 
 // NOTE: Consider providing this utility method as AX API
-int AccessibilityObject::index(const VisiblePosition& position) const
+int AccessibilityObject::index(const VisiblePosition&) const
 {
     return -1;
 }
 
 // Given a line number, the range of characters of the text associated with this accessibility
 // object that contains the line number.
-PlainTextRange AccessibilityObject::doAXRangeForLine(unsigned lineNumber) const
+PlainTextRange AccessibilityObject::doAXRangeForLine(unsigned) const
 {
     return PlainTextRange();
 }
@@ -879,7 +879,7 @@ PlainTextRange AccessibilityObject::doAXRangeForPosition(const IntPoint& point)
 // The composed character range in the text associated with this accessibility object that
 // is specified by the given index value. This parameterized attribute returns the complete
 // range of characters (including surrogate pairs of multi-byte glyphs) at the given index.
-PlainTextRange AccessibilityObject::doAXRangeForIndex(unsigned index) const
+PlainTextRange AccessibilityObject::doAXRangeForIndex(unsigned) const
 {
     return PlainTextRange();
 }
@@ -894,7 +894,7 @@ PlainTextRange AccessibilityObject::doAXStyleRangeForIndex(unsigned index) const
 
 // A substring of the text associated with this accessibility object that is
 // specified by the given character range.
-String AccessibilityObject::doAXStringForRange(const PlainTextRange& range) const
+String AccessibilityObject::doAXStringForRange(const PlainTextRange&) const
 {
     return String();
 }
@@ -902,7 +902,7 @@ String AccessibilityObject::doAXStringForRange(const PlainTextRange& range) cons
 // The bounding rectangle of the text associated with this accessibility object that is
 // specified by the given range. This is the bounding rectangle a sighted user would see
 // on the display screen, in pixels.
-IntRect AccessibilityObject::doAXBoundsForRange(const PlainTextRange& range) const
+IntRect AccessibilityObject::doAXBoundsForRange(const PlainTextRange&) const
 {
     return IntRect();
 }
@@ -926,7 +926,7 @@ FrameView* AccessibilityObject::documentFrameView() const
     return object->documentFrameView();
 }    
 
-AccessibilityObject* AccessibilityObject::doAccessibilityHitTest(const IntPoint& point) const
+AccessibilityObject* AccessibilityObject::doAccessibilityHitTest(const IntPoint&) const
 {
     return 0;
 }
index ecce5e9..1168c90 100644 (file)
@@ -436,8 +436,8 @@ void ChromeClient::enableSuddenTermination()
 }
 
 bool ChromeClient::paintCustomScrollbar(GraphicsContext*, const FloatRect&, ScrollbarControlSize, 
-                                        ScrollbarControlState, ScrollbarPart, bool vertical,
-                                        float value, float proportion, ScrollbarControlPartMask)
+                                        ScrollbarControlState, ScrollbarPart, bool,
+                                        float, float, ScrollbarControlPartMask)
 {
     return false;
 }
index a477308..42d2e90 100644 (file)
@@ -530,7 +530,7 @@ String DOMWindow::prompt(const String& message, const String& defaultValue)
     return String();
 }
 
-bool DOMWindow::find(const String& string, bool caseSensitive, bool backwards, bool wrap, bool wholeWord, bool searchInFrames, bool showDialog) const
+bool DOMWindow::find(const String& string, bool caseSensitive, bool backwards, bool wrap, bool /*wholeWord*/, bool /*searchInFrames*/, bool /*showDialog*/) const
 {
     if (!m_frame)
         return false;
index 4dc37d8..c3b59a5 100644 (file)
@@ -1519,7 +1519,7 @@ void EventHandler::updateMouseEventTargetNode(Node* targetNode, const PlatformMo
     }
 }
 
-bool EventHandler::dispatchMouseEvent(const AtomicString& eventType, Node* targetNode, bool cancelable, int clickCount, const PlatformMouseEvent& mouseEvent, bool setUnder)
+bool EventHandler::dispatchMouseEvent(const AtomicString& eventType, Node* targetNode, bool /*cancelable*/, int clickCount, const PlatformMouseEvent& mouseEvent, bool setUnder)
 {
     updateMouseEventTargetNode(targetNode, mouseEvent, setUnder);
 
index d0bb273..a4c0b57 100644 (file)
@@ -1285,7 +1285,7 @@ void Frame::revealCaret(const RenderLayer::ScrollAlignment& alignment) const
     }
 }
 
-void Frame::adjustPageHeight(float *newBottom, float oldTop, float oldBottom, float bottomLimit)
+void Frame::adjustPageHeight(float* newBottom, float oldTop, float oldBottom, float /*bottomLimit*/)
 {
     RenderView* root = contentRenderer();
     if (root) {
index db713f4..d751bb7 100644 (file)
@@ -63,17 +63,17 @@ static inline double solveCubicBezierFunction(double p1x, double p1y, double p2x
     return bezier.solve(t, solveEpsilon(duration));
 }
 
-static inline int blendFunc(const AnimationBase* anim, int from, int to, double progress)
+static inline int blendFunc(const AnimationBase*, int from, int to, double progress)
 {  
     return int(from + (to - from) * progress);
 }
 
-static inline double blendFunc(const AnimationBase* anim, double from, double to, double progress)
+static inline double blendFunc(const AnimationBase*, double from, double to, double progress)
 {  
     return from + (to - from) * progress;
 }
 
-static inline float blendFunc(const AnimationBase* anim, float from, float to, double progress)
+static inline float blendFunc(const AnimationBase*, float from, float to, double progress)
 {  
     return narrowPrecisionToFloat(from + (to - from) * progress);
 }
@@ -86,7 +86,7 @@ static inline Color blendFunc(const AnimationBase* anim, const Color& from, cons
                  blendFunc(anim, from.alpha(), to.alpha(), progress));
 }
 
-static inline Length blendFunc(const AnimationBase* anim, const Length& from, const Length& to, double progress)
+static inline Length blendFunc(const AnimationBase*, const Length& from, const Length& to, double progress)
 {  
     return to.blend(from, progress);
 }
index b72376c..23502d7 100644 (file)
@@ -122,8 +122,8 @@ public:
 
     double progress(double scale, double offset, const TimingFunction*) const;
 
-    virtual void animate(CompositeAnimation*, RenderObject*, const RenderStyle* currentStyle
-                         const RenderStyle* targetStyle, RefPtr<RenderStyle>& animatedStyle) { }
+    virtual void animate(CompositeAnimation*, RenderObject*, const RenderStyle* /*currentStyle*/
+        const RenderStyle* /*targetStyle*/, RefPtr<RenderStyle>& /*animatedStyle*/) { }
 
     virtual bool shouldFireEvents() const { return false; }
 
@@ -140,7 +140,7 @@ public:
     virtual bool overridden() const { return false; }
 
     // Does this animation/transition involve the given property?
-    virtual bool affectsProperty(int property) const { return false; }
+    virtual bool affectsProperty(int /*property*/) const { return false; }
     bool isAnimatingProperty(int property, bool isRunningNow) const
     {
         if (isRunningNow)
@@ -160,11 +160,11 @@ protected:
     CompositeAnimation* compositeAnimation() { return m_compAnim; }
 
     // These are called when the corresponding timer fires so subclasses can do any extra work
-    virtual void onAnimationStart(double elapsedTime) { }
-    virtual void onAnimationIteration(double elapsedTime) { }
-    virtual void onAnimationEnd(double elapsedTime) { }
-    virtual bool startAnimation(double beginTime) { return false; }
-    virtual void endAnimation(bool reset) { }
+    virtual void onAnimationStart(double /*elapsedTime*/) { }
+    virtual void onAnimationIteration(double /*elapsedTime*/) { }
+    virtual void onAnimationEnd(double /*elapsedTime*/) { }
+    virtual bool startAnimation(double /*beginTime*/) { return false; }
+    virtual void endAnimation(bool /*reset*/) { }
 
     void goIntoEndingOrLoopingState();
 
@@ -177,7 +177,6 @@ protected:
     
     static void setChanged(Node*);
 
-protected:
     AnimState m_animState;
 
     bool m_isAnimating;       // transition/animation requires continual timer firing
index 4dda7b6..d05d482 100644 (file)
@@ -216,7 +216,7 @@ void AnimationControllerPrivate::addEventToDispatch(PassRefPtr<Element> element,
     startUpdateRenderingDispatcher();
 }
 
-void AnimationControllerPrivate::animationTimerFired(Timer<AnimationControllerPrivate>* timer)
+void AnimationControllerPrivate::animationTimerFired(Timer<AnimationControllerPrivate>*)
 {
     // When the timer fires, all we do is call setChanged on all DOM nodes with running animations and then do an immediate
     // updateRendering.  It will then call back to us with new information.
index 50b9f68..fbd79ed 100644 (file)
@@ -352,7 +352,7 @@ PassRefPtr<KeyframeAnimation> CompositeAnimationPrivate::getAnimationForProperty
     return retval;
 }
 
-void CompositeAnimationPrivate::cleanupFinishedAnimations(RenderObject* renderer)
+void CompositeAnimationPrivate::cleanupFinishedAnimations(RenderObject*)
 {
     if (isSuspended())
         return;
index b995655..f984909 100644 (file)
@@ -60,8 +60,7 @@ bool ImplicitAnimation::shouldSendEventForListener(Document::ListenerType inList
     return m_object->document()->hasListenerType(inListenerType);
 }
 
-void ImplicitAnimation::animate(CompositeAnimation* animation, RenderObject* renderer, RenderStyle* currentStyle,
-                                RenderStyle* targetStyle, RefPtr<RenderStyle>& animatedStyle)
+void ImplicitAnimation::animate(CompositeAnimation*, RenderObject*, RenderStyle*, RenderStyle* targetStyle, RefPtr<RenderStyle>& animatedStyle)
 {
     // If we get this far and the animation is done, it means we are cleaning up a just finished animation.
     // So just return. Everything is already all cleaned up.
index 3b5f894..07fe984 100644 (file)
@@ -60,8 +60,7 @@ KeyframeAnimation::~KeyframeAnimation()
         updateStateMachine(AnimationStateInputEndAnimation, -1);
 }
 
-void KeyframeAnimation::animate(CompositeAnimation* animation, RenderObject* renderer, const RenderStyle* currentStyle, 
-                                    const RenderStyle* targetStyle, RefPtr<RenderStyle>& animatedStyle)
+void KeyframeAnimation::animate(CompositeAnimation*, RenderObject*, const RenderStyle*, const RenderStyle* targetStyle, RefPtr<RenderStyle>& animatedStyle)
 {
     // Fire the start timeout if needed
     fireAnimationEventsIfNeeded();
index 90f850f..697e73f 100644 (file)
@@ -90,8 +90,7 @@ static int CeilingLog2(unsigned int i) {
     return log2;
 }
 
-void InitArenaPool(ArenaPool *pool, const char *name, 
-                   unsigned int size, unsigned int align)
+void InitArenaPool(ArenaPool* pool, const char*, unsigned size, unsigned align)
 {
      if (align == 0)
          align = ARENA_DEFAULT_ALIGN;
index 1fdb2b2..e2aebc6 100644 (file)
@@ -40,7 +40,7 @@ class ScrollbarTheme {
 public:
     virtual ~ScrollbarTheme() {};
 
-    virtual bool paint(Scrollbar*, GraphicsContext*, const IntRect& damageRect) { return false; }
+    virtual bool paint(Scrollbar*, GraphicsContext*, const IntRect& /*damageRect*/) { return false; }
     virtual ScrollbarPart hitTest(Scrollbar*, const PlatformMouseEvent&) { return NoPart; }
     
     virtual int scrollbarThickness(ScrollbarControlSize = RegularScrollbar) { return 0; }
index 185c7ca..5cf4af9 100644 (file)
@@ -28,7 +28,7 @@
 
 namespace WebCore {
 
-LengthBox Theme::controlBorder(ControlPart part, const Font&, const LengthBox& zoomedBox, float zoomFactor) const
+LengthBox Theme::controlBorder(ControlPart part, const Font&, const LengthBox& zoomedBox, float) const
 {
     switch (part) {
         case PushButtonPart:
@@ -42,7 +42,7 @@ LengthBox Theme::controlBorder(ControlPart part, const Font&, const LengthBox& z
     }
 }
 
-LengthBox Theme::controlPadding(ControlPart part, const Font&, const LengthBox& zoomedBox, float zoomFactor) const
+LengthBox Theme::controlPadding(ControlPart part, const Font&, const LengthBox& zoomedBox, float) const
 {
     switch (part) {
         case MenulistPart:
index 79975ec..5574bd7 100644 (file)
@@ -81,13 +81,13 @@ public:
     // Methods used to adjust the RenderStyles of controls.
     
     // The font description result should have a zoomed font size.
-    virtual FontDescription controlFont(ControlPart, const Font& font, float zoomFactor) const { return font.fontDescription(); }
+    virtual FontDescription controlFont(ControlPart, const Font& font, float /*zoomFactor*/) const { return font.fontDescription(); }
     
     // The size here is in zoomed coordinates already.  If a new size is returned, it also needs to be in zoomed coordinates.
-    virtual LengthSize controlSize(ControlPart, const Font&, const LengthSize& zoomedSize, float zoomFactor) const { return zoomedSize; }
+    virtual LengthSize controlSize(ControlPart, const Font&, const LengthSize& zoomedSize, float /*zoomFactor*/) const { return zoomedSize; }
     
     // Returns the minimum size for a control in zoomed coordinates.  
-    virtual LengthSize minimumControlSize(ControlPart, const Font&, float zoomFactor) const { return LengthSize(Length(0, Fixed), Length(0, Fixed)); }
+    virtual LengthSize minimumControlSize(ControlPart, const Font&, float /*zoomFactor*/) const { return LengthSize(Length(0, Fixed), Length(0, Fixed)); }
     
     // Allows the theme to modify the existing padding/border.
     virtual LengthBox controlPadding(ControlPart, const Font&, const LengthBox& zoomedBox, float zoomFactor) const;
@@ -96,14 +96,14 @@ public:
     // Whether or not whitespace: pre should be forced on always.
     virtual bool controlRequiresPreWhiteSpace(ControlPart) const { return false; }
 
-    // Method for painting a control.  The rect is in zoomed coordinates.
-    virtual void paint(ControlPart, ControlStates, GraphicsContext*, const IntRect& zoomedRect, float zoomFactor, ScrollView*) const { };
+    // Method for painting a control. The rect is in zoomed coordinates.
+    virtual void paint(ControlPart, ControlStates, GraphicsContext*, const IntRect& /*zoomedRect*/, float /*zoomFactor*/, ScrollView*) const { }
 
     // Some controls may spill out of their containers (e.g., the check on an OS X checkbox).  When these controls repaint,
     // the theme needs to communicate this inflated rect to the engine so that it can invalidate the whole control.
     // The rect passed in is in zoomed coordinates, so the inflation should take that into account and make sure the inflation
     // amount is also scaled by the zoomFactor.
-    virtual void inflateControlPaintRect(ControlPart, ControlStates, IntRect& zoomedRect, float zoomFactor) const { }
+    virtual void inflateControlPaintRect(ControlPart, ControlStates, IntRect& /*zoomedRect*/, float /*zoomFactor*/) const { }
     
     // This method is called once, from RenderTheme::adjustDefaultStyleSheet(), to let each platform adjust
     // the default CSS rules in html4.css.
index 4ace9c8..1643e17 100644 (file)
@@ -380,7 +380,7 @@ void BitmapImage::resetAnimation()
     destroyDecodedDataIfNecessary(true);
 }
 
-void BitmapImage::advanceAnimation(Timer<BitmapImage>* timer)
+void BitmapImage::advanceAnimation(Timer<BitmapImage>*)
 {
     internalAdvanceAnimation(false);
     // At this point the image region has been marked dirty, and if it's
index 7ef5348..bb69ac3 100644 (file)
@@ -269,8 +269,7 @@ void Font::drawSimpleText(GraphicsContext* context, const TextRun& run, const Fl
     drawGlyphBuffer(context, glyphBuffer, run, startPoint);
 }
 
-void Font::drawGlyphBuffer(GraphicsContext* context, const GlyphBuffer& glyphBuffer, 
-                           const TextRun& run, const FloatPoint& point) const
+void Font::drawGlyphBuffer(GraphicsContext* context, const GlyphBuffer& glyphBuffer, const TextRun&, const FloatPoint& point) const
 {   
     // Draw each contiguous run of glyphs that use the same font data.
     const SimpleFontData* fontData = glyphBuffer.fontDataAt(0);
index ad4e5c2..71263b0 100644 (file)
@@ -53,7 +53,7 @@ public:
     virtual IntSize size() const { return m_size; }
 
     // Assume that generated content has no decoded data we need to worry about
-    virtual void destroyDecodedData(bool destroyAll = true) { }
+    virtual void destroyDecodedData(bool /*destroyAll*/ = true) { }
     virtual unsigned decodedSize() const { return 0; }
 
 protected:
@@ -61,7 +61,6 @@ protected:
     virtual void drawPattern(GraphicsContext*, const FloatRect& srcRect, const AffineTransform& patternTransform,
                              const FloatPoint& phase, CompositeOperator, const FloatRect& destRect);
     
-protected:
     GeneratedImage(PassRefPtr<Generator> generator, const IntSize& size)
         : m_generator(generator)
         , m_size(size)
index 876f27c..1223d29 100644 (file)
@@ -106,7 +106,7 @@ public:
     int height() const;
 
     bool setData(PassRefPtr<SharedBuffer> data, bool allDataReceived);
-    virtual bool dataChanged(bool allDataReceived) { return false; }
+    virtual bool dataChanged(bool /*allDataReceived*/) { return false; }
     
     virtual String filenameExtension() const { return String(); } // null string if unknown
 
@@ -159,7 +159,7 @@ protected:
     virtual bool mayFillWithSolidColor() const { return false; }
     virtual Color solidColor() const { return Color(); }
     
-    virtual void startAnimation(bool catchUpIfNecessary = true) { }
+    virtual void startAnimation(bool /*catchUpIfNecessary*/ = true) { }
     
     virtual void drawPattern(GraphicsContext*, const FloatRect& srcRect, const AffineTransform& patternTransform,
                              const FloatPoint& phase, CompositeOperator, const FloatRect& destRect);
index 2c5583b..c27d95e 100644 (file)
@@ -48,7 +48,7 @@ ImageSource::~ImageSource()
     clear(true);
 }
 
-void ImageSource::clear(bool destroyAll, size_t clearBeforeFrame)
+void ImageSource::clear(bool destroyAll, size_t)
 {
     if (destroyAll) {
         if (m_decoder) {
@@ -231,7 +231,7 @@ float ImageSource::frameDurationAtIndex(size_t index)
     return duration;
 }
 
-bool ImageSource::frameHasAlphaAtIndex(size_t index)
+bool ImageSource::frameHasAlphaAtIndex(size_t)
 {
     // Might be interesting to do this optimization on Mac some day, but for now we're just using this
     // for the Cairo source, since it uses our decoders, and our decoders can answer this question.
index 3a6db9f..130c12c 100644 (file)
@@ -42,7 +42,9 @@ namespace WebCore {
         {
             return adoptRef(new PDFDocumentImage);
         }
-        ~PDFDocumentImage();
+
+    private:
+        virtual ~PDFDocumentImage();
 
         virtual bool hasSingleSecurityOrigin() const { return true; }
 
@@ -50,12 +52,11 @@ namespace WebCore {
 
         // FIXME: PDF Images are underreporting decoded sizes and will be unable
         // to prune because these functions are not implemented yet.
-        virtual void destroyDecodedData(bool destroyAll = true) { }
+        virtual void destroyDecodedData(bool /*destroyAll*/ = true) { }
         virtual unsigned decodedSize() const { return 0; }
 
         virtual IntSize size() const;
 
-    private:
         PDFDocumentImage();
         virtual void draw(GraphicsContext*, const FloatRect& dstRect, const FloatRect& srcRect, CompositeOperator);
         
index 9a45c5a..52493e7 100644 (file)
@@ -154,8 +154,7 @@ static void initializeATSUStyle(const SimpleFontData* fontData)
     fontData->m_ATSUStyleInitialized = true;
 }
 
-static OSStatus overrideLayoutOperation(ATSULayoutOperationSelector iCurrentOperation, ATSULineRef iLineRef, URefCon iRefCon,
-                                        void *iOperationCallbackParameterPtr, ATSULayoutOperationCallbackStatus *oCallbackStatus)
+static OSStatus overrideLayoutOperation(ATSULayoutOperationSelector, ATSULineRef iLineRef, URefCon iRefCon, void*, ATSULayoutOperationCallbackStatus* oCallbackStatus)
 {
     ATSULayoutParameters* params = reinterpret_cast<ATSULayoutParameters*>(iRefCon);
     OSStatus status;
@@ -592,7 +591,7 @@ float Font::floatWidthForComplexText(const TextRun& run) const
            MIN(FixedToFloat(firstGlyphBounds.upperLeft.x), FixedToFloat(firstGlyphBounds.lowerLeft.x));
 }
 
-int Font::offsetForPositionForComplexText(const TextRun& run, int x, bool includePartialGlyphs) const
+int Font::offsetForPositionForComplexText(const TextRun& run, int x, bool /*includePartialGlyphs*/) const
 {
     OwnArrayPtr<UChar> charactersWithOverride;
     TextRun adjustedRun = copyRunForDirectionalOverrideIfNecessary(run, charactersWithOverride);
index 54c49a3..bd1b856 100644 (file)
@@ -36,6 +36,7 @@ public:
         return adoptRef(new IdentityTransformOperation());
     }
         
+private:
     virtual bool isIdentity() const { return true; }
     virtual OperationType getOperationType() const { return IDENTITY; }
     virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == IDENTITY; }
@@ -45,17 +46,16 @@ public:
         return isSameType(o);
     }
 
-    virtual bool apply(AffineTransform& transform, const IntSize& borderBoxSize) const
+    virtual bool apply(AffineTransform&, const IntSize&) const
     {
         return false;
     }
 
-    virtual PassRefPtr<TransformOperation> blend(const TransformOperation* from, double progress, bool blendToIdentity = false)
+    virtual PassRefPtr<TransformOperation> blend(const TransformOperation*, double, bool = false)
     {
         return this;
     }
 
-private:
     IdentityTransformOperation()
     {
     }
index 574f2a8..48bea19 100644 (file)
@@ -36,6 +36,7 @@ public:
         return adoptRef(new MatrixTransformOperation(a, b, c, d, e, f));
     }
 
+private:
     virtual bool isIdentity() const { return m_a == 1 && m_b == 0 && m_c == 0 && m_d == 1 && m_e == 0 && m_f == 0; }
     virtual OperationType getOperationType() const { return MATRIX; }
     virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == MATRIX; }
@@ -49,7 +50,7 @@ public:
         return m_a == m->m_a && m_b == m->m_b && m_c == m->m_c && m_d == m->m_d && m_e == m->m_e && m_f == m->m_f;
     }
 
-    virtual bool apply(AffineTransform& transform, const IntSize& borderBoxSize) const
+    virtual bool apply(AffineTransform& transform, const IntSize&) const
     {
         AffineTransform matrix(m_a, m_b, m_c, m_d, m_e, m_f);
         transform = matrix * transform;
@@ -58,7 +59,6 @@ public:
 
     virtual PassRefPtr<TransformOperation> blend(const TransformOperation* from, double progress, bool blendToIdentity = false);
     
-private:
     MatrixTransformOperation(double a, double b, double c, double d, double e, double f)
         : m_a(a)
         , m_b(b)
index 9cfc9c8..0b39662 100644 (file)
@@ -48,7 +48,7 @@ public:
         return m_angle == r->m_angle;
     }
 
-    virtual bool apply(AffineTransform& transform, const IntSize& borderBoxSize) const
+    virtual bool apply(AffineTransform& transform, const IntSize& /*borderBoxSize*/) const
     {
         transform.rotate(m_angle);
         return false;
index 787f067..07523a2 100644 (file)
@@ -36,6 +36,7 @@ public:
         return adoptRef(new ScaleTransformOperation(sx, sy, type));
     }
 
+private:
     virtual bool isIdentity() const { return m_x == 1 &&  m_y == 1; }
     virtual OperationType getOperationType() const { return m_type; }
     virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == m_type; }
@@ -48,7 +49,7 @@ public:
         return m_x == s->m_x && m_y == s->m_y;
     }
 
-    virtual bool apply(AffineTransform& transform, const IntSize& borderBoxSize) const
+    virtual bool apply(AffineTransform& transform, const IntSize&) const
     {
         transform.scale(m_x, m_y);
         return false;
@@ -56,7 +57,6 @@ public:
 
     virtual PassRefPtr<TransformOperation> blend(const TransformOperation* from, double progress, bool blendToIdentity = false);
 
-private:
     ScaleTransformOperation(double sx, double sy, OperationType type)
         : m_x(sx)
         , m_y(sy)
index a7879e8..e0553ef 100644 (file)
@@ -36,6 +36,7 @@ public:
         return adoptRef(new SkewTransformOperation(angleX, angleY, type));
     }
 
+private:
     virtual bool isIdentity() const { return m_angleX == 0 && m_angleY == 0; }
     virtual OperationType getOperationType() const { return m_type; }
     virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == m_type; }
@@ -48,7 +49,7 @@ public:
         return m_angleX == s->m_angleX && m_angleY == s->m_angleY;
     }
 
-    virtual bool apply(AffineTransform& transform, const IntSize& borderBoxSize) const
+    virtual bool apply(AffineTransform& transform, const IntSize&) const
     {
         transform.skew(m_angleX, m_angleY);
         return false;
@@ -56,7 +57,6 @@ public:
 
     virtual PassRefPtr<TransformOperation> blend(const TransformOperation* from, double progress, bool blendToIdentity = false);
     
-private:
     SkewTransformOperation(double angleX, double angleY, OperationType type)
         : m_angleX(angleX)
         , m_angleY(angleY)
index 1baa67d..e8ca012 100644 (file)
@@ -56,7 +56,7 @@ public:
     virtual PassRefPtr<TransformOperation> blend(const TransformOperation* from, double progress, bool blendToIdentity = false) = 0;
 
     virtual OperationType getOperationType() const = 0;
-    virtual bool isSameType(const TransformOperation& o) const { return false; }
+    virtual bool isSameType(const TransformOperation&) const { return false; }
 };
 
 } // namespace WebCore
index bd66787..5cf2e14 100644 (file)
@@ -122,7 +122,7 @@ String DragData::asURL(String* title) const
     return m_pasteboardHelper->urlFromPasteboard([m_platformDragData draggingPasteboard], title);
 }
 
-PassRefPtr<DocumentFragment> DragData::asFragment(Document* doc) const
+PassRefPtr<DocumentFragment> DragData::asFragment(Document*) const
 {
     return [m_pasteboardHelper->fragmentFromPasteboard([m_platformDragData draggingPasteboard]) _documentFragment];
 }
index 5810a6d..9d85866 100644 (file)
@@ -52,7 +52,7 @@ public:
 
 protected:
     // The AuthenticationChallenge subclass may "shadow" this method to compare platform specific fields
-    static bool platformCompare(const AuthenticationChallengeBase& a, const AuthenticationChallengeBase& b) { return true; }
+    static bool platformCompare(const AuthenticationChallengeBase&, const AuthenticationChallengeBase&) { return true; }
 
     bool m_isNull;
     ProtectionSpace m_protectionSpace;
index b2aac67..4631324 100644 (file)
@@ -70,7 +70,7 @@ protected:
     void platformLazyInit() {}
 
     // The ResourceError subclass may "shadow" this method to compare platform specific fields
-    static bool platformCompare(const ResourceError& a, const ResourceError& b) { return true; }
+    static bool platformCompare(const ResourceError&, const ResourceError&) { return true; }
 
     String m_domain;
     int m_errorCode;
index 58fef85..3668d88 100644 (file)
@@ -63,11 +63,11 @@ namespace WebCore {
         virtual ~ResourceHandleClient() { }
 
         // request may be modified
-        virtual void willSendRequest(ResourceHandle*, ResourceRequest&, const ResourceResponse& redirectResponse) { }
-        virtual void didSendData(ResourceHandle*, unsigned long long bytesSent, unsigned long long totalBytesToBeSent) { }
+        virtual void willSendRequest(ResourceHandle*, ResourceRequest&, const ResourceResponse& /*redirectResponse*/) { }
+        virtual void didSendData(ResourceHandle*, unsigned long long /*bytesSent*/, unsigned long long /*totalBytesToBeSent*/) { }
 
         virtual void didReceiveResponse(ResourceHandle*, const ResourceResponse&) { }
-        virtual void didReceiveData(ResourceHandle*, const char*, int, int lengthReceived) { }
+        virtual void didReceiveData(ResourceHandle*, const char*, int, int /*lengthReceived*/) { }
         virtual void didFinishLoading(ResourceHandle*) { }
         virtual void didFail(ResourceHandle*, const ResourceError&) { }
         virtual void wasBlocked(ResourceHandle*) { }
index 5f91d05..20df8af 100644 (file)
@@ -121,7 +121,7 @@ protected:
     void platformLazyInit() { }
 
     // The ResourceResponse subclass may "shadow" this method to compare platform specific fields
-    static bool platformCompare(const ResourceResponse& a, const ResourceResponse& b) { return true; }
+    static bool platformCompare(const ResourceResponse&, const ResourceResponse&) { return true; }
 
     KURL m_url;
     String m_mimeType;
index e571494..bf21ab1 100644 (file)
@@ -30,7 +30,7 @@
 
 namespace WebCore {
 
-void prefetchDNS(const String& hostname)
+void prefetchDNS(const String&)
 {
     notImplemented();
 }
index 134d05f..b618949 100644 (file)
@@ -223,7 +223,7 @@ static void formFinalize(CFReadStreamRef stream, void* context)
     delete form;
 }
 
-static Boolean formOpen(CFReadStreamRef stream, CFStreamError* error, Boolean* openComplete, void* context)
+static Boolean formOpen(CFReadStreamRef, CFStreamError* error, Boolean* openComplete, void* context)
 {
     FormStreamFields* form = static_cast<FormStreamFields*>(context);
 
@@ -279,14 +279,14 @@ static Boolean formCanRead(CFReadStreamRef stream, void* context)
     return CFReadStreamHasBytesAvailable(form->currentStream);
 }
 
-static void formClose(CFReadStreamRef stream, void* context)
+static void formClose(CFReadStreamRef, void* context)
 {
     FormStreamFields* form = static_cast<FormStreamFields*>(context);
 
     closeCurrentStream(form);
 }
 
-static void formSchedule(CFReadStreamRef stream, CFRunLoopRef runLoop, CFStringRef runLoopMode, void* context)
+static void formSchedule(CFReadStreamRef, CFRunLoopRef runLoop, CFStringRef runLoopMode, void* context)
 {
     FormStreamFields* form = static_cast<FormStreamFields*>(context);
 
@@ -295,7 +295,7 @@ static void formSchedule(CFReadStreamRef stream, CFRunLoopRef runLoop, CFStringR
     form->scheduledRunLoopPairs.add(SchedulePair::create(runLoop, runLoopMode));
 }
 
-static void formUnschedule(CFReadStreamRef stream, CFRunLoopRef runLoop, CFStringRef runLoopMode, void* context)
+static void formUnschedule(CFReadStreamRef, CFRunLoopRef runLoop, CFStringRef runLoopMode, void* context)
 {
     FormStreamFields* form = static_cast<FormStreamFields*>(context);
 
index 3cb6ba6..c0918a4 100644 (file)
@@ -76,7 +76,7 @@ void NetworkStateNotifier::updateState()
     }
 }
 
-void NetworkStateNotifier::dynamicStoreCallback(SCDynamicStoreRef store, CFArrayRef changedKeys, void *info) 
+void NetworkStateNotifier::dynamicStoreCallback(SCDynamicStoreRef, CFArrayRef, void* info) 
 {
     NetworkStateNotifier* notifier = static_cast<NetworkStateNotifier*>(info);
     
index 88e4e73..a4d0d28 100644 (file)
@@ -65,7 +65,7 @@ void TextCodecUTF16::registerCodecs(TextCodecRegistrar registrar)
     registrar("UTF-16BE", newStreamingTextDecoderUTF16BE, 0);
 }
 
-String TextCodecUTF16::decode(const char* bytes, size_t length, bool, bool stopOnError, bool& sawError)
+String TextCodecUTF16::decode(const char* bytes, size_t length, bool, bool, bool&)
 {
     if (!length)
         return String();
@@ -89,18 +89,19 @@ String TextCodecUTF16::decode(const char* bytes, size_t length, bool, bool stopO
         numChars -= 1;
     }
 
-    if (m_littleEndian)
+    if (m_littleEndian) {
         for (size_t i = 0; i < numChars; ++i) {
             UChar c = p[0] | (p[1] << 8);
             p += 2;
             *q++ = c;
         }
-    else
+    } else {
         for (size_t i = 0; i < numChars; ++i) {
             UChar c = (p[0] << 8) | p[1];
             p += 2;
             *q++ = c;
         }
+    }
 
     if (numBytes & 1) {
         ASSERT(!m_haveBufferedByte);
index 1dc66a9..d7c1293 100644 (file)
@@ -79,7 +79,7 @@ FixedTableLayout::FixedTableLayout(RenderTable* table)
 {
 }
 
-int FixedTableLayout::calcWidthArray(int tableWidth)
+int FixedTableLayout::calcWidthArray(int)
 {
     int usedWidth = 0;
 
index 5dc1f47..f5d0de5 100644 (file)
@@ -239,7 +239,7 @@ bool InlineBox::canAccommodateEllipsis(bool ltr, int blockEdge, int ellipsisWidt
     return !(boxRect.intersects(ellipsisRect));
 }
 
-int InlineBox::placeEllipsisBox(bool ltr, int blockEdge, int ellipsisWidth, bool&)
+int InlineBox::placeEllipsisBox(bool, int, int, bool&)
 {
     // Use -1 to mean "we didn't set the position."
     return -1;
index 9589603..30dad38 100644 (file)
@@ -124,8 +124,8 @@ public:
                               int& topPosition, int& bottomPosition, int& selectionTop, int& selectionBottom);
     void shrinkBoxesWithNoTextChildren(int topPosition, int bottomPosition);
     
-    virtual void setVerticalOverflowPositions(int top, int bottom) { }
-    virtual void setVerticalSelectionPositions(int top, int bottom) { }
+    virtual void setVerticalOverflowPositions(int /*top*/, int /*bottom*/) { }
+    virtual void setVerticalSelectionPositions(int /*top*/, int /*bottom*/) { }
     int maxHorizontalVisualOverflow() const { return m_maxHorizontalVisualOverflow; }
 
     void removeChild(InlineBox* child);
index a624a5b..0f7c29b 100644 (file)
@@ -41,8 +41,8 @@ public:
     void setNextLineBox(InlineRunBox* n) { m_nextLine = n; }
     void setPreviousLineBox(InlineRunBox* p) { m_prevLine = p; }
 
-    virtual void paintBoxDecorations(RenderObject::PaintInfo&, int tx, int ty) { }
-    virtual void paintTextDecorations(RenderObject::PaintInfo&, int tx, int ty, bool paintedChildren = false) { }
+    virtual void paintBoxDecorations(RenderObject::PaintInfo&, int /*tx*/, int /*ty*/) { }
+    virtual void paintTextDecorations(RenderObject::PaintInfo&, int /*tx*/, int /*ty*/, bool /*paintedChildren*/ = false) { }
     
 protected:
     InlineRunBox* m_prevLine;  // The previous box that also uses our RenderObject
index 0d15839..a0e9c5c 100644 (file)
@@ -219,7 +219,7 @@ bool InlineTextBox::isLineBreak() const
     return object()->isBR() || (object()->style()->preserveNewline() && len() == 1 && (*textObject()->text())[start()] == '\n');
 }
 
-bool InlineTextBox::nodeAtPoint(const HitTestRequest& request, HitTestResult& result, int x, int y, int tx, int ty)
+bool InlineTextBox::nodeAtPoint(const HitTestRequest&, HitTestResult& result, int x, int y, int tx, int ty)
 {
     if (isLineBreak())
         return false;
@@ -493,7 +493,7 @@ void InlineTextBox::selectionStartEnd(int& sPos, int& ePos)
     ePos = min(endPos - m_start, (int)m_len);
 }
 
-void InlineTextBox::paintSelection(GraphicsContext* context, int tx, int ty, RenderStyle* style, const Font* f)
+void InlineTextBox::paintSelection(GraphicsContext* context, int tx, int ty, RenderStyle* style, const Font*)
 {
     // See if we have a selection to paint at all.
     int sPos, ePos;
@@ -521,7 +521,7 @@ void InlineTextBox::paintSelection(GraphicsContext* context, int tx, int ty, Ren
     context->restore();
 }
 
-void InlineTextBox::paintCompositionBackground(GraphicsContext* context, int tx, int ty, RenderStyle* style, const Font* f, int startPos, int endPos)
+void InlineTextBox::paintCompositionBackground(GraphicsContext* context, int tx, int ty, RenderStyle* style, const Font*, int startPos, int endPos)
 {
     int offset = m_start;
     int sPos = max(startPos - offset, 0);
index 82b008a..2532c5b 100644 (file)
@@ -56,7 +56,7 @@ int RenderBR::baselinePosition(bool firstLine, bool isRootLineBox) const
     return RenderText::baselinePosition(firstLine, isRootLineBox);
 }
 
-int RenderBR::lineHeight(bool firstLine, bool isRootLineBox) const
+int RenderBR::lineHeight(bool firstLine, bool /*isRootLineBox*/) const
 {
     if (firstTextBox() && !firstTextBox()->isText())
         return 0;
index 5d210c7..9de4d8e 100644 (file)
@@ -2120,7 +2120,7 @@ IntRect RenderBlock::fillVerticalSelectionGap(int lastTop, int lastLeft, int las
 }
 
 IntRect RenderBlock::fillLeftSelectionGap(RenderObject* selObj, int xPos, int yPos, int height, RenderBlock* rootBlock,
-                                          int blockX, int blockY, int tx, int ty, const PaintInfo* paintInfo)
+                                          int blockX, int /*blockY*/, int tx, int ty, const PaintInfo* paintInfo)
 {
     int top = yPos + ty;
     int left = blockX + max(leftSelectionOffset(rootBlock, yPos), leftSelectionOffset(rootBlock, yPos + height));
@@ -2136,7 +2136,7 @@ IntRect RenderBlock::fillLeftSelectionGap(RenderObject* selObj, int xPos, int yP
 }
 
 IntRect RenderBlock::fillRightSelectionGap(RenderObject* selObj, int xPos, int yPos, int height, RenderBlock* rootBlock,
-                                           int blockX, int blockY, int tx, int ty, const PaintInfo* paintInfo)
+                                           int blockX, int /*blockY*/, int tx, int ty, const PaintInfo* paintInfo)
 {
     int left = max(xPos + tx, blockX + max(leftSelectionOffset(rootBlock, yPos), leftSelectionOffset(rootBlock, yPos + height)));
     int top = yPos + ty;
@@ -3089,7 +3089,7 @@ void RenderBlock::addVisualOverflow(const IntRect& r)
     m_overflowHeight = max(m_overflowHeight, r.bottom());
 }
 
-bool RenderBlock::isPointInOverflowControl(HitTestResult& result, int _x, int _y, int _tx, int _ty)
+bool RenderBlock::isPointInOverflowControl(HitTestResult& result, int, int, int, int)
 {
     if (!scrollsOverflow())
         return false;
index 5ae134e..902460d 100644 (file)
@@ -124,7 +124,7 @@ public:
     virtual void borderFitAdjust(int& x, int& w) const; // Shrink the box in which the border paints if border-fit is set.
 
     // Called to lay out the legend for a fieldset.
-    virtual RenderObject* layoutLegend(bool relayoutChildren) { return 0; };
+    virtual RenderObject* layoutLegend(bool /*relayoutChildren*/) { return 0; }
 
     // the implementation of the following functions is in bidi.cpp
     struct FloatWithRect {
index eec5ea7..997fde5 100644 (file)
@@ -2735,7 +2735,7 @@ IntRect RenderBox::localCaretRect(InlineBox* box, int caretOffset, int* extraWid
     return rect;
 }
 
-int RenderBox::lowestPosition(bool includeOverflowInterior, bool includeSelf) const
+int RenderBox::lowestPosition(bool /*includeOverflowInterior*/, bool includeSelf) const
 {
     if (!includeSelf || !m_width)
         return 0;
@@ -2745,7 +2745,7 @@ int RenderBox::lowestPosition(bool includeOverflowInterior, bool includeSelf) co
     return bottom;
 }
 
-int RenderBox::rightmostPosition(bool includeOverflowInterior, bool includeSelf) const
+int RenderBox::rightmostPosition(bool /*includeOverflowInterior*/, bool includeSelf) const
 {
     if (!includeSelf || !m_height)
         return 0;
@@ -2755,7 +2755,7 @@ int RenderBox::rightmostPosition(bool includeOverflowInterior, bool includeSelf)
     return right;
 }
 
-int RenderBox::leftmostPosition(bool includeOverflowInterior, bool includeSelf) const
+int RenderBox::leftmostPosition(bool /*includeOverflowInterior*/, bool includeSelf) const
 {
     if (!includeSelf || !m_height)
         return m_width;
index ee9d929..54349fa 100644 (file)
@@ -76,7 +76,7 @@ public:
     int calcContentBoxWidth(int width) const;
     int calcContentBoxHeight(int height) const;
 
-    virtual void borderFitAdjust(int& x, int& w) const {}; // Shrink the box in which the border paints if border-fit is set.
+    virtual void borderFitAdjust(int& /*x*/, int& /*w*/) const { } // Shrink the box in which the border paints if border-fit is set.
 
     // This method is now public so that centered objects like tables that are
     // shifted right by left-aligned floats can recompute their left and
index 042f5aa..9f6d737 100644 (file)
@@ -679,7 +679,7 @@ void RenderContainer::addLineBoxRects(Vector<IntRect>& rects, unsigned start, un
     }
 }
 
-void RenderContainer::collectAbsoluteLineBoxQuads(Vector<FloatQuad>& quads, unsigned start, unsigned end, bool useSelectionHeight)
+void RenderContainer::collectAbsoluteLineBoxQuads(Vector<FloatQuad>& quads, unsigned start, unsigned end, bool /*useSelectionHeight*/)
 {
     if (!m_firstChild && (isInline() || isAnonymousBlock())) {
         absoluteQuads(quads);
index 779b439..86a92f3 100644 (file)
@@ -310,7 +310,7 @@ void RenderFlow::dirtyLinesFromChangedChild(RenderObject* child)
     }
 }
 
-int RenderFlow::lineHeight(bool firstLine, bool isRootLineBox) const
+int RenderFlow::lineHeight(bool firstLine, bool /*isRootLineBox*/) const
 {
     if (firstLine) {
         RenderStyle* s = style(firstLine);
@@ -346,7 +346,7 @@ void RenderFlow::dirtyLineBoxes(bool fullLayout, bool isRootLineBox)
     }
 }
 
-InlineBox* RenderFlow::createInlineBox(bool makePlaceHolderBox, bool isRootLineBox, bool isOnlyRun)
+InlineBox* RenderFlow::createInlineBox(bool makePlaceHolderBox, bool isRootLineBox, bool /*isOnlyRun*/)
 {
     checkConsistency();
 
index 425add0..d93fa86 100644 (file)
@@ -661,7 +661,7 @@ int RenderFrameSet::hitTestSplit(const GridAxis& axis, int position) const
     return noSplit;
 }
 
-bool RenderFrameSet::isChildAllowed(RenderObject* child, RenderStyle* style) const
+bool RenderFrameSet::isChildAllowed(RenderObject* child, RenderStyle*) const
 {
     return child->isFrame() || child->isFrameSet();
 }
index 32f1216..fc6f19b 100644 (file)
@@ -171,7 +171,7 @@ bool RenderImageScaleObserver::shouldImagePaintAtLowQuality(RenderImage* image,
 
 HashMap<RenderImage*, RenderImageScaleData*>* RenderImageScaleObserver::gImages = 0;
 
-void RenderImage::highQualityRepaintTimerFired(Timer<RenderImage>* timer)
+void RenderImage::highQualityRepaintTimerFired(Timer<RenderImage>*)
 {
     RenderImageScaleObserver::highQualityRepaintTimerFired(this);
 }
index 512bcb3..71896d6 100644 (file)
@@ -72,7 +72,7 @@ public:
     void highQualityRepaintTimerFired(Timer<RenderImage>*);
 
 protected:
-    virtual Image* image(int w = 0, int h = 0) { return m_cachedImage ? m_cachedImage->image() : nullImage(); }
+    virtual Image* image(int /*w*/ = 0, int /*h*/ = 0) { return m_cachedImage ? m_cachedImage->image() : nullImage(); }
     virtual bool errorOccurred() const { return m_cachedImage && m_cachedImage->errorOccurred(); }
     virtual bool usesImageContainerSize() const { return m_cachedImage ? m_cachedImage->usesImageContainerSize() : false; }
     virtual void setImageContainerSize(const IntSize& size) const { if (m_cachedImage) m_cachedImage->setImageContainerSize(size); }
index c79ff17..7edca02 100644 (file)
@@ -2462,7 +2462,7 @@ bool RenderLayer::shouldBeOverflowOnly() const
            !isTransparent();
 }
 
-void RenderLayer::styleChanged(RenderStyle::Diff, const RenderStyle* oldStyle)
+void RenderLayer::styleChanged(RenderStyle::Diff, const RenderStyle*)
 {
     bool isOverflowOnly = shouldBeOverflowOnly();
     if (isOverflowOnly != m_isOverflowOnly) {
index 667a0f1..35b9451 100644 (file)
@@ -237,7 +237,7 @@ void RenderListBox::calcHeight()
     }
 }
 
-int RenderListBox::baselinePosition(bool b, bool isRootLineBox) const
+int RenderListBox::baselinePosition(bool, bool) const
 {
     return height() + marginTop() + marginBottom() - baselineAdjustment;
 }
index 9d7cb54..b9d2dff 100644 (file)
@@ -653,7 +653,7 @@ bool RenderObject::scroll(ScrollDirection direction, ScrollGranularity granulari
     return false;
 }
     
-bool RenderObject::canBeProgramaticallyScrolled(bool scrollToAnchor) const
+bool RenderObject::canBeProgramaticallyScrolled(bool) const
 {
     if (!layer())
         return false;
@@ -1702,7 +1702,7 @@ void RenderObject::paintBoxShadow(GraphicsContext* context, int tx, int ty, int
     }
 }
 
-void RenderObject::addLineBoxRects(Vector<IntRect>&, unsigned startOffset, unsigned endOffset, bool useSelectionHeight)
+void RenderObject::addLineBoxRects(Vector<IntRect>&, unsigned, unsigned, bool)
 {
 }
 
@@ -1751,7 +1751,7 @@ IntRect RenderObject::absoluteBoundingBoxRect(bool useTransforms)
     return result;
 }
 
-void RenderObject::collectAbsoluteLineBoxQuads(Vector<FloatQuad>& quads, unsigned startOffset, unsigned endOffset, bool useSelectionHeight)
+void RenderObject::collectAbsoluteLineBoxQuads(Vector<FloatQuad>&, unsigned, unsigned, bool)
 {
 }
 
@@ -1998,11 +1998,11 @@ bool RenderObject::repaintAfterLayoutIfNeeded(const IntRect& oldBounds, const In
     return false;
 }
 
-void RenderObject::repaintDuringLayoutIfMoved(const IntRect& rect)
+void RenderObject::repaintDuringLayoutIfMoved(const IntRect&)
 {
 }
 
-void RenderObject::repaintOverhangingFloats(bool paintAllDescendants)
+void RenderObject::repaintOverhangingFloats(bool)
 {
 }
 
@@ -2066,7 +2066,7 @@ void RenderObject::computeAbsoluteRepaintRect(IntRect& rect, bool fixed)
     }
 }
 
-void RenderObject::dirtyLinesFromChangedChild(RenderObject* child)
+void RenderObject::dirtyLinesFromChangedChild(RenderObject*)
 {
 }
 
@@ -2379,7 +2379,7 @@ void RenderObject::styleWillChange(RenderStyle::Diff diff, const RenderStyle* ne
     }
 }
 
-void RenderObject::styleDidChange(RenderStyle::Diff diff, const RenderStyle* oldStyle)
+void RenderObject::styleDidChange(RenderStyle::Diff diff, const RenderStyle*)
 {
     setHasBoxDecorations(m_style->hasBorder() || m_style->hasBackground() || m_style->hasAppearance() || m_style->boxShadow());
 
@@ -2481,7 +2481,7 @@ IntSize RenderObject::offsetFromContainer(RenderObject* o) const
     return offset;
 }
 
-IntRect RenderObject::localCaretRect(InlineBox* inlineBox, int caretOffset, int* extraWidthToEndOfLine)
+IntRect RenderObject::localCaretRect(InlineBox*, int, int* extraWidthToEndOfLine)
 {
    if (extraWidthToEndOfLine)
        *extraWidthToEndOfLine = 0;
@@ -2676,7 +2676,7 @@ void RenderObject::arenaDelete(RenderArena* arena, void* base)
     arena->free(*(size_t*)base, base);
 }
 
-VisiblePosition RenderObject::positionForCoordinates(int x, int y)
+VisiblePosition RenderObject::positionForCoordinates(int, int)
 {
     return VisiblePosition(element(), caretMinOffset(), DOWNSTREAM);
 }
index 01a922c..2a5c8ca 100644 (file)
@@ -483,11 +483,11 @@ public:
     void paintBoxShadow(GraphicsContext*, int tx, int ty, int w, int h, const RenderStyle*, bool begin = true, bool end = true);
 
     // RenderBox implements this.
-    virtual void paintBoxDecorations(PaintInfo&, int tx, int ty) { }
-    virtual void paintMask(PaintInfo&, int tx, int ty) { }
+    virtual void paintBoxDecorations(PaintInfo&, int /*tx*/, int /*ty*/) { }
+    virtual void paintMask(PaintInfo&, int /*tx*/, int /*ty*/) { }
     virtual void paintFillLayerExtended(const PaintInfo&, const Color&, const FillLayer*,
-                                        int clipy, int cliph, int tx, int ty, int width, int height,
-                                        InlineFlowBox* box = 0, CompositeOperator = CompositeSourceOver) { }
+                                        int /*clipY*/, int /*clipH*/, int /*tx*/, int /*ty*/, int /*width*/, int /*height*/,
+                                        InlineFlowBox* = 0, CompositeOperator = CompositeSourceOver) { }
 
     
     /*
@@ -885,8 +885,8 @@ public:
     virtual int previousOffset(int current) const;
     virtual int nextOffset(int current) const;
 
-    virtual void imageChanged(CachedImage* image, const IntRect* = 0);
-    virtual void imageChanged(WrappedImagePtr data, const IntRect* = 0) { };
+    virtual void imageChanged(CachedImage*, const IntRect* = 0);
+    virtual void imageChanged(WrappedImagePtr, const IntRect* = 0) { }
     virtual bool willRenderImage(CachedImage*);
 
     virtual void selectionStartEnd(int& spos, int& epos) const;
index e0e6cca..3e65aa1 100644 (file)
@@ -200,12 +200,12 @@ bool RenderPath::requiresLayer()
     return false;
 }
 
-int RenderPath::lineHeight(bool b, bool isRootLineBox) const
+int RenderPath::lineHeight(bool, bool) const
 {
     return relativeBBox(true).height();
 }
 
-int RenderPath::baselinePosition(bool b, bool isRootLineBox) const
+int RenderPath::baselinePosition(bool, bool) const
 {
     return relativeBBox(true).height();
 }
@@ -269,12 +269,12 @@ void RenderPath::absoluteRects(Vector<IntRect>& rects, int, int, bool)
     rects.append(absoluteClippedOverflowRect());
 }
 
-void RenderPath::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderPath::absoluteQuads(Vector<FloatQuad>& quads, bool)
 {
     quads.append(absoluteClippedOverflowRect());
 }
 
-bool RenderPath::nodeAtPoint(const HitTestRequest& request, HitTestResult& result, int _x, int _y, int, int, HitTestAction hitTestAction)
+bool RenderPath::nodeAtPoint(const HitTestRequest&, HitTestResult& result, int _x, int _y, int, int, HitTestAction hitTestAction)
 {
     // We only draw in the forground phase, so we only hit-test then.
     if (hitTestAction != HitTestForeground)
@@ -417,7 +417,7 @@ static void drawStartAndMidMarkers(void* info, const PathElement* element)
     data.elementIndex++;
 }
 
-FloatRect RenderPath::drawMarkersIfNeeded(GraphicsContext* context, const FloatRect& rect, const Path& path) const
+FloatRect RenderPath::drawMarkersIfNeeded(GraphicsContext* context, const FloatRect&, const Path& path) const
 {
     Document* doc = document();
 
index db94e3e..79df0f1 100644 (file)
@@ -345,7 +345,7 @@ void RenderReplaced::adjustOverflowForBoxShadow()
     }
 }
 
-int RenderReplaced::overflowHeight(bool includeInterior) const
+int RenderReplaced::overflowHeight(bool) const
 {
     if (m_hasOverflow) {
         IntRect *r = &gOverflowRectMap->find(this)->second;
@@ -355,7 +355,7 @@ int RenderReplaced::overflowHeight(bool includeInterior) const
     return height();
 }
 
-int RenderReplaced::overflowWidth(bool includeInterior) const
+int RenderReplaced::overflowWidth(bool) const
 {
     if (m_hasOverflow) {
         IntRect *r = &gOverflowRectMap->find(this)->second;
@@ -365,7 +365,7 @@ int RenderReplaced::overflowWidth(bool includeInterior) const
     return width();
 }
 
-int RenderReplaced::overflowLeft(bool includeInterior) const
+int RenderReplaced::overflowLeft(bool) const
 {
     if (m_hasOverflow)
         return gOverflowRectMap->get(this).x();
@@ -373,7 +373,7 @@ int RenderReplaced::overflowLeft(bool includeInterior) const
     return 0;
 }
 
-int RenderReplaced::overflowTop(bool includeInterior) const
+int RenderReplaced::overflowTop(bool) const
 {
     if (m_hasOverflow)
         return gOverflowRectMap->get(this).y();
@@ -381,7 +381,7 @@ int RenderReplaced::overflowTop(bool includeInterior) const
     return 0;
 }
 
-IntRect RenderReplaced::overflowRect(bool includeInterior) const
+IntRect RenderReplaced::overflowRect(bool) const
 {
     if (m_hasOverflow)
         return gOverflowRectMap->find(this)->second;
index dadddc8..3186718 100644 (file)
@@ -43,7 +43,7 @@ public:
     virtual int minimumReplacedHeight() const { return 0; }
 
     virtual void paint(PaintInfo&, int tx, int ty);
-    virtual void paintReplaced(PaintInfo&, int tx, int ty) { }
+    virtual void paintReplaced(PaintInfo&, int /*tx*/, int /*ty*/) { }
 
     virtual IntSize intrinsicSize() const;
 
index aa4f496..1829a0b 100644 (file)
@@ -209,12 +209,12 @@ bool RenderSVGContainer::requiresLayer()
     return false;
 }
 
-int RenderSVGContainer::lineHeight(bool b, bool isRootLineBox) const
+int RenderSVGContainer::lineHeight(bool, bool) const
 {
     return height() + marginTop() + marginBottom();
 }
 
-int RenderSVGContainer::baselinePosition(bool b, bool isRootLineBox) const
+int RenderSVGContainer::baselinePosition(bool, bool) const
 {
     return height() + marginTop() + marginBottom();
 }
@@ -299,7 +299,7 @@ void RenderSVGContainer::applyContentTransforms(PaintInfo& paintInfo)
         paintInfo.context->concatCTM(localTransform());
 }
 
-void RenderSVGContainer::applyAdditionalTransforms(PaintInfo& paintInfo)
+void RenderSVGContainer::applyAdditionalTransforms(PaintInfo&)
 {
     // no-op
 }
@@ -322,7 +322,7 @@ bool RenderSVGContainer::selfWillPaint() const
     return false;
 }
 
-void RenderSVGContainer::paint(PaintInfo& paintInfo, int parentX, int parentY)
+void RenderSVGContainer::paint(PaintInfo& paintInfo, int, int)
 {
     if (paintInfo.context->paintingDisabled() || !drawsContents())
         return;
@@ -383,7 +383,7 @@ IntRect RenderSVGContainer::absoluteClippedOverflowRect()
     return enclosingIntRect(repaintRect);
 }
 
-void RenderSVGContainer::addFocusRingRects(GraphicsContext* graphicsContext, int tx, int ty)
+void RenderSVGContainer::addFocusRingRects(GraphicsContext* graphicsContext, int, int)
 {
     graphicsContext->addFocusRingRect(m_absoluteBounds);
 }
@@ -393,7 +393,7 @@ void RenderSVGContainer::absoluteRects(Vector<IntRect>& rects, int, int, bool)
     rects.append(absoluteClippedOverflowRect());
 }
 
-void RenderSVGContainer::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderSVGContainer::absoluteQuads(Vector<FloatQuad>& quads, bool)
 {
     quads.append(absoluteClippedOverflowRect());
 }
index 5ef7719..ec7a05d 100644 (file)
@@ -44,12 +44,12 @@ bool RenderSVGHiddenContainer::requiresLayer()
     return false;
 }
 
-int RenderSVGHiddenContainer::lineHeight(bool b, bool isRootLineBox) const
+int RenderSVGHiddenContainer::lineHeight(bool, bool) const
 {
     return 0;
 }
 
-int RenderSVGHiddenContainer::baselinePosition(bool b, bool isRootLineBox) const
+int RenderSVGHiddenContainer::baselinePosition(bool, bool) const
 {
     return 0;
 }
@@ -81,12 +81,12 @@ IntRect RenderSVGHiddenContainer::absoluteClippedOverflowRect()
     return IntRect();
 }
 
-void RenderSVGHiddenContainer::absoluteRects(Vector<IntRect>& rects, int, int, bool)
+void RenderSVGHiddenContainer::absoluteRects(Vector<IntRect>&, int, int, bool)
 {
     // This subtree does not take up space or paint
 }
 
-void RenderSVGHiddenContainer::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderSVGHiddenContainer::absoluteQuads(Vector<FloatQuad>&, bool)
 {
     // This subtree does not take up space or paint
 }
@@ -101,12 +101,12 @@ AffineTransform RenderSVGHiddenContainer::localTransform() const
     return AffineTransform();
 }
 
-bool RenderSVGHiddenContainer::nodeAtPoint(const HitTestRequest& request, HitTestResult& result, int _x, int _y, int _tx, int _ty, HitTestAction hitTestAction)
+bool RenderSVGHiddenContainer::nodeAtPoint(const HitTestRequest&, HitTestResult&, int, int, int, int, HitTestAction)
 {
     return false;
 }
 
-FloatRect RenderSVGHiddenContainer::relativeBBox(bool includeStroke) const
+FloatRect RenderSVGHiddenContainer::relativeBBox(bool) const
 {
     return FloatRect();
 }
index c841964..87315cd 100644 (file)
@@ -193,7 +193,7 @@ void RenderSVGImage::paint(PaintInfo& paintInfo, int, int)
     paintInfo.context->restore();
 }
 
-bool RenderSVGImage::nodeAtPoint(const HitTestRequest& request, HitTestResult& result, int _x, int _y, int, int, HitTestAction hitTestAction)
+bool RenderSVGImage::nodeAtPoint(const HitTestRequest&, HitTestResult& result, int _x, int _y, int, int, HitTestAction hitTestAction)
 {
     // We only draw in the forground phase, so we only hit-test then.
     if (hitTestAction != HitTestForeground)
@@ -258,7 +258,7 @@ IntRect RenderSVGImage::absoluteClippedOverflowRect()
     return m_absoluteBounds;
 }
 
-void RenderSVGImage::addFocusRingRects(GraphicsContext* graphicsContext, int tx, int ty)
+void RenderSVGImage::addFocusRingRects(GraphicsContext* graphicsContext, int, int)
 {
     // this is called from paint() after the localTransform has already been applied
     IntRect contentRect = enclosingIntRect(relativeBBox());
@@ -270,7 +270,7 @@ void RenderSVGImage::absoluteRects(Vector<IntRect>& rects, int, int, bool)
     rects.append(absoluteClippedOverflowRect());
 }
 
-void RenderSVGImage::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderSVGImage::absoluteQuads(Vector<FloatQuad>& quads, bool)
 {
     quads.append(FloatRect(absoluteClippedOverflowRect()));
 }
index 385d19d..bfae64c 100644 (file)
@@ -58,7 +58,7 @@ void RenderSVGInlineText::absoluteRects(Vector<IntRect>& rects, int, int, bool)
     rects.append(computeAbsoluteRectForRange(0, textLength()));
 }
 
-void RenderSVGInlineText::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderSVGInlineText::absoluteQuads(Vector<FloatQuad>& quads, bool)
 {
     quads.append(FloatRect(computeAbsoluteRectForRange(0, textLength())));
 }
index bbd4e28..8f418ae 100644 (file)
@@ -53,12 +53,12 @@ RenderSVGRoot::~RenderSVGRoot()
 {
 }
 
-int RenderSVGRoot::lineHeight(bool b, bool isRootLineBox) const
+int RenderSVGRoot::lineHeight(bool, bool) const
 {
     return height() + marginTop() + marginBottom();
 }
 
-int RenderSVGRoot::baselinePosition(bool b, bool isRootLineBox) const
+int RenderSVGRoot::baselinePosition(bool, bool) const
 {
     return height() + marginTop() + marginBottom();
 }
@@ -259,7 +259,7 @@ void RenderSVGRoot::absoluteRects(Vector<IntRect>& rects, int, int)
         current->absoluteRects(rects, 0, 0);
 }
 
-void RenderSVGRoot::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderSVGRoot::absoluteQuads(Vector<FloatQuad>& quads, bool)
 {
     for (RenderObject* current = firstChild(); current != 0; current = current->nextSibling())
         current->absoluteQuads(quads);
index eb1b07b..fd2d3dd 100644 (file)
@@ -57,7 +57,7 @@ void RenderSVGTSpan::absoluteRects(Vector<IntRect>& rects, int, int, bool)
     }
 }
 
-void RenderSVGTSpan::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderSVGTSpan::absoluteQuads(Vector<FloatQuad>& quads, bool)
 {
     InlineRunBox* firstBox = firstLineBox();
 
index 0d23bcb..c53af66 100644 (file)
@@ -111,7 +111,7 @@ void RenderSVGText::layout()
     setNeedsLayout(false);
 }
 
-InlineBox* RenderSVGText::createInlineBox(bool makePlaceHolderBox, bool isRootLineBox, bool isOnlyRun)
+InlineBox* RenderSVGText::createInlineBox(bool, bool, bool)
 {
     ASSERT(!isInlineFlow());
     InlineFlowBox* flowBox = new (renderArena()) SVGRootInlineBox(this);
@@ -170,7 +170,7 @@ void RenderSVGText::absoluteRects(Vector<IntRect>& rects, int, int, bool)
     }
 }
 
-void RenderSVGText::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderSVGText::absoluteQuads(Vector<FloatQuad>& quads, bool)
 {
     RenderSVGRoot* root = findSVGRootObject(parent());
     if (!root)
index d5d8fde..aa0209c 100644 (file)
@@ -97,7 +97,7 @@ void RenderSVGTextPath::absoluteRects(Vector<IntRect>& rects, int, int)
     }
 }
 
-void RenderSVGTextPath::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderSVGTextPath::absoluteQuads(Vector<FloatQuad>& quads, bool)
 {
     InlineRunBox* firstBox = firstLineBox();
 
index 37ceac8..040fa31 100644 (file)
@@ -131,7 +131,7 @@ RenderSlider::~RenderSlider()
         m_thumb->detach();
 }
 
-int RenderSlider::baselinePosition(bool b, bool isRootLineBox) const
+int RenderSlider::baselinePosition(bool, bool) const
 {
     return height() + marginTop();
 }
index 58e17df..5c2a049 100644 (file)
@@ -83,7 +83,7 @@ IntRect RenderTableCol::absoluteClippedOverflowRect()
     return IntRect();
 }
 
-void RenderTableCol::imageChanged(WrappedImagePtr image, const IntRect*)
+void RenderTableCol::imageChanged(WrappedImagePtr, const IntRect*)
 {
     // FIXME: Repaint only the rect the image paints in.
     repaint();
index c37bff0..21ba91a 100644 (file)
@@ -210,7 +210,7 @@ void RenderTableRow::paint(PaintInfo& paintInfo, int tx, int ty)
     }
 }
 
-void RenderTableRow::imageChanged(WrappedImagePtr image, const IntRect*)
+void RenderTableRow::imageChanged(WrappedImagePtr, const IntRect*)
 {
     // FIXME: Examine cells and repaint only the rect the image paints in.
     repaint();
index c874d40..a65d0e9 100644 (file)
@@ -35,17 +35,18 @@ class RenderTableRow : public RenderContainer {
 public:
     RenderTableRow(Node*);
 
+    RenderTableSection* section() const { return static_cast<RenderTableSection*>(parent()); }
+    RenderTable* table() const { return static_cast<RenderTable*>(parent()->parent()); }
+
+private:
     virtual const char* renderName() const { return isAnonymous() ? "RenderTableRow (anonymous)" : "RenderTableRow"; }
 
     virtual bool isTableRow() const { return true; }
 
     virtual void destroy();
 
-    RenderTableSection* section() const { return static_cast<RenderTableSection*>(parent()); }
-    RenderTable* table() const { return static_cast<RenderTable*>(parent()->parent()); }
-
     virtual void addChild(RenderObject* child, RenderObject* beforeChild = 0);
-    virtual int lineHeight(bool firstLine, bool isRootLineBox = false) const { return 0; }
+    virtual int lineHeight(bool, bool) const { return 0; }
     virtual void position(InlineBox*) { }
     virtual void layout();
     virtual IntRect absoluteClippedOverflowRect();
@@ -57,7 +58,6 @@ public:
     virtual void paint(PaintInfo&, int tx, int ty);
     virtual void imageChanged(WrappedImagePtr, const IntRect* = 0);
 
-protected:
     virtual void styleWillChange(RenderStyle::Diff, const RenderStyle* newStyle);
 
 };
index eb37752..be43ab8 100644 (file)
@@ -973,7 +973,7 @@ void RenderTableSection::paint(PaintInfo& paintInfo, int tx, int ty)
     }
 }
 
-void RenderTableSection::imageChanged(WrappedImagePtr image, const IntRect*)
+void RenderTableSection::imageChanged(WrappedImagePtr, const IntRect*)
 {
     // FIXME: Examine cells and repaint only the rect the image paints in.
     repaint();
index f7a4e84..8d460cb 100644 (file)
@@ -47,9 +47,6 @@ public:
 
     virtual void addChild(RenderObject* child, RenderObject* beforeChild = 0);
 
-    virtual int lineHeight(bool firstLine, bool isRootLineBox = false) const { return 0; }
-    virtual void position(InlineBox*) { }
-
     virtual int getBaselineOfFirstLineBox() const;
 
     void addCell(RenderTableCell*, RenderObject* row);
@@ -126,6 +123,9 @@ public:
     virtual bool nodeAtPoint(const HitTestRequest&, HitTestResult&, int x, int y, int tx, int ty, HitTestAction);
 
 private:
+    virtual int lineHeight(bool, bool) const { return 0; }
+    virtual void position(InlineBox*) { }
+
     bool ensureRows(int);
     void clearGrid();
 
index 6e54c70..d7fdbb9 100644 (file)
@@ -247,7 +247,7 @@ void RenderText::addLineBoxRects(Vector<IntRect>& rects, unsigned start, unsigne
     }
 }
 
-void RenderText::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderText::absoluteQuads(Vector<FloatQuad>& quads, bool)
 {
     for (InlineTextBox* box = firstTextBox(); box; box = box->nextTextBox())
         quads.append(localToAbsoluteQuad(FloatRect(box->xPos(), box->yPos(), box->width(), box->height())));
index ac5102e..4c62638 100644 (file)
@@ -56,11 +56,6 @@ public:
     virtual InlineTextBox* createInlineTextBox();
     virtual void dirtyLineBoxes(bool fullLayout, bool isRootInlineBox = false);
 
-    virtual void paint(PaintInfo&, int tx, int ty) { ASSERT_NOT_REACHED(); }
-    virtual void layout() { ASSERT_NOT_REACHED(); }
-
-    virtual bool nodeAtPoint(const HitTestRequest&, HitTestResult&, int, int, int, int, HitTestAction) { ASSERT_NOT_REACHED(); return false; }
-
     virtual void absoluteRects(Vector<IntRect>&, int tx, int ty, bool topLevel = true);
     virtual void addLineBoxRects(Vector<IntRect>&, unsigned startOffset = 0, unsigned endOffset = UINT_MAX, bool useSelectionHeight = false);
 
@@ -144,6 +139,10 @@ private:
     // callers with a RenderObject* can continue to use length().
     virtual unsigned length() const { return textLength(); }
 
+    virtual void paint(PaintInfo&, int, int) { ASSERT_NOT_REACHED(); }
+    virtual void layout() { ASSERT_NOT_REACHED(); }
+    virtual bool nodeAtPoint(const HitTestRequest&, HitTestResult&, int, int, int, int, HitTestAction) { ASSERT_NOT_REACHED(); return false; }
+
     void deleteTextBoxes();
     bool containsOnlyWhitespace(unsigned from, unsigned len) const;
     int widthFromCache(const Font&, int start, int len, int xPos) const;
index 6d75141..4f52527 100644 (file)
@@ -655,7 +655,7 @@ bool RenderTextControlSingleLine::itemIsEnabled(unsigned listIndex) const
     return true;
 }
 
-PopupMenuStyle RenderTextControlSingleLine::itemStyle(unsigned listIndex) const
+PopupMenuStyle RenderTextControlSingleLine::itemStyle(unsigned) const
 {
     return menuStyle();
 }
index cb1cb5b..bd427c1 100644 (file)
@@ -592,7 +592,8 @@ bool RenderTheme::isDefault(const RenderObject* o) const
 }
 
 #if !USE(NEW_THEME)
-void RenderTheme::adjustCheckboxStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+
+void RenderTheme::adjustCheckboxStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
 {
     // A summary of the rules for checkbox designed to match WinIE:
     // width/height - honored (WinIE actually scales its control for small widths, but lets it overflow for small heights.)
@@ -609,7 +610,7 @@ void RenderTheme::adjustCheckboxStyle(CSSStyleSelector* selector, RenderStyle* s
     style->setBoxShadow(0);
 }
 
-void RenderTheme::adjustRadioStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustRadioStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
 {
     // A summary of the rules for checkbox designed to match WinIE:
     // width/height - honored (WinIE actually scales its control for small widths, but lets it overflow for small heights.)
@@ -626,39 +627,40 @@ void RenderTheme::adjustRadioStyle(CSSStyleSelector* selector, RenderStyle* styl
     style->setBoxShadow(0);
 }
 
-void RenderTheme::adjustButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustButtonStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
 {
     // Most platforms will completely honor all CSS, and so we have no need to adjust the style
     // at all by default.  We will still allow the theme a crack at setting up a desired vertical size.
     setButtonSize(style);
 }
+
 #endif
 
-void RenderTheme::adjustTextFieldStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustTextFieldStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
-void RenderTheme::adjustTextAreaStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustTextAreaStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
-void RenderTheme::adjustMenuListStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustMenuListStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
-void RenderTheme::adjustMenuListButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustMenuListButtonStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
-void RenderTheme::adjustButtonInnerStyle(RenderStyle* style) const
+void RenderTheme::adjustButtonInnerStyle(RenderStyle*) const
 {
 }
 
-void RenderTheme::adjustSliderTrackStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustSliderTrackStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
-void RenderTheme::adjustSliderThumbStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustSliderThumbStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
@@ -666,23 +668,23 @@ void RenderTheme::adjustSliderThumbSize(RenderObject*) const
 {
 }
 
-void RenderTheme::adjustSearchFieldStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustSearchFieldStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
-void RenderTheme::adjustSearchFieldCancelButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustSearchFieldCancelButtonStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
-void RenderTheme::adjustSearchFieldDecorationStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustSearchFieldDecorationStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
-void RenderTheme::adjustSearchFieldResultsDecorationStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustSearchFieldResultsDecorationStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
-void RenderTheme::adjustSearchFieldResultsButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderTheme::adjustSearchFieldResultsButtonStyle(CSSStyleSelector*, RenderStyle*, Element*) const
 {
 }
 
index c8c7138..09df200 100644 (file)
@@ -607,7 +607,7 @@ void RenderThemeMac::setSizeFromFont(RenderStyle* style, const IntSize* sizes) c
         style->setHeight(Length(size.height(), Fixed));
 }
 
-void RenderThemeMac::setFontFromControlSize(CSSStyleSelector* selector, RenderStyle* style, NSControlSize controlSize) const
+void RenderThemeMac::setFontFromControlSize(CSSStyleSelector*, RenderStyle* style, NSControlSize controlSize) const
 {
     FontDescription fontDescription;
     fontDescription.setIsAbsoluteSize(true);
@@ -646,7 +646,7 @@ void RenderThemeMac::adjustTextFieldStyle(CSSStyleSelector*, RenderStyle*, Eleme
 {
 }
 
-bool RenderThemeMac::paintCapsLockIndicator(RenderObject* o, const RenderObject::PaintInfo& paintInfo, const IntRect& r)
+bool RenderThemeMac::paintCapsLockIndicator(RenderObject*, const RenderObject::PaintInfo& paintInfo, const IntRect& r)
 {
     if (paintInfo.context->paintingDisabled())
         return true;
@@ -747,7 +747,7 @@ const int styledPopupPaddingLeft = 8;
 const int styledPopupPaddingTop = 1;
 const int styledPopupPaddingBottom = 2;
 
-static void TopGradientInterpolate(void* info, const CGFloat* inData, CGFloat* outData)
+static void TopGradientInterpolate(void*, const CGFloat* inData, CGFloat* outData)
 {
     static float dark[4] = { 1.0f, 1.0f, 1.0f, 0.4f };
     static float light[4] = { 1.0f, 1.0f, 1.0f, 0.15f };
@@ -757,7 +757,7 @@ static void TopGradientInterpolate(void* info, const CGFloat* inData, CGFloat* o
         outData[i] = (1.0f - a) * dark[i] + a * light[i];
 }
 
-static void BottomGradientInterpolate(void* info, const CGFloat* inData, CGFloat* outData)
+static void BottomGradientInterpolate(void*, const CGFloat* inData, CGFloat* outData)
 {
     static float dark[4] = { 1.0f, 1.0f, 1.0f, 0.0f };
     static float light[4] = { 1.0f, 1.0f, 1.0f, 0.3f };
@@ -767,7 +767,7 @@ static void BottomGradientInterpolate(void* info, const CGFloat* inData, CGFloat
         outData[i] = (1.0f - a) * dark[i] + a * light[i];
 }
 
-static void MainGradientInterpolate(void* info, const CGFloat* inData, CGFloat* outData)
+static void MainGradientInterpolate(void*, const CGFloat* inData, CGFloat* outData)
 {
     static float dark[4] = { 0.0f, 0.0f, 0.0f, 0.15f };
     static float light[4] = { 0.0f, 0.0f, 0.0f, 0.0f };
@@ -777,7 +777,7 @@ static void MainGradientInterpolate(void* info, const CGFloat* inData, CGFloat*
         outData[i] = (1.0f - a) * dark[i] + a * light[i];
 }
 
-static void TrackGradientInterpolate(void* info, const CGFloat* inData, CGFloat* outData)
+static void TrackGradientInterpolate(void*, const CGFloat* inData, CGFloat* outData)
 {
     static float dark[4] = { 0.0f, 0.0f, 0.0f, 0.678f };
     static float light[4] = { 0.0f, 0.0f, 0.0f, 0.13f };
@@ -986,7 +986,7 @@ int RenderThemeMac::popupInternalPaddingBottom(RenderStyle* style) const
     return 0;
 }
 
-void RenderThemeMac::adjustMenuListButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderThemeMac::adjustMenuListButtonStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
 {
     float fontScale = style->fontSize() / baseFontSize;
 
@@ -1027,7 +1027,7 @@ int RenderThemeMac::minimumMenuListSize(RenderStyle* style) const
 const int trackWidth = 5;
 const int trackRadius = 2;
 
-void RenderThemeMac::adjustSliderTrackStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderThemeMac::adjustSliderTrackStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
 {
     style->setBoxShadow(0);
 }
@@ -1071,7 +1071,7 @@ bool RenderThemeMac::paintSliderTrack(RenderObject* o, const RenderObject::Paint
     return false;
 }
 
-void RenderThemeMac::adjustSliderThumbStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderThemeMac::adjustSliderThumbStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
 {
     style->setBoxShadow(0);
 }
@@ -1193,7 +1193,7 @@ bool RenderThemeMac::paintSearchField(RenderObject* o, const RenderObject::Paint
     return false;
 }
 
-void RenderThemeMac::setSearchCellState(RenderObject* o, const IntRect& r)
+void RenderThemeMac::setSearchCellState(RenderObject* o, const IntRect&)
 {
     NSSearchFieldCell* search = this->search();
 
@@ -1220,7 +1220,7 @@ void RenderThemeMac::setSearchFieldSize(RenderStyle* style) const
     setSizeFromFont(style, searchFieldSizes());
 }
 
-void RenderThemeMac::adjustSearchFieldStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderThemeMac::adjustSearchFieldStyle(CSSStyleSelector* selector, RenderStyle* style, Element*) const
 {
     // Override border.
     style->resetBorder();
@@ -1289,7 +1289,7 @@ const IntSize* RenderThemeMac::cancelButtonSizes() const
     return sizes;
 }
 
-void RenderThemeMac::adjustSearchFieldCancelButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderThemeMac::adjustSearchFieldCancelButtonStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
 {
     IntSize size = sizeForSystemFont(style, cancelButtonSizes());
     style->setWidth(Length(size.width(), Fixed));
@@ -1304,7 +1304,7 @@ const IntSize* RenderThemeMac::resultsButtonSizes() const
 }
 
 const int emptyResultsOffset = 9;
-void RenderThemeMac::adjustSearchFieldDecorationStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderThemeMac::adjustSearchFieldDecorationStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
 {
     IntSize size = sizeForSystemFont(style, resultsButtonSizes());
     style->setWidth(Length(size.width() - emptyResultsOffset, Fixed));
@@ -1312,12 +1312,12 @@ void RenderThemeMac::adjustSearchFieldDecorationStyle(CSSStyleSelector* selector
     style->setBoxShadow(0);
 }
 
-bool RenderThemeMac::paintSearchFieldDecoration(RenderObject* o, const RenderObject::PaintInfo& paintInfo, const IntRect& r)
+bool RenderThemeMac::paintSearchFieldDecoration(RenderObject*, const RenderObject::PaintInfo&, const IntRect&)
 {
     return false;
 }
 
-void RenderThemeMac::adjustSearchFieldResultsDecorationStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderThemeMac::adjustSearchFieldResultsDecorationStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
 {
     IntSize size = sizeForSystemFont(style, resultsButtonSizes());
     style->setWidth(Length(size.width(), Fixed));
@@ -1325,7 +1325,7 @@ void RenderThemeMac::adjustSearchFieldResultsDecorationStyle(CSSStyleSelector* s
     style->setBoxShadow(0);
 }
 
-bool RenderThemeMac::paintSearchFieldResultsDecoration(RenderObject* o, const RenderObject::PaintInfo& paintInfo, const IntRect& r)
+bool RenderThemeMac::paintSearchFieldResultsDecoration(RenderObject* o, const RenderObject::PaintInfo&, const IntRect& r)
 {
     Node* input = o->node()->shadowAncestorNode();
     setSearchCellState(input->renderer(), r);
@@ -1344,7 +1344,7 @@ bool RenderThemeMac::paintSearchFieldResultsDecoration(RenderObject* o, const Re
 }
 
 const int resultsArrowWidth = 5;
-void RenderThemeMac::adjustSearchFieldResultsButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+void RenderThemeMac::adjustSearchFieldResultsButtonStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
 {
     IntSize size = sizeForSystemFont(style, resultsButtonSizes());
     style->setWidth(Length(size.width() + resultsArrowWidth, Fixed));
index 1008599..1715509 100644 (file)
@@ -131,7 +131,7 @@ void RenderView::layout()
     setNeedsLayout(false);
 }
 
-FloatPoint RenderView::localToAbsolute(FloatPoint localPoint, bool fixed, bool useTransforms) const
+FloatPoint RenderView::localToAbsolute(FloatPoint localPoint, bool fixed, bool) const
 {
     if (fixed && m_frameView)
         localPoint += m_frameView->scrollOffset();
@@ -139,7 +139,7 @@ FloatPoint RenderView::localToAbsolute(FloatPoint localPoint, bool fixed, bool u
     return localPoint;
 }
 
-FloatPoint RenderView::absoluteToLocal(FloatPoint containerPoint, bool fixed, bool useTransforms) const
+FloatPoint RenderView::absoluteToLocal(FloatPoint containerPoint, bool fixed, bool) const
 {
     if (fixed && m_frameView)
         containerPoint -= m_frameView->scrollOffset();
@@ -174,7 +174,7 @@ static inline bool rendererObscuresBackground(RenderObject* object)
     return object && object->style()->visibility() == VISIBLE && object->style()->opacity() == 1 && !object->style()->hasTransform();
 }
     
-void RenderView::paintBoxDecorations(PaintInfo& paintInfo, int tx, int ty)
+void RenderView::paintBoxDecorations(PaintInfo& paintInfo, int, int)
 {
     // Check to see if we are enclosed by a layer that requires complex painting rules.  If so, we cannot blit
     // when scrolling, and we need to use slow repaints.  Examples of layers that require this are transparent layers,
@@ -258,7 +258,7 @@ void RenderView::absoluteRects(Vector<IntRect>& rects, int tx, int ty, bool)
     rects.append(IntRect(tx, ty, m_layer->width(), m_layer->height()));
 }
 
-void RenderView::absoluteQuads(Vector<FloatQuad>& quads, bool topLevel)
+void RenderView::absoluteQuads(Vector<FloatQuad>& quads, bool)
 {
     quads.append(FloatRect(0, 0, m_layer->width(), m_layer->height()));
 }
index bbd8f4a..3ea1193 100644 (file)
@@ -32,18 +32,18 @@ namespace WebCore {
 
 using namespace SVGNames;
 
-void SVGInlineFlowBox::paint(RenderObject::PaintInfo& paintInfo, int tx, int ty)
+void SVGInlineFlowBox::paint(RenderObject::PaintInfo&, int, int)
 {
     ASSERT_NOT_REACHED();
 }
 
-int SVGInlineFlowBox::placeBoxesHorizontally(int x, int& leftPosition, int& rightPosition, bool& needsWordSpacing)
+int SVGInlineFlowBox::placeBoxesHorizontally(int, int&, int&, bool&)
 {
     // no-op
     return 0;
 }
 
-void SVGInlineFlowBox::verticallyAlignBoxes(int& heightOfBlock)
+void SVGInlineFlowBox::verticallyAlignBoxes(int&)
 {
     // no-op
 }
index ceb7b0e..f063222 100644 (file)
@@ -80,10 +80,8 @@ float SVGInlineTextBox::calculateGlyphWidth(RenderStyle* style, int offset, int
     return style->font().floatWidth(svgTextRunForInlineTextBox(textObject()->text()->characters() + offset, 1, style, this, 0), extraCharsAvailable, charsConsumed, glyphName);
 }
 
-float SVGInlineTextBox::calculateGlyphHeight(RenderStyle* style, int offset, int extraCharsAvailable) const
+float SVGInlineTextBox::calculateGlyphHeight(RenderStyle* style, int, int) const
 {
-    ASSERT(style);
-
     // This is just a guess, and the only purpose of this function is to centralize this hack.
     // In real-life top-top-bottom scripts this won't be enough, I fear.
     return style->font().ascent() + style->font().descent();
@@ -279,21 +277,21 @@ bool SVGInlineTextBox::svgCharacterHitsPosition(int x, int y, int& offset) const
     return true;
 }
 
-int SVGInlineTextBox::offsetForPosition(int x, bool includePartialGlyphs) const
+int SVGInlineTextBox::offsetForPosition(int, bool) const
 {
     // SVG doesn't use the offset <-> position selection system. 
     ASSERT_NOT_REACHED();
     return 0;
 }
 
-int SVGInlineTextBox::positionForOffset(int offset) const
+int SVGInlineTextBox::positionForOffset(int) const
 {
     // SVG doesn't use the offset <-> position selection system. 
     ASSERT_NOT_REACHED();
     return 0;
 }
 
-bool SVGInlineTextBox::nodeAtPoint(const HitTestRequest& request, HitTestResult& result, int x, int y, int tx, int ty)
+bool SVGInlineTextBox::nodeAtPoint(const HitTestRequest&, HitTestResult& result, int x, int y, int tx, int ty)
 {
     ASSERT(!isLineBreak());
 
@@ -436,7 +434,7 @@ void SVGInlineTextBox::paintCharacters(RenderObject::PaintInfo& paintInfo, int t
         paintInfo.context->concatCTM(ctm.inverse());
 }
 
-void SVGInlineTextBox::paintSelection(int boxStartOffset, const SVGChar& svgChar, const UChar* chars, int length, GraphicsContext* p, RenderStyle* style, const Font* f)
+void SVGInlineTextBox::paintSelection(int boxStartOffset, const SVGChar& svgChar, const UChar*, int length, GraphicsContext* p, RenderStyle* style, const Font* f)
 {
     if (selectionState() == RenderObject::SelectionNone)
         return;
index 8f0ad94..2dc5871 100644 (file)
@@ -120,8 +120,8 @@ public:
     static EFillRepeat initialFillRepeat(EFillLayerType) { return RepeatFill; }
     static CompositeOperator initialFillComposite(EFillLayerType) { return CompositeSourceOver; }
     static LengthSize initialFillSize(EFillLayerType) { return LengthSize(); }
-    static Length initialFillXPosition(EFillLayerType type) { return Length(0.0, Percent); }
-    static Length initialFillYPosition(EFillLayerType type) { return Length(0.0, Percent); }
+    static Length initialFillXPosition(EFillLayerType) { return Length(0.0, Percent); }
+    static Length initialFillYPosition(EFillLayerType) { return Length(0.0, Percent); }
     static StyleImage* initialFillImage(EFillLayerType) { return 0; }
 
 private:
index 17ebce9..b55c5b9 100644 (file)
@@ -84,7 +84,7 @@ void StyleCachedImage::removeClient(RenderObject* renderer)
     return m_image->removeClient(renderer);
 }
 
-Image* StyleCachedImage::image(RenderObject* renderer, const IntSize&) const
+Image* StyleCachedImage::image(RenderObject*, const IntSize&) const
 {
     return m_image->image();
 }
index 4bdba6e..cb90288 100644 (file)
@@ -48,7 +48,7 @@ public:
     
     virtual PassRefPtr<CSSValue> cssValue() = 0;
 
-    virtual bool canRender(float multiplier) const { return true; }
+    virtual bool canRender(float /*multiplier*/) const { return true; }
     virtual bool isLoaded() const { return true; }
     virtual bool errorOccurred() const { return false; }
     virtual IntSize imageSize(const RenderObject*, float multiplier) const = 0;
index 333dd52..ca0a7f2 100644 (file)
@@ -67,61 +67,61 @@ int DatabaseAuthorizer::dropTempTable(const String& tableName)
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::allowAlterTable(const String& databaseName, const String& tableName)
+int DatabaseAuthorizer::allowAlterTable(const String&, const String& tableName)
 {
     m_lastActionChangedDatabase = true;
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::createIndex(const String& indexName, const String& tableName)
+int DatabaseAuthorizer::createIndex(const String&, const String& tableName)
 {
     m_lastActionChangedDatabase = true;
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::createTempIndex(const String& indexName, const String& tableName)
+int DatabaseAuthorizer::createTempIndex(const String&, const String& tableName)
 {
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::dropIndex(const String& indexName, const String& tableName)
+int DatabaseAuthorizer::dropIndex(const String&, const String& tableName)
 {
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::dropTempIndex(const String& indexName, const String& tableName)
+int DatabaseAuthorizer::dropTempIndex(const String&, const String& tableName)
 {
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::createTrigger(const String& triggerName, const String& tableName)
+int DatabaseAuthorizer::createTrigger(const String&, const String& tableName)
 {
     m_lastActionChangedDatabase = true;
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::createTempTrigger(const String& triggerName, const String& tableName)
+int DatabaseAuthorizer::createTempTrigger(const String&, const String& tableName)
 {
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::dropTrigger(const String& triggerName, const String& tableName)
+int DatabaseAuthorizer::dropTrigger(const String&, const String& tableName)
 {
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::dropTempTrigger(const String& triggerName, const String& tableName)
+int DatabaseAuthorizer::dropTempTrigger(const String&, const String& tableName)
 {
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::createVTable(const String& tableName, const String& moduleName)
+int DatabaseAuthorizer::createVTable(const String&, const String&)
 {
     m_lastActionChangedDatabase = true;
     return m_securityEnabled ? SQLAuthDeny : SQLAuthAllow;
 }
 
-int DatabaseAuthorizer::dropVTable(const String& tableName, const String& moduleName)
+int DatabaseAuthorizer::dropVTable(const String&, const String&)
 {
     return m_securityEnabled ? SQLAuthDeny : SQLAuthAllow;
 }
@@ -138,7 +138,7 @@ int DatabaseAuthorizer::allowInsert(const String& tableName)
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::allowUpdate(const String& tableName, const String& columnName)
+int DatabaseAuthorizer::allowUpdate(const String& tableName, const String&)
 {
     m_lastActionChangedDatabase = true;
     return denyBasedOnTableName(tableName);
@@ -149,7 +149,7 @@ int DatabaseAuthorizer::allowTransaction()
     return m_securityEnabled ? SQLAuthDeny : SQLAuthAllow;
 }
 
-int DatabaseAuthorizer::allowRead(const String& tableName, const String& columnName)
+int DatabaseAuthorizer::allowRead(const String& tableName, const String&)
 {
     return denyBasedOnTableName(tableName);
 }
@@ -159,22 +159,22 @@ int DatabaseAuthorizer::allowAnalyze(const String& tableName)
     return denyBasedOnTableName(tableName);
 }
 
-int DatabaseAuthorizer::allowPragma(const String& pragmaName, const String& firstArgument)
+int DatabaseAuthorizer::allowPragma(const String&, const String&)
 {
     return m_securityEnabled ? SQLAuthDeny : SQLAuthAllow;
 }
 
-int DatabaseAuthorizer::allowAttach(const String& filename)
+int DatabaseAuthorizer::allowAttach(const String&)
 {
     return m_securityEnabled ? SQLAuthDeny : SQLAuthAllow;
 }
 
-int DatabaseAuthorizer::allowDetach(const String& databaseName)
+int DatabaseAuthorizer::allowDetach(const String&)
 {
     return m_securityEnabled ? SQLAuthDeny : SQLAuthAllow;
 }
 
-int DatabaseAuthorizer::allowFunction(const String& functionName)
+int DatabaseAuthorizer::allowFunction(const String&)
 {
     // FIXME: Are there any of these we need to prevent?  One might guess current_date, current_time, current_timestamp because
     // they would violate the "sandbox environment" part of 4.11.3, but scripts can generate the local client side information via
index d02d810..685bb97 100644 (file)
@@ -59,10 +59,10 @@ public:
     int dropTrigger(const String& triggerName, const String& tableName);
     int dropTempTrigger(const String& triggerName, const String& tableName);
 
-    int createView(const String& viewName) { return SQLAuthAllow; }
-    int createTempView(const String& viewName) { return SQLAuthAllow; }
-    int dropView(const String& viewName) { return SQLAuthAllow; }
-    int dropTempView(const String& viewName) { return SQLAuthAllow; }
+    int createView(const String& /*viewName*/) { return SQLAuthAllow; }
+    int createTempView(const String& /*viewName*/) { return SQLAuthAllow; }
+    int dropView(const String& /*viewName*/) { return SQLAuthAllow; }
+    int dropTempView(const String& /*viewName*/) { return SQLAuthAllow; }
 
     int createVTable(const String& tableName, const String& moduleName);
     int dropVTable(const String& tableName, const String& moduleName);
@@ -75,7 +75,7 @@ public:
     int allowSelect() { return SQLAuthAllow; }
     int allowRead(const String& tableName, const String& columnName);
 
-    int allowReindex(const String& indexName) { return SQLAuthAllow; }
+    int allowReindex(const String& /*indexName*/) { return SQLAuthAllow; }
     int allowAnalyze(const String& tableName);
     int allowFunction(const String& functionName);
     int allowPragma(const String& pragmaName, const String& firstArgument);
index 6a1b7aa..78b25c1 100644 (file)
@@ -71,7 +71,7 @@ String StorageArea::internalGetItem(const String& key) const
     return m_storageMap->getItem(key);
 }
 
-void StorageArea::internalSetItem(const String& key, const String& value, ExceptionCode& ec, Frame* frame)
+void StorageArea::internalSetItem(const String& key, const String& value, ExceptionCode&, Frame* frame)
 {
     ASSERT(!value.isNull());
     
index 9236af4..d06fbbc 100644 (file)
@@ -96,7 +96,7 @@ void SVGAElement::svgAttributeChanged(const QualifiedName& attrName)
     }
 }
 
-RenderObject* SVGAElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* SVGAElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     if (static_cast<SVGElement*>(parent())->isTextContent())
         return new (arena) RenderSVGInline(this);
index e88e9b9..d9da3f6 100644 (file)
@@ -42,7 +42,7 @@ SVGAltGlyphElement::~SVGAltGlyphElement()
 {
 }
 
-void SVGAltGlyphElement::setGlyphRef(const AtomicString& type, ExceptionCode& ec)
+void SVGAltGlyphElement::setGlyphRef(const AtomicString&, ExceptionCode& ec)
 {
     ec = NO_MODIFICATION_ALLOWED_ERR;
 }
@@ -52,7 +52,7 @@ const AtomicString& SVGAltGlyphElement::glyphRef() const
     return getAttribute(SVGNames::glyphRefAttr);
 }
 
-void SVGAltGlyphElement::setFormat(const AtomicString& type, ExceptionCode& ec)
+void SVGAltGlyphElement::setFormat(const AtomicString&, ExceptionCode& ec)
 {
     ec = NO_MODIFICATION_ALLOWED_ERR;
 }
index 4beb520..159a64c 100644 (file)
@@ -169,7 +169,7 @@ bool SVGAnimateMotionElement::calculateFromAndByValues(const String& fromString,
     return true;
 }
 
-void SVGAnimateMotionElement::calculateAnimatedValue(float percentage, unsigned repeat, SVGSMILElement*)
+void SVGAnimateMotionElement::calculateAnimatedValue(float percentage, unsigned, SVGSMILElement*)
 {
     SVGElement* target = targetElement();
     if (!target)
index 2a39828..649b74d 100644 (file)
@@ -159,7 +159,7 @@ bool SVGAnimationElement::beginElement(ExceptionCode& ec)
     return beginElementAt(0, ec);
 }
 
-bool SVGAnimationElement::beginElementAt(float offset, ExceptionCode& ec)
+bool SVGAnimationElement::beginElementAt(float offset, ExceptionCode&)
 {
     addBeginTime(elapsed() + offset);
     return true;
@@ -170,7 +170,7 @@ bool SVGAnimationElement::endElement(ExceptionCode& ec)
     return endElementAt(0, ec);
 }
 
-bool SVGAnimationElement::endElementAt(float offset, ExceptionCode& ec)
+bool SVGAnimationElement::endElementAt(float offset, ExceptionCode&)
 {
     if (offset < 0)
         return false;
index b4d4b78..120beb4 100644 (file)
@@ -99,7 +99,7 @@ namespace WebCore {
         virtual bool calculateFromAndToValues(const String& fromString, const String& toString) = 0;
         virtual bool calculateFromAndByValues(const String& fromString, const String& byString) = 0;
         virtual void calculateAnimatedValue(float percentage, unsigned repeat, SVGSMILElement* resultElement) = 0;
-        virtual float calculateDistance(const String& fromString, const String& toString) { return -1.f; }
+        virtual float calculateDistance(const String& /*fromString*/, const String& /*toString*/) { return -1.f; }
         virtual Path animationPath() const { return Path(); }
         
         void currentValuesForValuesAnimation(float percent, float& effectivePercent, String& from, String& to) const;
index b56a80e..5939b48 100644 (file)
@@ -88,12 +88,12 @@ Color SVGColor::colorFromRGBColorString(const String& colorString)
     return Color();
 }
 
-void SVGColor::setRGBColorICCColor(const String& /* rgbColor */, const String& /* iccColor */, ExceptionCode& ec)
+void SVGColor::setRGBColorICCColor(const String& /* rgbColor */, const String& /* iccColor */, ExceptionCode&)
 {
     // TODO: implement me!
 }
 
-void SVGColor::setColor(unsigned short colorType, const String& /* rgbColor */ , const String& /* iccColor */, ExceptionCode& ec)
+void SVGColor::setColor(unsigned short colorType, const String& /* rgbColor */ , const String& /* iccColor */, ExceptionCode&)
 {
     // TODO: implement me!
     m_colorType = colorType;
index 315e494..6211d89 100644 (file)
@@ -582,7 +582,7 @@ FloatRect Font::selectionRectForTextUsingSVGFont(const TextRun& run, const IntPo
                      point.y(), floatWidthOfSubStringUsingSVGFont(this, run, 0, from, to, charsConsumed, glyphName), height);
 }
 
-int Font::offsetForPositionForTextUsingSVGFont(const TextRun&, int position, bool includePartialGlyphs) const
+int Font::offsetForPositionForTextUsingSVGFont(const TextRun&, int, bool) const
 {
     // TODO: Fix text selection when HTML text is drawn using a SVG Font
     // We need to integrate the SVG text selection code in the offsetForPosition() framework.
index f7c4582..342a6f0 100644 (file)
@@ -151,7 +151,7 @@ void SVGForeignObjectElement::svgAttributeChanged(const QualifiedName& attrName)
         renderer()->setNeedsLayout(true);
 }
 
-RenderObject* SVGForeignObjectElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* SVGForeignObjectElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     return new (arena) RenderForeignObject(this);
 }
index 035dd28..c827c89 100644 (file)
@@ -118,7 +118,7 @@ bool SVGImageElement::hasRelativeValues() const
             y().isRelative() || height().isRelative());
 }
 
-RenderObject* SVGImageElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* SVGImageElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     return new (arena) RenderSVGImage(this);
 }
index 6145f7c..a89c478 100644 (file)
@@ -183,7 +183,7 @@ SVGResource* SVGMarkerElement::canvasResource()
     return m_marker.get();
 }
 
-RenderObject* SVGMarkerElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* SVGMarkerElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     RenderSVGViewportContainer* markerContainer = new (arena) RenderSVGViewportContainer(this);
     markerContainer->setDrawsContents(false); // Marker contents will be explicitly drawn.
index 894af80..b38960b 100644 (file)
@@ -203,8 +203,7 @@ bool pointsListFromSVGData(SVGPointList* pointsList, const String& points)
      * svg path data commands and parameters. In the second mode, it will convert all
      * relative coordinates to absolute ones, and convert all curves to cubic beziers.
      */
-    class SVGPathParser
-    {
+    class SVGPathParser {
     public:
         virtual ~SVGPathParser() { }
         bool parseSVG(const String& d, bool process = false);
@@ -212,14 +211,15 @@ bool pointsListFromSVGData(SVGPointList* pointsList, const String& points)
     protected:
         virtual void svgMoveTo(double x1, double y1, bool closed, bool abs = true) = 0;
         virtual void svgLineTo(double x1, double y1, bool abs = true) = 0;
-        virtual void svgLineToHorizontal(double x, bool abs = true) {}
-        virtual void svgLineToVertical(double y, bool abs = true) {}
+        virtual void svgLineToHorizontal(double, bool /*abs*/ = true) { }
+        virtual void svgLineToVertical(double /*y*/, bool /*abs*/ = true) { }
         virtual void svgCurveToCubic(double x1, double y1, double x2, double y2, double x, double y, bool abs = true) = 0;
-        virtual void svgCurveToCubicSmooth(double x, double y, double x2, double y2, bool abs = true) {}
-        virtual void svgCurveToQuadratic(double x, double y, double x1, double y1, bool abs = true) {}
-        virtual void svgCurveToQuadraticSmooth(double x, double y, bool abs = true) {}
-        virtual void svgArcTo(double x, double y, double r1, double r2, double angle, bool largeArcFlag, bool sweepFlag, bool abs = true) {}
+        virtual void svgCurveToCubicSmooth(double /*x*/, double /*y*/, double /*x2*/, double /*y2*/, bool /*abs*/ = true) { }
+        virtual void svgCurveToQuadratic(double /*x*/, double /*y*/, double /*x1*/, double /*y1*/, bool /*abs*/ = true) { }
+        virtual void svgCurveToQuadraticSmooth(double /*x*/, double /*y*/, bool /*abs*/ = true) { }
+        virtual void svgArcTo(double /*x*/, double /*y*/, double /*r1*/, double /*r2*/, double /*angle*/, bool /*largeArcFlag*/, bool /*sweepFlag*/, bool /*abs*/ = true) { }
         virtual void svgClosePath() = 0;
+
     private:
         void calculateArc(bool relative, double& curx, double& cury, double angle, double x, double y, double r1, double r2, bool largeArcFlag, bool sweepFlag);
     };
index 92a1de2..c4ac0f2 100644 (file)
@@ -292,7 +292,7 @@ unsigned long SVGSVGElement::suspendRedraw(unsigned long /* max_wait_millisecond
     return 0;
 }
 
-void SVGSVGElement::unsuspendRedraw(unsigned long /* suspend_handle_id */, ExceptionCode& ec)
+void SVGSVGElement::unsuspendRedraw(unsigned long /* suspend_handle_id */, ExceptionCode&)
 {
     // if suspend_handle_id is not found, throw exception
     // FIXME: Implement me (see bug 11275)
@@ -308,19 +308,19 @@ void SVGSVGElement::forceRedraw()
     // FIXME: Implement me (see bug 11275)
 }
 
-NodeList* SVGSVGElement::getIntersectionList(const FloatRect& rect, SVGElement*)
+NodeList* SVGSVGElement::getIntersectionList(const FloatRect&, SVGElement*)
 {
     // FIXME: Implement me (see bug 11274)
     return 0;
 }
 
-NodeList* SVGSVGElement::getEnclosureList(const FloatRect& rect, SVGElement*)
+NodeList* SVGSVGElement::getEnclosureList(const FloatRect&, SVGElement*)
 {
     // FIXME: Implement me (see bug 11274)
     return 0;
 }
 
-bool SVGSVGElement::checkIntersection(SVGElement* element, const FloatRect& rect)
+bool SVGSVGElement::checkIntersection(SVGElement*, const FloatRect& rect)
 {
     // TODO : take into account pointer-events?
     // FIXME: Why is element ignored??
@@ -328,7 +328,7 @@ bool SVGSVGElement::checkIntersection(SVGElement* element, const FloatRect& rect
     return rect.intersects(getBBox());
 }
 
-bool SVGSVGElement::checkEnclosure(SVGElement* element, const FloatRect& rect)
+bool SVGSVGElement::checkEnclosure(SVGElement*, const FloatRect& rect)
 {
     // TODO : take into account pointer-events?
     // FIXME: Why is element ignored??
index 61819ee..c4d7ad7 100644 (file)
@@ -125,7 +125,7 @@ struct SVGInlineTextBoxQueryWalker {
     {
     }
 
-    void chunkPortionCallback(SVGInlineTextBox* textBox, int startOffset, const AffineTransform& chunkCtm,
+    void chunkPortionCallback(SVGInlineTextBox* textBox, int startOffset, const AffineTransform&,
                               const Vector<SVGChar>::iterator& start, const Vector<SVGChar>::iterator& end)
     {
         RenderStyle* style = textBox->textObject()->style();
index 1ace1a9..ee23593 100644 (file)
@@ -102,7 +102,7 @@ AffineTransform* SVGTextElement::supplementalTransform()
     return m_supplementalTransform.get();
 }
 
-RenderObject* SVGTextElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* SVGTextElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     return new (arena) RenderSVGText(this);
 }
index d9c5f13..8bf5144 100644 (file)
@@ -71,7 +71,7 @@ void SVGTextPathElement::parseMappedAttribute(MappedAttribute* attr)
     }
 }
 
-RenderObject* SVGTextPathElement::createRenderer(RenderArena* arena, RenderStyle* style)
+RenderObject* SVGTextPathElement::createRenderer(RenderArena* arena, RenderStyle*)
 {
     return new (arena) RenderSVGTextPath(this);
 }
index 4aa281a..a2e2b0e 100644 (file)
@@ -66,7 +66,7 @@ public:
             m_eventBase->removeEventListener(m_condition->m_name, this, false);
     }
 
-    virtual void handleEvent(Event* event, bool isWindowEvent
+    virtual void handleEvent(Event* event, bool) 
     {
         m_animation->handleConditionEvent(event, m_condition);
     }
@@ -876,7 +876,7 @@ void SVGSMILElement::notifyDependentsIntervalChanged(NewOrExistingInterval newOr
     loopBreaker.remove(this);
 }
     
-void SVGSMILElement::createInstanceTimesFromSyncbase(SVGSMILElement* syncbase, NewOrExistingInterval newOrExisting)
+void SVGSMILElement::createInstanceTimesFromSyncbase(SVGSMILElement* syncbase, NewOrExistingInterval)
 {
     // FIXME: To be really correct, this should handle updating exising interval by changing 
     // the associated times instead of creating new ones.
@@ -911,7 +911,7 @@ void SVGSMILElement::removeTimeDependent(SVGSMILElement* animation)
     m_timeDependents.remove(animation);
 }
     
-void SVGSMILElement::handleConditionEvent(Event* event, Condition* condition)
+void SVGSMILElement::handleConditionEvent(Event*, Condition* condition)
 {
     if (condition->m_beginOrEnd == Begin)
         addBeginTime(elapsed() + condition->m_offset);
index 5bc7018..45d58ef 100644 (file)
@@ -59,18 +59,18 @@ public:
 
     SVGImage* image() const { return m_image; }
     
+private:
     virtual void chromeDestroyed()
     {
         m_image = 0;
     }
 
-    virtual void repaint(const IntRect& r, bool contentChanged, bool immediate, bool repaintContentOnly)
+    virtual void repaint(const IntRect& r, bool, bool, bool)
     {
         if (m_image && m_image->imageObserver())
             m_image->imageObserver()->changedInRect(m_image, r);
     }
 
-private:
     SVGImage* m_image;
 };
 
index b604353..062c0a2 100644 (file)
@@ -47,7 +47,9 @@ namespace WebCore {
         {
             return adoptRef(new SVGImage(observer));
         }
-        ~SVGImage();
+
+    private:
+        virtual ~SVGImage();
 
         virtual void setContainerSize(const IntSize&);
         virtual bool usesContainerSize() const;
@@ -60,12 +62,11 @@ namespace WebCore {
 
         // FIXME: SVGImages are underreporting decoded sizes and will be unable
         // to prune because these functions are not implemented yet.
-        virtual void destroyDecodedData(bool destroyAll = true) { }
+        virtual void destroyDecodedData(bool) { }
         virtual unsigned decodedSize() const { return 0; }
 
         virtual NativeImagePtr frameAtIndex(size_t) { return 0; }
         
-private:
         SVGImage(ImageObserver*);
         virtual void draw(GraphicsContext*, const FloatRect& fromRect, const FloatRect& toRect, CompositeOperator);
         
index 13a5967..485d2ea 100644 (file)
@@ -268,7 +268,7 @@ void SVGPaintServerGradient::renderPath(GraphicsContext*& context, const RenderO
         context->strokePath();
 }
 
-void SVGPaintServerGradient::teardown(GraphicsContext*& context, const RenderObject* object, SVGPaintTargetType type, bool isPaintingText) const
+void SVGPaintServerGradient::teardown(GraphicsContext*& context, const RenderObject* object, SVGPaintTargetType, bool isPaintingText) const
 {
 #if PLATFORM(CG)
     // renderPath() is not used when painting text, so we paint the gradient during teardown()
index 27ccb95..bb048a5 100644 (file)
@@ -38,7 +38,7 @@ void SVGPaintServer::draw(GraphicsContext*& context, const RenderObject* path, S
     teardown(context, path, type);
 }
 
-void SVGPaintServer::teardown(GraphicsContext*&, const RenderObject*, SVGPaintTargetType, bool isPaintingText) const
+void SVGPaintServer::teardown(GraphicsContext*&, const RenderObject*, SVGPaintTargetType, bool) const
 {
     // no-op
 }
index e575d6b..ca7fd12 100644 (file)
@@ -1154,7 +1154,7 @@ void XMLHttpRequest::processSyncLoadResults(const Vector<char>& data, const Reso
         ec = XMLHttpRequestException::NETWORK_ERR;
 }
 
-void XMLHttpRequest::didFail(SubresourceLoader* loader, const ResourceError& error)
+void XMLHttpRequest::didFail(SubresourceLoader*, const ResourceError& error)
 {
     // If we are already in an error state, for instance we called abort(), bail out early.
     if (m_error)
@@ -1204,7 +1204,7 @@ void XMLHttpRequest::didFinishLoading(SubresourceLoader* loader)
         dropProtection();
 }
 
-void XMLHttpRequest::didFinishLoadingPreflight(SubresourceLoader* loader)
+void XMLHttpRequest::didFinishLoadingPreflight(SubresourceLoader*)
 {
     ASSERT(m_inPreflight);
     ASSERT(!m_sameOriginRequest);
@@ -1217,7 +1217,7 @@ void XMLHttpRequest::didFinishLoadingPreflight(SubresourceLoader* loader)
         unsetPendingActivity(this);
 }
 
-void XMLHttpRequest::willSendRequest(SubresourceLoader*, ResourceRequest& request, const ResourceResponse& redirectResponse)
+void XMLHttpRequest::willSendRequest(SubresourceLoader*, ResourceRequest& request, const ResourceResponse&)
 {
     // FIXME: This needs to be fixed to follow the redirect correctly even for cross-domain requests.
     if (!scriptExecutionContext()->securityOrigin()->canRequest(request.url())) {
index 50a69c2..7b21038 100644 (file)
@@ -69,7 +69,7 @@ using namespace XPath;
 %{
 
 int xpathyylex(YYSTYPE* yylval) { return Parser::current()->lex(yylval); }
-void xpathyyerror(const char* str) { }
+void xpathyyerror(const char*) { }
     
 %}
 
index ed5d155..0d112a5 100644 (file)
@@ -133,7 +133,7 @@ DocLoader* XSLStyleSheet::docLoader()
     return m_ownerDocument->docLoader();
 }
 
-bool XSLStyleSheet::parseString(const String& string, bool strict)
+bool XSLStyleSheet::parseString(const String& string, bool)
 {
     // Parse in a single chunk into an xmlDocPtr
     const UChar BOM = 0xFEFF;
index 1949fd3..b644486 100644 (file)
@@ -74,7 +74,7 @@ SOFT_LINK(libxslt, xsltNextImport, xsltStylesheetPtr, (xsltStylesheetPtr style),
 
 namespace WebCore {
 
-void XSLTProcessor::genericErrorFunc(void* userData, const char* msg, ...)
+void XSLTProcessor::genericErrorFunc(void*, const char*, ...)
 {
     // It would be nice to do something with this error message.
 }
@@ -107,7 +107,7 @@ void XSLTProcessor::parseErrorFunc(void* userData, xmlError* error)
 static XSLTProcessor* globalProcessor = 0;
 static DocLoader* globalDocLoader = 0;
 static xmlDocPtr docLoaderFunc(const xmlChar* uri,
-                                    xmlDictPtr dict,
+                                    xmlDictPtr,
                                     int options,
                                     void* ctxt,
                                     xsltLoadType type)
@@ -436,21 +436,21 @@ PassRefPtr<DocumentFragment> XSLTProcessor::transformToFragment(Node* sourceNode
     return createFragmentFromSource(resultString, resultMIMEType, sourceNode, outputDoc);
 }
 
-void XSLTProcessor::setParameter(const String& namespaceURI, const String& localName, const String& value)
+void XSLTProcessor::setParameter(const String& /*namespaceURI*/, const String& localName, const String& value)
 {
     // FIXME: namespace support?
     // should make a QualifiedName here but we'd have to expose the impl
     m_parameters.set(localName, value);
 }
 
-String XSLTProcessor::getParameter(const String& namespaceURI, const String& localName) const
+String XSLTProcessor::getParameter(const String& /*namespaceURI*/, const String& localName) const
 {
     // FIXME: namespace support?
     // should make a QualifiedName here but we'd have to expose the impl
     return m_parameters.get(localName);
 }
 
-void XSLTProcessor::removeParameter(const String& namespaceURI, const String& localName)
+void XSLTProcessor::removeParameter(const String& /*namespaceURI*/, const String& localName)
 {
     // FIXME: namespace support?
     m_parameters.remove(localName);