cct: calls setClientDataHeaderForNewTab for hidden tabs
This adds a call to setClientDataHeaderForNewTab() for tabs created
when mayLaunchUrl is called. Without this, the right header is not
added if mayLaunchUrl() was called.
This also adds AppHooksImpl to the list of files that has multiple
paths, and changes the presubmit checking to prefer the public repo
in the case of multiple paths matching. This ensures that includes to
AppHooksImpl don't need to add DEPS to clank.
BUG=1207675
TEST=testMayLaunchUrlAddsClientDataHeader
Change-Id: I2ebbcb1b54ffae50203e0843f71eea5eb40b9fc3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2891644
Reviewed-by: Dirk Pranke <[email protected]>
Reviewed-by: Peter Conn <[email protected]>
Reviewed-by: Ted Choc <[email protected]>
Commit-Queue: Scott Violet <[email protected]>
Cr-Commit-Position: refs/heads/master@{#883000}
diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index 9444020..a75d8f9 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -1114,6 +1114,7 @@
)
_JAVA_MULTIPLE_DEFINITION_EXCLUDED_PATHS = [
+ r".*[\\/]AppHooksImpl\.java",
r".*[\\/]BuildHooksAndroidImpl\.java",
r".*[\\/]LicenseContentProvider\.java",
r".*[\\/]PlatformServiceBridgeImpl.java",
