Closed Bug 943292 Opened 9 years ago Closed 9 years ago
Camera icon/popup for g
UM access can get hidden
STR: 1. Install Talkilla & Log in 2. Open the conversation window with someone 3. Start a call with that person 4. As the call starts, click to another application - before the permissions dialog is presented. Actual Results No popup appears, no camera icon in the toolbar of the conversation window Expected Results Grey camera icon is shown in the toolbar Reproduced on Nightly 28.0a1 2013-11-25, but I've been seeing this for a while.
Additional comment: when I click the area where the icon should be, the icon appears and the popup is displayed.
This patch fixes the bug reported by Mark for the SocialAPI chat windows. I'm not sure if this bug applies to the social sidebars too (how are we displaying permission icons for the sidebar? Is it in the toolbar button that we are in the process of removing?), nor of how notification icons are displayed for social status panels. Only the chat window case seems to matter for Talkilla, but when looking at the patch, you may want to check that it doesn't break anything for other cases; if they worked now. Requesting feedback from Shane who wrote in bug 809085 the code I'm modifying. Will request review from Felipe later if Shane says the patch looks reasonable.
Attachment #8338533 - Flags: feedback?(mixedpuppy)
Attachment #8338533 - Flags: feedback?(mixedpuppy) → feedback+
Attachment #8338533 - Flags: review?(felipc)
Attachment #8338533 - Flags: review?(felipc) → review+
The patch was a bit old, so I pushed it to try to be sure it didn't get broken: https://tbpl.mozilla.org/?tree=Try&rev=84bf8628d59f The results were green enough, so: https://hg.mozilla.org/integration/fx-team/rev/00b675eb6551
Assignee: nobody → florian
OS: Mac OS X → All
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
You need to log in before you can comment on or make changes to this bug.