Prep for generated .vcproj files:...

Prep for generated .vcproj files:
Fix minor stylistic inconsistencies in some .vcproj files:
* /> tag termination:
    net\net_resources.vcproj
* Add empty <ToolFiles>, <References> and <Globals> sections:
    net\net_resources.vcproj
    third_party\zlib\zlib.vcproj
    third_party\libpng\libpng.vcproj
    chrome\tools\test\image_diff\image_diff.vcproj
* Remove .\ on file names where its use is inconsistent with the rest
  of the file names in the .vcproj:
    net\build\net.vcproj
    net\build\net_unittests.vcproj
    chrome\installer\util\util.vcproj
* Add explicit RootNamespace attribute:
    chrome\test\automation\automation.vcproj
    testing\gtest.vcproj
* Fix file order in four locales that don't match other locale .vcproj files:
    chrome\app\locales\da.vcproj
    chrome\app\locales\he.vcproj
    chrome\app\locales\zh-TW.vcproj
    chrome\app\locales\en-US.vcproj
* Remove duplicate files from the list:
    chrome\browser\views\browser_views.vcproj (event_utils.[ch])

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=8577

[sgk, 2009-01-23 17:41:11]

This updates some .vcproj files that have stylistic inconsistencies with (most
of) the rest, so when we move to generated .vcproj files we can have minimal
diffs and focus on the substance.

I'm pretty sure none of these introduces any substantive build changes, but let
me know if I'm overlooking something.

My trybot runs have built fine, but showed some test errors (different errors on
different runs).  I don't see them locally I'm pretty sure they're unrelated,
but am re-running through trybot to verify.

[Nicolas Sylvain, 2009-01-23 18:09:05]

LGTM, with a nit, but if you think it's right I have no problem with it.

http://codereview.chromium.org/18555/diff/1/3
File net/build/net.vcproj (right):

http://codereview.chromium.org/18555/diff/1/3#newcode384
Line 384: RelativePath="precompiled_net.cc"
Why do you remove the "." here while in the other files you add a "." ?

[sgk, 2009-01-23 18:26:09]

http://codereview.chromium.org/18555/diff/1/3
File net/build/net.vcproj (right):

http://codereview.chromium.org/18555/diff/1/3#newcode384
Line 384: RelativePath="precompiled_net.cc"
On 2009/01/23 18:09:05, Nicolas Sylvain wrote:
> Why do you remove the "." here while in the other files you add a "." ?

It's about trying to keep the diffs manageable in this round despite the fact
that our .vcproj files aren't consistent:  some use .\ as a prefix for local
files, some don't, and some have a mix.  The MSVSProject() generation function
lets you pass in options to specify what style you want, so we can generate
.vcproj files that diff pretty closely to the status quo.  What it can't do is
handle a mix of styles in the same file, so this CL is about making those
internally consistent.  Then a later CL will take care of making all the .vcproj
file lists consistent with each other, so any changes that might (inadvertently)
introduce will be obvious.

[M-A Ruel, 2009-01-23 18:28:11]

lgtm

[darin (slow to review), 2009-01-23 20:55:40]

LGTM