Fix some -Wdelete-non-virtual-dtor warnings on Windows.

Like https://chromium-review.googlesource.com/c/chromium/src/+/639732 but in a Windows build instead of in a Linux build. TBR=siggi Bug: 673171 Change-Id: I0966683235d7dcf1fc0ff8119a4f1b5b07be31e7 Reviewed-on: https://chromium-review.googlesource.com/646648 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by: Robert Liao <robliao@chromium.org> Reviewed-by: Hans Wennborg <hans@chromium.org> Cr-Commit-Position: refs/heads/master@{#498970}
Denger-Network · Aug 31, 2017 · 76fdda1 · 76fdda1
1 parent d791c03
commit 76fdda1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/components/browser_watcher/postmortem_report_collector_unittest.cc b/components/browser_watcher/postmortem_report_collector_unittest.cc
@@ -101,7 +101,7 @@ class MockCrashReportDatabase : public CrashReportDatabase {
                CrashReportDatabase::OperationStatus(const UUID& uuid));
 };
 
-class MockPostmortemReportCollector : public PostmortemReportCollector {
+class MockPostmortemReportCollector final : public PostmortemReportCollector {
  public:
   explicit MockPostmortemReportCollector(CrashReportDatabase* crash_database)
       : PostmortemReportCollector(kProductName,

diff --git a/ui/display/win/screen_win_unittest.cc b/ui/display/win/screen_win_unittest.cc
@@ -143,7 +143,7 @@ class TestScreenWinInitializer {
   virtual HWND CreateFakeHwnd(const gfx::Rect& bounds) = 0;
 };
 
-class TestScreenWinManager : public TestScreenWinInitializer {
+class TestScreenWinManager final : public TestScreenWinInitializer {
  public:
   TestScreenWinManager() = default;