Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(289)

Issue 406683005: Fix remaining -Wdelete-non-virtual-dtor violations on Windows. (Closed)

Created:
6 years, 5 months ago by Nico
Modified:
6 years, 5 months ago
Reviewers:
Bo Xu
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Visibility:
Public.

Description

Fix remaining -Wdelete-non-virtual-dtor violations on Windows. Follow-up to https://codereview.chromium.org/370853002/ BUG=none R=bo_xu@foxitsoftware.com Committed: https://pdfium.googlesource.com/pdfium/+/baca770

Patch Set 1 #

Patch Set 2 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M core/src/fxge/win32/fx_win32_device.cpp View 1 chunk +1 line, -1 line 0 comments Download
M core/src/fxge/win32/fx_win32_dwrite.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M core/src/fxge/win32/fx_win32_gdipext.cpp View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Nico
6 years, 5 months ago (2014-07-18 23:15:05 UTC) #1
Nico
Bo: ping
6 years, 5 months ago (2014-07-21 16:46:36 UTC) #2
Bo Xu
On 2014/07/21 16:46:36, Nico (away) wrote: > Bo: ping LGTM
6 years, 5 months ago (2014-07-21 16:48:37 UTC) #3
Nico
6 years, 5 months ago (2014-07-21 16:51:00 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 manually as rbaca770 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698