Commit 3c92b2ad authored by Alexey Baskakov's avatar Alexey Baskakov Committed by Commit Bot

WebApp: Use presets from WebApp icon generator.

Instead of extension-based sizes.

Bug: 901226
Change-Id: I4729ac0206fea143980db4cf43de67a71d83f7be
Reviewed-on: https://chromium-review.googlesource.com/c/1329819Reviewed-by: default avatarGiovanni Ortuño Urquidi <ortuno@chromium.org>
Commit-Queue: Alexey Baskakov <loyso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#607166}
parent 395cf32d
...@@ -70,8 +70,8 @@ void WebAppDataRetriever::GetIcons(const GURL& app_url, ...@@ -70,8 +70,8 @@ void WebAppDataRetriever::GetIcons(const GURL& app_url,
// Generate missing icons. // Generate missing icons.
static constexpr int kIconSizesToGenerate[] = { static constexpr int kIconSizesToGenerate[] = {
icon_size::k32, icon_size::k32 * 2, icon_size::k48, icon_size::k24, icon_size::k32, icon_size::k48,
icon_size::k48 * 2, icon_size::k128, icon_size::k128 * 2, icon_size::k128, icon_size::k256, icon_size::k512,
}; };
// Get the letter to use in the generated icon. // Get the letter to use in the generated icon.
......
...@@ -10,13 +10,13 @@ ...@@ -10,13 +10,13 @@
#include "base/optional.h" #include "base/optional.h"
#include "base/strings/utf_string_conversions.h" #include "base/strings/utf_string_conversions.h"
#include "chrome/browser/web_applications/components/web_app_icon_generator.h"
#include "chrome/test/base/chrome_render_view_host_test_harness.h" #include "chrome/test/base/chrome_render_view_host_test_harness.h"
#include "chrome/test/base/testing_profile.h" #include "chrome/test/base/testing_profile.h"
#include "content/public/browser/navigation_entry.h" #include "content/public/browser/navigation_entry.h"
#include "content/public/browser/site_instance.h" #include "content/public/browser/site_instance.h"
#include "content/public/test/test_browser_thread_bundle.h" #include "content/public/test/test_browser_thread_bundle.h"
#include "content/public/test/web_contents_tester.h" #include "content/public/test/web_contents_tester.h"
#include "extensions/common/constants.h"
#include "mojo/public/cpp/bindings/associated_binding.h" #include "mojo/public/cpp/bindings/associated_binding.h"
#include "testing/gtest/include/gtest/gtest.h" #include "testing/gtest/include/gtest/gtest.h"
#include "third_party/blink/public/common/associated_interfaces/associated_interface_provider.h" #include "third_party/blink/public/common/associated_interfaces/associated_interface_provider.h"
...@@ -30,14 +30,9 @@ const char kFooUrl2[] = "https://foo.example/bar"; ...@@ -30,14 +30,9 @@ const char kFooUrl2[] = "https://foo.example/bar";
const char kFooTitle[] = "Foo Title"; const char kFooTitle[] = "Foo Title";
const char kBarUrl[] = "https://bar.example"; const char kBarUrl[] = "https://bar.example";
constexpr int kIconSizesToGenerate[] = { constexpr int kIconSizesToGenerate[] = {icon_size::k24, icon_size::k32,
extension_misc::EXTENSION_ICON_SMALL, icon_size::k48, icon_size::k128,
extension_misc::EXTENSION_ICON_SMALL * 2, icon_size::k256, icon_size::k512};
extension_misc::EXTENSION_ICON_MEDIUM,
extension_misc::EXTENSION_ICON_MEDIUM * 2,
extension_misc::EXTENSION_ICON_LARGE,
extension_misc::EXTENSION_ICON_LARGE * 2,
};
} // namespace } // namespace
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment