958087 - nsAndroidHistory::SetURITitle is called for iframes and embeds

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

Since DocShell calls SetURITitle for iframes and other embedded content, we try to save titles to the DB, even though it will fail. For cnn.com, for example, this leads to 15 SetURITitle calls. We try to update the DB 15 times, and it can take ~100ms on some mid range phones.

Desktop keeps a list of recent "emdbed" URIs in it's History implementation. If a URI is in the "embed" list they do not try to save the title change. We could do something similar.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: Ignore embed URIs v0.1

This patch mimics what desktop does, as best we can. We allow AndroidHistory to cache the last 128 non-toplevel URIs. We use that list to ignore SetURITitle changes.

I was profiling an HTC Desire HD, loading cnn.com, GlobalHistory.update() times:
before: 98ms
after: 10ms

Pages that have a lot of iframes, mainly for social buttons and ads, are affected more than nice mobile pages.

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

Comment on attachment 8357812 [details] [diff] [review]
Ignore embed URIs v0.1

Review of attachment 8357812 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/components/build/nsAndroidHistory.cpp
@@ +132,5 @@
> +  bool equals = false;
> +  EmbedArray::index_type i;
> +  EmbedArray::size_type length = mEmbedURIs.Length();
> +  for (i = 0; i < length && !equals; ++i) {
> +    aURI->Equals(mEmbedURIs.ElementAt(i), &equals);

As I mentioned on irc, we might want to make this a sorted list and do a binary search.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

https://hg.mozilla.org/integration/fx-team/rev/75f5cbf9c085

If we decide to add the sorting, we should try it for the RecentlyVisitedURIs array too.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/75f5cbf9c085