Skip to content

Commit

Permalink
[EOC] Move chrome://snippets-internals to its own directory
Browse files Browse the repository at this point in the history
Moving the backend & frontend code to their own folders. This CL contains no
functional changes.

Bug: 830931
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ia6147c93063214143c8de29ca945cc3df8892bfa
Reviewed-on: https://chromium-review.googlesource.com/1023114
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Commit-Queue: Brandon Wylie <wylieb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552896}
  • Loading branch information
Brandon Wylie authored and Commit Bot committed Apr 23, 2018
1 parent aeacdd5 commit 1b4e121
Show file tree
Hide file tree
Showing 13 changed files with 22 additions and 18 deletions.
6 changes: 3 additions & 3 deletions chrome/browser/browser_resources.grd
Original file line number Diff line number Diff line change
Expand Up @@ -174,9 +174,9 @@
<include name="IDR_OFFLINE_INTERNALS_CSS" file="resources\offline_pages\offline_internals.css" type="BINDATA" compress="gzip" />
<include name="IDR_OFFLINE_INTERNALS_JS" file="resources\offline_pages\offline_internals.js" type="BINDATA" compress="gzip" />
<include name="IDR_OFFLINE_INTERNALS_BROWSER_PROXY_JS" file="resources\offline_pages\offline_internals_browser_proxy.js" type="BINDATA" compress="gzip" />
<include name="IDR_SNIPPETS_INTERNALS_HTML" file="resources\snippets_internals.html" allowexternalscript="true" compress="gzip" type="BINDATA" />
<include name="IDR_SNIPPETS_INTERNALS_CSS" file="resources\snippets_internals.css" compress="gzip" type="BINDATA" />
<include name="IDR_SNIPPETS_INTERNALS_JS" file="resources\snippets_internals.js" compress="gzip" type="BINDATA" />
<include name="IDR_SNIPPETS_INTERNALS_HTML" file="resources\snippets_internals\snippets_internals.html" allowexternalscript="true" compress="gzip" type="BINDATA" />
<include name="IDR_SNIPPETS_INTERNALS_CSS" file="resources\snippets_internals\snippets_internals.css" compress="gzip" type="BINDATA" />
<include name="IDR_SNIPPETS_INTERNALS_JS" file="resources\snippets_internals\snippets_internals.js" compress="gzip" type="BINDATA" />
</if>
<include name="IDR_SUPERVISED_USER_INTERNALS_HTML" file="resources\supervised_user_internals.html" allowexternalscript="true" compress="gzip" type="BINDATA" />
<include name="IDR_SUPERVISED_USER_INTERNALS_CSS" file="resources\supervised_user_internals.css" compress="gzip" type="BINDATA" />
Expand Down
1 change: 1 addition & 0 deletions chrome/browser/resources/snippets_internals/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
file://components/ntp_snippets/OWNERS
8 changes: 4 additions & 4 deletions chrome/browser/ui/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -1209,10 +1209,10 @@ split_static_library("ui") {
"webui/offline/offline_internals_ui_message_handler.h",
"webui/physical_web/physical_web_ui.cc",
"webui/physical_web/physical_web_ui.h",
"webui/snippets_internals_message_handler.cc",
"webui/snippets_internals_message_handler.h",
"webui/snippets_internals_ui.cc",
"webui/snippets_internals_ui.h",
"webui/snippets_internals/snippets_internals_message_handler.cc",
"webui/snippets_internals/snippets_internals_message_handler.h",
"webui/snippets_internals/snippets_internals_ui.cc",
"webui/snippets_internals/snippets_internals_ui.h",
"webui/webapks_handler.cc",
"webui/webapks_handler.h",
"webui/webapks_ui.cc",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@

#if defined(OS_ANDROID)
#include "chrome/browser/ui/webui/offline/offline_internals_ui.h"
#include "chrome/browser/ui/webui/snippets_internals_ui.h"
#include "chrome/browser/ui/webui/snippets_internals/snippets_internals_ui.h"
#include "chrome/browser/ui/webui/webapks_ui.h"
#else
#include "chrome/browser/ui/webui/devtools_ui.h"
Expand Down
1 change: 1 addition & 0 deletions chrome/browser/ui/webui/snippets_internals/OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
file://components/ntp_snippets/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
// 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/snippets_internals_message_handler.h"
#include "chrome/browser/ui/webui/snippets_internals/snippets_internals_message_handler.h"

#include <memory>
#include <set>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#ifndef CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_MESSAGE_HANDLER_H_
#define CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_MESSAGE_HANDLER_H_
#ifndef CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_SNIPPETS_INTERNALS_MESSAGE_HANDLER_H_
#define CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_SNIPPETS_INTERNALS_MESSAGE_HANDLER_H_

#include <map>
#include <string>
Expand Down Expand Up @@ -115,4 +115,4 @@ class SnippetsInternalsMessageHandler
DISALLOW_COPY_AND_ASSIGN(SnippetsInternalsMessageHandler);
};

#endif // CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_MESSAGE_HANDLER_H_
#endif // CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_SNIPPETS_INTERNALS_MESSAGE_HANDLER_H_
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,14 @@
// 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/snippets_internals_ui.h"
#include "chrome/browser/ui/webui/snippets_internals/snippets_internals_ui.h"

#include <memory>

#include "build/build_config.h"
#include "chrome/browser/ntp_snippets/content_suggestions_service_factory.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/webui/snippets_internals_message_handler.h"
#include "chrome/browser/ui/webui/snippets_internals/snippets_internals_message_handler.h"
#include "chrome/common/url_constants.h"
#include "chrome/grit/browser_resources.h"
#include "content/public/browser/web_ui.h"
Expand All @@ -33,7 +33,7 @@ content::WebUIDataSource* CreateSnippetsInternalsHTMLSource() {
return source;
}

} // namespace
} // namespace

SnippetsInternalsUI::SnippetsInternalsUI(content::WebUI* web_ui)
: WebUIController(web_ui) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

#ifndef CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_UI_H_
#define CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_UI_H_
#ifndef CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_SNIPPETS_INTERNALS_UI_H_
#define CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_SNIPPETS_INTERNALS_UI_H_

#include "base/macros.h"
#include "content/public/browser/web_ui_controller.h"
Expand All @@ -18,4 +18,4 @@ class SnippetsInternalsUI : public content::WebUIController {
DISALLOW_COPY_AND_ASSIGN(SnippetsInternalsUI);
};

#endif // CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_UI_H_
#endif // CHROME_BROWSER_UI_WEBUI_SNIPPETS_INTERNALS_SNIPPETS_INTERNALS_UI_H_
2 changes: 2 additions & 0 deletions components/ntp_snippets/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -13,3 +13,5 @@ vitaliii@chromium.org

# For ios:
noyau@chromium.org

# COMPONENT: UI>Browser>ContentSuggestions

0 comments on commit 1b4e121

Please sign in to comment.