0

PRESUBMIT: Fix raw strings to use the "r" prefix

Bug: 397963615, 400761433
Change-Id: I3bcba6816cd2f91a5685bd7d8187eae31d574492
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6302852
Auto-Submit: Riley Wong <rgw@google.com>
Reviewed-by: Josip Sokcevic <sokcevic@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@google.com>
Cr-Commit-Position: refs/heads/main@{#1429458}
This commit is contained in:
Riley Wong
2025-03-07 06:18:57 -08:00
committed by Chromium LUCI CQ
parent dc2e49d2e0
commit f4ee964fb1
3 changed files with 25 additions and 25 deletions
buildtools/checkdeps
chrome/browser

@@ -141,7 +141,7 @@ class DepsBuilder(object):
being_tested=False, being_tested=False,
ignore_temp_rules=False, ignore_temp_rules=False,
ignore_specific_rules=False): ignore_specific_rules=False):
"""Creates a new DepsBuilder. r"""Creates a new DepsBuilder.
Args: Args:
base_directory: local path to root of checkout, e.g. C:\chr\src. base_directory: local path to root of checkout, e.g. C:\chr\src.
@@ -185,7 +185,7 @@ class DepsBuilder(object):
def _ApplyRules(self, existing_rules, includes, specific_includes, def _ApplyRules(self, existing_rules, includes, specific_includes,
cur_dir_norm): cur_dir_norm):
"""Applies the given include rules, returning the new rules. r"""Applies the given include rules, returning the new rules.
Args: Args:
existing_rules: A set of existing rules that will be combined. existing_rules: A set of existing rules that will be combined.

