WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
58648
Expose unified Quota API if QUOTA build flag is enabled
https://bugs.webkit.org/show_bug.cgi?id=58648
Summary
Expose unified Quota API if QUOTA build flag is enabled
Kinuko Yasuda
Reported
Friday, April 15, 2011 9:56:39 AM UTC
Per public discussion on public-webapps, we want expose the unified quota API that allows webapps to request/query the current storage usage and quota.
http://lists.w3.org/Archives/Public/public-webapps/2011JanMar/0346.html
The API is not standardized yet so it needs to be vendor-prefixed.
Attachments
Patch
(34.53 KB, patch)
2011-04-15 02:08 PDT
,
Kinuko Yasuda
no flags
Details
Formatted Diff
Diff
Patch
(44.78 KB, patch)
2011-04-15 04:03 PDT
,
Kinuko Yasuda
no flags
Details
Formatted Diff
Diff
Patch
(46.87 KB, patch)
2011-04-15 07:26 PDT
,
Kinuko Yasuda
no flags
Details
Formatted Diff
Diff
Patch
(47.41 KB, patch)
2011-04-15 10:36 PDT
,
Kinuko Yasuda
no flags
Details
Formatted Diff
Diff
Patch
(47.62 KB, patch)
2011-04-15 12:41 PDT
,
Kinuko Yasuda
levin
: review+
Details
Formatted Diff
Diff
Show Obsolete
(4)
View All
Add attachment
proposed patch, testcase, etc.
Kinuko Yasuda
Comment 1
Friday, April 15, 2011 10:08:11 AM UTC
Created
attachment 89756
[details]
Patch
Kinuko Yasuda
Comment 2
Friday, April 15, 2011 12:03:18 PM UTC
Created
attachment 89763
[details]
Patch
Early Warning System Bot
Comment 3
Friday, April 15, 2011 2:50:28 PM UTC
Attachment 89763
[details]
did not build on qt: Build output:
http://queues.webkit.org/results/8459151
Collabora GTK+ EWS bot
Comment 4
Friday, April 15, 2011 3:01:46 PM UTC
Attachment 89763
[details]
did not build on gtk: Build output:
http://queues.webkit.org/results/8452151
Kinuko Yasuda
Comment 5
Friday, April 15, 2011 3:26:20 PM UTC
Created
attachment 89784
[details]
Patch
Kinuko Yasuda
Comment 6
Friday, April 15, 2011 6:36:28 PM UTC
Created
attachment 89807
[details]
Patch
Kinuko Yasuda
Comment 7
Friday, April 15, 2011 8:41:26 PM UTC
Created
attachment 89830
[details]
Patch Rebased.
David Levin
Comment 8
Friday, April 15, 2011 10:24:04 PM UTC
Comment on
attachment 89830
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=89830&action=review
ok. A few minor nits.
> Source/WebCore/page/DOMWindow.cpp:791 > +
remove the blank line.
> Source/WebCore/storage/StorageInfo.idl:2 > + * Copyright (C) 2008, 2009 Apple Inc. All rights reserved.
This copyright line looks incorrect.
David Levin
Comment 9
Friday, April 15, 2011 10:25:23 PM UTC
(In reply to
comment #0
)
> Per public discussion on public-webapps, we want expose the unified quota API that allows webapps to request/query the current storage usage and quota. >
http://lists.w3.org/Archives/Public/public-webapps/2011JanMar/0346.html
> > The API is not standardized yet so it needs to be vendor-prefixed.
And prefix it before check in.
Kinuko Yasuda
Comment 10
Friday, April 15, 2011 10:30:09 PM UTC
Thanks! (In reply to
comment #9
)
> (In reply to
comment #0
) > > Per public discussion on public-webapps, we want expose the unified quota API that allows webapps to request/query the current storage usage and quota. > >
http://lists.w3.org/Archives/Public/public-webapps/2011JanMar/0346.html
> > > > The API is not standardized yet so it needs to be vendor-prefixed. > > And prefix it before check in.
The entry interface (StorageInfo) is prefixed by webkit (i.e. webkitStorageInfo) in DOMWindow.idl.
David Levin
Comment 11
Friday, April 15, 2011 10:52:45 PM UTC
(In reply to
comment #10
)
> (In reply to
comment #9
) > > And prefix it before check in. > The entry interface (StorageInfo) is prefixed by webkit (i.e. webkitStorageInfo) in DOMWindow.idl.
Ah ok.
Kinuko Yasuda
Comment 12
Saturday, April 16, 2011 12:59:07 AM UTC
Committed
r84053
: <
http://trac.webkit.org/changeset/84053
>
WebKit Review Bot
Comment 13
Saturday, April 16, 2011 2:22:10 AM UTC
http://trac.webkit.org/changeset/84053
might have broken WinCairo Debug (Build)
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