Closed Bug 982152 Opened 12 years ago Closed 8 years ago

[Contacts] Transition search adapters to data objects

Categories

(Firefox OS Graveyard :: Gaia::Contacts, defect, P4)

Product:
Firefox OS Graveyard
Component:
Gaia::Contacts
Platform:
ARM
Gonk (Firefox OS)
Type:
defect

Tracking

(tracking-b2g:backlog)

Status:
RESOLVED WONTFIX
Project Flags:
tracking-b2g backlog

People

(Reporter: kgrandon, Unassigned)

References

Details

(Keywords: perf, Whiteboard: [c= p=5 s= u=] [priority])

Currently contact search adapters work on DOM objects, and the entire DOM needs to be loaded into the page to search all contacts. Instead of using DOM objects, we should back the source with a data model backed by normal objects. I would imagine that the performance should be about the same considering we will lose the cloneNode niceness, but gain some performance in other areas. This is also necessary for improving the performance of the main list as seen in bug 975680.
Blocks: 975680
Francisco, Jose - if you guys have any thoughts or concerns please share them. AFAIK the list and import are the two current consumers of search, are there more? List.js looks fairly easy to refactor and it seems import.js is more tricky.
Kevin, I'm assigning this to you since it blocks bug 975680 which blocks bug 967884, both assigned to you. Where are you with the proposal you outlined in this bug?
Assignee: nobody → kgrandon
Status: NEW → ASSIGNED
OS: Mac OS X → Gonk (Firefox OS)
Priority: -- → P1
Hardware: x86 → ARM
I'll comment in bug 967884. We have a prototype in bug 975680, but it has a long way to go and I don't think this approach is possible for me to do for 1.4. The graphics guys are looking at contact scroll today and have filed a number of other bugs, it may be possible that some of these could make contacts better, but I'm not sure that they will solve checkerboarding all together.
Whiteboard: [c= p= s= u=] → [c= p=5 s= u=]
Depends on: 982081
Blocks: 968098
blocking-b2g: --- → backlog
Whiteboard: [c= p=5 s= u=] → [c= p=5 s= u=][priority]
I might revisit this once bug 968098 is resolved.
Assignee: kgrandon → nobody
Whiteboard: [c= p=5 s= u=][priority] → [c= p=5 s= u=backlog][priority]
Priority: P1 → P4
Whiteboard: [c= p=5 s= u=backlog][priority] → [c= p=5 s= u=] [priority]
blocking-b2g: backlog → ---
tracking-b2g: --- → backlog
Firefox OS is not being worked on
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.