grit: qualify includes for shell_resources.h, virtual_driver_setup_resources.h, bluetooth_strings.h

chromecast/app/BUILD.gn

Index: chromecast/app/BUILD.gn
diff --git a/chromecast/app/BUILD.gn b/chromecast/app/BUILD.gn
index 8de02c89df413b0c02b2e1024e420b4607d987f6..55c82872acf7249b9aac59cafb5f60295085629a 100644
--- a/chromecast/app/BUILD.gn
+++ b/chromecast/app/BUILD.gn
@@ -92,6 +92,7 @@ source_set("unittests") {
 
 grit("resources") {
   source = "//chromecast/app/resources/shell_resources.grd"
+  use_qualified_include = true
 
   resource_ids = "//chromecast/app/resources/resource_ids"
 
@@ -104,6 +105,8 @@ grit("resources") {
 grit("chromecast_settings") {
   source = "//chromecast/app/resources/chromecast_settings.grd"
   resource_ids = "//chromecast/app/resources/resource_ids"
+
+  # TODO(thakis): Consider removing this in favor of the default directory.

[slan, 2017/02/27 16:01:34]

Are you interested in moving this for consistency's sake, or is there another
advantage as well? If we do leave a TODO here, could we take out a bug as well?

[Nico, 2017/02/27 16:05:59]

There's another advantage: With this output_dir, once we make this target have
use_qualified_include=true too, the .h file needs to be included via #include
chromecast_strings/grit/chromecast_settings.h" which in turn requires a
+chromecast_strings entry in a DEPS file, which in return needs an LGTM from a
chromecast_strings/OWNER for the new dep -- but that file doesn't exist.

Using the default output dir nicely makes grit outputs match the existing DEPS
system.

Do you remember why this line was needed? Almost all grit() targets don't have
it.

   output_dir = "$root_gen_dir/chromecast_strings"
 
   outputs = [