[SCR][Rocketbar] Can we do a reduction of flashing via a comparison with this found set versus last found set?

RESOLVED FIXED

Status

Firefox OS
Gaia::Search
--
enhancement
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: nhirata, Unassigned)

Tracking

unspecified
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(b2g-v2.0 affected, b2g-v2.1 affected)

Details

(Whiteboard: [perf-reviewed])

1. tap Rocketbar
2. type t
3. type e
4. type s
5. type t

Expected: It wouldn't refresh so much after each type because it does a compare/diff from the last found set

Actual: refreshes each time there's a letter change

Gaia      83844c7679b3b9f6e7f1116c1eeec2d1e7a64eec
Gecko     https://hg.mozilla.org/releases/mozilla-aurora/rev/55679dc2e72b
BuildID   20140618000202
Version   32.0a2
ro.build.version.incremental=108
ro.build.date=Tue Jun 10 19:40:40 CST 2014
flame

Marking as an enhancement

Updated

4 years ago
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking-]
To go with the previous perf issue.
status-b2g-v2.0: --- → affected
status-b2g-v2.1: --- → affected
Flags: needinfo?(kgrandon)
Keywords: perf
I would assume that we should be able to leverage platform caching here. We should do an investigation to see why we currently wouldn't be leveraging that. Keeping a manual cache in javascript seems to be asking for problems, but maybe we can investigate it.
Flags: needinfo?(kgrandon)

Comment 3

4 years ago
Removing perf keyword as the flashing described is a functional issue with Rocketbar's implementation and not a performance issue.
Keywords: perf
Whiteboard: [perf-reviewed]
(In reply to Kevin Grandon :kgrandon from comment #2)
> I would assume that we should be able to leverage platform caching here. We
> should do an investigation to see why we currently wouldn't be leveraging
> that. Keeping a manual cache in javascript seems to be asking for problems,
> but maybe we can investigate it.

In the browser app we kept a cache of icons in IndexedDB and a "result cache" in memory as you type. It actually worked quite well.

I think the challenges for the search app are a bit different because most of the results currently come remotely from EverythingMe and Firefox Marketplace rather than from local browsing history, but I don't think relying on the platform's browser cache is working too well here.

We need to do something smarter because this is currently painful to use, especially on a Flame configured for 319MB. Part of the solution might be a different UI, which UX are currently exploring.
Refreshing is not an issue with the current one line implementation
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.