1520907 - Rename QueryContext to UrlbarQueryContext

Status: RESOLVED FIXED

(Firefox :: Address Bar, enhancement, P3)

Description

[Drew Willcoxon :adw]

Most class names start with "Urlbar", but QueryContext doesn't, so we should rename it to be consistent. Marco suggests UrlbarContext -- leaving out "Query" -- since we've added many properties to it and some aren't necessarily strictly related to a query, and the term "query" is used elsewhere and may be confusing.

I'm marking this blocking the main quantumbar bug, but maybe it's better for it to block the three model, view, and controller bugs? Feel free to change.

Comment 1

[Drew Willcoxon :adw]

Attachment: Bug 1520907 - Rename QueryContext to UrlbarContext.

There are still lots of local uses of "query context" and "queryContext", and I didn't change them. In some places those terms still makes sense because the code is clearly dealing with a query/search. In other places they make less sense, but this patch ended up being a lot bigger than I expected and I didn't really want to change more blame for this.

Comment 2

[Mike de Boer [:mikedeboer]]

I think the UrlbarQueryContext name was nodded to, because UrlbarContext is yet again too generic. Just sayin'.

Comment 3

[Dão Gottwald [::dao]]

(In reply to Mike de Boer [:mikedeboer] from comment #2)

I think the UrlbarQueryContext name was nodded to, because UrlbarContext is yet again too generic. Just sayin'.

Yeah, I too think UrlbarContext is too ambiguous.

Comment 4

[Drew Willcoxon :adw]

I thought we ended up on UrlbarContext for the reasons in comment 0?

Comment 5

[Mark Banner (:standard8)]

I'm in two minds about this. In some ways "Context" isn't the context for the whole urlbar (if it was, I think I'd expect it to be a singleton), it is the context for a particular query / search results. The fact it does contain other items I'm not sure matters too much - they're context for the search, and a different view could possibly vary these depending on what it fancies (silly or not(?) idea - time of day).

Comment 6

[Drew Willcoxon :adw]

OK, it sounds like most of us think Query should be in the name. I think I do too.

Ayes: Mike, Dão, Me, 1/2 of Mark
Noes: Marco (I think -- maybe I misunderstood?), 1/2 of Mark

I'll post a new revision using UrlbarQueryContext.

Comment 7

[Drew Willcoxon :adw]

Attachment: Bug 1520907 - Rename QueryContext to UrlbarQueryContext.

Comment 8

[Marco Bonardo [:mak]]

I'm fine either way, go for it.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/efa95aa6623a
Rename QueryContext to UrlbarQueryContext. r=Standard8

Comment 10

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/efa95aa6623a