Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(234)

Side by Side Diff: PRESUBMIT.py

Issue 2953473002: Use leveldb_env::OpenDB() to open leveldb databases. (Closed)
Patch Set: Don't dump in background mode Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """Top-level presubmit script for Chromium. 5 """Top-level presubmit script for Chromium.
6 6
7 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts 7 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
8 for more details about the presubmit API built into depot_tools. 8 for more details about the presubmit API built into depot_tools.
9 """ 9 """
10 10
(...skipping 366 matching lines...) Expand 10 before | Expand all | Expand 10 after
377 'AllowJavascript(), OnJavascriptAllowed()/OnJavascriptDisallowed(),', 377 'AllowJavascript(), OnJavascriptAllowed()/OnJavascriptDisallowed(),',
378 'and CallJavascriptFunction(). See https://goo.gl/qivavq.', 378 'and CallJavascriptFunction(). See https://goo.gl/qivavq.',
379 ), 379 ),
380 False, 380 False,
381 ( 381 (
382 r'^content[\\\/]browser[\\\/]webui[\\\/]web_ui_impl\.(cc|h)$', 382 r'^content[\\\/]browser[\\\/]webui[\\\/]web_ui_impl\.(cc|h)$',
383 r'^content[\\\/]public[\\\/]browser[\\\/]web_ui\.h$', 383 r'^content[\\\/]public[\\\/]browser[\\\/]web_ui\.h$',
384 r'^content[\\\/]public[\\\/]test[\\\/]test_web_ui\.(cc|h)$', 384 r'^content[\\\/]public[\\\/]test[\\\/]test_web_ui\.(cc|h)$',
385 ), 385 ),
386 ), 386 ),
387 (
388 'leveldb::DB::Open',
389 (
390 'Instead of leveldb::DB::Open() use leveldb_env::OpenDB() from',
391 'third_party/leveldatabase/env_chromium.h. It exposes databases to',
392 "Chrome's tracing, making their memory usage visible.",
393 ),
394 True,
395 (
396 r'^third_party/leveldatabase/.*\.(cc|h)$',
397 ),
398 )
387 ) 399 )
388 400
389 401
390 _IPC_ENUM_TRAITS_DEPRECATED = ( 402 _IPC_ENUM_TRAITS_DEPRECATED = (
391 'You are using IPC_ENUM_TRAITS() in your code. It has been deprecated.\n' 403 'You are using IPC_ENUM_TRAITS() in your code. It has been deprecated.\n'
392 'See http://www.chromium.org/Home/chromium-security/education/security-tips- for-ipc') 404 'See http://www.chromium.org/Home/chromium-security/education/security-tips- for-ipc')
393 405
394 406
395 _VALID_OS_MACROS = ( 407 _VALID_OS_MACROS = (
396 # Please keep sorted. 408 # Please keep sorted.
(...skipping 2015 matching lines...) Expand 10 before | Expand all | Expand 10 after
2412 output_api, 2424 output_api,
2413 json_url='http://chromium-status.appspot.com/current?format=json')) 2425 json_url='http://chromium-status.appspot.com/current?format=json'))
2414 2426
2415 results.extend( 2427 results.extend(
2416 input_api.canned_checks.CheckPatchFormatted(input_api, output_api)) 2428 input_api.canned_checks.CheckPatchFormatted(input_api, output_api))
2417 results.extend(input_api.canned_checks.CheckChangeHasBugField( 2429 results.extend(input_api.canned_checks.CheckChangeHasBugField(
2418 input_api, output_api)) 2430 input_api, output_api))
2419 results.extend(input_api.canned_checks.CheckChangeHasDescription( 2431 results.extend(input_api.canned_checks.CheckChangeHasDescription(
2420 input_api, output_api)) 2432 input_api, output_api))
2421 return results 2433 return results
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698