Ensure that Synced Tab result appears above History result for the same item

RESOLVED FIXED in Firefox 60

Status

()

enhancement
P1
normal
RESOLVED FIXED
Last year
Last year

People

(Reporter: rfeeley, Assigned: mak)

Tracking

(Blocks 1 bug)

55 Branch
Firefox 60
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox60 fixed)

Details

(Whiteboard: [fxsearch])

Attachments

(1 attachment)

Reporter

Description

Last year
Currently when you enter in a fragment of a title or URL of a page that exists in history, and is also open on another device, the History item shows first.

We should probably bump up the prominence of the Synced Tab result, as an open tab is likely more relevant than History.
I've been meaning to file this bug myself! P1 assuming it's a simple weight-bumping change, but we can reconsider if it's more involved than that.
Blocks: 1425029
Component: Search → Address Bar
Priority: -- → P1
Whiteboard: [fxsearch]
Assignee

Comment 2

Last year
it's not a simple weight bump, because synced tabs come from an alien source that cannot mixed up with frecency. We can decide to present them before or after other results. We currently do that after, moving it before other results would be trivial. Though, I suppose it would be wrong to present this before an adaptive result (that comes from previous user selections), thus what we could do is: adaptive, remote tab, others. This should be easy enough to do.
Assignee

Updated

Last year
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Assignee

Updated

Last year
Blocks: 1263806
Assignee

Updated

Last year
Blocks: 1222435
Comment hidden (mozreview-request)
Assignee

Comment 4

Last year
I still have to check Try.
It's possible this somehow bitrots with the autofill patch. We can delay this in case.
Assignee

Updated

Last year
Attachment #8950924 - Flags: review?(adw)
Comment hidden (mozreview-request)

Comment 7

Last year
mozreview-review
Comment on attachment 8950924 [details]
Bug 1433938 - Move Synced Tab matches above general history matches in the Address Bar.

https://reviewboard.mozilla.org/r/220182/#review226996

::: toolkit/components/places/PlacesUtils.jsm:434
(Diff revision 2)
> +  parseActionUrl(url) {
> +    if (url instanceof Ci.nsIURI)
> +      url = url.spec;
> +    else if (url instanceof URL)
> +      url = url.href;
> +    if (!url.startsWith("moz-action:"))

If you're going to add a MOZ_ACTION_REGEX const/getter, then maybe add one for "moz-action:" that you can use here and above in mozActionURI()?

Actually why not just skip this startsWith() here and let the match() handle this case below?
Attachment #8950924 - Flags: review?(adw) → review+
Assignee

Comment 8

Last year
I'd just expect startsWith being more efficient than a regex. I didn't verify that though.
Comment hidden (mozreview-request)

Comment 10

Last year
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/6cfa62837419
Move Synced Tab matches above general history matches in the Address Bar. r=adw

Comment 11

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/6cfa62837419
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
Depends on: 1439787

Comment 12

Last year
Can reproduce this issue in Firefox 59.0.3 (20180427210249) on Windows 10x64.

This issue can no longer be reproduced on Firefox 61.0b1 (20180428110614) on Windows 10x64.

[bugday-20180502]
You need to log in before you can comment on or make changes to this bug.