gn desc: printing public_deps without --all and --tree

tools/gn/command_desc.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <set>
 #include <sstream>
 
@@ skipping 354 matching lines @@
   Configs can have child configs. Specifying --tree will show the hierarchy.
 
 Printing outputs
 
   The "outputs" section will list all outputs that apply, including the outputs
   computed from the tool definition (eg for "executable", "static_library", ...
   targets).
 
 Printing deps
 
-  Deps will include all public, private, and data deps (TODO this could be
-  clarified and enhanced) sorted in order applying. The following may be used:
+  Deps could include public, private and data deps.
+
+  If the command is invoked with --tree or --all, deps will include all public,

[brettw, 2017/05/23 17:23:05]

Documentation formatting: Can you wrap this together in one paragraph. Separate
paragraphs should be separated by blank lines (as opposed to newlines but no
blank like you did).

[mbonadei1, 2017/05/26 09:29:44]

Done.

+  private and data deps.
+  If the command is invoked without --tree and --all deps will only include
+  private and data deps. In this case public deps will be printed as part of

[brettw, 2017/05/23 17:23:05]

This says "deps" will include data deps, is that right? If this is right, can we
change it? Data deps are very different and aren't linked. It doesn't make sense
to include this but not public_deps.

Can we separate out data_deps as well like you did for public and private?

[mbonadei1, 2017/05/26 09:29:44]

I am not sure I am following, let me try to rephrase to be sure we
are on the same page.

If we are splitting private and public deps it does not make sense
to include data_deps in the private deps (this change will be done
in the function Target::GetDeps).

After this change we can also add another field to the target
description, this field will be called data_deps and it will always
include data deps. Is this right?

+  the section "public_deps".
+  Deps will be sorted in order applying. The following may be used:
 
   --all
       Collects all recursive dependencies and prints a sorted flat list. Also
       usable with --tree (see below).
 )"
 
     TARGET_PRINTING_MODE_COMMAND_LINE_HELP
 "\n"
     TARGET_TESTONLY_FILTER_COMMAND_LINE_HELP
 
@@ skipping 113 matching lines @@
 
       if (!PrintConfig(config, what_to_print, !multiple_outputs))
         return 1;
     }
   }
 
   return 0;
 }
 
 }  // namespace commands