[MD settings] Scale avatar icon.

chrome/browser/resources/settings/people_page/people_page.html

Index: chrome/browser/resources/settings/people_page/people_page.html
diff --git a/chrome/browser/resources/settings/people_page/people_page.html b/chrome/browser/resources/settings/people_page/people_page.html
index 80a0c9f23b793aff165185514de6eec598430437..2543a127655e5682ca29d6693f5e04cfdf71cd4a 100644
--- a/chrome/browser/resources/settings/people_page/people_page.html
+++ b/chrome/browser/resources/settings/people_page/people_page.html
@@ -45,7 +45,7 @@
         background-position: center;
         background-repeat: no-repeat;
         background-size: cover;
-        border-radius: 20px;
+        clip-path: circle(19px at center);

[tommycli, 2017/03/21 22:29:53]

Looks like the root cause is that the data URL image is 38x31, and not quite
centered horizontally.

Since you actually need a square, I recommend changing the call to
GetAvatarIconForWebUI within ProfileInfoHandler to GetSizedAvatarIcon, and just
specify the 40x40 image you actually want.

I think that will fix this problem, as well as the image blurriness.

In fact -- I'm not entirely sure why we even do the image scaling within C++...
it seems entirely necessary. -- but that's a sidebar.

[dschuyler, 2017/03/21 23:27:15]

That's true, it's documented in the bug (in comment #9).
 
That appears to work too. Doing the scaling c++ is ok with me if you prefer.
 
It doesn't actually. It's about the same as patch #1.
With this new change I think we are now scaling the image -- i.e. I'm not sure
if I follow. We can chat about it off-line if it's not critical to this CL.

done.

         flex-shrink: 0;
         height: 40px;
         width: 40px;