ash: HiDPI user avatar for SessionController

chrome/browser/ui/ash/session_controller_client.cc

Index: chrome/browser/ui/ash/session_controller_client.cc
diff --git a/chrome/browser/ui/ash/session_controller_client.cc b/chrome/browser/ui/ash/session_controller_client.cc
index 4e503c120a7db267f4db2277d63fa3dc267192d6..ffc85f44c17e10f2607d45cc5d9247cdec4eb2cd 100644
--- a/chrome/browser/ui/ash/session_controller_client.cc
+++ b/chrome/browser/ui/ash/session_controller_client.cc
@@ -62,12 +62,13 @@ ash::mojom::UserSessionPtr UserToUserSession(const User& user) {
   session->display_name = base::UTF16ToUTF8(user.display_name());
   session->display_email = user.display_email();
 
-  // TODO(xiyuan): Support multiple scale factor.
-  session->avatar = *user.GetImage().bitmap();
+  // Make sure all representations are loaded before sending via mojo.

[msw, 2017/03/23 20:47:51]

Hmm, I wonder if there's a way to ensure we always do this; can this be moved
into the struct traits? (also, do we need to call SetReadOnly() or
MakeThreadSafe() before allowing shared memory access?)

[xiyuan, 2017/03/24 05:45:19]

I left this part out of traits because only caller knows exactly what should be
transferred and can limit memory usage by only loading what are needed. However,
I agree it would make it error prone and harder to write code.

Moved EnsureRepsForSupportedScales() to struct trait. 

MakeThreadSafe() is not need to for using shared memory. A SkBitmap in
ImageSkia/ImageSkiaRep is immutable [1] so copying its pixel data should not be
problem.

Also MakeThreadSafe() has a side effect that removes ImageSkiaSource from
storage. This might not be the right thing. A few ImageSkia is implemented to
get representation asynchronously, e.g extensions::IconImage::Source[2].
EnsureRepsForSupportedScales would trigger the load and the ImageSkia would be
filled with correct reps later. But if MakeThreadSafe is used, the source is
deleted at the call and we get stub image reps instead.

[1]:
https://cs.chromium.org/chromium/src/ui/gfx/image/image_skia_rep.cc?rcl=d62fe...
[2]:
https://cs.chromium.org/chromium/src/extensions/browser/extension_icon_image....

+  user.GetImage().EnsureRepsForSupportedScales();
+
+  session->avatar = user.GetImage();
   if (session->avatar.isNull()) {
-    session->avatar = *ResourceBundle::GetSharedInstance()
-                           .GetImageSkiaNamed(IDR_PROFILE_PICTURE_LOADING)
-                           ->bitmap();
+    session->avatar = *ResourceBundle::GetSharedInstance().GetImageSkiaNamed(
+        IDR_PROFILE_PICTURE_LOADING);
   }
 
   return session;