Closed
Bug 937730
Opened 12 years ago
Closed 11 years ago
Contacts should not have avatar images for Eideticker
Categories
(Testing Graveyard :: Eideticker, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wlach, Assigned: davehunt)
References
Details
(Keywords: perf, Whiteboard: [c=automation p= s=2014.01.17 u=])
Attachments
(1 file)
1.39 KB,
patch
|
wlach
:
review+
|
Details | Diff | Splinter Review |
According to :bkelly, contact avatars are designed to be loaded asynchrously, and we should not count on them appearing in a timely fashion when the contacts app starts up.
To stabilize the load numbers for eideticker, let's just remove any images for contacts avatars for now.
Updated•12 years ago
|
Assignee | ||
Comment 1•12 years ago
|
||
The reference workloads [1] we use have contact images, so we'll either need and alternative workload or see if we can simply omit the pictures. Jon: Are you aware of any issues that might come from using the contacts database without the pictures?
[1] https://github.com/mozilla-b2g/gaia/tree/master/test_media/reference-workload
Flags: needinfo?(jhylands)
Comment 2•12 years ago
|
||
If you simply delete the numbered files in the subdirectory with the same name as the database, the pictures will not show up. I assume they will still have an impact, because it will try and load each one and fail, but I suspect it will be a much lower impact than loading the actual images.
Flags: needinfo?(jhylands)
Reporter | ||
Comment 3•12 years ago
|
||
So I believe the solution we decided was to modify eideticker to have load and/or modify its workload at the beginning of each test. Dave is working on this at the moment.
Assignee | ||
Comment 4•12 years ago
|
||
Once the blocking bugs are fixed, we'll need to patch eideticker to use the latest version of b2gpopulate and pass the argument so that the contact pictures are not populated.
Assignee | ||
Comment 5•11 years ago
|
||
Attachment #8356581 -
Flags: review?(wlachance)
Reporter | ||
Comment 6•11 years ago
|
||
Comment on attachment 8356581 [details] [diff] [review]
Do not include pictures when populating contacts. v1.0
Thanks!
Attachment #8356581 -
Flags: review?(wlachance) → review+
Reporter | ||
Comment 7•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Whiteboard: [c=automation p= s= u=] → [c=automation p= s=2014.01.17 u=]
Updated•8 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•