win: Only call GetFileVersionInfo() once per module

snapshot/win/module_snapshot_win.h

 // Copyright 2015 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,
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
 #ifndef CRASHPAD_SNAPSHOT_WIN_MODULE_SNAPSHOT_WIN_H_
 #define CRASHPAD_SNAPSHOT_WIN_MODULE_SNAPSHOT_WIN_H_
 
 #include <stdint.h>
 #include <sys/types.h>
+#include <windows.h>
 
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "snapshot/crashpad_info_client_options.h"
 #include "snapshot/module_snapshot.h"
 #include "snapshot/win/process_reader_win.h"
+#include "util/misc/initialization_state.h"
 #include "util/misc/initialization_state_dcheck.h"
 #include "util/win/process_info.h"
 
 namespace crashpad {
 
 class PEImageReader;
 struct UUID;
 
 namespace internal {
 
@@ skipping 42 matching lines @@
   ModuleType GetModuleType() const override;
   void UUIDAndAge(crashpad::UUID* uuid, uint32_t* age) const override;
   std::string DebugFileName() const override;
   std::vector<std::string> AnnotationsVector() const override;
   std::map<std::string, std::string> AnnotationsSimpleMap() const override;
 
  private:
   template <class Traits>
   void GetCrashpadOptionsInternal(CrashpadInfoClientOptions* options);
 
+  // Initializes vs_fixed_file_info_ if it has not yet been initialized, and
+  // returns a pointer to it. Returns nullptr on failure, with a message logged
+  // on the first call.
+  const VS_FIXEDFILEINFO* VSFixedFileInfo() const;
+
   std::wstring name_;
   std::string pdb_name_;
   UUID uuid_;
   scoped_ptr<PEImageReader> pe_image_reader_;
   ProcessReaderWin* process_reader_;  // weak
   time_t timestamp_;
   uint32_t age_;
   InitializationStateDcheck initialized_;
 
+  // VSFixedFileInfo() is logically const, but updates these members on the
+  // the call. See https://crashpad.chromium.org/bug/9.
+  mutable VS_FIXEDFILEINFO vs_fixed_file_info_;
+  mutable InitializationState initialized_vs_fixed_file_info_;
+
   DISALLOW_COPY_AND_ASSIGN(ModuleSnapshotWin);
 };
 
 }  // namespace internal
 }  // namespace crashpad
 
 #endif  // CRASHPAD_SNAPSHOT_WIN_MODULE_SNAPSHOT_WIN_H_