Comment 5 for bug 1452766

Understood.

We should definitely let users bookmark pages before they finish loading.

I think in cases where the page isn't yet known, we can use the URL as the bookmark name/ in place of a page title (I believe bookmarks currently work this way already).
--- --- ---
Also, when I experimented with this yesterday I found that the interval before page loading is considered to have failed was very long (certainly long enough to appear to 'never finish loading'):

1. Enter URL for large website with lots of images (e.g. theverge.com).
2. Before page loads, switch to airplane mode.
3. Page loading appears to continue without end.
4. 'Favourite/Star' icon is never shown in address bar

Recommended behaviour:
1. Enter URL for large website with lots of images (e.g. theverge.com).
2. Before page loads, switch to airplane mode.
3. After a reasonable period (15-30 seconds) 'Network Error' page is shown.
4. 'Favourite/Star' icon is available in address bar throughout.