[net] Clang-format filename_util_unittest.cc

[net] Clang-format filename_util_unittest.cc

Upcoming changes to filename_util are unreviewable if they include
material code changes and formatting changes. This CL just includes
formatting changes and is the result of running:

  'clang-format -i --style=Chromium filename_util_unittest.cc'.

Depends on: https://codereview.chromium.org/869823003
Needed by : https://codereview.chromium.org/869233006

BUG=444102
Committed: https://crrev.com/13cc94a3430af79bb6fae21576bc688e7431b94b
Cr-Commit-Position: refs/heads/master@{#314399}

[asanka, 2015-01-30 23:41:46]

asanka@chromium.org changed reviewers:
+ jkarlin@chromium.org

[asanka, 2015-01-30 23:42:13]

Can you check if this is the intended formatting?

[jkarlin, 2015-01-31 00:59:43]

lgtm with nits, though I'm not an owner.

https://codereview.chromium.org/873843002/diff/20001/net/base/filename_util_u...
File net/base/filename_util_unittest.cc (right):

https://codereview.chromium.org/873843002/diff/20001/net/base/filename_util_u...
net/base/filename_util_unittest.cc:251: //{L"/foo%5Cbar.txt",
"file://foo\\bar.txt"},
This indentation level seems unfortunate, but is still readable.

Can you add a space after the below //'s here and below?

https://codereview.chromium.org/873843002/diff/20001/net/base/filename_util_u...
net/base/filename_util_unittest.cc:346: #else   // !defined(OS_WIN)
I wonder why it added a third space. That seems like a bug.

Can you file a bug using the the link at the bottom of
https://code.google.com/p/chromium/wiki/ClangFormat ?

[asanka, 2015-02-03 19:49:46]

Thanks!

https://codereview.chromium.org/873843002/diff/20001/net/base/filename_util_u...
File net/base/filename_util_unittest.cc (right):

https://codereview.chromium.org/873843002/diff/20001/net/base/filename_util_u...
net/base/filename_util_unittest.cc:251: //{L"/foo%5Cbar.txt",
"file://foo\\bar.txt"},
On 2015/01/31 at 00:59:43, jkarlin wrote:
> This indentation level seems unfortunate, but is still readable.
> 
> Can you add a space after the below //'s here and below?

Done.

https://codereview.chromium.org/873843002/diff/20001/net/base/filename_util_u...
net/base/filename_util_unittest.cc:346: #else   // !defined(OS_WIN)
On 2015/01/31 at 00:59:43, jkarlin wrote:
> I wonder why it added a third space. That seems like a bug.
> 
> Can you file a bug using the the link at the bottom of
https://code.google.com/p/chromium/wiki/ClangFormat ?

The extra space appears to be to keep this comment aligned with the comment on
line 429/379. If I remove that comment, then this one collapses to two spaces.
If it still seems like a bug, I'll be happy to file one.

[asanka, 2015-02-03 19:49:52]

The CQ bit was checked by asanka@chromium.org

[commit-bot: I haz the power, 2015-02-03 19:51:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/873843002/40001

[commit-bot: I haz the power, 2015-02-03 20:32:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-03 20:35:25]

Patchset 3 (id:??) landed as
https://crrev.com/13cc94a3430af79bb6fae21576bc688e7431b94b
Cr-Commit-Position: refs/heads/master@{#314399}