Enable history search on master

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: daleharvey, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [systemsfe])

Attachments

(1 attachment)

Comment hidden (empty)
(Reporter)

Comment 1

4 years ago
Probably in search
Component: Gaia::System::Browser Chrome → Gaia::Search
(Reporter)

Updated

4 years ago
Blocks: 948303
(Reporter)

Updated

4 years ago
Depends on: 968156
(Reporter)

Updated

4 years ago
Depends on: 1041623
(Reporter)

Updated

4 years ago
No longer depends on: 968156
(Reporter)

Updated

4 years ago
Depends on: 968156
See Also: → bug 1041985
Duplicate of this bug: 1041985
I have a patch in progress here: https://github.com/mozilla-b2g/gaia/pull/22024

Not sure if I will continue to work on this or not though as I'm not sure if we can turn this on yet due to the whole MozActivity thing. Let's discuss options today.
Created attachment 8460161 [details] [review]
Pull request - enable places in master (no haida flag)

Hey - can you guys review this? I understand that we may want to block this until we have the icon sizes list fixed, in the meantime I think we should have this reviewed so we can be ready to go.

Also we may consider just wiping the places DB before launch, so maybe it's not a big deal to land early?
Attachment #8460161 - Flags: review?(dale)
Attachment #8460161 - Flags: review?(21)
(Reporter)

Comment 5

4 years ago
So https://bugzilla.mozilla.org/show_bug.cgi?id=968156 is the bug I wanna block this on, its in review but data storage format has changed several times so would definitely prefer it to land prior to this, the places db is never persisted, but the datastore is

Checking this out now though, can review and land straight after
(Reporter)

Updated

4 years ago
Depends on: 1042116
(Reporter)

Comment 6

4 years ago
Comment on attachment 8460161 [details] [review]
Pull request - enable places in master (no haida flag)

Works well thanks, happy to land once the dependency does
Attachment #8460161 - Flags: review?(dale) → review+
Comment on attachment 8460161 [details] [review]
Pull request - enable places in master (no haida flag)

Will go with Dale's review for now. Vivien - feel free to leave any comments.
Attachment #8460161 - Flags: review?(21)
Enabled in master: https://github.com/mozilla-b2g/gaia/commit/b9240ad3dbb861c2e98150573ce6e4a64b300abb

Going to unblock 1041623 and 1042116, and block the parent user story with them.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
No longer depends on: 1041623, 1042116
Resolution: --- → FIXED
Whiteboard: [systemsfe]
hi Kevin, I think this patch introduced a small performance regression to the Calendar app (startup & moz-chrome-dom-loaded), do you think it's possible? see this comment for more info: https://bugzilla.mozilla.org/show_bug.cgi?id=1059349#c3 Thanks!
Flags: needinfo?(kgrandon)
Depends on: 1063789
(In reply to Miller Medeiros [:millermedeiros] from comment #9)
> hi Kevin, I think this patch introduced a small performance regression to
> the Calendar app (startup & moz-chrome-dom-loaded), do you think it's
> possible? see this comment for more info:
> https://bugzilla.mozilla.org/show_bug.cgi?id=1059349#c3 Thanks!

I've filed bug 1063789 to fix places performance.
Flags: needinfo?(kgrandon)
You need to log in before you can comment on or make changes to this bug.