Bug 109109 – webdatabase: Introduce SQLTransactionBackend and SQLTransactionBackendSync.

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Mark Lam]

The majority of the functionality in SQLTransaction belongs in the back-end.  We should rename SQLTransaction as SQLTransactionBackend, and introduce a SQLTransaction to wrap it.  Eventually, we'll want to move the front-end (i.e. script related) functionality back to the front-end side.  This patch is mostly a rename operation to keep the change simple though it hits many files.

[Mark Lam]

Created attachment 186979 [details]
the patch.

[EFL EWS Bot]

Comment on attachment 186979 [details]
the patch.

Attachment 186979 [details] did not pass efl-ews (efl):
Output: http://queues.webkit.org/results/16416026

[WebKit Review Bot]

Attachment 186979 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/Modules/webdatabase/Database.cpp', u'Source/WebCore/Modules/webdatabase/Database.h', u'Source/WebCore/Modules/webdatabase/DatabaseBackend.h', u'Source/WebCore/Modules/webdatabase/DatabaseTask.cpp', u'Source/WebCore/Modules/webdatabase/DatabaseTask.h', u'Source/WebCore/Modules/webdatabase/SQLTransaction.cpp', u'Source/WebCore/Modules/webdatabase/SQLTransaction.h', u'Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp', u'Source/WebCore/Modules/webdatabase/SQLTransactionBackend.h', u'Source/WebCore/Modules/webdatabase/SQLTransactionBackendSync.cpp', u'Source/WebCore/Modules/webdatabase/SQLTransactionBackendSync.h', u'Source/WebCore/Modules/webdatabase/SQLTransactionCoordinator.cpp', u'Source/WebCore/Modules/webdatabase/SQLTransactionCoordinator.h', u'Source/WebCore/Modules/webdatabase/SQLTransactionSync.cpp', u'Source/WebCore/Modules/webdatabase/SQLTransactionSync.h', u'Source/WebCore/Target.pri', u'Source/WebCore/WebCore.gypi', u'Source/WebCore/WebCore.vcproj/WebCore.vcproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj']" exit_code: 1
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.h:67:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackendSync.cpp:96:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackendSync.cpp:98:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackendSync.cpp:160:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackendSync.cpp:172:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackendSync.cpp:177:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackendSync.cpp:215:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:40:  Alphabetical sorting problem.  [build/include_order] [4]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:42:  Alphabetical sorting problem.  [build/include_order] [4]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:267:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:280:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:288:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:358:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:370:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:383:  One line control clauses should not use braces.  [whitespace/braces] [4]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:502:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:579:  One line control clauses should not use braces.  [whitespace/braces] [4]
Source/WebCore/Modules/webdatabase/SQLTransactionBackend.cpp:617:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Total errors found: 18 in 22 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[kov's GTK+ EWS bot]

Comment on attachment 186979 [details]
the patch.

Attachment 186979 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/16404036

[Build Bot]

Comment on attachment 186979 [details]
the patch.

Attachment 186979 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://queues.webkit.org/results/16417047

[Mark Lam]

Comment on attachment 186979 [details]
the patch.

The patch mechanism is not ordered.  Hence, my renaming a file (e.g. SQLTransaction.h to SQLTransactionBackend.h), and then creating a new one (E.G. SQLTransaction.h) with the same old name can result in badness.  Reworking the patch to just add the file with the different name (e.g. SQLTransactionBackend.h) as a new file.

[Mark Lam]

Created attachment 186982 [details]
Reworked patch to make the patch application process work. Also fixed all the other style violations that the ews bot was complaining about.

[Mark Lam]

Thanks for the review.  Landed in 2 parts:
Part 1 r142193: <http://trac.webkit.org/changeset/142193>
Part 2 r142194: <http://trac.webkit.org/changeset/142194>