Fix the .ycm_extra_conf.py and SublimeText documentation.

tools/vim/chromium.ycm_extra_conf.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Autocompletion config for YouCompleteMe in Chromium.
 #
 # USAGE:
 #
 #   1. Install YCM [https://github.com/Valloric/YouCompleteMe]
 #          (Googlers should check out [go/ycm])
@@ skipping 144 matching lines @@
 
   Returns:
     (List of Strings) List of target names. Will return [] if |filename| doesn't
         yield any .o or .obj outputs.
   """
   # Ninja needs the path to the source file relative to the output build
   # directory.
   rel_filename = os.path.relpath(os.path.realpath(filename), out_dir)
 
   p = subprocess.Popen(['ninja', '-C', out_dir, '-t', 'query', rel_filename],
-                       stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
+                       stdout=subprocess.PIPE, stderr=subprocess.STDOUT,
+                       universal_newlines=True)
   stdout, _ = p.communicate()
-  if p.returncode:
+  if p.returncode != 0:
     return []
 
   # The output looks like:
   #   ../../relative/path/to/source.cc:
   #     outputs:
   #       obj/reative/path/to/target.source.o
   #       obj/some/other/target2.source.o
   #       another/target.txt
   #
   outputs_text = stdout.partition('\n  outputs:\n')[2]
@@ skipping 11 matching lines @@
   Args:
     out_dir: (String) Absolute path to ninja build output directory.
     build_target: (String) A build target understood by ninja
 
   Returns:
     (String or None) Clang command line or None if a Clang command line couldn't
         be determined.
   """
   p = subprocess.Popen(['ninja', '-v', '-C', out_dir,
                         '-t', 'commands', build_target],
-                       stdout=subprocess.PIPE)
+                       stdout=subprocess.PIPE, universal_newlines=True)
   stdout, stderr = p.communicate()
-  if p.returncode:
+  if p.returncode != 0:
     return None
 
   # Ninja will return multiple build steps for all dependencies up to
   # |build_target|. The build step we want is the last Clang invocation, which
   # is expected to be the one that outputs |build_target|.
   for line in reversed(stdout.split('\n')):
     if 'clang' in line:
       return line
   return None
 
@@ skipping 112 matching lines @@
   clang_line = GetClangCommandLineFromNinjaForSource(
       out_dir, GetBuildableSourceFile(chrome_root, filename))
   if not clang_line:
     # If ninja didn't know about filename or it's companion files, then try a
     # default build target. It is possible that the file is new, or build.ninja
     # is stale.
     clang_line = GetClangCommandLineFromNinjaForSource(
         out_dir, GetDefaultSourceFile(chrome_root, filename))
 
   if not clang_line:
-    return (additional_flags, [])
+    return additional_flags
 
   return GetClangOptionsFromCommandLine(clang_line, out_dir, additional_flags)
 
 
 def FlagsForFile(filename):
   """This is the main entry point for YCM. Its interface is fixed.
 
   Args:
     filename: (String) Path to source file being edited.
 
@@ skipping 10 matching lines @@
   # transient failure. Preventing YCM from caching the flags allows us to try to
   # determine the flags again.
   should_cache_flags_for_file = bool(clang_flags)
 
   final_flags = _default_flags + clang_flags
 
   return {
     'flags': final_flags,
     'do_cache': should_cache_flags_for_file
   }