Clean up some JS documentation.

FossilOrigin-Name: 925fdbfc6c1f9f06951346bc0d4fb81ffb5797d5f1123728931973f6cb1a0efa

ext/wasm/api/sqlite3-api-oo1.js

@@ -505,20 +505,19 @@ self.sqlite3ApiBootstrap.initializers.push(function(sqlite3){
 
        - `.callback` = a function which gets called for each row of
        the result set, but only if that statement has any result
-       _rows_. The callback's "this" is the options object. The second
-       argument passed to the callback is always the current Stmt
-       object (so that the caller may collect column names, or
-       similar). The 2nd argument to the callback is always the Stmt
-       instance, as it's needed if the caller wants to fetch the
-       column names or some such (noting that they could also be
-       fetched via `this.columnNames`, if the client provides the
-       `columnNames` option).
+       _rows_. The callback's "this" is the options object, noting
+       that this function synthesizes one if the caller does not pass
+       one to exec(). The second argument passed to the callback is
+       always the current Stmt object, as it's needed if the caller
+       wants to fetch the column names or some such (noting that they
+       could also be fetched via `this.columnNames`, if the client
+       provides the `columnNames` option).
 
        ACHTUNG: The callback MUST NOT modify the Stmt object. Calling
        any of the Stmt.get() variants, Stmt.getColumnName(), or
        similar, is legal, but calling step() or finalize() is
-       not. Routines which are illegal in this context will trigger an
-       exception.
+       not. Member methods which are illegal in this context will
+       trigger an exception.
 
        The first argument passed to the callback defaults to an array of
        values from the current result row but may be changed with ...

ext/wasm/kvvfs1.js

@@ -105,7 +105,7 @@
       log("sqlite3_web_kvvfs_size(",dbStorage,") says", capi.sqlite3_web_kvvfs_size(dbStorage),
          "bytes");
     });
-    log("Storage backend:",db.filename /* note that the name was internally translated */);
+    log("Storage backend:",db.filename);
     if(0===db.selectValue('select count(*) from sqlite_master')){
       log("DB is empty. Use the init button to populate it.");
     }else{

ext/wasm/testing-worker1-promiser.js

@@ -145,10 +145,6 @@
       mustNotReach
     ).catch((e)=>{
       warn("Intentional error:",e);
-      // Why does the browser report console.error "Uncaught (in
-      // promise)" when we catch(), and does so _twice_ if we don't
-      // catch()? According to all docs, that error must be supressed
-      // if we explicitly catch().
     });
 
     await wtest('exec',{