Closed Bug 909245 Opened 11 years ago Closed 6 years ago

[Activity] Generalize Activity Error Reason

Categories

(Firefox OS Graveyard :: General, defect)

Product:
Firefox OS Graveyard
Component:
General
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: alive, Unassigned)

References

Details

There're four error message name of web activities now:
* NotUserInput
* USER_ABORT
* NO_PROVIDER
* ActivityCanceled

These names are inconsistent, which may let developer confuse when they want to diagnose the reason why activity is failed.

And AFAIK, activity.postError send back the message written by the user app. Isn't this overlapped with gecko's error message?

Fabrice, thought?
Flags: needinfo?(fabrice)
I agree that we should not have THIS_KIND_OF_ERRORS and move everything to CamelCase. As for the messages themselves, gecko does not override messages sent by apps, but can send some by itself.

I don't have a strong opinion on turning all the gecko ones to be a single "SystemCancel". If having different messages confuses developers, let's change that.
Flags: needinfo?(fabrice)
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.