276318 - Context menu search does not obey "browser.tabs.loadInBackground" pref (tabs are not selected)

Status: VERIFIED FIXED

(Firefox :: Search, defect)

Description

[:Gavin Sharp [email: [e-mail]]]

The pref is supposed to be checked at:
http://lxr.mozilla.org/mozilla/source/browser/base/content/browser.js#2409

but the pref variable is not defined. Patch coming up.

Comment 1

[:Gavin Sharp [email: [e-mail]]]

Attachment: Patch

Comment 2

[O. Atsushi (Torisugari)]

(In reply to comment #0)
> but the pref variable is not defined.

It seems that gPrefService is defined as expected.
http://lxr.mozilla.org/mozilla/source/browser/base/content/browser.js#696

Comment 3

[:Gavin Sharp [email: [e-mail]]]

Attachment: Patch 2

Major oopsies :). That was a little presumptuous of me wasn't it? Thanks to
Callek and Torisugari for pointing out my mistake.

Comment 4

[Asaf Romano (gone)]

Checking in browser/base/content/browser.js;
/cvsroot/mozilla/browser/base/content/browser.js,v  <--  browser.js
new revision: 1.375; previous revision: 1.374
done

Comment 5

[:Gavin Sharp [email: [e-mail]]]

*** Bug 251777 has been marked as a duplicate of this bug. ***