Fix the .ycm_extra_conf.py and SublimeText documentation.

tools/vim/tests/chromium.ycm_extra_conf_unittest.py

Index: tools/vim/tests/chromium.ycm_extra_conf_unittest.py
diff --git a/tools/vim/tests/chromium.ycm_extra_conf_unittest.py b/tools/vim/tests/chromium.ycm_extra_conf_unittest.py
index b9a1ff4b7996ba3b446369b0dee0cda2cf7191be..c2fbd9f830d3c42847c73f9f556330e180ae570f 100755
--- a/tools/vim/tests/chromium.ycm_extra_conf_unittest.py
+++ b/tools/vim/tests/chromium.ycm_extra_conf_unittest.py
@@ -53,15 +53,6 @@ def CreateFile(path,
     statinfo = os.stat(path)
     os.chmod(path, statinfo.st_mode | stat.S_IXUSR)
 
-@unittest.skipIf(sys.platform.startswith('linux'),
-                 'Tests are only valid on Linux.')
-class Chromium_ycmExtraConfTest_NotOnLinux(unittest.TestCase):
-  def testAlwaysFailsIfNotRunningOnLinux(self):
-    self.fail('Changes to chromium.ycm_extra_conf.py currently need to be ' \
-              'uploaded from Linux since the tests only run on Linux.')
-
-@unittest.skipUnless(sys.platform.startswith('linux'),
-                     'Tests are only valid on Linux.')
 class Chromium_ycmExtraConfTest(unittest.TestCase):
 
   def SetUpFakeChromeTreeBelowPath(self):
@@ -104,7 +95,7 @@ class Chromium_ycmExtraConfTest(unittest.TestCase):
 
   def NormalizeString(self, string):
     return string.replace(self.out_dir, '[OUT]').\
-        replace(self.chrome_root, '[SRC]')
+        replace(self.chrome_root, '[SRC]').replace('\\', '/')
 
   def NormalizeStringsInList(self, list_of_strings):
     return [self.NormalizeString(s) for s in list_of_strings]
@@ -121,7 +112,7 @@ class Chromium_ycmExtraConfTest(unittest.TestCase):
     self.SetUpFakeChromeTreeBelowPath()
 
   def tearDown(self):
-    if self.test_root:
+    if False and self.test_root:

[scottmg, 2016/06/06 16:32:11]

?

[pastarmovj, 2016/06/07 08:52:29]

Sorry I needed to preserve the test files while fixing the test. I reverted
this. :)

       shutil.rmtree(self.test_root)
 
   def testNinjaIsAvailable(self):
@@ -172,8 +163,10 @@ class Chromium_ycmExtraConfTest(unittest.TestCase):
     clang_options = \
         self.ycm_extra_conf.GetClangOptionsFromNinjaForFilename(
             self.chrome_root, os.path.join(self.chrome_root, 'one.cpp'))
-    self.assertIn('-I%s/a' % out_dir, clang_options)
-    self.assertIn('-I%s/tag-one' % out_dir, clang_options)
+    self.assertIn('-I%s/a' % self.NormalizeString(out_dir),
+        self.NormalizeStringsInList(clang_options))
+    self.assertIn('-I%s/tag-one' % self.NormalizeString(out_dir),
+        self.NormalizeStringsInList(clang_options))
 
   def testGetFlagsForFileForKnownCppFile(self):
     result = self.ycm_extra_conf.FlagsForFile(
@@ -313,7 +306,8 @@ class Chromium_ycmExtraConfTest(unittest.TestCase):
     # output.
     p = subprocess.Popen(['ninja', '-C', self.out_dir, '-t',
                           'query', '../../four.cc'],
-                         stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
+                         stdout=subprocess.PIPE, stderr=subprocess.STDOUT,
+                         universal_newlines=True)
     stdout, _ = p.communicate()
     self.assertFalse(p.returncode)
     self.assertEquals(stdout,