Add proper path separator(\) for testing AX test cases on windows.

Add proper path separator(\) for testing AX test cases on windows.

This CL would make the path separator uniform on windows for AX tests.

Presently the test path is coming as follow:This will be executed because windows supports both(\,/)
Testing: e:\b\swarm_slave\work\run_tha_testzigska\content\test\data\accessibility/aria\aria-disabled.html

But proper would be to keep it uniform.
Testing: e:\b\swarm_slave\work\run_tha_testzigska\content\test\data\accessibility\aria\aria-disabled.html

BUG=none

Committed: https://crrev.com/845a27e777ee5b8ec20deda87001c86ce59fa13e
Cr-Commit-Position: refs/heads/master@{#311693}

[shreeramk, 2014-12-30 05:28:58]

shreeram.k@samsung.com changed reviewers:
+ aboxhall@chromium.org, dmazzoni@chromium.org

[shreeramk, 2014-12-30 05:28:59]

Please review. Thanks!!

[dmazzoni, 2015-01-05 09:04:59]

Thanks for the fix, but would be better to reuse existing file path support
rather than an ifdef

https://codereview.chromium.org/826323002/diff/1/content/browser/accessibilit...
File content/browser/accessibility/dump_accessibility_events_browsertest.cc
(right):

https://codereview.chromium.org/826323002/diff/1/content/browser/accessibilit...
content/browser/accessibility/dump_accessibility_events_browsertest.cc:142:
const char* file_dir = "accessibility/event";
Use the FilePath class (base/files/file_path.h) to append directories with the
right separator rather than an ifdef.

https://codereview.chromium.org/826323002/diff/1/content/browser/accessibilit...
content/browser/accessibility/dump_accessibility_events_browsertest.cc:149:
base::FilePath event_file = test_path.Append(base::FilePath(file_path));
It might be easiest to rewrite the code to append "accessibility" and then
append "event"

[dmazzoni, 2015-01-05 09:05:33]

Oh, and thanks for your patience - I'm back from vacation and catching up on
reviews.

[shreeramk, 2015-01-12 07:42:31]

@Dominic: PTAL. Thanks!

[dmazzoni, 2015-01-14 18:13:53]

lgtm

https://codereview.chromium.org/826323002/diff/40001/content/browser/accessib...
File content/browser/accessibility/dump_accessibility_tree_browsertest.cc
(right):

https://codereview.chromium.org/826323002/diff/40001/content/browser/accessib...
content/browser/accessibility/dump_accessibility_tree_browsertest.cc:73:
base::FilePath event_file = test_path.Append(base::FilePath(file_path));
nit: this should be aria_file, not event_file

https://codereview.chromium.org/826323002/diff/40001/content/browser/accessib...
content/browser/accessibility/dump_accessibility_tree_browsertest.cc:84:
base::FilePath event_file = test_path.Append(base::FilePath(file_path));
should be html_file

[shreeramk, 2015-01-15 03:53:33]

The CQ bit was checked by shreeram.k@samsung.com

[commit-bot: I haz the power, 2015-01-15 03:54:01]

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

[commit-bot: I haz the power, 2015-01-15 04:24:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-01-15 04:24:49]

Try jobs failed on following builders:
  android_clang_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)

[shreeramk, 2015-01-15 17:58:41]

The CQ bit was checked by shreeram.k@samsung.com

[commit-bot: I haz the power, 2015-01-15 17:59:52]

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

[commit-bot: I haz the power, 2015-01-15 18:09:38]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-01-15 18:10:54]

Patchset 4 (id:??) landed as
https://crrev.com/845a27e777ee5b8ec20deda87001c86ce59fa13e
Cr-Commit-Position: refs/heads/master@{#311693}