Closed
Bug 856732
Opened 12 years ago
Closed 12 years ago
[email] Update manifest to ask for storage
Categories
(Firefox OS Graveyard :: Gaia::E-Mail, defect)
Tracking
(blocking-b2g:tef+, b2g18 verified, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 verified)
VERIFIED
FIXED
blocking-b2g | tef+ |
People
(Reporter: jrburke, Unassigned)
Details
Attachments
(1 file)
A result from this dev-gaia list about asking for storage permission in the manifest if needing more than 5MB of IndexedDB storage, which email does need:
http://groups.google.com/group/mozilla.dev.gaia/browse_thread/thread/5a4cb46725d6b3ad#
Reporter | ||
Comment 1•12 years ago
|
||
Asking for tef+ since the above linked thread indicates it is needed for all branches including 1.0.1.
blocking-b2g: --- → tef?
Reporter | ||
Comment 2•12 years ago
|
||
Pointer to Github pull-request
Reporter | ||
Comment 3•12 years ago
|
||
Comment on attachment 731959 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/8917
Adds `storage: {}` to manifest.
Updated•12 years ago
|
Attachment #731959 -
Flags: review+
Updated•12 years ago
|
blocking-b2g: tef? → tef+
status-b2g18:
--- → affected
status-b2g18-v1.0.0:
--- → wontfix
status-b2g18-v1.0.1:
--- → affected
Can you please provide steps to verify this fix - as we will blackbox test from the UI?
Comment 5•12 years ago
|
||
Uplifted commit 1f7c4553be6f41e679528645e982a81176154700 as:
v1-train: c4d306d08b6806896a6c51d920b568ef45675a1c
v1.0.1: 12305ddbafe09c1f77dfa0da329917a2d8fd3f3d
Reporter | ||
Comment 6•12 years ago
|
||
Steps to verify:
* Configure an account. Any account type will do, as long as it has a decent amount of email.
* Scroll through list of messages.
* View message bodies.
This change was to make sure we could access more than 5MB of IndexedDB storage given the new gecko rules around restriction to 5MB unless the app specifies needing the "storage" permission, which this change does.
It may be hard to trigger storage over 5MB limit, but viewing messages and even configuring more than one account would be a way to try to get to that limit.
Verified fixed on
Unagi Build ID: 20130403070201
Kernel Date: Dec 5
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/512258bc00a3
Gaia: daea430624ec02f8d36a12d581fc4a3278c27cb7
and
Unagi Build ID: 20130403070204
Kernel Date: Dec 5
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/d467369d1b0c
Gaia: 06e0e5ce42bdfb62bdbe38271de6b5b2d9e40e75
Able to have four email accounts on devices and view access more then 5MB.
You need to log in
before you can comment on or make changes to this bug.
Description
•