Gather metrics on hosted app unlimitedStorage usage

Gather metrics on hosted app unlimitedStorage usage

BUG=416691
Committed: https://crrev.com/bc8979a1b57ae00eee1c730fdfc41f558a921055
Cr-Commit-Position: refs/heads/master@{#297323}

[Devlin, 2014-09-29 20:57:21]

rdevlin.cronin@chromium.org changed reviewers:
+ jyasskin@chromium.org

[Devlin, 2014-09-29 20:57:22]

Jeffrey, PTAL.  The approach I took in
https://codereview.chromium.org/595683002/ was patently wrong, since hosted apps
don't go through the extension storage api (d'oh).

[Jeffrey Yasskin, 2014-09-29 21:50:22]

lgtm

Sorry for not knowing the system well enough to catch that.

https://codereview.chromium.org/612983004/diff/1/chrome/browser/extensions/ex...
File chrome/browser/extensions/extension_special_storage_policy.cc (right):

https://codereview.chromium.org/612983004/diff/1/chrome/browser/extensions/ex...
chrome/browser/extensions/extension_special_storage_policy.cc:39: void
ReportQuotaUsage(storage::QuotaStatusCode code, int64 usage, int64 quota) {
Handle status codes other than kQuotaStatusOk, probably by skipping the report.

[Devlin, 2014-09-29 22:44:31]

On 2014/09/29 21:50:22, Jeffrey Yasskin wrote:
> lgtm
> 
> Sorry for not knowing the system well enough to catch that.

No worries - clearly I didn't either.  Now we both know better. :)

https://codereview.chromium.org/612983004/diff/1/chrome/browser/extensions/ex...
File chrome/browser/extensions/extension_special_storage_policy.cc (right):

https://codereview.chromium.org/612983004/diff/1/chrome/browser/extensions/ex...
chrome/browser/extensions/extension_special_storage_policy.cc:39: void
ReportQuotaUsage(storage::QuotaStatusCode code, int64 usage, int64 quota) {
On 2014/09/29 21:50:22, Jeffrey Yasskin wrote:
> Handle status codes other than kQuotaStatusOk, probably by skipping the
report.

Done.

[Devlin, 2014-09-29 22:44:35]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2014-09-29 22:45:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/612983004/20001

[commit-bot: I haz the power, 2014-09-29 23:24:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-29 23:24:47]

Try jobs failed on following builders:
  linux_chromium_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Devlin, 2014-09-30 00:02:28]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2014-09-30 00:03:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/612983004/30001

[commit-bot: I haz the power, 2014-09-30 01:08:57]

Committed patchset #3 (id:30001) as 5f4b1afd2880c9a0bfb45cc37181254d589580e2

[commit-bot: I haz the power, 2014-09-30 01:10:08]

Patchset 3 (id:??) landed as
https://crrev.com/bc8979a1b57ae00eee1c730fdfc41f558a921055
Cr-Commit-Position: refs/heads/master@{#297323}