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

This reverts commit a099d2db.

Reason for revert: NewTabPageModulesDummyModuleTest.All failing. bot failures on linux-chromeos-chrome: https://ci.chromium.org/p/chrome/builders/ci/linux-chromeos-chrome

First failure: https://ci.chromium.org/p/chrome/builders/ci/linux-chromeos-chrome/8711

Original change's description:
> [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/+/2411671
> Reviewed-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}

TBR=tiborg@chromium.org,aee@chromium.org,yuezhanggg@chromium.org

Change-Id: I4537af0a34a5cc61a1f7b228a5047127862e2a0f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1130814
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2425207Reviewed-by: Darwin Huang <huangdarwin@chromium.org>
Commit-Queue: Darwin Huang <huangdarwin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809564}

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 + [ "is_official_build=$is_official_build" ]
+    defines = chrome_grit_defines
     outputs = [
       "grit/new_tab_page_resources.h",
       "grit/new_tab_page_resources_map.cc",

chrome/browser/resources/new_tab_page/BUILD.gn

@@ -3,7 +3,6 @@
 # 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")
@@ -278,7 +277,6 @@ 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,9 +8,7 @@
 
 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';
@@ -22,9 +20,7 @@ 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" preprocess="true"/>
+      file="modules/modules.js" type="BINDATA" compress="false" />
   <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,9 +16,7 @@ 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" preprocess="true"/>
+          file="new_tab_page.js" type="BINDATA" compress="false" />
       <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"