WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
15592
DOM Exception opening up client-side database example for the first time
https://bugs.webkit.org/show_bug.cgi?id=15592
Summary
DOM Exception opening up client-side database example for the first time
David Kilzer (:ddkilzer)
Reported
2007-10-21 10:33:02 PDT
* SUMMARY "DOM Exception 12" is reported the very first time the DatabaseExample.html page is opened. * STEPS TO REPRODUCE 1. Remove ~/Libraries/WebKit/Databases if it exists. 2. Launch Safari + WebKit. 3. Open URL:
http://webkit.org/misc/DatabaseExample.html
4. Open JavaScript Console. * EXPECTED RESULTS No "DOM Exception 12" should be reported in the JavaScript Console. * ACTUAL RESULTS A "DOM Exception 12" is reported in the JavaScript Console: SYNTAX_ERR: DOM Exception 12
http://webkit.org/misc/DatabaseExample.html
Line: 264 * REGRESSION Not applicable as client-side database support is a new feature. * NOTES Tested with a local debug build of WebKit
r26847
with Safari 3 Public Beta v. 3.0.3 (522.12.1) on Mac OS X 10.4.10 (8R218).
Attachments
Patch v1
(2.20 KB, patch)
2007-10-21 11:33 PDT
,
David Kilzer (:ddkilzer)
darin
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1
2007-10-21 10:34:23 PDT
Also, I believe this "DOM Exception 12" prevents a new blank note from being created the first time the page is loaded.
David Kilzer (:ddkilzer)
Comment 2
2007-10-21 11:07:12 PDT
(In reply to
comment #1
)
> Also, I believe this "DOM Exception 12" prevents a new blank note from being > created the first time the page is loaded.
The problem is that there is a race condition between creating the WebKitStickyNotes table for the first time and querying the table in the loaded() function. The loaded() function may get called before the WebKitStickyNotes table is created.
David Kilzer (:ddkilzer)
Comment 3
2007-10-21 11:33:30 PDT
Created
attachment 16767
[details]
Patch v1 Proposed fix.
Darin Adler
Comment 4
2007-10-21 11:45:06 PDT
Comment on
attachment 16767
[details]
Patch v1 r=me
David Kilzer (:ddkilzer)
Comment 5
2007-10-21 11:53:42 PDT
Committed revision 26851.
Timothy Hatcher
Comment 6
2007-10-21 12:49:54 PDT
Oops, I fixed this for the original example. But it wasn't fixed in Adam's version.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug