Android: Update version of retrace.jar to 6.0.3

The existing version was found to infinite loop at times.
Also moves the jars to CIPD

NOTRY=true # windows bot failing

Bug: 876539
Change-Id: If822837f4958434372ef0cb40799aec32ec7cd37
Reviewed-on: https://chromium-review.googlesource.com/1187003
Commit-Queue: agrieve <agrieve@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#585887}

DEPS

@@ -950,6 +950,17 @@ deps = {
       'condition': 'checkout_win',
   },
 
+  'src/third_party/proguard': {
+      'packages': [
+          {
+              'package': 'chromium/third_party/proguard',
+              'version': '3bd778c422ea5496de2ef25c007a517dbb5ce5ca',
+          },
+      ],
+      'condition': 'checkout_android',
+      'dep_type': 'cipd',
+  },
+
   # Dependency of chromite.git and skia.
   'src/third_party/pyelftools': {
       'url': Var('chromium_git') + '/chromiumos/third_party/pyelftools.git' + '@' + '19b3e610c86fcadb837d252c794cb5e8008826ae',

build/android/pylib/utils/proguard.py

@@ -32,14 +32,14 @@ _PROGUARD_INDENT_WIDTH = 2
 _PROGUARD_ANNOTATION_VALUE_RE = re.compile(r'^(\s*?)- \S+? \[(.*)\]$')
 
 _PROGUARD_PATH_SDK = os.path.join(
-    constants.PROGUARD_ROOT, 'lib', 'proguard.jar')
+    constants.PROGUARD_ROOT, 'lib', 'proguard603.jar')
 _PROGUARD_PATH_BUILT = (
     os.path.join(os.environ['ANDROID_BUILD_TOP'], 'external', 'proguard',
                  'lib', 'proguard.jar')
     if 'ANDROID_BUILD_TOP' in os.environ else None)
 _PROGUARD_PATH = (
     _PROGUARD_PATH_SDK if os.path.exists(_PROGUARD_PATH_SDK)
-    else _PROGUARD_PATH_BUILT)
+    else _PROGUARD_PATH_BUILT or _PROGUARD_PATH_SDK)
 
 
 def Dump(jar_path):

third_party/.gitignore

@@ -167,6 +167,7 @@
 /perfetto
 /perl
 /ppapi
+/proguard/lib/
 /psyco_win32
 /pthreads-win32
 /py_trace_event/src

third_party/proguard/BUILD.gn

@@ -4,19 +4,22 @@
 
 import("//build/config/android/rules.gni")
 
-java_prebuilt("proguard_java") {
-  jar_path = "lib/proguard.jar"
+# This target is used only by the test runner to list tests as well as to
+# deobfuscate via retrace. proguard.jar (v5.2.1) is used when building, and is
+# reference directly via "java -jar proguar.jar"
+java_prebuilt("proguard603_java") {
+  jar_path = "lib/proguard603.jar"
   data = [
-    "$root_build_dir/lib.java/third_party/proguard/proguard.jar",
+    "$root_build_dir/lib.java/third_party/proguard/proguard603.jar",
   ]
 }
 
 java_prebuilt("retrace_java") {
-  jar_path = "lib/retrace.jar"
+  jar_path = "lib/retrace603.jar"
   deps = [
-    ":proguard_java",
+    ":proguard603_java",
   ]
   data = [
-    "$root_build_dir/lib.java/third_party/proguard/retrace.jar",
+    "$root_build_dir/lib.java/third_party/proguard/retrace603.jar",
   ]
 }

third_party/proguard/README.chromium

 Name: Proguard
 URL: http://proguard.sourceforge.net/
-Version: 5.2.1
+Version: 5.2.1_and_6.0.3
 Date: June 22, 2016
 License: GPL v2
 License File: NOT_SHIPPED
@@ -10,3 +10,7 @@ Description:
 This directory includes proguard.jar to allow Chromium to shrink, optimize java
 classes for Android, as well as retrace.jar, to allow for deobfuscation of stack
 traces.
+
+Proguard is kept at 5.2.1 because all newer versions were found to optimize in a
+way that breaks Chrome. Retrace is at 6.0.3 because version 5.2.1 was found to
+infinite loop when deobfuscating certainly strings.

third_party/proguard/cipd.yaml

+# Copyright 2018 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.
+
+# To create CIPD package run the following command.
+# cipd create --pkg-def cipd.yaml
+package: chromium/third_party/proguard
+data:
+  - file: lib/proguard.jar
+  - file: lib/proguard603.jar
+  - file: lib/retrace603.jar