| OLD | NEW |
| 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "overridden_methods.h" | 5 #include "overridden_methods.h" |
| 6 | 6 |
| 7 // Fill in the implementations | 7 // Fill in the implementations |
| 8 void DerivedClass::SomeMethod() {} | 8 void DerivedClass::SomeMethod() {} |
| 9 void DerivedClass::SomeOtherMethod() {} | 9 void DerivedClass::SomeOtherMethod() {} |
| 10 void DerivedClass::WebKitModifiedSomething() {} | 10 void DerivedClass::WebKitModifiedSomething() {} |
| 11 | 11 |
| 12 class ImplementationInterimClass : public BaseClass { |
| 13 public: |
| 14 // Should not warn about pure virtual methods. |
| 15 virtual void SomeMethod() = 0; |
| 16 }; |
| 17 |
| 18 class ImplementationDerivedClass : public ImplementationInterimClass, |
| 19 public webkit_glue::WebKitObserverImpl { |
| 20 public: |
| 21 // Should not warn about destructors. |
| 22 virtual ~ImplementationDerivedClass() {} |
| 23 // Should warn. |
| 24 virtual void SomeMethod(); |
| 25 // Should not warn if marked as override. |
| 26 virtual void SomeOtherMethod() override; |
| 27 // Should not warn for inline implementations in implementation files. |
| 28 virtual void SomeInlineMethod() {} |
| 29 // Should not warn if overriding a method whose origin is WebKit. |
| 30 virtual void WebKitModifiedSomething(); |
| 31 // Should warn if overridden method isn't pure. |
| 32 virtual void SomeNonPureBaseMethod() {} |
| 33 }; |
| 34 |
| 12 int main() { | 35 int main() { |
| 13 DerivedClass something; | 36 DerivedClass something; |
| 37 ImplementationDerivedClass something_else; |
| 14 } | 38 } |
| OLD | NEW |