Only include unique lines when patching orderfile.

tools/cygprofile/patch_orderfile_unittest.py

 #!/usr/bin/python
 # Copyright 2015 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 patch_orderfile
 import unittest
 
 
 class TestPatchOrderFile(unittest.TestCase):
@@ skipping 83 matching lines @@
                patch_orderfile.SymbolInfo(0x42, 0x12, symbol_name2)]}
     symbol_names = patch_orderfile._ExpandSymbolsWithDupsFromSameOffset(
         symbols, offset_to_symbol_infos)
     self.assertEquals(len(symbol_names), 2)
     self.assertEquals(symbol_names[0], symbol_name)
     self.assertEquals(symbol_names[1], symbol_name2)
 
   def testPrintSymbolWithPrefixes(self):
     class FakeOutputFile(object):
       def __init__(self):
-        self.output = []
+        self.output = ''
       def write(self, s):
-        self.output.append(s)
+        self.output = self.output + s

[pasko, 2015/01/22 17:37:14]

did the old version not work? if so, why?

[azarchs, 2015/01/22 17:44:30]

The old one resulted in extra '\n's between output lines.  This did not cause a
test failure in the old version because only one call was made to write().

     test_symbol = "dummySymbol"
     symbol_names = [test_symbol]
     fake_output = FakeOutputFile()
     patch_orderfile._PrintSymbolsWithPrefixes(symbol_names, fake_output)
     expected_output = """.text.startup.dummySymbol
 .text.hot.dummySymbol
 .text.unlikely.dummySymbol
 .text.dummySymbol
 """
-    self.assertEquals("\n".join(fake_output.output), expected_output)
+    self.assertEquals(fake_output.output, expected_output)
 
 
 if __name__ == "__main__":
   unittest.main()