snapshot: Make destructors of interface classes public and virtual

snapshot/module_snapshot.h

 // Copyright 2014 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 12 matching lines @@
 #include <vector>
 
 #include "util/misc/uuid.h"
 
 namespace crashpad {
 
 //! \brief An abstract interface to a snapshot representing a code module
 //!     (binary image) loaded into a snapshot process.
 class ModuleSnapshot {
  public:
+  virtual ~ModuleSnapshot() {}
+
   //! \brief A module’s type.
   enum ModuleType {
     //! \brief The module’s type is unknown.
     kModuleTypeUnknown = 0,
 
     //! \brief The module is a main executable.
     kModuleTypeExecutable,
 
     //! \brief The module is a shared library.
     //!
@@ skipping 95 matching lines @@
   //! keys and values are strings. These are referred to in Chrome as “crash
   //! keys.”
   //!
   //! For Mac OS X snapshots, these annotations are found by interpreting the
   //! `__DATA, __crashpad_info` section as `CrashpadInfo`. Clients can use the
   //! Crashpad client interface to store annotations in this structure.
   //!
   //! The annotations returned by this method do not duplicate those returned by
   //! AnnotationsVector().
   virtual std::map<std::string, std::string> AnnotationsSimpleMap() const = 0;
-
- protected:
-  ~ModuleSnapshot() {}
 };
 
 }  // namespace crashpad
 
 #endif  // CRASHPAD_SNAPSHOT_MODULE_SNAPSHOT_H_