Added missing override specifiers under Source/WebCore.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 Mar 2016 23:28:22 +0000 (23:28 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 Mar 2016 23:28:22 +0000 (23:28 +0000)
https://bugs.webkit.org/show_bug.cgi?id=155021

Patch by Konstantin Tokarev <annulen@yandex.ru> on 2016-03-04
Reviewed by Michael Catanzaro.

No new tests needed.

* bridge/NP_jsobject.cpp:
* css/CSSStyleSheet.cpp:
* dom/MutationObserver.cpp:
* editing/DictationCommand.cpp:
* editing/EditingStyle.cpp:
* loader/EmptyClients.cpp:
* loader/SinkDocument.cpp:
* page/DOMWindow.cpp:
* page/animation/CSSPropertyAnimation.cpp:
* platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
Also added #if(SOUP) guard to
ResourceHandleStreamingClient::getOrCreateReadBuffer
because it is not an overridden method otherwise.
* platform/text/LineEnding.cpp:
* xml/parser/XMLDocumentParserLibxml2.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@197591 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 files changed:
Source/WebCore/ChangeLog
Source/WebCore/bridge/NP_jsobject.cpp
Source/WebCore/css/CSSStyleSheet.cpp
Source/WebCore/dom/MutationObserver.cpp
Source/WebCore/editing/DictationCommand.cpp
Source/WebCore/editing/EditingStyle.cpp
Source/WebCore/loader/EmptyClients.cpp
Source/WebCore/loader/SinkDocument.cpp
Source/WebCore/page/DOMWindow.cpp
Source/WebCore/page/animation/CSSPropertyAnimation.cpp
Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp
Source/WebCore/platform/text/LineEnding.cpp
Source/WebCore/xml/parser/XMLDocumentParserLibxml2.cpp

index 01a339e..dbd541d 100644 (file)
@@ -1,3 +1,28 @@
+2016-03-04  Konstantin Tokarev  <annulen@yandex.ru>
+
+        Added missing override specifiers under Source/WebCore.
+        https://bugs.webkit.org/show_bug.cgi?id=155021
+
+        Reviewed by Michael Catanzaro.
+
+        No new tests needed.
+
+        * bridge/NP_jsobject.cpp:
+        * css/CSSStyleSheet.cpp:
+        * dom/MutationObserver.cpp:
+        * editing/DictationCommand.cpp:
+        * editing/EditingStyle.cpp:
+        * loader/EmptyClients.cpp:
+        * loader/SinkDocument.cpp:
+        * page/DOMWindow.cpp:
+        * page/animation/CSSPropertyAnimation.cpp:
+        * platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
+        Also added #if(SOUP) guard to
+        ResourceHandleStreamingClient::getOrCreateReadBuffer
+        because it is not an overridden method otherwise.
+        * platform/text/LineEnding.cpp:
+        * xml/parser/XMLDocumentParserLibxml2.cpp:
+
 2016-03-04  Gavin Barraclough  <barraclough@apple.com>
 
         Max out timer throttling immediately for pre-render pages
index 40ebb80..a3985d7 100644 (file)
@@ -88,7 +88,7 @@ public:
 
 private:
     struct RootObjectInvalidationCallback : public RootObject::InvalidationCallback {
-        virtual void operator()(RootObject*);
+        void operator()(RootObject*) override;
     };
     RootObjectInvalidationCallback m_invalidationCallback;
 
index 1fc8939..545fa52 100644 (file)
@@ -52,13 +52,13 @@ public:
     StyleSheetCSSRuleList(CSSStyleSheet* sheet) : m_styleSheet(sheet) { }
     
 private:
-    virtual void ref() { m_styleSheet->ref(); }
-    virtual void deref() { m_styleSheet->deref(); }
-    
-    virtual unsigned length() const { return m_styleSheet->length(); }
-    virtual CSSRule* item(unsigned index) const { return m_styleSheet->item(index); }
-    
-    virtual CSSStyleSheet* styleSheet() const { return m_styleSheet; }
+    void ref() override { m_styleSheet->ref(); }
+    void deref() override { m_styleSheet->deref(); }
+
+    unsigned length() const override { return m_styleSheet->length(); }
+    CSSRule* item(unsigned index) const override { return m_styleSheet->item(index); }
+
+    CSSStyleSheet* styleSheet() const override { return m_styleSheet; }
 
     CSSStyleSheet* m_styleSheet;
 };
index 87fbbbd..b27a4c1 100644 (file)
@@ -167,8 +167,8 @@ public:
     {
     }
 
-private:    
-    virtual Result run()
+private:
+    Result run() override
     {
         mutationObserverCompoundMicrotaskQueuedFlag = false;
 
index bc5b6a5..99295c6 100644 (file)
@@ -60,7 +60,7 @@ public:
         return adoptRef(*new DictationMarkerSupplier(alternatives));
     }
 
