If this was lost in MD, that's probably unintentional.

I think Evan was the person refactoring the content settings icon code for MD?

Are you sure that it worked in m52 or is that conjecture? I found the fix but it looks to me like it's been broken since 2013 or so.

Works for me in M-52.

Deleted: M-52.png 21.3 KB

	M-52.png 21.3 KB View Download

I did not actually check Chrome 52, I just disabled the material design in Chrome 55 canary and it worked. Since Chrome 53 brought material design to everyone (or most of the users?), I let myself state Chrome 52 as working, even though I did not really try it. Good enough, though, right? :)

Then how it broke may remain a mystery not worth solving. The fix is incoming one way or another.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/31c20ef1d76caf6e6f1a5489c30ef5136d56cf37

commit 31c20ef1d76caf6e6f1a5489c30ef5136d56cf37
Author: estade <estade@chromium.org>
Date: Fri Sep 09 19:35:47 2016

Fix tooltip text for ContentSettingImageView.

The image is non-interactive, so it doesn't display the tooltip. Set the tooltip on the ContentSettingImageView instead.

BUG= 645140 

Review-Url: https://codereview.chromium.org/2323973002
Cr-Commit-Position: refs/heads/master@{#417670}

[modify] https://crrev.com/31c20ef1d76caf6e6f1a5489c30ef5136d56cf37/chrome/browser/ui/views/location_bar/content_setting_image_view.cc
[modify] https://crrev.com/31c20ef1d76caf6e6f1a5489c30ef5136d56cf37/chrome/browser/ui/views/location_bar/content_setting_image_view.h

Looks like the mixed content icon also does not have a tooltip (the one that opens a bubble about scripts from unauthenticated sources and lets you "Load unsafe scripts"). Do you want me to create an issue for it (can you just make sure all of the icons have tooltips?)?

As a tangent, the icons are so small that their meaning are entirely unclear to me. I would not have to look for the tooltip if the icons were recognizable.
I think that little red-and-white x is covering a lot of the already-small icon and makes it unclear, but the problem may be present even without it.

Are you saying it doesn't have a tooltip even with the patch that landed above? i.e. you can now see one for cookies but not mixed content? The fix should apply to all content settings icons.

I have not tested your change yet (does the latest canary have it?). However, I am not sure the mixed content icon is related to content settings (it is not configurable by the content settings section of the settings).

yes, it is.

https://cs.chromium.org/chromium/src/components/content_settings/core/common/content_settings_types.h?rcl=1473664647&l=31

 Issue 649028  has been merged into this issue.