Changeset 53401 in webkit


Ignore:
Timestamp:
Jan 18, 2010 4:54:08 AM (14 years ago)
Author:
benm@google.com
Message:

[Android] DOMWrapperMap::Visitor needs virtual destructor
​https://bugs.webkit.org/show_bug.cgi?id=33675

Patch by Andrei Popescu <​andreip@google.com> on 2010-01-18
Reviewed by Adam Barth.

Add protected virtual dtor to DOMWrapperMap::Visitor. See bug for further discussion.

No new tests needed, functionality not changed.

  • bindings/v8/V8DOMMap.h:

(WebCore::DOMWrapperMap::Visitor::~Visitor): Added.

Location:
trunk/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/WebCore/ChangeLog

    r53394 r53401  
     12010-01-18  Andrei Popescu  <andreip@google.com>
     2
     3        Reviewed by Adam Barth.
     4
     5        [Android] DOMWrapperMap::Visitor needs virtual destructor
     6        https://bugs.webkit.org/show_bug.cgi?id=33675
     7
     8        Add protected virtual dtor to DOMWrapperMap::Visitor. See bug for further discussion.
     9
     10        No new tests needed, functionality not changed.
     11
     12        * bindings/v8/V8DOMMap.h:
     13        (WebCore::DOMWrapperMap::Visitor::~Visitor): Added.
     14
    1152010-01-18  Simon Hausmann  <simon.hausmann@nokia.com>
    216

  • trunk/WebCore/bindings/v8/V8DOMMap.h

    r51998 r53401  
    103103        public:
    104104            virtual void visitDOMWrapper(KeyType*, v8::Persistent<ValueType>) = 0;
     105        protected:
     106            virtual ~Visitor() { }
    105107        };
    106108
Note: See TracChangeset for help on using the changeset viewer.