Bug 190271 – Add a switch for Web SQL

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Sihui Liu]

This could help check whether websites are broken without Web SQL.

[Sihui Liu]

Created attachment 351575 [details]
Patch

[Chris Dumez]

Comment on attachment 351575 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=351575&action=review

> Source/WebKit/Shared/WebPreferences.yaml:1364
> +  webcoreBinding: RuntimeEnabledFeatures

Don't we need a category? What's the default category if you do not specify one?

> Source/WebKit/UIProcess/API/C/WKPreferencesRef.h:332
> +WK_EXPORT bool WKPreferencesGetWebSQLEnabled(WKPreferencesRef preferencesRef);

Do we really need this SPI?

[Sihui Liu]

Created attachment 352406 [details]
Patch

[Sihui Liu]

(In reply to Chris Dumez from comment #2)
> Comment on attachment 351575 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=351575&action=review
> 
> > Source/WebKit/Shared/WebPreferences.yaml:1364
> > +  webcoreBinding: RuntimeEnabledFeatures
> 
> Don't we need a category? What's the default category if you do not specify
> one?
> 
It is said default category means displaying in nowhere. I changed it to experimental.

> > Source/WebKit/UIProcess/API/C/WKPreferencesRef.h:332
> > +WK_EXPORT bool WKPreferencesGetWebSQLEnabled(WKPreferencesRef preferencesRef);
> 
> Do we really need this SPI?
Yes. We are going to toggle this setting, making "Disable Web SQL" an experimental feature.

Uploaded a new patch with a more appropriate variable name: disableWebSQLEnabled.

[EWS Watchlist]

Comment on attachment 352406 [details]
Patch

Attachment 352406 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/9604452

Number of test failures exceeded the failure limit.

[EWS Watchlist]

Created attachment 352421 [details]
Archive of layout-test-results from ews107 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6

[EWS Watchlist]

Comment on attachment 352406 [details]
Patch

Attachment 352406 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/9605026

Number of test failures exceeded the failure limit.

[EWS Watchlist]

Created attachment 352424 [details]
Archive of layout-test-results from ews124 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews124  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.6

[Ryosuke Niwa]

Appears to be failing to build on MacOS?

[50093/50123] storage/websql/empty-statement.html failed unexpectedly (test timed out, text diff)
[50094/50123] storage/websql/execute-sql-args.html failed unexpectedly (test timed out, text diff)
[50095/50123] storage/websql/execute-sql-rowsAffected.html failed unexpectedly (text diff)
[50096/50123] storage/websql/executesql-accepts-only-one-statement.html failed unexpectedly (test timed out, text diff)
[50097/50123] storage/websql/hash-change-with-xhr.html failed unexpectedly (test timed out, text diff)
[50098/50123] storage/websql/multiple-transactions-on-different-handles.html failed unexpectedly (test timed out, text diff)
[50099/50123] storage/websql/multiple-transactions.html failed unexpectedly (test timed out, text diff)
[50100/50123] storage/websql/null-callbacks.html failed unexpectedly (text diff)
[50101/50123] storage/websql/open-database-creation-callback-isolated-world.html failed unexpectedly (test timed out, text diff)

This might be because WTR tries to enable all experimental features...

[Ryosuke Niwa]

Comment on attachment 352406 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=352406&action=review

> Source/WebKit/Shared/WebPreferences.yaml:1297
> +DisabledWebSQLEnabled:

This is a very strangely named preference. Can we just call it WebSQLDisabled instead?

> Source/WebKit/Shared/WebPreferences.yaml:1303
> +  category: experimental

Does it really make sense to expose it as an experimental feature??

[Ryosuke Niwa]

Comment on attachment 352406 [details]
Patch

r- because tests are failing in WK2.

[Sihui Liu]

(In reply to Ryosuke Niwa from comment #10)
> Comment on attachment 352406 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=352406&action=review
> 
> > Source/WebKit/Shared/WebPreferences.yaml:1297
> > +DisabledWebSQLEnabled:
> 
> This is a very strangely named preference. Can we just call it
> WebSQLDisabled instead?
> 
Okay.

> > Source/WebKit/Shared/WebPreferences.yaml:1303
> > +  category: experimental
> 
> Does it really make sense to expose it as an experimental feature??
Making it experimental so we could expose "Disable WebSQL" to the web developer, letting them check if their pages broken under the feature. Is there another way to achieve this?

[Sihui Liu]

Created attachment 352461 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 352461 [details]
Patch

Clearing flags on attachment: 352461

Committed r237210: <https://trac.webkit.org/changeset/237210>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/45323549>