Correction in Chrome Version,

Chrome Version : 55.0.2883.11 (Official Build) 709e70a37ad5ea7e4155b38134d00530605f11bb-refs/branch-heads/2883@{#79} 64-bit

Adding RB label as this is a recent regression.

estade@, could you please take a look and fix this ASAP as this is marked as M55 stable blocker.

Thank you.

I guess this[1] isn't working as expected, or perhaps this is a

Elly, would you mind having a look while I'm ooo?

[1] https://cs.chromium.org/chromium/src/chrome/browser/ui/cocoa/website_settings/permission_bubble_controller.mm?rcl=0&l=517

@ellyjones: gentle Ping, can we have an update on this issue

Since this issue is marked as a stable blocker issue, can we get any latest update on it? Already M55 has moved to beta and we are planning to push M55 to stable version soon.

ellyjones@ is OOO. estade@ may be out for part of this week (can't tell) but if so he will be back before ellyjones@. Assigning back to him.

we're pushing m55 to stable soon? Calendar says early December.

On chrome 56.0.2899.0 canary (64-bit) on my laptop, I do get the icon. OSX 10.11.6


estade: Is it possible that https://codereview.chromium.org/2385063005 fixed this?

yea, that seems likely. I don't look forward to merging that :/

merge request for https://codereview.chromium.org/2385063005

**** Bulk edit -  please ignore if not applicable ****

A friendly reminder that M55 Stable is launch is coming soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!

Your change meets the bar and is auto-approved for M55 (branch: 2883)

Your change meets the bar and is auto-approved for M55 (branch: 2883)

Rechecked this on chrome canary version 56.0.2901.4 on MAC 10.11.6. Fix is working as intended. Able to see the Microphone icon in the permission bubble.

Need to verify on M55 branch once the merge is in.

Adding TE-Verified labels for Canary.

Thanks.!

merge seems to have gone through cleanly, thankfully.

estade@, could you please confirm whether CL listed at comment #10 (https://codereview.chromium.org/2385063005) is merged to M55? I don't see any bugdroid update for merge so double checking. Thank you.

ask whoever's in charge of bugdroid why it's flaky

here's the commit: https://chromium.googlesource.com/chromium/src/+/089cf6d16f3733ff8c78767c56bd8856e6695d09

Thank you estade@.

https://chromium.googlesource.com/chromium/src/+/089cf6d16f3733ff8c78767c56bd8856e6695d09 (listed BUG= 651270 ). And this bug is 655925, this is the reason we don't see bugdroid update in this bug.

If nothing is pending for M55, please remove "Merge-Approved-55" label and apply "merge-merged-2883" label. Thank you.

ah, makes sense

Verified the issue on chrome latest M55-55.0.2883.34 by followings steps mentioned in the original comment. Observed the microphone icon is present in the permission bubble as expected. Hence adding TE-Verified label.

Deleted: Screen Shot 2016-11-01 at 11.22.36 AM.png 20.7 KB

	Screen Shot 2016-11-01 at 11.22.36 AM.png 20.7 KB View Download