Closed
Bug 961468
Opened 10 years ago
Closed 10 years ago
No need to check file existence twice in _readManifests
Categories
(Core Graveyard :: DOM: Apps, defect)
Core Graveyard
DOM: Apps
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla29
People
(Reporter: marco, Assigned: marco)
References
Details
(Whiteboard: [qa-])
Attachments
(1 file, 1 obsolete file)
1.91 KB,
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
I've just noticed that _readManifests is calling nsIFile::exists on the same file twice.
Attachment #8362206 -
Flags: review?(fabrice)
Updated•10 years ago
|
Attachment #8362206 -
Flags: review?(fabrice) → review+
Assignee | ||
Comment 1•10 years ago
|
||
This is an alternative patch that avoids completely to check for file existence in _readManifests (because it's already done by loadJSONAsync, that returns null if the file doesn't exist).
Assignee: nobody → mar.castelluccio
Status: NEW → ASSIGNED
Attachment #8367604 -
Flags: review?(fabrice)
Updated•10 years ago
|
Attachment #8367604 -
Flags: review?(fabrice) → review+
Assignee | ||
Updated•10 years ago
|
Attachment #8362206 -
Attachment is obsolete: true
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 2•10 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/a2c030eb1c50
Keywords: checkin-needed
Comment 3•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a2c030eb1c50
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Updated•10 years ago
|
Whiteboard: [qa-]
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•