diff options
author | thakis <thakis@chromium.org> | 2015-05-01 18:28:44 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-05-02 01:29:23 +0000 |
commit | efa363a5b868a17049852fa9cd86679120041853 (patch) | |
tree | d51b9fcc4597a6b7536fa5d721b75fcf0625da39 /content/common/font_cache_dispatcher_win.cc | |
parent | 2045be86fa3510f263a5780903dacb96859ec890 (diff) | |
download | chromium_src-efa363a5b868a17049852fa9cd86679120041853.zip chromium_src-efa363a5b868a17049852fa9cd86679120041853.tar.gz chromium_src-efa363a5b868a17049852fa9cd86679120041853.tar.bz2 |
win: Ref-counted classes should have non-public destructors.
BUG=123295,467287
TBR=avi,rsleevi,kbr,grt
Review URL: https://codereview.chromium.org/1120203004
Cr-Commit-Position: refs/heads/master@{#328040}
Diffstat (limited to 'content/common/font_cache_dispatcher_win.cc')
-rw-r--r-- | content/common/font_cache_dispatcher_win.cc | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/content/common/font_cache_dispatcher_win.cc b/content/common/font_cache_dispatcher_win.cc index 69aa7ae..880f0f4 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 |