Bug 35289

Summary:

Webkit Sticky Notes Demo For Firefox Browsers

Product:

WebKit

Reporter:

Hish <pop2.bugwebkit>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED WONTFIX

Severity:

Enhancement

CC:

sam

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

URL:

http://webkit.org/demos/sticky-notes/index.html

Attachments:

Description	Flags
Check for gecko browsers and offers user option to download SQL addon	sam: review-
Check for gecko browsers and offers user option to download SQL addon #2	sam: review-

Hish

Reported 2010-02-23 03:05:29 PST

Created attachment 49272 [details] Check for gecko browsers and offers user option to download SQL addon The SQL API does not work on Firefox and other Gecko based browsers. With the attached patch, the user has the option to be directed to an addon which enables Firefox to handle the Sticky Notes SQL example.

Attachments
Check for gecko browsers and offers user option to download SQL addon (838 bytes, patch) 2010-02-23 03:05 PST, Hish	sam: review-	Details Formatted Diff Diff
Check for gecko browsers and offers user option to download SQL addon #2 (1.52 KB, patch) 2010-02-24 12:02 PST, Hish	sam: review-	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Alexey Proskuryakov

Comment 1 2010-02-23 10:16:45 PST

Would you be willing to submit a patch for review, as described in <http://webkit.org/coding/contributing.html>?

Hish

Comment 2 2010-02-24 02:53:10 PST

(In reply to comment #1) I already uploaded the patch above. But it seems that I didn't check the question mark as described in http://webkit.org/coding/contributing.html#commitqueue.

Sam Weinig

Comment 3 2010-02-24 04:43:26 PST

Comment on attachment 49272 [details] Check for gecko browsers and offers user option to download SQL addon This is missing a ChangeLog and has some style issues (spaces after if, around !=). r-.

Hish

Comment 4 2010-02-24 12:02:51 PST

Created attachment 49419 [details] Check for gecko browsers and offers user option to download SQL addon #2

Eric Seidel (no email)

Comment 5 2010-03-15 15:51:48 PDT

Comment on attachment 49419 [details] Check for gecko browsers and offers user option to download SQL addon #2 I'm slightly confused as to if this change is just adding marketing for this AddOn, or if this is really the only way to run this demo inside any version of Firefox?

Hish

Comment 6 2010-03-17 00:41:36 PDT

I would be happy if this addon would be obsolete. But unfortunately Firefox does not provide the SQL API. Only the HTML5 key-value pair storage.

Hish

Comment 7 2010-03-17 04:20:53 PDT

(In reply to comment #5) https://developer.mozilla.org/en/Storage says that the SQL API is only exposed to extensions and not to web pages. There is also a lot of discussion about this topic: http://blog.vlad1.com/2009/04/06/html5-web-storage-and-sql/ Currently it seems that the SQL API won't be exposed to web pages in favour of a W3C non-relational storage syntax.

Sam Weinig

Comment 8 2010-03-17 17:36:00 PDT

Comment on attachment 49419 [details] Check for gecko browsers and offers user option to download SQL addon #2 There is not a good reason to take this change. This test is supposed to be demo of browser behavior, not a advertisement for an optional add on.

Note You need to log in before you can comment on or make changes to this bug.