-    virtual void addMarkersToTextNode(Text* textNode, unsigned offsetOfInsertion, const String& textToBeInserted)
+    void addMarkersToTextNode(Text* textNode, unsigned offsetOfInsertion, const String& textToBeInserted) override
     {
         DocumentMarkerController& markerController = textNode->document().markers();
         for (auto& alternative : m_alternatives) {
index 4c79103..2314de5 100644 (file)
@@ -251,10 +251,10 @@ public:
     HTMLAttributeEquivalent(CSSPropertyID, const QualifiedName& tagName, const QualifiedName& attrName);
     HTMLAttributeEquivalent(CSSPropertyID, const QualifiedName& attrName);
 
-    bool matches(const Element& element) const { return HTMLElementEquivalent::matches(element) && element.hasAttribute(m_attrName); }
-    virtual bool hasAttribute() const { return true; }
-    virtual bool valueIsPresentInStyle(Element&, const EditingStyle&) const;
-    virtual void addToStyle(Element*, EditingStyle*) const;
+    bool matches(const Element& element) const override { return HTMLElementEquivalent::matches(element) && element.hasAttribute(m_attrName); }
+    bool hasAttribute() const override { return true; }
+    bool valueIsPresentInStyle(Element&, const EditingStyle&) const override;
+    void addToStyle(Element*, EditingStyle*) const override;
     virtual PassRefPtr<CSSValue> attributeValueAsCSSValue(Element*) const;
     inline const QualifiedName& attributeName() const { return m_attrName; }
 
@@ -305,7 +305,7 @@ class HTMLFontSizeEquivalent : public HTMLAttributeEquivalent {
 public:
     HTMLFontSizeEquivalent();
 
-    virtual PassRefPtr<CSSValue> attributeValueAsCSSValue(Element*) const;
+    PassRefPtr<CSSValue> attributeValueAsCSSValue(Element*) const override;
 };
 
 HTMLFontSizeEquivalent::HTMLFontSizeEquivalent()
index 6cc4d26..04e8cc6 100644 (file)
@@ -141,18 +141,18 @@ void fillWithEmptyClients(PageConfiguration& pageConfiguration)
 
 class EmptyPopupMenu : public PopupMenu {
 public:
-    virtual void show(const IntRect&, FrameView*, int) { }
-    virtual void hide() { }
-    virtual void updateFromElement() { }
-    virtual void disconnectClient() { }
+    void show(const IntRect&, FrameView*, int) override { }
+    void hide() override { }
+    void updateFromElement() override { }
+    void disconnectClient() override { }
 };
 
 class EmptySearchPopupMenu : public SearchPopupMenu {
 public:
-    virtual PopupMenu* popupMenu() { return m_popup.get(); }
-    virtual void saveRecentSearches(const AtomicString&, const Vector<RecentSearch>&) { }
-    virtual void loadRecentSearches(const AtomicString&, Vector<RecentSearch>&) { }
-    virtual bool enabled() { return false; }
+    PopupMenu* popupMenu() override { return m_popup.get(); }
+    void saveRecentSearches(const AtomicString&, const Vector<RecentSearch>&) override { }
+    void loadRecentSearches(const AtomicString&, Vector<RecentSearch>&) override { }
+    bool enabled() override { return false; }
 
 private:
     RefPtr<EmptyPopupMenu> m_popup;
index a530fb0..a1bf52a 100644 (file)
@@ -44,7 +44,7 @@ private:
     }
 
     // Ignore all data.
-    virtual void appendBytes(DocumentWriter&, const char*, size_t)
+    void appendBytes(DocumentWriter&, const char*, size_t) override
     {
     }
 };
index d91420c..d177bc7 100644 (file)
@@ -165,7 +165,7 @@ public:
     ScriptCallStack* stackTrace() const { return m_stackTrace.get(); }
 
 private:
-    virtual void fired()
+    void fired() override
     {
         // This object gets deleted when std::unique_ptr falls out of scope..
         std::unique_ptr<PostMessageTimer> timer(this);
index 455c750..84b7c64 100644 (file)
@@ -401,7 +401,7 @@ public:
     {
     }
 
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
         if (a == b)
             return true;
@@ -479,7 +479,7 @@ public:
     {
     }
 
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
         // If the style pointers are the same, don't bother doing the test.
         // If either is null, return false. If both are null, return true.
@@ -507,7 +507,7 @@ public:
     {
     }
 
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
         // If the style pointers are the same, don't bother doing the test.
         // If either is null, return false. If both are null, return true.
@@ -535,7 +535,7 @@ public:
     {
     }
 
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
        if (a == b)
            return true;
@@ -557,7 +557,7 @@ public:
     {
     }
 
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         (dst->*m_setter)(blendFunc(anim, (a->*PropertyWrapperGetter<Color>::m_getter)(), (b->*PropertyWrapperGetter<Color>::m_getter)(), progress));
     }
@@ -575,9 +575,9 @@ public:
     {
     }
 
-    virtual bool animationIsAccelerated() const { return true; }
+    bool animationIsAccelerated() const override { return true; }
 
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         float fromOpacity = a->opacity();
 
@@ -594,9 +594,9 @@ public:
     {
     }
 
-    virtual bool animationIsAccelerated() const { return true; }
+    bool animationIsAccelerated() const override { return true; }
 
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         dst->setTransform(blendFunc(anim, a->transform(), b->transform(), progress));
     }
