Fix presubmit script for .cc files which include system files first.

PRESUBMIT_test.py

 #!/usr/bin/env python
 # 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.
 
 import glob
 import json
 import os
 import re
 import subprocess
@@ skipping 116 matching lines @@
   def testSpecialFirstInclude6(self):
     mock_input_api = MockInputApi()
     contents = ['#include "some/other/path/foo_win.h"',
                 '#include <set>',
                 '#include "a/header.h"']
     mock_file = MockFile('some/path/foo_unittest_win.h', contents)
     warnings = PRESUBMIT._CheckIncludeOrderInFile(
         mock_input_api, mock_file, range(1, len(contents) + 1))
     self.assertEqual(0, len(warnings))
 
+  def testSpecialFirstInclude7(self):
+    mock_input_api = MockInputApi()
+    contents = ['#include "ipc/some_macros.h"',

[Paweł Hajdan Jr., 2015/09/03 13:23:52]

This doesn't look like a realistic scenario we'd like to support.

That makes it less suitable for a test.

Do you possibly have a better scenario? Looks like with maybe fixing
gpu_channel.cc no changes would be necessary here. Are any other files in the
tree affected?

[David Yen, 2015/09/03 16:39:27]

Done. Changed the example to windows.h to be more realistic.

+                '#include "some/other/path/foo.h"',
+                '#include <set>',
+                '#include "a/header.h"']
+    mock_file = MockFile('some/path/foo.cc', contents)
+    warnings = PRESUBMIT._CheckIncludeOrderInFile(
+        mock_input_api, mock_file, range(1, len(contents) + 1))
+    print warnings
+    self.assertEqual(0, len(warnings))
+
   def testOrderAlreadyWrong(self):
     scope = [(1, '#include "b.h"'),
              (2, '#include "a.h"'),
              (3, '#include "c.h"')]
     mock_input_api = MockInputApi()
     warnings = PRESUBMIT._CheckIncludeOrderForScope(scope, mock_input_api,
                                                     '', [3])
     self.assertEqual(0, len(warnings))
 
   def testConflictAdded1(self):
@@ skipping 772 matching lines @@
     self.assertTrue('HasInlineTag.java:4' in msgs[2].items)
 
     # Util Log usage
     self.assertEqual(2, len(msgs[3].items))
     self.assertTrue('HasAndroidLog.java:3' in msgs[3].items)
     self.assertTrue('IsInBasePackageButImportsLog.java:4' in msgs[3].items)
 
 
 if __name__ == '__main__':
   unittest.main()