Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(260)

Issue 208743003: Remove queryUsageAndQuota API (Closed)

Created:
6 years, 9 months ago by vsm
Modified:
6 years, 9 months ago
Reviewers:
blois
CC:
reviews_dartlang.org, ricow1, ahe
Visibility:
Public.

Description

Remove queryUsageAndQuota API This is gone in Chrome 34. It's breaking the dart2js analyze_api test. TBR'ing to get the bots green. TBR=blois@google.com Committed: https://code.google.com/p/dart/source/detail?r=34376

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -73 lines) Patch
M sdk/lib/html/dart2js/html_dart2js.dart View 2 chunks +2 lines, -22 lines 3 comments Download
M sdk/lib/html/dartium/html_dartium.dart View 2 chunks +4 lines, -21 lines 0 comments Download
M tools/dom/scripts/htmlrenamer.py View 1 chunk +0 lines, -1 line 0 comments Download
D tools/dom/templates/html/impl/impl_StorageInfo.darttemplate View 1 chunk +0 lines, -29 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
vsm
6 years, 9 months ago (2014-03-25 17:54:17 UTC) #1
vsm
6 years, 9 months ago (2014-03-25 18:06:16 UTC) #2
vsm
Committed patchset #1 manually as r34376 (presubmit successful).
6 years, 9 months ago (2014-03-25 18:06:36 UTC) #3
blois
https://chromiumcodereview.appspot.com/208743003/diff/1/sdk/lib/html/dart2js/html_dart2js.dart File sdk/lib/html/dart2js/html_dart2js.dart (left): https://chromiumcodereview.appspot.com/208743003/diff/1/sdk/lib/html/dart2js/html_dart2js.dart#oldcode24468 sdk/lib/html/dart2js/html_dart2js.dart:24468: - Future<StorageInfoUsage> queryUsageAndQuota(int storageType) { I'm pretty sure that ...
6 years, 9 months ago (2014-03-25 19:06:01 UTC) #4
vsm
https://chromiumcodereview.appspot.com/208743003/diff/1/sdk/lib/html/dart2js/html_dart2js.dart File sdk/lib/html/dart2js/html_dart2js.dart (left): https://chromiumcodereview.appspot.com/208743003/diff/1/sdk/lib/html/dart2js/html_dart2js.dart#oldcode24468 sdk/lib/html/dart2js/html_dart2js.dart:24468: - Future<StorageInfoUsage> queryUsageAndQuota(int storageType) { StorageInfo is the type ...
6 years, 9 months ago (2014-03-25 19:50:11 UTC) #5
blois
6 years, 9 months ago (2014-03-25 21:42:01 UTC) #6
Message was sent while issue was closed.
https://chromiumcodereview.appspot.com/208743003/diff/1/sdk/lib/html/dart2js/...
File sdk/lib/html/dart2js/html_dart2js.dart (left):

https://chromiumcodereview.appspot.com/208743003/diff/1/sdk/lib/html/dart2js/...
sdk/lib/html/dart2js/html_dart2js.dart:24468: -  Future<StorageInfoUsage>
queryUsageAndQuota(int storageType) {
On 2014/03/25 19:50:11, vsm wrote:
> StorageInfo is the type changed here, not StorageQuota.  Here are the IDL
> changes.  It appears that usage and quota are just attributes now:
> 
>  ] interface StorageInfo {
> -    const unsigned short TEMPORARY = 0;
> -    const unsigned short PERSISTENT = 1;
> -
> -    [CallWith=ExecutionContext] void queryUsageAndQuota(unsigned short
> storageType, optional StorageUsageCallback usageCallback, optional
> StorageErrorCallba
> -    [CallWith=ExecutionContext] void requestQuota(unsigned short storageType,
> unsigned long long newQuotaInBytes, optional StorageQuotaCallback quotaCallbac
> +    readonly attribute unsigned long long usage;
> +    readonly attribute unsigned long long quota;
>  };
> 
> 
> On 2014/03/25 19:06:02, blois wrote:
> > I'm pretty sure that people are using this API right now. The replacement
API
> > uses Promises and isn't being generated.
> > 
> > http://www.w3.org/TR/quota-api/#idl-def-StorageQuota
> > 
> > What needs to be done to get this working?
> 
The API to query the quota changed from StorageInfo.queryUsageAndQuota to
StorageQuota.queryInfo, which is not exposed because it uses promises.

As it stands now, there's no way to get a StorageInfo object, or to request
additional persistent quota.

Powered by Google App Engine
This is Rietveld 408576698