Closed
Bug 726189
Opened 12 years ago
Closed 12 years ago
get rid of search engine "used" attribute, since it causes unnecessary I/O
Categories
(Firefox :: Search, defect)
Firefox
Search
Tracking
()
RESOLVED
FIXED
Firefox 15
People
(Reporter: Gavin, Assigned: raymondlee)
Details
Attachments
(1 file, 1 obsolete file)
2.77 KB,
patch
|
Details | Diff | Splinter Review |
Bug 467890 added a "used" annotation in engineMetaDataService to support tracking the first-use of default search plugins, so that we can trigger an update check. We haven't made use of that functionality (bug 511017), and in the mean time, this is causing us to hit the disk the first time a default search engine is used. and also increasing the odds of needing to write metadata, since it's the only use of the engineMetaDataService that's triggered by a common user action (simply using the search bar). I think we should get rid of the "used" attribute for the moment. It's easy to re-introduce if we ever do end up investing in the work required to complete bug 511017.
Reporter | ||
Comment 1•12 years ago
|
||
The changeset that added it: http://hg.mozilla.org/mozilla-central/rev/ecba698e87b5
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #625906 -
Flags: review?(gavin.sharp)
Reporter | ||
Comment 3•12 years ago
|
||
Comment on attachment 625906 [details] [diff] [review] v1 Thanks!
Attachment #625906 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Comment 4•12 years ago
|
||
Pushed to try and waiting for results https://tbpl.mozilla.org/?tree=Try&rev=84b3c632fa1d
Assignee: nobody → raymond
Status: NEW → ASSIGNED
Assignee | ||
Updated•12 years ago
|
Attachment #625906 -
Attachment is obsolete: true
Assignee | ||
Comment 5•12 years ago
|
||
Passed Try
Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Comment 6•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/611663a9cb81
Keywords: checkin-needed
Target Milestone: --- → Firefox 15
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/611663a9cb81
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•