@@ -610,9 +610,9 @@ public:
     {
     }
 
-    virtual bool animationIsAccelerated() const { return true; }
+    bool animationIsAccelerated() const override { return true; }
 
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         dst->setFilter(blendFunc(anim, a->filter(), b->filter(), progress));
     }
@@ -670,7 +670,7 @@ public:
     {
     }
 
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
         if (a == b)
             return true;
@@ -699,7 +699,7 @@ public:
         return true;
     }
 
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         const ShadowData* shadowA = (a->*m_getter)();
         const ShadowData* shadowB = (b->*m_getter)();
@@ -791,7 +791,7 @@ public:
     {
     }
 
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
         if (a == b)
             return true;
@@ -812,7 +812,7 @@ public:
         return fromColor == toColor;
     }
 
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         Color fromColor = (a->*m_getter)();
         Color toColor = (b->*m_getter)();
@@ -851,11 +851,11 @@ public:
         , m_visitedWrapper(std::make_unique<PropertyWrapperMaybeInvalidColor>(prop, visitedGetter, visitedSetter))
     {
     }
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
         return m_wrapper->equals(a, b) && m_visitedWrapper->equals(a, b);
     }
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         m_wrapper->blend(anim, dst, a, b, progress);
         m_visitedWrapper->blend(anim, dst, a, b, progress);
@@ -890,7 +890,7 @@ public:
     {
     }
 
-    virtual bool equals(const FillLayer* a, const FillLayer* b) const
+    bool equals(const FillLayer* a, const FillLayer* b) const override
     {
         if (a == b)
             return true;
@@ -949,7 +949,7 @@ public:
     {
     }
 
-    virtual bool equals(const FillLayer* a, const FillLayer* b) const
+    bool equals(const FillLayer* a, const FillLayer* b) const override
     {
        if (a == b)
            return true;
@@ -995,7 +995,7 @@ public:
         }
     }
 
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
         if (a == b)
             return true;
@@ -1016,7 +1016,7 @@ public:
         return true;
     }
 
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         const FillLayer* aLayer = (a->*m_layersGetter)();
         const FillLayer* bLayer = (b->*m_layersGetter)();
@@ -1046,9 +1046,9 @@ public:
     {
     }
 
-    virtual bool isShorthandWrapper() const { return true; }
+    bool isShorthandWrapper() const override { return true; }
 
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
         if (a == b)
             return true;
@@ -1062,7 +1062,7 @@ public:
         return true;
     }
 
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         for (auto& wrapper : m_propertyWrappers)
             wrapper->blend(anim, dst, a, b, progress);
@@ -1082,7 +1082,7 @@ public:
     {
     }
 
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
         if (a == b)
             return true;
@@ -1092,7 +1092,7 @@ public:
         return a->flexBasis() == b->flexBasis() && a->flexGrow() == b->flexGrow() && a->flexShrink() == b->flexShrink();
     }
 
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         dst->setFlexBasis(blendFunc(anim, a->flexBasis(), b->flexBasis(), progress));
         dst->setFlexGrow(blendFunc(anim, a->flexGrow(), b->flexGrow(), progress));
@@ -1111,7 +1111,7 @@ public:
     {
     }
 
-    virtual bool equals(const RenderStyle* a, const RenderStyle* b) const
+    bool equals(const RenderStyle* a, const RenderStyle* b) const override
     {
         if (a == b)
             return true;
@@ -1141,7 +1141,7 @@ public:
         return true;
     }
 
