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

Side by Side Diff: third_party/WebKit/Source/modules/webdatabase/ChangeVersionWrapper.h

Issue 2050123002: Remove OwnPtr from Blink. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: First attempt to land. Created 4 years, 6 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2007, 2013 Apple Inc. All rights reserved. 2 * Copyright (C) 2007, 2013 Apple Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 13 matching lines...) Expand all
24 * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 24 * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
25 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF 25 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
26 * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 26 * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
27 */ 27 */
28 #ifndef ChangeVersionWrapper_h 28 #ifndef ChangeVersionWrapper_h
29 #define ChangeVersionWrapper_h 29 #define ChangeVersionWrapper_h
30 30
31 #include "modules/webdatabase/SQLTransactionBackend.h" 31 #include "modules/webdatabase/SQLTransactionBackend.h"
32 #include "platform/heap/Handle.h" 32 #include "platform/heap/Handle.h"
33 #include "wtf/Forward.h" 33 #include "wtf/Forward.h"
34 #include <memory>
34 35
35 namespace blink { 36 namespace blink {
36 37
37 class SQLErrorData; 38 class SQLErrorData;
38 39
39 class ChangeVersionWrapper final : public SQLTransactionWrapper { 40 class ChangeVersionWrapper final : public SQLTransactionWrapper {
40 public: 41 public:
41 static ChangeVersionWrapper* create(const String& oldVersion, const String& newVersion) { return new ChangeVersionWrapper(oldVersion, newVersion); } 42 static ChangeVersionWrapper* create(const String& oldVersion, const String& newVersion) { return new ChangeVersionWrapper(oldVersion, newVersion); }
42 43
43 bool performPreflight(SQLTransactionBackend*) override; 44 bool performPreflight(SQLTransactionBackend*) override;
44 bool performPostflight(SQLTransactionBackend*) override; 45 bool performPostflight(SQLTransactionBackend*) override;
45 SQLErrorData* sqlError() const override { return m_sqlError.get(); } 46 SQLErrorData* sqlError() const override { return m_sqlError.get(); }
46 void handleCommitFailedAfterPostflight(SQLTransactionBackend*) override; 47 void handleCommitFailedAfterPostflight(SQLTransactionBackend*) override;
47 48
48 private: 49 private:
49 ChangeVersionWrapper(const String& oldVersion, const String& newVersion); 50 ChangeVersionWrapper(const String& oldVersion, const String& newVersion);
50 51
51 String m_oldVersion; 52 String m_oldVersion;
52 String m_newVersion; 53 String m_newVersion;
53 OwnPtr<SQLErrorData> m_sqlError; 54 std::unique_ptr<SQLErrorData> m_sqlError;
54 }; 55 };
55 56
56 } // namespace blink 57 } // namespace blink
57 58
58 #endif // ChangeVersionWrapper_h 59 #endif // ChangeVersionWrapper_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698