Refactor generate engine manifest to accept blacklist as an argument.

blimp/engine/engine-manifest.txt

Index: blimp/engine/engine-manifest.txt
diff --git a/blimp/engine/engine-manifest.txt b/blimp/engine/engine-manifest.txt
index 8943ec4ff0fdd9cc3ce65f5b6e7d2eee8e06f704..b350d6e260dc26c5b569b084bfe8ec22f0aa9059 100644
--- a/blimp/engine/engine-manifest.txt
+++ b/blimp/engine/engine-manifest.txt
@@ -1,13 +1,39 @@
 # Runtime dependencies for the Blimp Engine
 #
 # This file was generated by running:
-# generate-engine-manifest.py --build-dir out-chromeos/Debug --target //blimp/engine --output blimp/engine/engine-manifest.txt
+# generate-engine-manifest.py --build-dir out-linux/Debug --target //blimp/engine --output blimp/engine/engine-manifest.txt --blacklist blimp/tools/manifest-blacklist.txt
 #
-# Note: Any unnecessary dependencies should be added to
-#       manifest-blacklist.txt and this file should be regenerated.
+# Note: Any unnecessary dependencies should be added to provided
+#       manifest-blacklist and this file should be regenerated.
+Mojo
+Applications/media/manifest.json
+Mojo
+Applications/user/manifest.json
 ./blimp_engine_app
 icudtl.dat
+Mojo
+Applications/catalog/manifest.json
+Mojo
+Applications/shell/manifest.json
 natives_blob.bin
+Mojo
+Mojo

[maniscalco, 2016/05/03 21:57:54]

Mojo is repeated several times... something seems odd here.  Looks like the
generation of the manifest isn't working properly anymore (this is probably why
some of the trybots are failing).  Might be best to not regen the manifest as
part of this change.  Or maybe add these new items to the blacklist?

[nyquist, 2016/05/03 22:44:59]

I don't think our bundle needs these. Could you just add */manifest.json to the
blacklist? The same with "Mojo".

There is some wildcard-support in the blacklist, but just verify that none of
these new dependencies sneak in.

[Jess, 2016/05/09 23:42:55]

Agree this behavior is odd which is why I included these changes. From comments
it appears the blacklist should be updated. That change and resulting engine
manifest are reflected in patch set 2.

[Jess, 2016/05/09 23:42:55]

Acknowledged.

+Mojo
+Mojo
+Applications/tracing/manifest.json
+Mojo
+Applications/resource_provider/manifest.json
+Mojo
+Applications/mus/manifest.json
+Mojo
+Applications/mus/resources/mus_app_resources_strings.pak
+Mojo
+Applications/mus/resources/mus_app_resources_100.pak
+Mojo
+Applications/mus/resources/mus_app_resources_200.pak
+Mojo
+Mojo
+Applications/font_service/manifest.json
 blimp_engine.pak
 ./chrome_sandbox
 gen/third_party/blimp_fonts/LICENSE
@@ -214,4 +240,4 @@ gen/third_party/blimp_fonts/RobotoCondensed-Italic.ttf
 gen/third_party/blimp_fonts/RobotoCondensed-Light.ttf
 gen/third_party/blimp_fonts/RobotoCondensed-LightItalic.ttf
 gen/third_party/blimp_fonts/RobotoCondensed-Regular.ttf
-gen/third_party/blimp_fonts/fonts.xml
+gen/third_party/blimp_fonts/fonts.xml