-    virtual void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const
+    void blend(const AnimationBase* anim, RenderStyle* dst, const RenderStyle* a, const RenderStyle* b, double progress) const override
     {
         if ((a->*m_paintTypeGetter)() != SVGPaint::SVG_PAINTTYPE_RGBCOLOR
             || (b->*m_paintTypeGetter)() != SVGPaint::SVG_PAINTTYPE_RGBCOLOR)
index 3c55474..6cd0323 100644 (file)
@@ -92,15 +92,17 @@ class ResourceHandleStreamingClient : public ResourceHandleClient, public Stream
 
     private:
         // ResourceHandleClient virtual methods.
-        virtual char* getOrCreateReadBuffer(size_t requestedSize, size_t& actualSize);
-        virtual void willSendRequest(ResourceHandle*, ResourceRequest&, const ResourceResponse&);
-        virtual void didReceiveResponse(ResourceHandle*, const ResourceResponse&);
-        virtual void didReceiveData(ResourceHandle*, const char*, unsigned, int);
-        virtual void didReceiveBuffer(ResourceHandle*, PassRefPtr<SharedBuffer>, int encodedLength);
-        virtual void didFinishLoading(ResourceHandle*, double /*finishTime*/);
-        virtual void didFail(ResourceHandle*, const ResourceError&);
-        virtual void wasBlocked(ResourceHandle*);
-        virtual void cannotShowURL(ResourceHandle*);
+#if USE(SOUP)
+        char* getOrCreateReadBuffer(size_t requestedSize, size_t& actualSize) override;
+#endif
+        void willSendRequest(ResourceHandle*, ResourceRequest&, const ResourceResponse&) override;
+        void didReceiveResponse(ResourceHandle*, const ResourceResponse&) override;
+        void didReceiveData(ResourceHandle*, const char*, unsigned, int) override;
+        void didReceiveBuffer(ResourceHandle*, PassRefPtr<SharedBuffer>, int encodedLength) override;
+        void didFinishLoading(ResourceHandle*, double /*finishTime*/) override;
+        void didFail(ResourceHandle*, const ResourceError&) override;
+        void wasBlocked(ResourceHandle*) override;
+        void cannotShowURL(ResourceHandle*) override;
 
         RefPtr<ResourceHandle> m_resource;
 };
index ae24fb0..c0b0765 100644 (file)
@@ -52,14 +52,14 @@ public:
     }
     virtual ~CStringBuffer() { }
 
-    virtual char* allocate(size_t size)
+    char* allocate(size_t size) override
     {
         char* ptr;
         m_buffer = CString::newUninitialized(size, ptr);
         return ptr;
     }
 
-    virtual void copy(const CString& source)
+    void copy(const CString& source) override
     {
         m_buffer = source;
     }
@@ -78,14 +78,14 @@ public:
     }
     virtual ~VectorCharAppendBuffer() { }
 
-    virtual char* allocate(size_t size)
+    char* allocate(size_t size) override
     {
         size_t oldSize = m_buffer.size();
         m_buffer.grow(oldSize + size);
         return m_buffer.data() + oldSize;
     }
 
-    virtual void copy(const CString& source)
+    void copy(const CString& source) override
     {
         m_buffer.append(source.data(), source.length());
     }
index 0e77eac..1462945 100644 (file)
@@ -227,7 +227,7 @@ private:
             xmlFree(attributes);
         }
 
-        virtual void call(XMLDocumentParser* parser)
+        void call(XMLDocumentParser* parser) override
         {
             parser->startElementNs(xmlLocalName, xmlPrefix, xmlURI,
                                       nb_namespaces, const_cast<const xmlChar**>(namespaces),
@@ -245,7 +245,7 @@ private:
     };
 
     struct PendingEndElementNSCallback : public PendingCallback {
-        virtual void call(XMLDocumentParser* parser)
+        void call(XMLDocumentParser* parser) override
         {
             parser->endElementNs();
         }
@@ -257,7 +257,7 @@ private:
             xmlFree(s);
         }
 
-        virtual void call(XMLDocumentParser* parser)
+        void call(XMLDocumentParser* parser) override
         {
             parser->characters(s, len);
         }
@@ -273,7 +273,7 @@ private:
             xmlFree(data);
         }
 
-        virtual void call(XMLDocumentParser* parser)
+        void call(XMLDocumentParser* parser) override
         {
             parser->processingInstruction(target, data);
         }
@@ -288,7 +288,7 @@ private:
             xmlFree(s);
         }
 
-        virtual void call(XMLDocumentParser* parser)
+        void call(XMLDocumentParser* parser) override
         {
             parser->cdataBlock(s, len);
         }
@@ -303,7 +303,7 @@ private:
             xmlFree(s);
         }
 
-        virtual void call(XMLDocumentParser* parser)
+        void call(XMLDocumentParser* parser) override
         {
             parser->comment(s);
         }
@@ -319,7 +319,7 @@ private:
             xmlFree(systemID);
         }
 
-        virtual void call(XMLDocumentParser* parser)
+        void call(XMLDocumentParser* parser) override
         {
             parser->internalSubset(name, externalID, systemID);
         }
@@ -335,7 +335,7 @@ private:
             xmlFree(message);
         }
 
-        virtual void call(XMLDocumentParser* parser)
+        void call(XMLDocumentParser* parser) override
         {
             parser->handleError(type, reinterpret_cast<char*>(message), TextPosition(lineNumber, columnNumber));
         }