[Findit] Improve output format and cherry-pick bugs fix.

tools/findit/chromium_deps_unittest.py

Index: tools/findit/chromium_deps_unittest.py
diff --git a/tools/findit/chromium_deps_unittest.py b/tools/findit/chromium_deps_unittest.py
index a26ac9c6e87232f2f130016ffa9473ae11b76b32..cf9ea649a80fc4bf486b3773c6cd0f4671e03d05 100644
--- a/tools/findit/chromium_deps_unittest.py
+++ b/tools/findit/chromium_deps_unittest.py
@@ -5,6 +5,7 @@
 import unittest
 
 import chromium_deps
+from common import utils
 
 
 class ChromiumDEPSTest(unittest.TestCase):
@@ -38,12 +39,13 @@ deps_os = {
 
   def testGetChromiumComponents(self):

[aarya, 2014/08/26 02:13:53]

Can you do the testing on these. these test an interesting case where webkit
revision is svn on one, and git on another. if you are using .DEPS.git
correctly, then both should be git hashes.

chromium
745dac28f8e454766824d6fefcd8063f1f11b1cb:744746cc51ef81c8f8d727fafa46b14d1c03fe44

for
https://chromium.googlesource.com/chromium/src/+/745dac28f8e454766824d6fefcd8...
"webkit_revision": "180779",

for
https://chromium.googlesource.com/chromium/src/+/744746cc51ef81c8f8d727fafa46...
'webkit_revision': '04b6022fdfb31a848d67683b58cd785aaeba9a69',

[stgao, 2014/08/26 18:35:24]

Good suggestion.
I added a new test testComponentRangeCrossGitMigrationPoint.

     chromium_revision = '283296'
+    chromium_revision_git_hash = 'b041fda2e8493dcb26aac08deb493943df240cbb'
     webkit_revision = '178200'
     breakpad_revision = '1345'
     liblouis_commit_hashcode = '3c2daee56250162e5a75830871601d74328d39f5'
 
     def _GetContentOfDEPS(chromium_revision_tmp):
-      self.assertEqual(chromium_revision_tmp, chromium_revision)
+      self.assertEqual(chromium_revision_tmp, chromium_revision_git_hash)
       return self.DEPS_TEMPLATE % (webkit_revision, breakpad_revision,
                                    liblouis_commit_hashcode)
 
@@ -65,10 +67,10 @@ deps_os = {
         },
         'src/': {
             'path': 'src/',
-            'repository_type': 'svn',
+            'repository_type': 'git',
             'name': 'chromium',
-            'repository': 'https://src.chromium.org/chrome/trunk',
-            'revision': chromium_revision
+            'repository': 'https://chromium.googlesource.com/chromium/src/',
+            'revision': chromium_revision_git_hash
         },
         'src/third_party/WebKit/': {
             'path': 'src/third_party/WebKit/',
@@ -85,22 +87,24 @@ deps_os = {
 
   def testGetChromiumComponentRange(self):
     chromium_revision1 = '283200'
+    chromium_revision_git_hash1 = 'c53c387f46a2ff0cf7c072222b826cff0817a80f'
     webkit_revision1 = '178084'
     breakpad_revision1 = '1345'
     liblouis_commit_hashcode1 = '3c2daee56250162e5a75830871601d74328d39f5'
 
     chromium_revision2 = '283296'
+    chromium_revision_git_hash2 = 'b041fda2e8493dcb26aac08deb493943df240cbb'
     webkit_revision2 = '178200'
     breakpad_revision2 = '1345'
     liblouis_commit_hashcode2 = '3c2daee56250162e5a75830871601d74328d39f5'
 
     def _GetContentOfDEPS(chromium_revision):
       chromium_revision = str(chromium_revision)
-      if chromium_revision == chromium_revision1:
+      if chromium_revision == chromium_revision_git_hash1:
         return self.DEPS_TEMPLATE % (webkit_revision1, breakpad_revision1,
                                      liblouis_commit_hashcode1)
       else:
-        self.assertEqual(chromium_revision2, chromium_revision)
+        self.assertEqual(chromium_revision, chromium_revision_git_hash2)
         return self.DEPS_TEMPLATE % (webkit_revision2, breakpad_revision2,
                                      liblouis_commit_hashcode2)
 
@@ -125,13 +129,13 @@ deps_os = {
             'repository_type': 'git'
         },
         'src/': {
-            'old_revision': chromium_revision1,
+            'old_revision': chromium_revision_git_hash1,
             'name': 'chromium',
-            'repository': 'https://src.chromium.org/chrome/trunk',
+            'repository': 'https://chromium.googlesource.com/chromium/src/',
             'rolled': True,
-            'new_revision': chromium_revision2,
+            'new_revision': chromium_revision_git_hash2,
             'path': 'src/',
-            'repository_type': 'svn'
+            'repository_type': 'git'
         },
         'src/third_party/WebKit/': {
             'old_revision': webkit_revision1,
@@ -149,6 +153,24 @@ deps_os = {
         deps_file_downloader=_GetContentOfDEPS)
     self.assertEqual(expected_results, components)
 
+  def _VerifyGitHashForAllComponents(self, deps):
+    self.assertTrue(isinstance(deps, dict))
+    for component in deps.values():
+      self.assertTrue(utils.IsGitHash(component['revision']))
+
   def testGetSvnRevision(self):
+    # For this case, svn revision needs converting to git hash and there will be
+    # .DEPS.git and DEPS.
     deps = chromium_deps.GetChromiumComponents(284750)
-    self.assertTrue(isinstance(deps, dict))
+    self._VerifyGitHashForAllComponents(deps)
+
+  def testGetGitRevision(self):
+    # For this case, there is only DEPS, not .DEPS.git.
+    deps = chromium_deps.GetChromiumComponents(
+        'f8b3fe9660d8dda318800f55d5e29799bbfd43f7')
+    self._VerifyGitHashForAllComponents(deps)
+
+    # For this case, there will be .DEPS.git.
+    deps = chromium_deps.GetChromiumComponents(
+        '8ae88241aa9f224e8ce97250f32469d616e437aa')
+    self._VerifyGitHashForAllComponents(deps)