- 27 Jul, 2020 40 commits
-
-
Richard He authored
Use vpython for `generate_gradle.py` and `list_java_targets.py` Change-Id: I39bb601add8cc1c019a3aab9bca60bee0e341084 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315400 Commit-Queue: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#791917}
-
Devlin Cronin authored
Update tests under //chrome/browser/pdf to use WebContentsConsoleObserver instead of ConsoleObserverDelegate. This allows us to observe console messages added without overriding the existing delegate of the WebContents. Bug: 1034150 Change-Id: I03badfdb5956066553b4d6d278aae6b34ad94715 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2316702Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#791916}
-
Devlin Cronin authored
Update remaining tests under //chrome/browser to use WebContentsConsoleObserver instead of ConsoleObserverDelegate. This allows us to observe console messages added without overriding the existing delegate of the WebContents. Bug: 1034150 Change-Id: Ib6da6799bb906b7547ab8d4fce4066a0a976df24 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2316948Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#791915}
-
John Lee authored
Bug: 1084620 Change-Id: Idde2fe2bc1ce4f387a20a00d01aa7c8ba13354a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2311039 Commit-Queue: John Lee <johntlee@chromium.org> Reviewed-by:
dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#791914}
-
Sophie Chang authored
Bug: 1108891 Change-Id: I627b0ca7aa08dbe3717654ff6ba1cb9233237540 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2316324Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Sophie Chang <sophiechang@chromium.org> Cr-Commit-Position: refs/heads/master@{#791913}
-
Guillaume Jenkins authored
In preparation for bringing Chrome Browser Cloud Management to iOS, this adds a command-line switch to control whether CBCM is enabled. It needs to be a command-line flag because policy infrastructure is initialized before Finch / about flags and field trials. Bug: 1066495 Change-Id: I0bd27e5ee863deda949afa64b248ca5445f9ef8e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2316620 Commit-Queue: Guillaume Jenkins <gujen@google.com> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#791912}
-
Liviu Tinta authored
Because word is highlighted the cursor position is lost. The fix moves highlighting after code knows if the event was 'preventdefaulted'. Bug: 1076078 Change-Id: I08ebece22beee7cc71dd6e8241cb42f7be2e0331 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2183174Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Liviu Tinta <liviutinta@chromium.org> Cr-Commit-Position: refs/heads/master@{#791911}
-
Ian Kilpatrick authored
TBR: atotic@chromium.org Bug: 1108224 Change-Id: I29901956f130097aec56b1f37def05595c59664f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2318550Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#791910}
-
Lijin Shen authored
Update email address in presubmit warning message Bug: 1109248 Change-Id: If55f0651bf1606a596dfb65733d01d23411e7db9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317845Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Lijin Shen <lazzzis@google.com> Cr-Commit-Position: refs/heads/master@{#791909}
-
Alexei Svitkine authored
Bug: None Change-Id: Id0981a264b247208a78ba0cddad59166528c53fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304974 Auto-Submit: Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Commit-Queue: Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#791908}
-
David Benjamin authored
Instead parent the sheet to the specified gfx::NativeWindow directly. This avoids a host of bugs around the native constrained dialogs machinery, but does mean the dialog is now window-modal rather than tab-modal. On Windows, where we also use the native dialog, the sheet is similarly window-modal and the certificate viewer only ever opens in response to the user clicking in browser-owned UI or DevTools, so there shouldn't be an abuse concern. See https://crbug.com/1098786#c4 Bug: 762915, 1078158, 1098786 Change-Id: Iafeee58fd04aff9f8b88e8c2f030fe18913f81f2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2313160Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: David Benjamin <davidben@chromium.org> Cr-Commit-Position: refs/heads/master@{#791907}
-
Ben Kelly authored
Previously BlobBytesConsumer would read the BlobData object by first create a DataPipeGetter and then passing the resulting pipe to a FormDataBytesConsumer. The FormDataBytesConsumer would then internally use a DataPipeBytesConsumer to read the pipe. This CL attempts to simply things by having BlobBytesConsumer use a DataPipeBytesConsumer directly. In addition, it uses BlobData->ReadAll() avoiding the need to create a DataPipeGetter. This removes one mojo call. Bug: 1076081 Change-Id: I0f270cd99c9808c85d43523ac700aafe75d49c7a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2171017Reviewed-by:
Tsuyoshi Horo <horo@chromium.org> Commit-Queue: Ben Kelly <wanderview@chromium.org> Cr-Commit-Position: refs/heads/master@{#791906}
-
Hiroki Nakagawa authored
As a part of the Onion Soup project, this CL moves PreviewsState from content/public/common/ to blink/public/common/. In addition, this merges WebURLRequest::PreviewsState into that. Notable changes in this CL: - PreviewTypes enums define both FOO_BAR and kFooBar styles to avoid non-essential diffs. FOO_BAR is used for the content side, and kFooBar is used for the blink side. Follow-up CLs will merge FOO_BAR into kFooBar. - This CL adds a new dependency rule in component/previews/content/DEPS to allow to include blink/public/common/previews_state.h from there. Change-Id: I5ce7e8bb844416ac2482143665925e08b7858746 Bug: 860403 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2319489Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Reviewed-by:
Cait Phillips <caitkp@chromium.org> Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#791905}
-
Evan Stade authored
Bug: 1097216 Change-Id: Ibdb9a50ea5f6a7313bd65f45cf9bd38744eb9469 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317685Reviewed-by:
Dirk Pranke <dpranke@google.com> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#791904}
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1595789207-a5f1c4250bc5f6d42bdf1b980d8df879d82cd223.profdata to chrome-win32-master-1595851043-c14119a80137ce7090e196461287429806779e74.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win32-chromium Please CC sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: chrome/try:win-chrome Tbr: sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com Change-Id: Id4a39f0baf4669bf2233c14ea24208e3fe4eb4cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320191Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#791903}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/60cbf70944cf..4fd830acab6f 2020-07-27 landrey@webrtc.org Add possibility to adjust frames before comparison in pc level tests If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: I05f3f0b46f26c18e8755e1f1e24387e8ff5273ec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320043Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#791902}
-
arthursonzogni authored
A python server is used has a communication channel in between documents from different browsing context group and potentially cross-origin. There are two methods: - one for writing a message. - one for reading the message. I initially supposed we won't need to store more than one message in the stash. It turns out there are tests doing two write in a row without any read in the middle. It was throwing an exception when this happened. This patch makes the server to maintain a queue of messages instead. It doesn't throw errors anymore. Bug: chromium:1109194 Change-Id: I25f8dc2c2a032781a4527a0994e94e46f85e12c9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317865Reviewed-by:
Pâris Meuleman <pmeuleman@chromium.org> Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Cr-Commit-Position: refs/heads/master@{#791901}
-
Adithya Srinivasan authored
Bug: 1109714 Change-Id: If2ed226e624c78a97c8a7fe5434520464295eba9 No-Try: True Tbr: achuith@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320129Reviewed-by:
Adithya Srinivasan <adithyas@chromium.org> Reviewed-by:
Achuith Bhandarkar <achuith@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#791900}
-
Luis G Garcia authored
Update README.chromium to include instructions on creating a cloud bucket such as "chromium-webxr-test". Change-Id: I0e3e575be60d35436246dc6e67eb8cb104742552 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2314198Reviewed-by:
Klaus Weidner <klausw@chromium.org> Commit-Queue: Luis Garcia <luisggarcia@google.com> Cr-Commit-Position: refs/heads/master@{#791899}
-
Marc Treib authored
The original plan was to put all account-storage-related code into such subfolders, but that hasn't happened, and this factory was in fact the only thing in such a subfolder. So for consistency with all the other code, move it out of the subfolder as well. Bug: none Change-Id: I5dcccbb960203cafd46f13460ad784b1ca6de85c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317962Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#791898}
-
Richard Coles authored
This reverts commit e44d3738. Reason for revert: It looks to have been necessary after all; crbug.com/1107830 Original change's description: > webview: don't use UI thread pool unnecessarily. > > Just use the general pool at best-effort priority for logging command > line and field trials during startup, since it doesn't need to be on the > UI thread. > > Change-Id: I73dfb14ae67bcc953fa7c90b5840354d5b4a84b3 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302946 > Reviewed-by: Nate Fischer <ntfschr@chromium.org> > Commit-Queue: Richard Coles <torne@chromium.org> > Cr-Commit-Position: refs/heads/master@{#789538} TBR=torne@chromium.org,ntfschr@chromium.org,hazems@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Fixed: 1107830 Change-Id: If9f8038787cc0a16377aca599cbcb18efa055463 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2316414Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Nate Fischer <ntfschr@chromium.org> Commit-Queue: Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#791897}
-
Tanmoy Mollik authored
The parent cl should fix the failure Bug: 1102296 Change-Id: Iea806b0dc067a2966651482c112845d6c2be4c4b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317189Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org> Cr-Commit-Position: refs/heads/master@{#791896}
-
Nodir Turakulov authored
This builder is superseded by metadata-exporter builder. Depends on https://chromium-review.googlesource.com/c/chromium/tools/build/+/2313785 Bug: 1106176 Change-Id: Ia391aea0fc36fb55260f1289d30b789f23fb02a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2314032Reviewed-by:
Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Garrett Beaty <gbeaty@chromium.org> Auto-Submit: Nodir Turakulov <nodir@chromium.org> Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Cr-Commit-Position: refs/heads/master@{#791895}
-
Olivier Robin authored
After discussion with chrome-privacy, this can be enabled for users who have opted-in UMA. Bug: 1017434 Change-Id: I140dac0327206846127be4befa3e6c9fca8780fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2292510 Commit-Queue: Olivier Robin <olivierrobin@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#791894}
-
Maksim Sadym authored
Move IdleManager to RenderFrameHost for better isolation of simulating Idle Status. Bug: 1090802 Change-Id: Ia0e3ca436e612aaba29dc9880c69ae54305f13d3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2308722Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Mathias Bynens <mathias@chromium.org> Reviewed-by:
Alex Rudenko <alexrudenko@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Auto-Submit: Maksim Sadym <sadym@chromium.org> Cr-Commit-Position: refs/heads/master@{#791893}
-
Justin Cohen authored
Bug: 672917 Change-Id: Ie28e6de56bcfea09bf9462236665c31497e2494b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2311116 Auto-Submit: Justin Cohen <justincohen@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#791892}
-
Mark Brand authored
https://chromium-review.googlesource.com/c/chromium/src/+/2302609 revealed that the dependencies are broken for the MojoLPM bindings generators. Suspect that this is also the root cause of the linked bug Bug: 1108739 Change-Id: I1cac1fbf2a2cc139f972f7e6e605bd4a70323c33 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315340Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Mark Brand <markbrand@google.com> Cr-Commit-Position: refs/heads/master@{#791891}
-
Kyle Horimoto authored
Provides a simpler mechanism for checking whether the associated flag is enabled. Bug: 1106937 Change-Id: I7b54cae69f52d25ea6aa50843ab2e0b7e88ed7c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317768Reviewed-by:
Josh Nohle <nohle@chromium.org> Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#791890}
-
Leonard Grey authored
This reverts commit 33253a15. Reason for revert: Broke PWA content fullscreen https://crbug.com/1109608 Original change's description: > Mac: modernize menu reveal tracking > > The fullscreen toolbar controller currently uses undocumented Carbon > events to track the menubar's reveal amount in fullscreen. In recent > macOS versions, the intermediate values are no longer send and we > only receive events when the menubar is fully shown or fully hidden. > > The Mac immersive code which is still behind a flag uses a different > technique, observing the reveal amount on a hidden titlebar accessory > view. This *does* provide correct intermediate values. > > This change extracts the reveal code from the immersive controller > and uses it in the current fullscreen code rather than the Carbon > event. Unfortunately, despite receiving the correct progress > notifications, the topchrome still does a discontinuous jump, but > we'll cross that bridge later. > > Bug: 1063417 > Change-Id: I6827cafbcad58eeaabf98c6facaeb85ce3439997 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2314998 > Reviewed-by: Avi Drissman <avi@chromium.org> > Commit-Queue: Leonard Grey <lgrey@chromium.org> > Cr-Commit-Position: refs/heads/master@{#791273} TBR=avi@chromium.org,lgrey@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1063417 Change-Id: Ie70a1ce26feddaa6aec2e0f061b8dd7656fcf55e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320409Reviewed-by:
Leonard Grey <lgrey@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Leonard Grey <lgrey@chromium.org> Cr-Commit-Position: refs/heads/master@{#791889}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/f23e4df5..2f54b1a2 Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,vahl@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: If7c449219f01f3304d184bded2e8d3e7f34edf04 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320348Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#791888}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/84ab9b3dd3ae..f2780c629ddc 2020-07-27 hjd@google.com Merge "Investigate SaveTraceIntoDropboxAndIncidentOrCrash" If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:linux-perfetto-rel Bug: None Tbr: perfetto-bugs@google.com Change-Id: Icc6d3b45d0c355afece4410dc436bb3a22c57c32 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320037Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#791887}
-
sauski authored
Create a generated preference to support the Password Leak Detection toggle within chrome://settings/security. This toggle has additional behavior based on profile state that currently requires additional front-end JS logic. This CL recreates this logic within C++ code and exposes it via a generated preference. Bug: 1063265 Change-Id: Iacf35ae323bfb22fbc24702e369bee866babb982 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2283610Reviewed-by:
dpapad <dpapad@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Commit-Queue: Theodore Olsauskas-Warren <sauski@google.com> Cr-Commit-Position: refs/heads/master@{#791886}
-
Aaron Colwell authored
This reverts commit 592b68f4. Reason for revert: Caused a significant number of crashes in canary builds. http://crbug.com/1109463 Original change's description: > Crash if a BrowserContext is referenced by a RenderProcessHost at destruction. > > Adding an extra sanity check in the BrowserContext destructor to > immediately terminate if we detect that a RenderProcessHost still > references the object being destroyed. > > > Bug: 1099998 > Change-Id: I84af3b5d5f09274fa551b63903101ab5710d073d > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2305674 > Auto-Submit: Aaron Colwell <acolwell@chromium.org> > Reviewed-by: Nasko Oskov <nasko@chromium.org> > Commit-Queue: Aaron Colwell <acolwell@chromium.org> > Cr-Commit-Position: refs/heads/master@{#791282} TBR=nasko@chromium.org,acolwell@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1099998, 1109463 Change-Id: Id15033cbe5880c92d6985d199962c199aa7e0a80 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320410Reviewed-by:
Aaron Colwell <acolwell@chromium.org> Commit-Queue: Aaron Colwell <acolwell@chromium.org> Cr-Commit-Position: refs/heads/master@{#791885}
-
Edward Lesmes authored
checkteamtags is no longer necessary. - Add a check to validate DIR_METADATA files. This obviates the need fo checkteamtags, as `dirmd` validates metadata in OWNERS files, and we no longer need to enforce a unique team email for all (component, OS) pair. - Add a check to enforce that OWNERS files contain no metadata if a DIR_METADATA file is present in the same directory. Bug: 1102997 Change-Id: I661f7a1958824e7d20642d65534c2502d1a050e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317838Reviewed-by:
Nodir Turakulov <nodir@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@google.com> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Cr-Commit-Position: refs/heads/master@{#791884}
-
arthursonzogni authored
Patch [3/N] modified blink windows's binding to call CheckCoopAccess() whenever a "CrossOrigin" attribute/methods was accessed. So far, so good. Nevertheless, some attributes are missing, because they have a custom implementation. CheckCoopAccess needs to be added manually for them. This patch fixes the accesses when setting "opener". COOP access reporting: [ 1/N] https://chromium-review.googlesource.com/c/chromium/src/+/2264294 [ 2/N] https://chromium-review.googlesource.com/c/chromium/src/+/2270185 [ 3/N] https://chromium-review.googlesource.com/c/chromium/src/+/2270472 [ 4/N] https://chromium-review.googlesource.com/c/chromium/src/+/2273120 [ 5/N] https://chromium-review.googlesource.com/c/chromium/src/+/2309433 [ 6/N] https://chromium-review.googlesource.com/c/chromium/src/+/2308715 [ 7/N] https://chromium-review.googlesource.com/c/chromium/src/+/2309697 [ 8/N] https://chromium-review.googlesource.com/c/chromium/src/+/2275889 [ 9/N] https://chromium-review.googlesource.com/c/chromium/src/+/2310530 [10/N] this patch. Bug: 1090273 Change-Id: I4a5357d123db9e04ac4ac4fcf119ac3cacd93688 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315024Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Cr-Commit-Position: refs/heads/master@{#791883}
-
Hwanseung Lee authored
Add Troy card which card number have prefix 2205 or 9792 for autofill Bug: 1026289 Skip-Translation-Screenshots-Check: True Change-Id: I72b7b363c2ec63cb5ba1192b996773bcdf2d2e67 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1952628Reviewed-by:
Dominic Battré <battre@chromium.org> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Hwanseung Lee <hs1217.lee@samsung.com> Cr-Commit-Position: refs/heads/master@{#791882}
-
Dylan Cutler authored
This is part of a larger cleanup effort of Chrome's use of cookies. The eventual goal is to make the constructor private and to have other parts of Chrome use CreateSanitizedCookie(). Bug: 1102874 Change-Id: Id27150b987d2303fc670aac8cdf83300d2238877 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303300 Commit-Queue: Dylan Cutler <dylancutler@google.com> Reviewed-by:
Lily Chen <chlily@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Cr-Commit-Position: refs/heads/master@{#791881}
-
Xi Han authored
The chrome://history page is a native page, and currently TabListMediator and TabListFaviconProvider return the default round favicon for all native pages. To fix this issue, both classes will only check if it is a NTP, and treat other native pages the same as regular URLs, and load real favicon for them. Bug: 1104811 Change-Id: Ia93cfb77e41f2fdd2da9eb06c12fd797c4b34a82 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2310151 Commit-Queue: Xi Han <hanxi@chromium.org> Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Cr-Commit-Position: refs/heads/master@{#791880}
-
Sreeja Kamishetty authored
Test: VirtualBrowsingContextGroupTest.HistoryNavigation android-bot: https://ci.chromium.org/p/chromium/builders/ci/android-bfcache-rel linux-bot: https://ci.chromium.org/p/chromium/builders/ci/linux-bfcache-rel Bug: 1109648 Change-Id: If153c4143b1ad9c6e0fc2f684ef46d41882e624c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320126Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Commit-Queue: Sreeja Kamishetty <sreejakshetty@chromium.org> Cr-Commit-Position: refs/heads/master@{#791879}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast Tbr: cr-fuchsia+bot@chromium.org Change-Id: Ia9a830eb7acec89ff253da36f7d64be47e0b17b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320197Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#791878}
-