[ntp][modules] Disable dummy modules in NTP for official build

Bug: 1130814
Change-Id: If8d93fb7c134637f399735637ff031abbe31ebc2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2411671Reviewed-by: Esmael Elmoslimany <aee@chromium.org>
Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
Commit-Queue: Yue Zhang <yuezhanggg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809463}

chrome/browser/resources/BUILD.gn

@@ -234,7 +234,7 @@ if (!is_android) {
       "root_gen_dir=" + rebase_path(root_gen_dir, root_build_dir),
     ]
 
-    defines = chrome_grit_defines
+    defines = chrome_grit_defines + [ "is_official_build=$is_official_build" ]
     outputs = [
       "grit/new_tab_page_resources.h",
       "grit/new_tab_page_resources_map.cc",

chrome/browser/resources/new_tab_page/BUILD.gn

@@ -3,6 +3,7 @@
 # found in the LICENSE file.
 
 import("//chrome/browser/resources/optimize_webui.gni")
+import("//chrome/common/features.gni")
 import("//third_party/closure_compiler/compile_js.gni")
 import("//tools/grit/grit_rule.gni")
 import("//tools/polymer/html_to_js.gni")
@@ -277,6 +278,7 @@ if (optimize_webui) {
       "root_gen_dir=" + rebase_path(root_gen_dir, root_build_dir),
     ]
 
+    defines = chrome_grit_defines + [ "is_official_build=$is_official_build" ]
     outputs = [
       "grit/new_tab_page_resources.h",
       "grit/new_tab_page_resources_map.cc",

chrome/browser/resources/new_tab_page/modules/modules.js

@@ -8,7 +8,9 @@
 
 import {loadTimeData} from 'chrome://resources/js/load_time_data.m.js';
 
+// <if expr="not is_official_build">
 import {dummyDescriptor, dummyDescriptor2} from './dummy/module.js';
+// </if>
 import {kaleidoscopeDescriptor} from './kaleidoscope/module.js';
 import {ModuleDescriptor} from './module_descriptor.js';
 import {ModuleRegistry} from './module_registry.js';
@@ -20,7 +22,9 @@ if (loadTimeData.getBoolean('kaleidoscopeModuleEnabled')) {
   descriptors.push(kaleidoscopeDescriptor);
 }
 
+// <if expr="not is_official_build">
 descriptors.push(dummyDescriptor);
 descriptors.push(dummyDescriptor2);
+// </if>
 
 ModuleRegistry.getInstance().registerModules(descriptors);

chrome/browser/resources/new_tab_page/modules/modules_resources.grdp

 <?xml version="1.0" encoding="UTF-8"?>
 <grit-part>
+<if expr="not is_official_build">
   <include name="IDR_NEW_TAB_PAGE_MODULES_DUMMY_MODULE_JS"
       file="${root_gen_dir}/chrome/browser/resources/new_tab_page/modules/dummy/module.js"
       use_base_dir="false" type="BINDATA" compress="false" />
+</if>
   <include name="IDR_NEW_TAB_PAGE_MODULES_KALEIDOSCOPE_MODULE_JS"
       file="modules/kaleidoscope/module.js"
       type="BINDATA" compress="false" />
   <include name="IDR_NEW_TAB_PAGE_MODULES_MODULE_DESCRIPTOR_JS"
       file="modules/module_descriptor.js" type="BINDATA" compress="false" />
   <include name="IDR_NEW_TAB_PAGE_MODULES_MODULES_JS"
-      file="modules/modules.js" type="BINDATA" compress="false" />
+      file="modules/modules.js" type="BINDATA" compress="false" preprocess="true"/>
   <include name="IDR_NEW_TAB_PAGE_MODULES_MODULE_REGISTRY_JS"
       file="modules/module_registry.js" type="BINDATA" compress="false" />
   <include name="IDR_NEW_TAB_PAGE_MODULE_WRAPPER_JS"

chrome/browser/resources/new_tab_page/new_tab_page.js

@@ -16,7 +16,9 @@ export {BackgroundManager} from './background_manager.js';
 export {BrowserProxy} from './browser_proxy.js';
 export {BackgroundSelectionType} from './customize_dialog.js';
 export {ImgElement} from './img.js';
+// <if expr="not is_official_build">
 export {dummyDescriptor} from './modules/dummy/module.js';
+// </if>
 export {kaleidoscopeDescriptor} from './modules/kaleidoscope/module.js';
 export {ModuleDescriptor} from './modules/module_descriptor.js';
 export {ModuleRegistry} from './modules/module_registry.js';

chrome/browser/resources/new_tab_page/new_tab_page_resources.grd

@@ -13,7 +13,7 @@
   <release seq="1">
     <includes>
       <include name="IDR_NEW_TAB_PAGE_NEW_TAB_PAGE_JS"
-          file="new_tab_page.js" type="BINDATA" compress="false" />
+          file="new_tab_page.js" type="BINDATA" compress="false" preprocess="true"/>
       <include name="IDR_NEW_TAB_PAGE_APP_JS"
           file="${root_gen_dir}/chrome/browser/resources/new_tab_page/app.js"
           use_base_dir="false" type="BINDATA" compress="false"