DEPS: Update leveldatabase from 1.18 to 1.20.

storage/browser/BUILD.gn

 # Copyright 2014 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("//testing/test.gni")
 
 component("browser") {
   output_name = "storage_browser"
   sources = [
     "blob/blob_data_builder.cc",
     "blob/blob_data_builder.h",
@@ skipping 219 matching lines @@
     # This file will be removed when the real tests are moved over from
     # content_unittests to this target.
     "crbug653751_unittest.cc",
   ]
 
   deps = [
     "//base/test:run_all_unittests",
     "//base/test:test_support",
     "//testing/gtest",
   ]
+
+  # Trick for getting leveldb tests compiled on the CQ. http://crbug.com/697788

[Nico, 2017/03/02 14:11:27]

dpranke: ^

[Dirk Pranke, 2017/03/02 16:35:00]

Don't do this, it's making the build incorrect by claiming that there are
dependencies that aren't really there.
In addition, you're trying to subvert policies that were put in place
intentionally.

If you want to ensure that particular targets are built as part of the CQ, there
is an approved way to do that: add them to the `additional_compile_targets`
entries for a given builder in the //testing/buildbot/*.json files.

+  if (!is_android) {
+    deps += [ "//third_party/leveldatabase:leveldb_test_targets" ]
+  }
 }