RESOLVED FIXED 139558
Move storage code from WebCore to WebKit 
https://bugs.webkit.org/show_bug.cgi?id=139558
Summary Move storage code from WebCore to WebKit
Anders Carlsson
Reported 2014-12-11 12:57:18 PST
Move storage code from WebCore to WebKit
Attachments
Patch (222.15 KB, patch)
2014-12-11 13:00 PST, Anders Carlsson 		no flags
DetailsFormatted DiffDiff
Patch (226.29 KB, patch)
2014-12-29 12:55 PST, Anders Carlsson 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2014-12-11 13:00:58 PST
Created attachment 243140 [details] Patch
WebKit Commit Bot
Comment 2 2014-12-11 13:03:43 PST
Attachment 243140 [details] did not pass style-queue: ERROR: Source/WebKit/Storage/StorageAreaImpl.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/mac/Storage/WebStorageManager.mm:28: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageThread.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageThread.cpp:76: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebKit/Storage/StorageThread.cpp:80: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebKit/Storage/StorageThread.cpp:92: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebKit/Storage/StorageAreaSync.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageAreaSync.cpp:44: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebKit/Storage/StorageAreaSync.cpp:93: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebKit/Storage/StorageAreaSync.cpp:172: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebKit/Storage/StorageAreaSync.cpp:197: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebKit/Storage/StorageAreaSync.cpp:373: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebKit/Storage/StorageTrackerClient.h:25: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/Storage/StorageNamespaceImpl.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageTracker.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageNamespaceImpl.h:29: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageSyncManager.h:45: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebKit/Storage/StorageSyncManager.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageSyncManager.cpp:66: Extra space before ( in function call [whitespace/parens] [4] Total errors found: 19 in 26 files If any of these errors are false positives, please file a bug against check-webkit-style.
Anders Carlsson
Comment 3 2014-12-29 12:55:10 PST
Created attachment 243808 [details] Patch
WebKit Commit Bot
Comment 4 2014-12-29 12:57:07 PST
Attachment 243808 [details] did not pass style-queue: ERROR: Source/WebKit/Storage/StorageAreaImpl.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/mac/Storage/WebStorageManager.mm:28: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageThread.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageThread.cpp:76: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebKit/Storage/StorageThread.cpp:80: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebKit/Storage/StorageThread.cpp:92: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebKit/Storage/StorageAreaSync.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageAreaSync.cpp:44: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebKit/Storage/StorageAreaSync.cpp:91: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebKit/Storage/StorageTrackerClient.h:25: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/Storage/StorageNamespaceImpl.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageTracker.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageSyncManager.h:45: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WebKit/Storage/StorageSyncManager.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebKit/Storage/StorageSyncManager.cpp:66: Extra space before ( in function call [whitespace/parens] [4] Total errors found: 15 in 26 files If any of these errors are false positives, please file a bug against check-webkit-style.
Anders Carlsson
Comment 5 2014-12-29 13:12:29 PST
Committed r177813: <http://trac.webkit.org/changeset/177813>
Csaba Osztrogonác
Comment 6 2014-12-30 01:37:46 PST
Comment on attachment 243808 [details] Patch remove r? from landed patch
Note You need to log in before you can comment on or make changes to this bug.