946
edits
Places:Performance: Difference between revisions
Jump to navigation
Jump to search
no edit summary
No edit summary |
No edit summary |
||
| Line 1: | Line 1: | ||
Our history implementation can have a non-trivial impact on the browser's performance, not only when actively browsing history or bookmarks, but also in other areas such as (for example) page load. | Our history implementation can have a non-trivial impact on the browser's performance, not only when actively browsing history or bookmarks, but also in other areas such as (for example) page load, or browser startup / shutdown. | ||
In the case of page load, one reason for the performance hit is that links must be colored as visited / not visited when displaying a page, which requires going through history (see [https://bugzilla.mozilla.org/show_bug.cgi?id=329816 Bug #329816]). | = Known Problem Areas = | ||
From previous experience, the most-affected areas are outlined below. Special care should be taken to ensure there are no unacceptable regressions in them. However, there may be other performance implications that we do not yet know or anticipate (ideas, anyone?). Please update this page as new information (or new doubts/questions) come up. | |||
== Page Load == | |||
In the case of page load, one reason for the performance hit is that links must be colored as visited / not visited when displaying a page, which requires going through history (see [https://bugzilla.mozilla.org/show_bug.cgi?id=329816 Bug #329816]). | |||
== Browser Startup == | |||
Brett Wilson's experience here is that places somewhat regresses startup speed when there's no history, but significantly improves it when there is large history. | |||
== Browser Shutdown == | |||
Browser shutdown can be negatively impacted because expiring history is (or used to be) slow. | |||
== Bookmarks/History UI == | |||
The "Clear history" button can be slow. | |||
= Testing Environments = | = Testing Environments = | ||
| Line 27: | Line 45: | ||
= Test Data = | = Test Data = | ||
== Page Load == | |||
Brett Wilson spent a good chunk of time optimizing Places. This bug has some test data: [https://bugzilla.mozilla.org/show_bug.cgi?id=329816 Bug #329816]. Note that he times the IsVisited call directly, as the page load tests have too much noise to be useful. | Brett Wilson spent a good chunk of time optimizing Places. This bug has some test data: [https://bugzilla.mozilla.org/show_bug.cgi?id=329816 Bug #329816]. Note that he times the IsVisited call directly, as the page load tests have too much noise to be useful. | ||
| Line 47: | Line 67: | ||
Comments welcome. | Comments welcome. | ||
== Startup == | |||
TBD | |||
== Shutdown == | |||
TBD | |||
== Other == | |||
TBD | |||