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

Unified Diff: client/settings.h

Issue 988063003: Define the Settings interface for a CrashReportDatabase and provide a Mac implementation. (Closed) Base URL: https://chromium.googlesource.com/crashpad/crashpad@master
Patch Set: For landing Created 5 years, 9 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 | « client/crash_report_database_win.cc ('k') | client/settings.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: client/settings.h
diff --git a/client/settings.h b/client/settings.h
new file mode 100644
index 0000000000000000000000000000000000000000..741561be4925ae2a7e5d17d53439f7ca4df725d7
--- /dev/null
+++ b/client/settings.h
@@ -0,0 +1,147 @@
+// 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_CLIENT_SETTINGS_H_
+#define CRASHPAD_CLIENT_SETTINGS_H_
+
+#include <time.h>
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/files/file_path.h"
+#include "util/file/file_io.h"
+#include "util/misc/initialization_state_dcheck.h"
+#include "util/misc/uuid.h"
+
+namespace crashpad {
+
+//! \brief An interface for accessing and modifying the settings of a
+//! CrashReportDatabase.
+//!
+//! This class must not be instantiated directly, but rather an instance of it
+//! should be retrieved via CrashReportDatabase::GetSettings().
+class Settings {
+ public:
+ explicit Settings(const base::FilePath& file_path);
+ ~Settings();
+
+ bool Initialize();
+
+ //! \brief Retrieves the immutable identifier for this client, which is used
+ //! on a server to locate all crash reports from a specific Crashpad
+ //! database.
+ //!
+ //! This is automatically initialized when the database is created.
+ //!
+ //! \param[out] client_id The unique client identifier.
+ //!
+ //! \return On success, returns `true`, otherwise returns `false` with an
+ //! error logged.
+ bool GetClientID(UUID* client_id);
+
+ //! \brief Retrieves the user’s preference for submitting crash reports to a
+ //! collection server.
+ //!
+ //! The default value is `false`.
+ //!
+ //! \param[out] enabled Whether crash reports should be uploaded.
+ //!
+ //! \return On success, returns `true`, otherwise returns `false` with an
+ //! error logged.
+ bool GetUploadsEnabled(bool* enabled);
+
+ //! \brief Sets the user’s preference for submitting crash reports to a
+ //! collection server.
+ //!
+ //! \param[in] enabled Whether crash reports should be uploaded.
+ //!
+ //! \return On success, returns `true`, otherwise returns `false` with an
+ //! error logged.
+ bool SetUploadsEnabled(bool enabled);
+
+ //! \brief Retrieves the last time at which a report was attempted to be
+ //! uploaded.
+ //!
+ //! The default value is `0` if it has never been set before.
+ //!
+ //! \param[out] time The last time at which a report was uploaded.
+ //!
+ //! \return On success, returns `true`, otherwise returns `false` with an
+ //! error logged.
+ bool GetLastUploadAttemptTime(time_t* time);
+
+ //! \brief Sets the last time at which a report was attempted to be uploaded.
+ //!
+ //! This is only meant to be used internally by the CrashReportDatabase.
+ //!
+ //! \param[in] time The last time at which a report was uploaded.
+ //!
+ //! \return On success, returns `true`, otherwise returns `false` with an
+ //! error logged.
+ bool SetLastUploadAttemptTime(time_t time);
+
+ private:
+ struct Data;
+
+ // Opens the settings file for reading. On error, logs a message and returns
+ // the invalid handle.
+ ScopedFileHandle OpenForReading();
+
+ // Opens the settings file for reading and writing. On error, logs a message
+ // and returns the invalid handle.
+ ScopedFileHandle OpenForReadingAndWriting();
+
+ // Opens the settings file and reads the data. If that fails, an error will
+ // be logged and the settings will be recovered and re-initialized. If that
+ // also fails, returns false with additional log data from recovery.
+ bool OpenAndReadSettings(Data* out_data);
+
+ // Opens the settings file for writing and reads the data. If reading fails,
+ // recovery is attempted. Returns the opened file handle on success, or the
+ // invalid file handle on failure, with an error logged.
+ ScopedFileHandle OpenForWritingAndReadSettings(Data* out_data);
+
+ // Reads the settings from |handle|. Logs an error and returns false on
+ // failure. This does not perform recovery.
+ bool ReadSettings(FileHandle handle, Data* out_data);
+
+ // Writes the settings to |handle|. Logs an error and returns false on
+ // failure. This does not perform recovery.
+ bool WriteSettings(FileHandle handle, const Data& data);
+
+ // Recovers the settings file by re-initializing the data. If |handle| is the
+ // invalid handle, this will open the file; if it is not, then it must be the
+ // result of OpenForReadingAndWriting(). If the invalid handle is passed, the
+ // caller must not be holding the handle. The new settings data are stored in
+ // |out_data|. Returns true on success and false on failure, with an error
+ // logged.
+ bool RecoverSettings(FileHandle handle, Data* out_data);
+
+ // Initializes a settings file and writes the data to |handle|. Returns true
+ // on success and false on failure, with an error logged.
+ bool InitializeSettings(FileHandle handle);
+
+ const char* file_path() { return file_path_.value().c_str(); }
+
+ base::FilePath file_path_;
+
+ InitializationStateDcheck initialized_;
+
+ DISALLOW_COPY_AND_ASSIGN(Settings);
+};
+
+} // namespace crashpad
+
+#endif // CRASHPAD_CLIENT_SETTINGS_H_
« no previous file with comments | « client/crash_report_database_win.cc ('k') | client/settings.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698