GitLab

Previously the code relied on some fragile coincidence to work:
 1) flags.html was using |flattenhtml| to inline flags.css at build time.
 2) flags.html was using absolute chrome://flags/strings.js and
    chrome://flags/flags.js URLs, to prevent those files from being
    inlined (which would violate CSP and result in a runtime error).

1 and 2 above resulted in a breakage (runtime error) if one changes
<script src="chrome://flags/flags.js"></script>
to
<script src="flags.js"></script>

which is confusing, since those URLs are equivalent and should be treated
exactly the same.

Removing |flattenhtml| from flags.js GRIT entry fixes the weirdness.

This is in preparation of converting this page to use JS modules.

Bug: 1029094
Change-Id: I394ce31f082372758ea6d34ce607c0794e27b4c4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1941207
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Auto-Submit: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721109}