Closed Bug 1309604 Opened 3 years ago Closed 3 years ago

The webcam favicon is very faded when the call is in a background tab


(Firefox :: Theme, defect)

51 Branch
Not set



Firefox 52
52.3 - Nov 14
Tracking Status
firefox49 --- unaffected
firefox50 --- unaffected
firefox51 --- verified
firefox52 --- verified


(Reporter: cornel_ionce, Assigned: dao)



(Keywords: regression, Whiteboard: [fxprivacy])


(1 file)

- Screenshot:

[Affected versions]:
- latest 52.0a1 Nightly
- latest 51.0a2 Aurora

[Affected platforms]:
- Windows 10 x64
- Ubuntu 14.04 x86
- Mac OS X 10.12

[Steps to reproduce]:
1. Open Firefox
2. Go to
3. Select "Audio & Video" and share the devices.
4. Open a new tab.
5. Observe the favicon of the tab where the call is active.

[Expected result]:
- The favicon changes to a webcam and back.

[Actual result]:
- The webcam favicon is very faded compared to the one from the identity panel. See the screenshot for a better understanding of the issue.

[Regression range]:
- This is not a regression.
(In reply to Cornel Ionce [QA] (:cornel_ionce) from comment #0)

> [Regression range]:
> - This is not a regression.

Looks a lot like a regression to me, I'm pretty sure this sharing icon wasn't partially transparent when I landed it in bug 1275262. Could you please find the regression range?
Blocks: 1275262
Flags: needinfo?(cornel.ionce)
Whiteboard: [fxprivacy][triage]
Mozregression narrowed down the regression range to this push:

Bug 1304708 was uplifted to Aurora51 as well, so that seems like the most likely candidate.
Blocks: 1304708
Flags: needinfo?(cornel.ionce) → needinfo?(dao+bmo)
Version: Trunk → 51 Branch
The fix will likely be similar to what we did in bug 1303339.
Assignee: nobody → dao+bmo
Flags: needinfo?(dao+bmo)
Attached patch patchSplinter Review
Attachment #8802181 - Flags: review?(florian)
Comment on attachment 8802181 [details] [diff] [review]

Review of attachment 8802181 [details] [diff] [review]:

Attachment #8802181 - Flags: review?(florian) → review+
Pushed by
Stop using the fill filter for tab-sharing-icon-overlay, use designated sharing icon variants instead. r=florian
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
No longer blocks: 1206233, 1275262
Comment on attachment 8802181 [details] [diff] [review]

Approval Request Comment
[Feature/regressing bug #]: bug 1304708
[User impact if declined]: see comment 0
[Describe test coverage new/current, TreeHerder]: n/a
[Risks and why]: I missed updating this use case of notification-icons.svg in bug 1304708. Simple fix, low risk.
[String/UUID change made/needed]: none
Attachment #8802181 - Flags: approval-mozilla-aurora?
Comment on attachment 8802181 [details] [diff] [review]

Fix a regression related to webcam favicon. Take it in 51 aurora.

Hi :cornel_ionce,
Can you help to verify this?
Flags: needinfo?(cornel.ionce)
Attachment #8802181 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

I'm confirming the fix for this issue across platforms using:
* latest 51.0a2 Aurora, build ID 20161021004016
* latest 52.0a1 Nightly, build ID 20161020030211.
QA Contact: cornel.ionce
Iteration: --- → 52.3 - Nov 7
Whiteboard: [fxprivacy][triage] → [fxprivacy]
Flags: needinfo?(cornel.ionce)
You need to log in before you can comment on or make changes to this bug.