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

Unified Diff: content/common/font_cache_dispatcher_win.cc

Issue 1120203004: win: Ref-counted classes should have non-public destructors. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/font_cache_dispatcher_win.cc
diff --git a/content/common/font_cache_dispatcher_win.cc b/content/common/font_cache_dispatcher_win.cc
index 69aa7ae73a6e8ffb4e41b21424420b94c46c8f1b..880f0f473761a94ef9d7885dccd568aee111968d 100644
--- a/content/common/font_cache_dispatcher_win.cc
+++ b/content/common/font_cache_dispatcher_win.cc
@@ -143,6 +143,14 @@ FontCacheDispatcher::FontCacheDispatcher()
: sender_(NULL) {
}
+bool FontCacheDispatcher::Send(IPC::Message* message) {
+ if (sender_)
+ return sender_->Send(message);
+
+ delete message;
+ return false;
+}
+
FontCacheDispatcher::~FontCacheDispatcher() {
}
@@ -165,14 +173,6 @@ void FontCacheDispatcher::OnChannelClosing() {
sender_ = NULL;
}
-bool FontCacheDispatcher::Send(IPC::Message* message) {
- if (sender_)
- return sender_->Send(message);
-
- delete message;
- return false;
-}
-
void FontCacheDispatcher::OnPreCacheFont(const LOGFONT& font) {
// If a child process is running in a sandbox, GetTextMetrics()
// can sometimes fail. If a font has not been loaded

Powered by Google App Engine
This is Rietveld 408576698