win: Ref-counted classes should have non-public destructors.

win8/viewer/metro_viewer_process_host.h

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef WIN8_VIEWER_METRO_VIEWER_PROCESS_HOST_H_
 #define WIN8_VIEWER_METRO_VIEWER_PROCESS_HOST_H_
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
@@ skipping 173 matching lines @@
   // MetroViewerProcessHost from having to pump messages on their own message
   // loop.
   class InternalMessageFilter : public IPC::MessageFilter {
    public:
     InternalMessageFilter(MetroViewerProcessHost* owner);
 
     // IPC::MessageFilter implementation.
     void OnChannelConnected(int32 peer_pid) override;
 
    private:
+    ~InternalMessageFilter();

[Ryan Sleevi, 2015/05/02 01:28:26]

no override?

+
     MetroViewerProcessHost* owner_;
     DISALLOW_COPY_AND_ASSIGN(InternalMessageFilter);
   };
 
   scoped_ptr<IPC::ChannelProxy> channel_;
   scoped_ptr<base::WaitableEvent> channel_connected_event_;
   scoped_refptr<InternalMessageFilter> message_filter_;
 
   static MetroViewerProcessHost* instance_;
 
   // Saved callbacks which inform the caller about the result of the open file/
   // save file/select operations.
   OpenFileCompletion file_open_completion_callback_;
   OpenMultipleFilesCompletion multi_file_open_completion_callback_;
   SaveFileCompletion file_saveas_completion_callback_;
   SelectFolderCompletion select_folder_completion_callback_;
   FileSelectionCanceled failure_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(MetroViewerProcessHost);
 };
 
 }  // namespace win8
 
 #endif  // WIN8_VIEWER_METRO_VIEWER_PROCESS_HOST_H_