Site icon from media control notification is not displayed on tablets

RESOLVED FIXED in Firefox 52

Status

()

Firefox for Android
Audio/Video
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: sorina, Assigned: sebastian)

Tracking

51 Branch
Firefox 52
ARM
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox51 wontfix, firefox52 verified)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

2 years ago
Created attachment 8791921 [details]
Screenshot_20160916-114554.png

Environment: 
Device: Nexus 9 (Android 6.0.1);
Build: Nightly 51.0a1 (2016-09-15);

Steps to reproduce:
1. Launch Firefox and play a video;
2. Go to the android notification bar;
3. Observe the media control notification.

Expected result:
Site icon is displayed on media control notification.

Actual result:
Site icon is not displayed on media control notification.

Notes:
Please see the attached screenshot.
This is reproducible also on Asus ZenPad 8 (Android 5.0.2) and Nexus 7 (Android 5.1.1).

Comment 1

2 years ago
Hi, Sebastian,
Could you help me check this icon issue?
Thanks :)
Flags: needinfo?(s.kaspari)
Sebastian is on PTO until Thursday.
(Assignee)

Comment 3

2 years ago
Okay, I'm back and can help debugging this.
Assignee: nobody → s.kaspari
Status: NEW → ASSIGNED
Flags: needinfo?(s.kaspari)
(Assignee)

Updated

2 years ago
status-firefox52: --- → affected
Comment hidden (mozreview-request)

Comment 5

2 years ago
mozreview-review
Comment on attachment 8794157 [details]
Bug 1303292 - Media notification: Use icons that have at least the target size.

https://reviewboard.mozilla.org/r/80724/#review79530
Attachment #8794157 - Flags: review?(ahunt) → review+

Comment 6

2 years ago
Pushed by s.kaspari@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/a574544a7c7e
Media notification: Use icons that have at least the target size. r=ahunt

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a574544a7c7e
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox52: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
(Reporter)

Updated

2 years ago
Depends on: 1305430
Mark 51 as won't fix as this will be shipped in 52.
status-firefox51: affected → wontfix
(Reporter)

Comment 9

2 years ago
Tested using:
Device: Nexus 6 (Android 6.0.1) and Asus ZenPad 8 (Android 5.0.2);
Build: Nightly 52.0a1 (2016-10-09);
Favicon from media control notification is displayed on tablets.
status-firefox52: fixed → verified
You need to log in before you can comment on or make changes to this bug.