Vicamo Yang [:vicamo][:vyang] Reporter
	Description • 12 years ago

      No description provided.

	Andrew Overholt [:overholt]
	Comment 1 • 12 years ago

Are you going to do the work here, philikon?

	Vicamo Yang [:vicamo][:vyang] Reporter
	Comment 2 • 12 years ago

(In reply to Andrew Overholt [:overholt] from comment #1)
> blocking-b2g: --- → tef?

Why does this issue deserve a tef?? This task affects almost nothing but more accurate error handling of sms db. Any other reasons?

	Andrew Overholt [:overholt]
	Comment 3 • 12 years ago

(In reply to Vicamo Yang [:vicamo][:vyang] from comment #2)
> (In reply to Andrew Overholt [:overholt] from comment #1)
> > blocking-b2g: --- → tef?
> 
> Why does this issue deserve a tef?? This task affects almost nothing but
> more accurate error handling of sms db. Any other reasons?

This is marked as blocking bug 823010 which in turns blocks bug 808607.  If this doesn't actually block bug 823010 I'm more than happy to remove the tef? :)

	Gene Lian (I already quit Mozilla)
	Comment 4 • 12 years ago

Since the DB is shared for SMS and MMS now, the MMS DB callbacks would have the same issue. Please refer to nsIRilMobileMessageDatabaseCallback.

	at-kitamura
	Comment 5 • 11 years ago

Hi all,

I think that not only the checking event.target.errorCode but also the notifying user has to consider it about this bug.
Because the user cannot notice the reception of the message by the present implementation.
(And error continues being repaid to a server.)

In addition, I think that I have to notify a user without transmitting a message when preservation to DB failed.

Thanks.

	Ken Chang[:kenkai]
	Comment 6 • 11 years ago

It is necessary for Madai project. Set 1.4+.

	Ken Chang[:kenkai]
	Comment 7 • 11 years ago

Bevis, after discussing with partners, this is a 1.4+ bug. Please help to fix it.

	Ken Chang[:kenkai]
	Comment 8 • 11 years ago

To change internal ril interface.

	Ken Chang[:kenkai]
	Comment 9 • 11 years ago

Blocking LTE feature.

	Ken Chang[:kenkai]
	Comment 10 • 11 years ago

This is a 1.4+ bug.

	Szu-Yu Chen [:aknow] (inactive)
	Comment 11 • 11 years ago

Take from Bevis.

	Bevis Tseng[:bevis][:btseng](Exited)
	Comment 12 • 11 years ago

After confirmed with Szu-Yu, I'd like to take back from Szu-Yu to follow up.

	Bevis Tseng[:bevis][:btseng](Exited)
	Comment 13 • 11 years ago

For checking the error condition of full storage,
the alternative way is to provide a storage monitor to observe "disk-space-watcher" notification.
Then, always consult the storage monitor before storing sms/mms.

This will be consistent to what user aware from the notification tray and will be helpful to support RIL_REQUEST_REPORT_SMS_MEMORY_STATUS mentioned in bug#739143 in the future.

I'd prefer to apply the alternative one to support the feature in bug#891344 instead.

	Bevis Tseng[:bevis][:btseng](Exited)
	Comment 15 • 11 years ago

Pre-condition:
1. According to bug#861903 and bug#853350, we know that when device storage is low, indexedDB only allows us for read access.
2. While testing, I found that event.target.errorCode is undefined when we access the indexedDB in storage full condition. No proper error code is available to identify the problem.

Summary of the patch:
1. The main purpose is to add error handling of sending/receiving message while device storage is full.
2. Hence, instead of checking event.target.errorCode, the solution is to apply DiskSpaceWatcher.isDiskFull before any |Write| access to the MobileMessageDatabase and notify STORAGE_FULL_ERROR to the caller.

	Vicamo Yang [:vicamo][:vyang] Reporter
	Comment 16 • 11 years ago

Comment on attachment 8378808 [details] [diff] [review]
Patch Part 1: Add new error type STORAGE_FULL_ERROR. r=vyang

Review of attachment 8378808 [details] [diff] [review]:
-----------------------------------------------------------------

This bug is not for storage full event.  Please move to either bug 891344 or bug 739143.

	Bevis Tseng[:bevis][:btseng](Exited)
	Comment 17 • 11 years ago

Per discussion offline, 
1. bug 891344 is dedicated for Gaia to fix the problem of 
sending/receiving message while message is full.
2. bug 739143 is more about adopting UNSOLICITED_SIM_SMS_STORAGE_FULL & 
   RIL_REQUEST_REPORT_SMS_MEMORY_STATUS for modem.

I'll fire another bug to provide the fix in gecko side for the problem of bug 891344.

	Bevis Tseng[:bevis][:btseng](Exited)
	Comment 18 • 11 years ago

No longer a 1.4+ blocking bug.
Check Bug 974820 instead.

	Ken Chang[:kenkai]
	Comment 19 • 11 years ago

Put this bug into backlog.

	BMO Automation
	Comment 20 • 7 years ago

Firefox OS is not being worked on