Last Comment Bug 726189 - get rid of search engine "used" attribute, since it causes unnecessary I/O
: get rid of search engine "used" attribute, since it causes unnecessary I/O
Product: Firefox
Classification: Client Software
Component: Search (show other bugs)
: Trunk
: All All
-- normal (vote)
: Firefox 15
Assigned To: Raymond Lee [:raymondlee]
: Florian Quèze [:florian] [:flo] (PTO until February 27)
Depends on:
  Show dependency treegraph
Reported: 2012-02-10 15:06 PST by :Gavin Sharp [email:]
Modified: 2012-05-26 05:21 PDT (History)
3 users (show)
ryanvm: in‑testsuite-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

v1 (2.49 KB, patch)
2012-05-21 23:45 PDT, Raymond Lee [:raymondlee] review+
Details | Diff | Splinter Review
Patch for checkin (2.77 KB, patch)
2012-05-23 21:00 PDT, Raymond Lee [:raymondlee]
no flags Details | Diff | Splinter Review

Description User image :Gavin Sharp [email:] 2012-02-10 15:06:04 PST
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.
Comment 1 User image :Gavin Sharp [email:] 2012-02-29 12:15:57 PST
The changeset that added it:
Comment 2 User image Raymond Lee [:raymondlee] 2012-05-21 23:45:44 PDT
Created attachment 625906 [details] [diff] [review]
Comment 3 User image :Gavin Sharp [email:] 2012-05-22 11:10:16 PDT
Comment on attachment 625906 [details] [diff] [review]

Comment 4 User image Raymond Lee [:raymondlee] 2012-05-23 21:00:17 PDT
Created attachment 626686 [details] [diff] [review]
Patch for checkin

Pushed to try and waiting for results
Comment 5 User image Raymond Lee [:raymondlee] 2012-05-24 20:27:12 PDT
Passed Try
Comment 6 User image Daniel Holbert [:dholbert] (vacation, returning 2/27) 2012-05-25 12:30:02 PDT
Comment 7 User image Ryan VanderMeulen [:RyanVM] 2012-05-26 05:21:02 PDT

Note You need to log in before you can comment on or make changes to this bug.