search

chromium / suspicious-site-reporter

Extension for reporting suspicious sites to Safe Browsing.
Apache License 2.0
96 stars 22 forks source link

Performance: Simplify UX and improve performance #42

Open ericlaw1979 opened 5 years ago

ericlaw1979 commented 5 years ago

Opening the popup window can take quite a bit of time. The popup performs expensive operations onload:

      popup.generateScreenshotPreview_(currentTab);
      popup.retrieveDom_(currentTab);
      popup.generateReferrerChainPreview_(currentTab);

...even in the common case where this data isn't going to be used because the user isn't going to submit a report.

It might be useful to have a simpler UX, whereby we initially show the user the alerts with only a "Report this site" button below. When the user clicks that button, we swap the UI for a reporting UI which computes the report content (screenshot, dom, and referrer chain) for the user to review before they click the "Send report" button.