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

Unified Diff: snapshot/system_snapshot_mac.h

Issue 666483002: Create snapshot/mac and move some files from snapshot and util to there (Closed) Base URL: https://chromium.googlesource.com/crashpad/crashpad/+/master
Patch Set: Move process_reader, process_types, and mach_o_image*_reader from util/mac to snapshot/mac Created 6 years, 2 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
« no previous file with comments | « snapshot/snapshot.gyp ('k') | snapshot/system_snapshot_mac.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: snapshot/system_snapshot_mac.h
diff --git a/snapshot/system_snapshot_mac.h b/snapshot/system_snapshot_mac.h
deleted file mode 100644
index 3eadc372952fb03641bcd686e466bcbe9e8ff297..0000000000000000000000000000000000000000
--- a/snapshot/system_snapshot_mac.h
+++ /dev/null
@@ -1,101 +0,0 @@
-// 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,
-// 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_SYSTEM_SNAPSHOT_MAC_H_
-#define CRASHPAD_SNAPSHOT_SYSTEM_SNAPSHOT_MAC_H_
-
-#include <stdint.h>
-
-#include <string>
-
-#include "base/basictypes.h"
-#include "snapshot/system_snapshot.h"
-#include "util/misc/initialization_state_dcheck.h"
-
-namespace crashpad {
-
-class ProcessReader;
-
-namespace internal {
-
-//! \brief A SystemSnapshot of the running system, when the system runs Mac OS
-//! X.
-class SystemSnapshotMac final : public SystemSnapshot {
- public:
- SystemSnapshotMac();
- ~SystemSnapshotMac();
-
- //! \brief Initializes the object.
- //!
- //! \param[in] process_reader A reader for the process being snapshotted.
- //! \n\n
- //! It seems odd that a system snapshot implementation would need a
- //! ProcessReader, but some of the information reported about the system
- //! depends on the process it’s being reported for. For example, the
- //! architecture returned by GetCPUArchitecture() should be the
- //! architecture of the process, which may be different than the native
- //! architecture of the system: an x86_64 system can run both x86_64 and
- //! 32-bit x86 processes.
- //! \param[in] snapshot_time The time of the snapshot being taken.
- //! \n\n
- //! This parameter is necessary for TimeZone() to determine whether
- //! daylight saving time was in effect at the time the snapshot was taken.
- //! Otherwise, it would need to base its determination on the current
- //! time, which may be different than the snapshot time for snapshots
- //! generated around the daylight saving transition time.
- void Initialize(ProcessReader* process_reader, const timeval* snapshot_time);
-
- // SystemSnapshot:
-
- CPUArchitecture GetCPUArchitecture() const override;
- uint32_t CPURevision() const override;
- uint8_t CPUCount() const override;
- std::string CPUVendor() const override;
- void CPUFrequency(uint64_t* current_hz, uint64_t* max_hz) const override;
- uint32_t CPUX86Signature() const override;
- uint64_t CPUX86Features() const override;
- uint64_t CPUX86ExtendedFeatures() const override;
- uint32_t CPUX86Leaf7Features() const override;
- bool CPUX86SupportsDAZ() const override;
- OperatingSystem GetOperatingSystem() const override;
- bool OSServer() const override;
- void OSVersion(
- int* major, int* minor, int* bugfix, std::string* build) const override;
- std::string OSVersionFull() const override;
- bool NXEnabled() const override;
- std::string MachineDescription() const override;
- void TimeZone(DaylightSavingTimeStatus* dst_status,
- int* standard_offset_seconds,
- int* daylight_offset_seconds,
- std::string* standard_name,
- std::string* daylight_name) const override;
-
- private:
- std::string os_version_full_;
- std::string os_version_build_;
- ProcessReader* process_reader_; // weak
- const timeval* snapshot_time_; // weak
- int os_version_major_;
- int os_version_minor_;
- int os_version_bugfix_;
- bool os_server_;
- InitializationStateDcheck initialized_;
-
- DISALLOW_COPY_AND_ASSIGN(SystemSnapshotMac);
-};
-
-} // namespace internal
-} // namespace crashpad
-
-#endif // CRASHPAD_SNAPSHOT_SYSTEM_SNAPSHOT_MAC_H_
« no previous file with comments | « snapshot/snapshot.gyp ('k') | snapshot/system_snapshot_mac.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698