Closed
Bug 338045
Opened 18 years ago
Closed 18 years ago
safe browsing slows down page load in non-enhanced mode
Categories
(Toolkit :: Safe Browsing, defect)
Toolkit
Safe Browsing
Tracking
()
RESOLVED
FIXED
People
(Reporter: tony, Assigned: tony)
Details
(Keywords: fixed1.8.1)
Attachments
(2 files, 1 obsolete file)
15.30 KB,
patch
|
brettw
:
review+
bryner
:
approval-branch-1.8.1+
|
Details | Diff | Splinter Review |
15.20 KB,
patch
|
Details | Diff | Splinter Review |
We spend a considerable amount of time processing the url on page load. We don't need to do that upfront, so we should defer until later.
Assignee | ||
Comment 1•18 years ago
|
||
Use a G_Alarm to defer the local list lookup until after docnavstart. Also break a JS circular reference between MultiTableQuerier and the callback. Start to refactor the code included in browser.xul to be namespaced in a single object.
Attachment #222091 -
Flags: review?(brettw)
Attachment #222091 -
Flags: approval-branch-1.8.1?
Assignee | ||
Updated•18 years ago
|
Attachment #222091 -
Flags: approval-branch-1.8.1? → approval-branch-1.8.1?(bryner)
Assignee | ||
Comment 2•18 years ago
|
||
Same as the last patch except we remove some abstraction layers (move webprogresslistener directly into phishing warden and get rid of EventRegistrar).
Attachment #222091 -
Attachment is obsolete: true
Attachment #222116 -
Flags: review?(brettw)
Attachment #222116 -
Flags: approval-branch-1.8.1?(bryner)
Attachment #222091 -
Flags: review?(brettw)
Attachment #222091 -
Flags: approval-branch-1.8.1?(bryner)
Updated•18 years ago
|
Attachment #222116 -
Flags: approval-branch-1.8.1?(bryner) → approval-branch-1.8.1+
Updated•18 years ago
|
Attachment #222116 -
Flags: review?(brettw) → review+
Assignee | ||
Comment 3•18 years ago
|
||
tweak web progress listener to listen for only document requests rather than all requests
Comment 4•18 years ago
|
||
Fixed on trunk and 1.8 branch.
Updated•10 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•