Add more phony rules to GN build

Add more phony rules to GN build.

For a target //foo/bar:bar this adds the additional phony rules: "foo/bar:bar" and "foo/bar" to build.ninja. This is on top of the phony rules "bar" which will exist for all targets with a unique name "bar". This allows one to disambiguate targets with different paths but the same short name.

R=scottmg@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266680

[scottmg, 2014-04-28 18:45:21]

https://codereview.chromium.org/247663006/diff/40001/tools/gn/ninja_build_wri...
File tools/gn/ninja_build_writer.cc (right):

https://codereview.chromium.org/247663006/diff/40001/tools/gn/ninja_build_wri...
tools/gn/ninja_build_writer.cc:261: ReplaceSubstringsAfterOffset(&escaped, 0,
":", "$:");
shouldn't escape_ninja be doing this?

[brettw, 2014-04-28 19:28:11]

I was thinking not doing the escaping of ':' by default since the colon is
actually specific syntax to rule writing and only happens if you're writing
something on the left.

However, I can't see a disadvantage of doing this, so I changed it.

[scottmg, 2014-04-28 19:50:53]

lgtm

https://codereview.chromium.org/247663006/diff/60001/tools/gn/escape.cc
File tools/gn/escape.cc (right):

https://codereview.chromium.org/247663006/diff/60001/tools/gn/escape.cc#newco...
tools/gn/escape.cc:63: dest->push_back(':');
test for this

[brettw, 2014-04-28 22:40:11]

Committed patchset #5 manually as r266680 (presubmit successful).