For pages like file:/// that don't have a title/favicon, set the title/favicon doorhanger elements as GONE.

Margaret, want to upload your xpi for testing?

(In reply to Chenxia Liu [:liuche] from comment #1) > Created attachment 8625784 [details] > Screenshot: Doorhanger with blank title/favicon > > Margaret, want to upload your xpi for testing? I don't think this has to do with the XPI. Any XPI hosted locally on your file system would do it. You could just use the boilerplate add-on for this: https://github.com/mozilla/firefox-for-android-addons/tree/master/boilerplate/native-ui-boilerplate

This has annoyed me a lot recently while developing add-ons and APIs. I'm going to fix that. :)

Comment on attachment 8646920 [details] [diff] [review] 1177124-doorhanger-title.patch Review of attachment 8646920 [details] [diff] [review]: ----------------------------------------------------------------- One comment - up to you what you decide though. ::: mobile/android/base/DoorHangerPopup.java @@ +309,5 @@ > show(); > return; > } > > + final Bitmap favicon = tab.getFavicon(); I wonder if this should be in Doorhanger.showTitle - I think we can assume that if both the favicon is null and the string is empty, we don't want to just show a blank space. I'm not sure what we'd want to do if there is no title but a favicon exists - but that seems like an edge-enough case that I don't care one way or the other. (e.g., who remembers to add a favicon but forgets the title?)

url: https://hg.mozilla.org/integration/fx-team/rev/0bfda075f48486c4b6bdda5750cb307d59c1d3e9 changeset: 0bfda075f48486c4b6bdda5750cb307d59c1d3e9 user: Sebastian Kaspari <s.kaspari@gmail.com> date: Wed Aug 12 15:30:57 2015 +0200 description: Bug 1177124 - Doorhanger: Hide title/favicon if they don't exist. r=liuche