1177124 - Hide title/favicon title in the doorhanger if they don't exist

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Chenxia Liu [:liuche]]

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

Comment 1

[Chenxia Liu [:liuche]]

Attachment: Screenshot: Doorhanger with blank title/favicon

Margaret, want to upload your xpi for testing?

Comment 2

[:Margaret Leibovic]

(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

Comment 3

[Sebastian Kaspari (:sebastian; :pocmo)]

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

Comment 4

[Sebastian Kaspari (:sebastian; :pocmo)]

Attachment: 1177124-doorhanger-title.patch

Comment 5

[Chenxia Liu [:liuche]]

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?)

Comment 6

[Sebastian Kaspari (:sebastian; :pocmo)]

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

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/0bfda075f484