Split web_app::AppId out of web_app_helpers.h and update includes.
Many places were including the extra methods when they only needed
AppId, and it was potentially confusing that AppId is provided by
web_app_helpers.
Admittedly not our biggest problem, but a very easy fix that makes
life better for future readers of the codebase. :)
Mostly just a straight find/replace of includes when methods defined
in web_app_helpers were not present. A few (mostly test) files had to
have a new include of web_app_helpers because methods were previously
pulled in through transitive includes.
Change-Id: Ia19c13365733868c50ab3b9ec5affdb114aa5eb3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016818
Reviewed-by: Alexey Baskakov <loyso@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Glen Robertson <glenrob@chromium.org>
Cr-Commit-Position: refs/heads/master@{#735662}
diff --git a/chrome/browser/web_applications/components/external_install_options.h b/chrome/browser/web_applications/components/external_install_options.h
index b472ccb..1a69eb50 100644
--- a/chrome/browser/web_applications/components/external_install_options.h
+++ b/chrome/browser/web_applications/components/external_install_options.h
@@ -9,7 +9,7 @@
#include "chrome/browser/web_applications/components/install_manager.h"
#include "chrome/browser/web_applications/components/web_app_constants.h"
-#include "chrome/browser/web_applications/components/web_app_helpers.h"
+#include "chrome/browser/web_applications/components/web_app_id.h"
#include "url/gurl.h"
namespace web_app {
