Fix the image size option in the content url

?sz={n} query parameter to specify the Google profile image size doesn't work. It may not be supported anymore. This change fixes it as well as adds logging in a couple of places.

Bug: 1019032
Change-Id: I4fe794ac5e8f04b3624de2d021dd2a526e565294
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1885346Reviewed-by: Tien Mai <tienmai@chromium.org>
Commit-Queue: Yusuf Sengul <yusufsn@google.com>
Cr-Commit-Position: refs/heads/master@{#710579}

chrome/credential_provider/gaiacp/password_recovery_manager.cc

@@ -202,7 +202,7 @@ class EscrowServiceRequest {
 
     HRESULT hr = requester->fetcher_->Fetch(&requester->response_);
     if (FAILED(hr))
-      LOGFN(INFO) << "fetcher.Fetch hr=" << putHR(hr);
+      LOGFN(ERROR) << "fetcher.Fetch hr=" << putHR(hr);
 
     requester->ProcessingDone();
     return 0;
@@ -710,6 +710,8 @@ HRESULT PasswordRecoveryManager::StoreWindowsPasswordIfNeeded(
         LOGFN(ERROR) << "StorePrivateData hr=" << putHR(hr);
         return hr;
       }
+
+      LOGFN(INFO) << "Encrypted and stored secret for sid=" << sid;
     } else {
       LOGFN(ERROR) << "base::JSONWriter::Write failed";
       return E_FAIL;
@@ -770,6 +772,8 @@ HRESULT PasswordRecoveryManager::RecoverWindowsPasswordIfPossible(
     *recovered_password = decrypted_password;
   SecurelyClearString(decrypted_password);
 
+  LOGFN(INFO) << "Decrypted the secret for sid=" << sid;
+
   return hr;
 }
 

chrome/credential_provider/gaiacp/scoped_user_profile.cc

@@ -325,8 +325,8 @@ HRESULT UpdateProfilePicturesForWindows8AndNewer(
       continue;
     }
 
-    std::string current_picture_url = base::UTF16ToUTF8(picture_url) +
-                                      base::StringPrintf("?sz=%i", image_size);
+    std::string current_picture_url =
+        base::UTF16ToUTF8(picture_url) + base::StringPrintf("=s%i", image_size);
 
     auto fetcher = WinHttpUrlFetcher::Create(GURL(current_picture_url));
     if (!fetcher) {