Store hashes of current and previous shipped V8 version

tools/release/releases.py

Index: tools/release/releases.py
diff --git a/tools/release/releases.py b/tools/release/releases.py
index e7000ac9fcf83e0be7fc5aa58b36664c34178164..e12d3436abbf3494e978567562f031c8bf246f6e 100755
--- a/tools/release/releases.py
+++ b/tools/release/releases.py
@@ -450,19 +450,44 @@ class RetrieveInformationOnChromeReleases(Step):
 
     for current_os in recent_releases:
       for current_version in current_os["versions"]:
+
+        if current_version["channel"] != "canary":
+          continue
+
+        url_to_call = "http://omahaproxy.appspot.com/v8.json?version=" + current_version["previous_version"]

[Michael Achenbach, 2015/04/16 11:32:50]

nit: 80 chars

+        result_raw = self.ReadURL(
+                             url_to_call,
+                             params,
+                             wait_plan=[5, 20]
+                             )
+        previous_v8_version_wrapper = json.loads(result_raw)

[Michael Achenbach, 2015/04/16 11:32:50]

inline local variable previous_v8_version_wrapper

[Michael Hablich, 2015/04/16 12:01:02]

What do you mean by that?

[Michael Achenbach, 2015/04/16 12:17:56]

I mean:
previous_v8_version = json.loads(result_raw)["v8_version"]

+        previous_v8_version = previous_v8_version_wrapper["v8_version"]
+        v8_previous_version_hash = self._GetGitHashForV8Version(previous_v8_version)
+
+        current_V8_version = current_version["v8_version"]

[Michael Achenbach, 2015/04/16 11:32:50]

nit: current_v8_version

[Michael Hablich, 2015/04/16 12:01:02]

Acknowledged.

+        v8_version_hash = self._GetGitHashForV8Version(current_V8_version)
+
         current_candidate = {
                             "chrome_version": current_version["version"],
-                           "os": current_version["os"],
-                           "release_date": current_version["current_reldate"],
-                           "v8_version": current_version["v8_version"],
+                            "os": current_version["os"],
+                            "release_date": current_version["current_reldate"],
+                            "v8_version": current_V8_version,
+                            "v8_version_hash": v8_version_hash,
+                            "v8_previous_version": previous_v8_version,
+                            "v8_previous_version_hash": v8_previous_version_hash,
                            }
 
-        if current_version["channel"] == "canary":
-          canaries.append(current_candidate)
+        canaries.append(current_candidate)
 
     chrome_releases = {"canaries": canaries}
     self["chrome_releases"] = chrome_releases
 
+  def _GetGitHashForV8Version(self, v8_version):
+    if v8_version.split(".")[3]== "0":
+        return self.GitGetHashOfTag(v8_version[:-2])

[Michael Achenbach, 2015/04/16 11:32:50]

nit: 2 space indentation + space before ==

[Michael Hablich, 2015/04/16 12:01:02]

Acknowledged.

+
+    return self.GitGetHashOfTag(v8_version)
+
 
 class CleanUp(Step):
   MESSAGE = "Clean up."