@@ -579,17 +579,17 @@ include_rules = [
] ]
specific_include_rules = { specific_include_rules = {
'.*Test\.java': [ r'.*Test\.java': [
# Android instrumentation tests depend on the whole app. # Android instrumentation tests depend on the whole app.
"+chrome/android", "+chrome/android",
"+chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java", "+chrome/android/java/src/org/chromium/chrome/browser/app/ChromeActivity.java",
"+chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java", "+chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java",
], ],
'.*TestUtils\.java': [ r'.*TestUtils\.java': [
"+chrome/android/java/src/org/chromium/chrome/browser/searchwidget/SearchActivity.java", "+chrome/android/java/src/org/chromium/chrome/browser/searchwidget/SearchActivity.java",
"+chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java", "+chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java",
], ],
"platform_util_ash\.cc": [ r"platform_util_ash\.cc": [
"+ash/wm/window_pin_util.h", "+ash/wm/window_pin_util.h",
], ],
"platform_util_linux.cc": [ "platform_util_linux.cc": [
@@ -597,7 +597,7 @@ specific_include_rules = {
# DBus interface to highlight a file within its parent folder # DBus interface to highlight a file within its parent folder
"+dbus" "+dbus"
], ],
"exo_parts\.cc": [ r"exo_parts\.cc": [
"+ash/shell.h", "+ash/shell.h",
], ],
# TODO(estade): the message center is initialized and shut down here, but it # TODO(estade): the message center is initialized and shut down here, but it
@@ -613,18 +613,18 @@ specific_include_rules = {
"+services/network/network_context.h", "+services/network/network_context.h",
"+services/network/network_service.h", "+services/network/network_service.h",
], ],
"bound_session_cookie_observer_unittest\.cc": [ r"bound_session_cookie_observer_unittest\.cc": [
"+services/network/cookie_manager.h", "+services/network/cookie_manager.h",
"+services/network/network_context.h", "+services/network/network_context.h",
"+services/network/network_service.h", "+services/network/network_service.h",
], ],
"chrome_authenticator_request_delegate_unittest\.cc": [ r"chrome_authenticator_request_delegate_unittest\.cc": [
"+third_party/microsoft_webauthn/webauthn.h", "+third_party/microsoft_webauthn/webauthn.h",
], ],
"chrome_content_browser_client_receiver_bindings\.cc" : [ r"chrome_content_browser_client_receiver_bindings\.cc" : [
"+chrome/browser/performance_manager/mechanisms/userspace_swap_chromeos.h", "+chrome/browser/performance_manager/mechanisms/userspace_swap_chromeos.h",
], ],
"chrome_content_browser_client\.cc" : [ r"chrome_content_browser_client\.cc" : [
"+ash/shell.h", "+ash/shell.h",
"+ash/multi_capture/multi_capture_service.h", "+ash/multi_capture/multi_capture_service.h",
"+content/public/browser/tts_controller.h", "+content/public/browser/tts_controller.h",
@@ -632,12 +632,12 @@ specific_include_rules = {
"chrome_navigation_browsertest.cc" : [ "chrome_navigation_browsertest.cc" : [
"+content/common/content_navigation_policy.h", "+content/common/content_navigation_policy.h",
], ],
"about_flags\.cc" : [ r"about_flags\.cc" : [
"+mojo/core/embedder/features.h", "+mojo/core/embedder/features.h",
"+sandbox/policy/features.h", "+sandbox/policy/features.h",
"+skia/rusty_png_feature.h", "+skia/rusty_png_feature.h",
], ],
"dbus_memory_pressure_evaluator_linux(_unittest)?\.(cc|h)" : [ r"dbus_memory_pressure_evaluator_linux(_unittest)?\.(cc|h)" : [
"+dbus", "+dbus",
], ],
"permission_element_browsertest.cc": [ "permission_element_browsertest.cc": [
@@ -646,7 +646,7 @@ specific_include_rules = {
"browsing_data_model_browsertest.cc" : [ "browsing_data_model_browsertest.cc" : [
"+components/services/storage/shared_storage/shared_storage_manager.h", "+components/services/storage/shared_storage/shared_storage_manager.h",
], ],
"ash_session_restore_page_load_metrics_observer_browsertest\.cc": [ r"ash_session_restore_page_load_metrics_observer_browsertest\.cc": [
"+ash/wm/window_restore/window_restore_util.h", "+ash/wm/window_restore/window_restore_util.h",
], ],
"render_view_context_menu_browsertest.cc" : [ "render_view_context_menu_browsertest.cc" : [

@@ -45,46 +45,46 @@ include_rules = [
] ]
specific_include_rules = { specific_include_rules = {
"application_launch_browsertest\.cc": [ r"application_launch_browsertest\.cc": [
"+ash/shell.h", "+ash/shell.h",
], ],
"browser_command_controller_browsertest\.cc": [ r"browser_command_controller_browsertest\.cc": [
"+ash/wm/window_pin_util.h", "+ash/wm/window_pin_util.h",
], ],
"browser_finder_chromeos_browsertest\.cc": [ r"browser_finder_chromeos_browsertest\.cc": [
"+ash/wm/desks/desk.h", "+ash/wm/desks/desk.h",
"+ash/wm/desks/desks_controller.h", "+ash/wm/desks/desks_controller.h",
"+ash/wm/desks/desks_test_util.h", "+ash/wm/desks/desks_test_util.h",
], ],
"browser_navigator_browsertest\.cc": [ r"browser_navigator_browsertest\.cc": [
"+ash/shell.h", "+ash/shell.h",
], ],
"browser_navigator_browsertest_chromeos\.cc": [ r"browser_navigator_browsertest_chromeos\.cc": [
"+ash/wm/window_pin_util.h", "+ash/wm/window_pin_util.h",
], ],
"browser_tabstrip_browsertest_chromeos\.cc": [ r"browser_tabstrip_browsertest_chromeos\.cc": [
"+ash/wm/window_pin_util.h", "+ash/wm/window_pin_util.h",
], ],
"dark_mode_manager_linux(_unittest)?\.cc": [ r"dark_mode_manager_linux(_unittest)?\.cc": [
"+dbus", "+dbus",
], ],
"fullscreen_controller_browsertest\.cc": [ r"fullscreen_controller_browsertest\.cc": [
"+ash/wm/window_pin_util.h", "+ash/wm/window_pin_util.h",
], ],
"fullscreen_controller_interactive_browsertest\.cc": [ r"fullscreen_controller_interactive_browsertest\.cc": [
"+ash/shell.h", "+ash/shell.h",
"+ash/wm/window_pin_util.h", "+ash/wm/window_pin_util.h",
], ],
"popup_browsertest\.cc": [ r"popup_browsertest\.cc": [
"+ash/shell.h", "+ash/shell.h",
], ],
"sad_tab_helper.cc": [ "sad_tab_helper.cc": [
"+content/common/content_navigation_policy.h", "+content/common/content_navigation_policy.h",
], ],
"settings_window_manager_chromeos\.cc": [ r"settings_window_manager_chromeos\.cc": [
"+ash/wm/window_properties.h", "+ash/wm/window_properties.h",
], ],
"hid_chooser_controller_unittest\.cc": [ r"hid_chooser_controller_unittest\.cc": [
"+services/device/hid", "+services/device/hid",
], ],
} }