Commit a8a2861b authored by Roman Arora's avatar Roman Arora Committed by Commit Bot

Tab Search: WebUI source migration

Tab search WebUI source migration effort. Will be followed by a CL that
removes tab_search references from src-internal/DEPS and another CL to
rename the tab_search_merge references back to tab_search.

Bug: 1134167
Change-Id: I4d44201b651b60c13125ffaffece668ce66df28e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2492544Reviewed-by: default avatarThomas Lukaszewicz <tluk@chromium.org>
Reviewed-by: default avatarYuheng Huang <yuhengh@chromium.org>
Reviewed-by: default avatarJohn Lee <johntlee@chromium.org>
Commit-Queue: Roman Arora <romanarora@chromium.org>
Cr-Commit-Position: refs/heads/master@{#820389}
parent 27e795c1
...@@ -76,7 +76,7 @@ if (enable_js_type_check) { ...@@ -76,7 +76,7 @@ if (enable_js_type_check) {
deps += [ "about_nacl:closure_compile" ] deps += [ "about_nacl:closure_compile" ]
} }
if (enable_tab_search) { if (enable_tab_search) {
deps += [ "tab_search:closure_compile" ] deps += [ "tab_search_merge:closure_compile" ]
} }
if (enable_webui_tab_strip) { if (enable_webui_tab_strip) {
deps += [ "tab_strip:closure_compile" ] deps += [ "tab_strip:closure_compile" ]
...@@ -634,9 +634,9 @@ if (enable_tab_search) { ...@@ -634,9 +634,9 @@ if (enable_tab_search) {
"-E", "-E",
"root_gen_dir=" + rebase_path(root_gen_dir, root_build_dir), "root_gen_dir=" + rebase_path(root_gen_dir, root_build_dir),
] ]
source = "tab_search/tab_search_resources.grd" source = "tab_search_merge/tab_search_resources.grd"
deps = [ deps = [
"//chrome/browser/resources/tab_search:web_components", "//chrome/browser/resources/tab_search_merge:web_components",
"//chrome/browser/ui/webui/tab_search:mojo_bindings_js", "//chrome/browser/ui/webui/tab_search:mojo_bindings_js",
] ]
defines = chrome_grit_defines defines = chrome_grit_defines
......
...@@ -13,7 +13,7 @@ ...@@ -13,7 +13,7 @@
<release seq="1"> <release seq="1">
<includes> <includes>
<include name="IDR_APP_JS" <include name="IDR_APP_JS"
file="${root_gen_dir}/chrome/browser/resources/tab_search/app.js" file="${root_gen_dir}/chrome/browser/resources/tab_search_merge/app.js"
type="BINDATA" type="BINDATA"
use_base_dir="false" /> use_base_dir="false" />
<include name="IDR_FUSE_JS" <include name="IDR_FUSE_JS"
...@@ -29,7 +29,7 @@ ...@@ -29,7 +29,7 @@
file="tab_search_api_proxy.js" file="tab_search_api_proxy.js"
type="BINDATA" /> type="BINDATA" />
<include name="IDR_TAB_SEARCH_ITEM_JS" <include name="IDR_TAB_SEARCH_ITEM_JS"
file="${root_gen_dir}/chrome/browser/resources/tab_search/tab_search_item.js" file="${root_gen_dir}/chrome/browser/resources/tab_search_merge/tab_search_item.js"
type="BINDATA" type="BINDATA"
use_base_dir="false"/> use_base_dir="false"/>
<include name="IDR_TAB_SEARCH_MOJO_LITE_JS" <include name="IDR_TAB_SEARCH_MOJO_LITE_JS"
...@@ -40,7 +40,7 @@ ...@@ -40,7 +40,7 @@
file="tab_search_page.html" file="tab_search_page.html"
type="BINDATA" /> type="BINDATA" />
<include name="IDR_TAB_SEARCH_SEARCH_FIELD_JS" <include name="IDR_TAB_SEARCH_SEARCH_FIELD_JS"
file="${root_gen_dir}/chrome/browser/resources/tab_search/tab_search_search_field.js" file="${root_gen_dir}/chrome/browser/resources/tab_search_merge/tab_search_search_field.js"
type="BINDATA" type="BINDATA"
use_base_dir="false" /> use_base_dir="false" />
</includes> </includes>
......
...@@ -28,7 +28,7 @@ ...@@ -28,7 +28,7 @@
#if BUILDFLAG(ENABLE_TAB_SEARCH) #if BUILDFLAG(ENABLE_TAB_SEARCH)
namespace { namespace {
constexpr char kGeneratedPath[] = constexpr char kGeneratedPath[] =
"@out_folder@/gen/chrome/browser/resources/tab_search/"; "@out_folder@/gen/chrome/browser/resources/tab_search_merge/";
} }
#endif // BUILDFLAG(ENABLE_TAB_SEARCH) #endif // BUILDFLAG(ENABLE_TAB_SEARCH)
......
...@@ -29,7 +29,7 @@ if (include_js_tests) { ...@@ -29,7 +29,7 @@ if (include_js_tests) {
] ]
if (enable_tab_search) { if (enable_tab_search) {
sources += [ "tab_search/test/tab_search_interactive_ui_tests.js" ] sources += [ "tab_search_merge/tab_search_interactive_ui_tests.js" ]
} }
gen_include_files = [ gen_include_files = [
...@@ -180,7 +180,7 @@ if (include_js_tests) { ...@@ -180,7 +180,7 @@ if (include_js_tests) {
sources += [ "print_preview/print_preview_ui_browsertest.js" ] sources += [ "print_preview/print_preview_ui_browsertest.js" ]
} }
if (enable_tab_search) { if (enable_tab_search) {
sources += [ "tab_search/test/tab_search_browsertest.js" ] sources += [ "tab_search_merge/tab_search_browsertest.js" ]
} }
if (enable_webui_tab_strip) { if (enable_webui_tab_strip) {
sources += [ "tab_strip/tab_strip_browsertest.js" ] sources += [ "tab_strip/tab_strip_browsertest.js" ]
...@@ -502,7 +502,7 @@ group("closure_compile") { ...@@ -502,7 +502,7 @@ group("closure_compile") {
] ]
} }
if (enable_tab_search) { if (enable_tab_search) {
deps += [ "tab_search/test:closure_compile" ] deps += [ "tab_search_merge:closure_compile" ]
} }
} }
......
...@@ -7,7 +7,7 @@ import("//third_party/closure_compiler/compile_js.gni") ...@@ -7,7 +7,7 @@ import("//third_party/closure_compiler/compile_js.gni")
js_type_check("closure_compile") { js_type_check("closure_compile") {
is_polymer3 = true is_polymer3 = true
closure_flags = default_closure_args + [ closure_flags = default_closure_args + [
"browser_resolver_prefix_replacements=\"chrome://tab-search/=../../chrome/browser/resources/tab-search/\"", "browser_resolver_prefix_replacements=\"chrome://tab-search/=../../chrome/browser/resources/tab-search-merge/\"",
"js_module_root=../../chrome/test/data/webui/", "js_module_root=../../chrome/test/data/webui/",
"js_module_root=./gen/chrome/test/data/webui/", "js_module_root=./gen/chrome/test/data/webui/",
] ]
...@@ -22,7 +22,7 @@ js_type_check("closure_compile") { ...@@ -22,7 +22,7 @@ js_type_check("closure_compile") {
js_library("fuzzy_search_test") { js_library("fuzzy_search_test") {
deps = [ deps = [
":test_tab_search_api_proxy", ":test_tab_search_api_proxy",
"../..:chai_assert", "..:chai_assert",
] ]
externs_list = [ "$externs_path/mocha-2.5.js" ] externs_list = [ "$externs_path/mocha-2.5.js" ]
} }
...@@ -30,8 +30,8 @@ js_library("fuzzy_search_test") { ...@@ -30,8 +30,8 @@ js_library("fuzzy_search_test") {
js_library("tab_search_app_test") { js_library("tab_search_app_test") {
deps = [ deps = [
":test_tab_search_api_proxy", ":test_tab_search_api_proxy",
"../..:chai_assert", "..:chai_assert",
"//chrome/browser/resources/tab_search:app", "//chrome/browser/resources/tab_search_merge:app",
] ]
externs_list = [ "$externs_path/mocha-2.5.js" ] externs_list = [ "$externs_path/mocha-2.5.js" ]
} }
...@@ -39,24 +39,24 @@ js_library("tab_search_app_test") { ...@@ -39,24 +39,24 @@ js_library("tab_search_app_test") {
js_library("tab_search_app_focus_test") { js_library("tab_search_app_focus_test") {
deps = [ deps = [
":test_tab_search_api_proxy", ":test_tab_search_api_proxy",
"../..:chai_assert", "..:chai_assert",
"//chrome/browser/resources/tab_search:app", "//chrome/browser/resources/tab_search_merge:app",
] ]
externs_list = [ "$externs_path/mocha-2.5.js" ] externs_list = [ "$externs_path/mocha-2.5.js" ]
} }
js_library("test_tab_search_api_proxy") { js_library("test_tab_search_api_proxy") {
deps = [ deps = [
"//chrome/browser/resources/tab_search:tab_search_api_proxy", "..:test_browser_proxy.m",
"//chrome/browser/resources/tab_search_merge:tab_search_api_proxy",
"//chrome/browser/ui/webui/tab_search:mojo_bindings_js_library_for_compile", "//chrome/browser/ui/webui/tab_search:mojo_bindings_js_library_for_compile",
"//chrome/test/data/webui:test_browser_proxy.m",
] ]
} }
js_library("tab_search_item_test") { js_library("tab_search_item_test") {
deps = [ deps = [
"../..:chai_assert", "..:chai_assert",
"//chrome/browser/resources/tab_search:tab_search_item", "//chrome/browser/resources/tab_search_merge:tab_search_item",
] ]
externs_list = [ "$externs_path/mocha-2.5.js" ] externs_list = [ "$externs_path/mocha-2.5.js" ]
} }
...@@ -176,8 +176,8 @@ suite('FuzzySearchTest', () => { ...@@ -176,8 +176,8 @@ suite('FuzzySearchTest', () => {
assertDeepEquals(records, fuzzySearch('', records, options)); assertDeepEquals(records, fuzzySearch('', records, options));
assertDeepEquals(archMatchedRecords, fuzzySearch('arch', records, options)); assertDeepEquals(archMatchedRecords, fuzzySearch('arch', records, options));
assertDeepEquals(searchMatchedRecords, assertDeepEquals(
fuzzySearch('search', records, options)); searchMatchedRecords, fuzzySearch('search', records, options));
// No matches should return an empty list. // No matches should return an empty list.
assertDeepEquals([], fuzzySearch('archh', records, options)); assertDeepEquals([], fuzzySearch('archh', records, options));
...@@ -214,10 +214,10 @@ suite('FuzzySearchTest', () => { ...@@ -214,10 +214,10 @@ suite('FuzzySearchTest', () => {
}, },
]; ];
assertDeepEquals(backslashMatchedRecords, assertDeepEquals(
fuzzySearch('\\test', records, options)); backslashMatchedRecords, fuzzySearch('\\test', records, options));
assertDeepEquals(quoteMatchedRecords, assertDeepEquals(
fuzzySearch('\"end', records, options)); quoteMatchedRecords, fuzzySearch('\"end', records, options));
}); });
test('Test exact match result scoring accounts for match position.', () => { test('Test exact match result scoring accounts for match position.', () => {
......
...@@ -6,7 +6,7 @@ import {loadTimeData} from 'chrome://resources/js/load_time_data.m.js'; ...@@ -6,7 +6,7 @@ import {loadTimeData} from 'chrome://resources/js/load_time_data.m.js';
import {getDeepActiveElement} from 'chrome://resources/js/util.m.js'; import {getDeepActiveElement} from 'chrome://resources/js/util.m.js';
import {keyDownOn} from 'chrome://resources/polymer/v3_0/iron-test-helpers/mock-interactions.js'; import {keyDownOn} from 'chrome://resources/polymer/v3_0/iron-test-helpers/mock-interactions.js';
import {TabSearchAppElement} from 'chrome://tab-search/app.js'; import {TabSearchAppElement} from 'chrome://tab-search/app.js';
import {TabSearchApiProxy, TabSearchApiProxyImpl} from 'chrome://tab-search/tab_search_api_proxy.js' import {TabSearchApiProxy, TabSearchApiProxyImpl} from 'chrome://tab-search/tab_search_api_proxy.js';
import {TabSearchItem} from 'chrome://tab-search/tab_search_item.js'; import {TabSearchItem} from 'chrome://tab-search/tab_search_item.js';
import {TabSearchSearchField} from 'chrome://tab-search/tab_search_search_field.js'; import {TabSearchSearchField} from 'chrome://tab-search/tab_search_search_field.js';
...@@ -53,7 +53,7 @@ suite('TabSearchAppFocusTest', () => { ...@@ -53,7 +53,7 @@ suite('TabSearchAppFocusTest', () => {
const searchField = /** @type {!TabSearchSearchField} */ const searchField = /** @type {!TabSearchSearchField} */
(tabSearchApp.shadowRoot.querySelector('#searchField')); (tabSearchApp.shadowRoot.querySelector('#searchField'));
const searchInput = /** @type {!HTMLInputElement} */ const searchInput = /** @type {!HTMLInputElement} */
(searchField.shadowRoot.querySelector('#searchInput')) (searchField.shadowRoot.querySelector('#searchInput'));
assertEquals(searchInput, getDeepActiveElement()); assertEquals(searchInput, getDeepActiveElement());
const tabSearchItems = /** @type {!NodeList<!HTMLElement>} */ const tabSearchItems = /** @type {!NodeList<!HTMLElement>} */
......
...@@ -5,7 +5,7 @@ ...@@ -5,7 +5,7 @@
import {loadTimeData} from 'chrome://resources/js/load_time_data.m.js'; import {loadTimeData} from 'chrome://resources/js/load_time_data.m.js';
import {keyDownOn} from 'chrome://resources/polymer/v3_0/iron-test-helpers/mock-interactions.js'; import {keyDownOn} from 'chrome://resources/polymer/v3_0/iron-test-helpers/mock-interactions.js';
import {TabSearchAppElement} from 'chrome://tab-search/app.js'; import {TabSearchAppElement} from 'chrome://tab-search/app.js';
import {TabSearchApiProxy, TabSearchApiProxyImpl} from 'chrome://tab-search/tab_search_api_proxy.js' import {TabSearchApiProxy, TabSearchApiProxyImpl} from 'chrome://tab-search/tab_search_api_proxy.js';
import {TabSearchItem} from 'chrome://tab-search/tab_search_item.js'; import {TabSearchItem} from 'chrome://tab-search/tab_search_item.js';
import {TabSearchSearchField} from 'chrome://tab-search/tab_search_search_field.js'; import {TabSearchSearchField} from 'chrome://tab-search/tab_search_search_field.js';
......
...@@ -36,7 +36,7 @@ class TabSearchBrowserTest extends PolymerTest { ...@@ -36,7 +36,7 @@ class TabSearchBrowserTest extends PolymerTest {
var TabSearchAppTest = class extends TabSearchBrowserTest { var TabSearchAppTest = class extends TabSearchBrowserTest {
/** @override */ /** @override */
get browsePreload() { get browsePreload() {
return 'chrome://tab-search/test_loader.html?module=tab_search/test/tab_search_app_test.js'; return 'chrome://tab-search/test_loader.html?module=tab_search_merge/tab_search_app_test.js';
} }
}; };
...@@ -48,7 +48,7 @@ TEST_F('TabSearchAppTest', 'All', function() { ...@@ -48,7 +48,7 @@ TEST_F('TabSearchAppTest', 'All', function() {
var FuzzySearchTest = class extends TabSearchBrowserTest { var FuzzySearchTest = class extends TabSearchBrowserTest {
/** @override */ /** @override */
get browsePreload() { get browsePreload() {
return 'chrome://tab-search/test_loader.html?module=tab_search/test/fuzzy_search_test.js'; return 'chrome://tab-search/test_loader.html?module=tab_search_merge/fuzzy_search_test.js';
} }
}; };
...@@ -60,10 +60,10 @@ TEST_F('FuzzySearchTest', 'All', function() { ...@@ -60,10 +60,10 @@ TEST_F('FuzzySearchTest', 'All', function() {
var TabSearchItemTest = class extends TabSearchBrowserTest { var TabSearchItemTest = class extends TabSearchBrowserTest {
/** @override */ /** @override */
get browsePreload() { get browsePreload() {
return 'chrome://tab-search/test_loader.html?module=tab_search/test/tab_search_item_test.js'; return 'chrome://tab-search/test_loader.html?module=tab_search_merge/tab_search_item_test.js';
} }
}; };
TEST_F('TabSearchItemTest', 'All', function() { TEST_F('TabSearchItemTest', 'All', function() {
mocha.run(); mocha.run();
}); });
\ No newline at end of file
...@@ -14,7 +14,7 @@ GEN('#include "services/network/public/cpp/features.h"'); ...@@ -14,7 +14,7 @@ GEN('#include "services/network/public/cpp/features.h"');
var TabSearchInteractiveUITest = class extends PolymerInteractiveUITest { var TabSearchInteractiveUITest = class extends PolymerInteractiveUITest {
/** @override */ /** @override */
get browsePreload() { get browsePreload() {
return 'chrome://tab-search/test_loader.html?module=tab_search/test/tab_search_app_focus_test.js'; return 'chrome://tab-search/test_loader.html?module=tab_search_merge/tab_search_app_focus_test.js';
} }
get extraLibraries() { get extraLibraries() {
...@@ -32,7 +32,7 @@ var TabSearchInteractiveUITest = class extends PolymerInteractiveUITest { ...@@ -32,7 +32,7 @@ var TabSearchInteractiveUITest = class extends PolymerInteractiveUITest {
] ]
}; };
} }
} };
TEST_F('TabSearchInteractiveUITest', 'All', function() { TEST_F('TabSearchInteractiveUITest', 'All', function() {
mocha.run(); mocha.run();
......
...@@ -3,7 +3,7 @@ ...@@ -3,7 +3,7 @@
// found in the LICENSE file. // found in the LICENSE file.
import {TabSearchApiProxy} from 'chrome://tab-search/tab_search_api_proxy.js'; import {TabSearchApiProxy} from 'chrome://tab-search/tab_search_api_proxy.js';
import {TestBrowserProxy} from '../../test_browser_proxy.m.js'; import {TestBrowserProxy} from '../test_browser_proxy.m.js';
/** @implements {TabSearchApiProxy} */ /** @implements {TabSearchApiProxy} */
export class TestTabSearchApiProxy extends TestBrowserProxy { export class TestTabSearchApiProxy extends TestBrowserProxy {
......
...@@ -179,7 +179,7 @@ ...@@ -179,7 +179,7 @@
"META": {"sizes": {"includes": [50],}}, "META": {"sizes": {"includes": [50],}},
"includes": [1850], "includes": [1850],
}, },
"chrome/browser/resources/tab_search/tab_search_resources.grd": { "chrome/browser/resources/tab_search_merge/tab_search_resources.grd": {
"includes": [1880], "includes": [1880],
}, },
"<(SHARED_INTERMEDIATE_DIR)/chrome/browser/resources/tab_strip/tab_strip_resources.grd": { "<(SHARED_INTERMEDIATE_DIR)/chrome/browser/resources/tab_strip/tab_strip_resources.grd": {
......
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