Adds more metadata to the about:webapks page

chrome/browser/ui/webui/webapks_handler.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/webapks_handler.h"
 
+#include <string>
+
 #include "base/callback_forward.h"
+#include "base/strings/stringprintf.h"
 #include "base/values.h"
 #include "chrome/browser/android/shortcut_helper.h"
 #include "content/public/browser/web_ui.h"
+#include "content/public/common/manifest_util.h"
 
 WebApksHandler::WebApksHandler() : weak_ptr_factory_(this) {}
 
 WebApksHandler::~WebApksHandler() {}
 
 void WebApksHandler::RegisterMessages() {
   web_ui()->RegisterMessageCallback(
       "requestWebApksInfo",
       base::Bind(&WebApksHandler::HandleRequestWebApksInfo,
                  base::Unretained(this)));
 }
 
 void WebApksHandler::HandleRequestWebApksInfo(const base::ListValue* args) {
   AllowJavascript();
   ShortcutHelper::RetrieveWebApks(base::Bind(
       &WebApksHandler::OnWebApkInfoRetrieved, weak_ptr_factory_.GetWeakPtr()));
 }
 
+// Converts a color from the format specified in content::Manifest to a CSS
+// string.
+std::string ColorToString(int64_t color) {
+  if (color == content::Manifest::kInvalidOrMissingColor)
+    return "";
+
+  SkColor sk_color = reinterpret_cast<uint32_t&>(color);
+  int r = SkColorGetR(sk_color);
+  int g = SkColorGetG(sk_color);
+  int b = SkColorGetB(sk_color);
+  double a = SkColorGetA(sk_color) / 255.0;
+  return base::StringPrintf("rgba(%d,%d,%d,%.2f)", r, g, b, a);
+}

[Dan Beam, 2017/02/24 18:36:16]

can this share code with

https://cs.chromium.org/chromium/src/chrome/browser/ui/webui/ntp/ntp_resource...

[gonzalon, 2017/02/24 22:30:52]

Yes, we probably can.
https://cs.chromium.org/chromium/src/chrome/browser/android/webapk/webapk_ins...
also uses similar code. My question is where you think this code should live?
There are so many directories in this codebase, so I have no idea what the
correct place would be. Or should I just leave it in one of these classes?

[Dan Beam, 2017/02/24 22:33:52]

i think they both share chrome/browser/ui/webui as a common ancestor, so maybe
in some util that already exists there or a new one if they're all different?

+
 void WebApksHandler::OnWebApkInfoRetrieved(
     const std::vector<WebApkInfo>& webapks_list) {
   if (!IsJavascriptAllowed())
     return;
   base::ListValue list;
   for (const auto& webapk_info : webapks_list) {
     std::unique_ptr<base::DictionaryValue> result(new base::DictionaryValue());
+    result->SetString("name", webapk_info.name);
     result->SetString("shortName", webapk_info.short_name);
     result->SetString("packageName", webapk_info.package_name);
     result->SetInteger("shellApkVersion", webapk_info.shell_apk_version);
     result->SetInteger("versionCode", webapk_info.version_code);
+    result->SetString("uri", webapk_info.uri);
+    result->SetString("scope", webapk_info.scope);
+    result->SetString("manifestUrl", webapk_info.manifest_url);
+    result->SetString("manifestStartUrl", webapk_info.manifest_start_url);
+    result->SetString("displayMode",
+                      content::WebDisplayModeToString(webapk_info.display));
+    result->SetString(
+        "orientation",
+        content::WebScreenOrientationLockTypeToString(webapk_info.orientation));
+    result->SetString("themeColor", ColorToString(webapk_info.theme_color));
+    result->SetString("backgroundColor",
+                      ColorToString(webapk_info.background_color));
     list.Append(std::move(result));
   }
 
   CallJavascriptFunction("returnWebApksInfo", list);
 }