Remove Web SQL from Lighthouse web test

adamraine · Devtools-frontend LUCI CQ · commit 287be69dde16 · 2021-06-01T19:28:06.000Z
This web test has become flaky downstream in Lighthouse. Web SQL is not always surfaced in the warning message. The warning message can still be tested with just IndexedDB, so we can remove Web SQL from this test. Bug: none Change-Id: Ia73ad928bcb724049ee3578722b29beb3e57108f Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2929758 Commit-Queue: Paul Irish <paulirish@chromium.org> Reviewed-by: Paul Irish <paulirish@chromium.org>
diff --git a/test/webtests/http/tests/devtools/lighthouse/lighthouse-clear-data-warning-expected.txt b/test/webtests/http/tests/devtools/lighthouse/lighthouse-clear-data-warning-expected.txt
@@ -11,5 +11,5 @@ Tests that Lighthouse panel displays a warning when important data may affect pe
 [x] Clear storage
 [x] Simulated throttling
 Generate report: enabled visible
-Warning Text: There may be stored data affecting loading performance in these locations: Web SQL, IndexedDB. Audit this page in an incognito window to prevent those resources from affecting your scores.
+Warning Text: There may be stored data affecting loading performance in this location: IndexedDB. Audit this page in an incognito window to prevent those resources from affecting your scores.
 
diff --git a/test/webtests/http/tests/devtools/lighthouse/resources/lighthouse-storage.html b/test/webtests/http/tests/devtools/lighthouse/resources/lighthouse-storage.html
@@ -4,13 +4,6 @@
   <script>
     // Indexeddb
     window.indexedDB.open("DataBase", 3);
-
-    // WebSQL
-    var db = openDatabase('mydb', '1.0', 'Test', 2 * 1024 * 1024);
-    db.transaction(function (tx) {
-       tx.executeSql('CREATE TABLE IF NOT EXISTS LOGS (id unique, log)');
-       tx.executeSql('INSERT INTO LOGS (id, log) VALUES (1, "foobar")');
-    });
   </script>
 </head>
 <body>
