Closed Bug 1024874 Opened 10 years ago Closed 10 years ago

[System] Use shared uuid.js

Categories

(Firefox OS Graveyard :: Gaia::System, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
2.0 S4 (20june)

People

(Reporter: kgrandon, Assigned: kgrandon)

Details

(Whiteboard: [p=1],[systemsfe])

Attachments

(1 file)

46 bytes, text/x-github-pull-request
timdream
: review+
Details | Review
In bug 1022976 we decided to move uuid into shared/ for use across apps. The system app uses the same file, so to reduce confusion let's move it.
Whiteboard: [p=1],[systemsfe]
Attached file Github pull request
Comment on attachment 8439712 [details] [review]
Github pull request

A fairly simple change to use the shared uuid.js which we landed for the vertical homescreen. (Calendar also uses this library)
Attachment #8439712 - Flags: review?(timdream)
Attachment #8439712 - Flags: review?(alive)
Comment on attachment 8439712 [details] [review]
Github pull request

Not sure if it's possible but do we have unit test for in this script?
Attachment #8439712 - Flags: review?(timdream)
Attachment #8439712 - Flags: review?(alive)
Attachment #8439712 - Flags: review+
(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TPE) (please ni?) from comment #3)
> Comment on attachment 8439712 [details] [review]
> Github pull request
> 
> Not sure if it's possible but do we have unit test for in this script?

It's an external script, and it seems pretty hard, but we may be able to create some. I do know we have several unit tests and marionette tests that rely on this script, and would fail if it breaks in some cases.


Landed: https://github.com/mozilla-b2g/gaia/commit/1396f543d02bb2908a9243451bd6ee6fb4a08b8b
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.0 S4 (20june)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: