- 16 Sep, 2020 40 commits
-
-
Anne Lim authored
This is a reland of b0398ace Original change's description: > [Autofill] Update Sync Bridge to save instrument_id > > Please see bug for design doc. > > Bug: 1121806 > Change-Id: I9c67bc3487a64eb1b07b4d0334a63081f85c66fa > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2404947 > Reviewed-by: Jared Saul <jsaul@google.com> > Reviewed-by: Marc Treib <treib@chromium.org> > Reviewed-by: Dominic Battré <battre@chromium.org> > Commit-Queue: Anne Lim <annelim@google.com> > Cr-Commit-Position: refs/heads/master@{#806731} Bug: 1121806 Change-Id: Ic0c1303e8284a9c00b49b4e54121ec86c8a37645 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412887Reviewed-by:
Dominic Battré <battre@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Anne Lim <annelim@google.com> Cr-Commit-Position: refs/heads/master@{#807501}
-
Liquan (Max) Gu authored
Change: * In PRImpl#initAndValidate(), refactors the mSpec parsing block to unwind the UI logic and the business logic. This way, mPaymentUIsManager.updateDetailsOnPaymentRequestUI() have a chance to be combined into onPaymentRequestParamsInitiated(). * Do the same unwinding for other cases in the code Bug: 1102522 Change-Id: I6939e934a554a2a252f1f0cb71208c245a738486 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2405617 Commit-Queue: Liquan (Max) Gu <maxlg@chromium.org> Reviewed-by:
Sahel Sharify <sahel@chromium.org> Cr-Commit-Position: refs/heads/master@{#807500}
-
David Black authored
Previously we watched the backing file for each holding space item in order to determine when to remove a stale holding space item. Now, we'll instead watch the parent directory of the backing file so that a file system watch can be reused across multiple holding space items. Bug: 1122418 Change-Id: I2b18f51c06dba36027c9c8b1562803f756980b99 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2410978Reviewed-by:
Ahmed Mehfooz <amehfooz@chromium.org> Commit-Queue: David Black <dmblack@google.com> Cr-Commit-Position: refs/heads/master@{#807499}
-
Dominique Fauteux-Chapleau authored
This is done by storing the response in the DownloadItem user data. Bug: 1128046 Change-Id: I8c9711e428d94208a4965bfce400a2a36ef71a7a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2410587Reviewed-by:
Daniel Rubery <drubery@chromium.org> Commit-Queue: Dominique Fauteux-Chapleau <domfc@chromium.org> Cr-Commit-Position: refs/heads/master@{#807498}
-
James Cook authored
Previously the crostini window tracking code was identifying lacros as an "unmatched" crostini app. Correctly setting the shelf ID fixes several issues: * No more penguin icon for lacros windows * Lacros icon shows correct active/running indicator * Lacros context menu has list of open windows when there is more than one window open Also change the Lacros BrowserManager to wait a little longer before starting the browser during session restore. This ensures that the shelf (ChromeLauncherController and objects it owns) are created before the first lacros-chrome browser window. There will probably be more shelf fixes required, but this is a start. Bug: 1115263 Test: added to browser_tests Change-Id: I323c7d9b213affba4de59130380585d769de19e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2411607 Commit-Queue: James Cook <jamescook@chromium.org> Auto-Submit: James Cook <jamescook@chromium.org> Reviewed-by:
Nancy Wang <nancylingwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#807497}
-
rbpotter authored
Two up mode is implemented in the plugin but not the ink controller, and triggering it causes a crash if the plugin is not on the page. Also fixing the following case: 1. Rotate PDF. Annotation is disabled. 2. Set 2 page view. 3. Rotate PDF back to upright. Currently, annotation is re-enabled, despite still being in 2 page view. After this CL, annotation remains disabled, until two page view is turned off. Bug: 1048812 Change-Id: I26c0cb43645c82905e1fe5b53398332573d8a774 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2411284Reviewed-by:
dpapad <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#807496}
-
rbpotter authored
Bug: 1128583 Change-Id: Id7e714eb94b2ff08a346d4d0166d97e93654265d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412139Reviewed-by:
dpapad <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#807495}
-
mhasank authored
The UMA is sent in session manager daemon when starting ARC fails and we create backup of ARC bug report. BUG=b:168630900 TEST=deploy to dut, opt in to arc, ensure suffix shows up in chrome://histograms Change-Id: Iab49253ab966ff98c73a2da573bcc73dd1289cfe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412780Reviewed-by:
Brian White <bcwhite@chromium.org> Reviewed-by:
Hidehiko Abe <hidehiko@chromium.org> Reviewed-by:
Yury Khmel <khmel@chromium.org> Commit-Queue: Muhammad Hasan Khan <mhasank@chromium.org> Cr-Commit-Position: refs/heads/master@{#807494}
-
Hazem Ashmawy authored
Add tests for crash collection consent error message. This also handles a silent bug where calling assertNoUnverifiedIntents in the teardown of a skipped test, will cause a crash, by checking that the activity is launched. Fixed: 1106315 Test: run_webview_instrumentation_test_apk -f "*android_webview.devui*" Change-Id: Ie0252d898f848f69dee90a6f526691dfab495289 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2411943 Commit-Queue: Hazem Ashmawy <hazems@chromium.org> Reviewed-by:
Nate Fischer <ntfschr@chromium.org> Cr-Commit-Position: refs/heads/master@{#807493}
-
David Grogan authored
* Don't make all tables in row flexboxes go through the min-width:auto path, which was added in https://chromium-review.googlesource.com/c/chromium/src/+/2392444 * Always make flex item's min/max widths encompass table's min intrinsic size, even if the table has a definite min-width. No behavior change intended. Change-Id: I92456b5c4e20272887066593d166809e6b98ab54 Bug: 1123100,1128262 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2410596Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: David Grogan <dgrogan@chromium.org> Cr-Commit-Position: refs/heads/master@{#807492}
-
Matt Jones authored
This patch ensures that the bottom sheet is not currently suppressed prior to showing content. This patch also ensures that higher-priority content replaces existing content when suppressed. Bug: 1114584 Change-Id: I99f5ed696e269012908bc68f9968c09ae7ee818b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401855 Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by:
Patrick Noland <pnoland@chromium.org> Cr-Commit-Position: refs/heads/master@{#807491}
-
Andrew Grieve authored
This reverts commit 90feb41b. Reason for revert: Broke disassembling with ndk for apks Original change's description: > SuperSize: Fix console's Disassemble not using stored toolchain > > Just something that came up when I was trying to use it. > > Bug: 999351 > Change-Id: I31c7c6a3b0ae6baaa7205b59cf1e6fff1bb2a517 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412369 > Auto-Submit: Andrew Grieve <agrieve@chromium.org> > Reviewed-by: Samuel Huang <huangs@chromium.org> > Commit-Queue: Samuel Huang <huangs@chromium.org> > Cr-Commit-Position: refs/heads/master@{#807185} TBR=huangs@chromium.org,agrieve@chromium.org Change-Id: I13c23a145096865627c080910b4901ebff293901 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 999351 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2414648Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#807490}
-
Gayane Petrosyan authored
Add prefix and suffix to text fragment selector when only exact text selector is not enough. Gradually add words from available prefix and suffix range until unique match is found. Bug: 1102382 Change-Id: I377deca1315b3f2ea781a2dce37d04bb5a57dd69 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2374369 Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Reviewed-by:
Tommy Martino <tmartino@chromium.org> Cr-Commit-Position: refs/heads/master@{#807489}
-
Erik Staab authored
This is a reland of 11c26e5e Original change's description: > Change the mac builder default to be either 10.13 or 10.15. > > Once the hosts are all upgraded to 10.15 we will remove the 10.13 option. > > Bug: 1121185 > Change-Id: I3177749f79ddd16b9d725693f21e79ebd752c22c > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412757 > Commit-Queue: Erik Staab <estaab@chromium.org> > Auto-Submit: Erik Staab <estaab@chromium.org> > Reviewed-by: Garrett Beaty <gbeaty@chromium.org> > Cr-Commit-Position: refs/heads/master@{#807218} Bug: 1121185 Change-Id: I0b4e69122fc7b80165ca2abbd2542c24ba88d0d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2413529 Auto-Submit: Erik Staab <estaab@chromium.org> Reviewed-by:
Garrett Beaty <gbeaty@chromium.org> Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Cr-Commit-Position: refs/heads/master@{#807488}
-
Guillaume Jenkins authored
When opening multiple URLs via Siri Shortcuts while the tab switcher was opened and empty, only the first 3 URLs were successfully opened. The cause is as follows: - URL 1 opens and causes the tab switcher to dismiss asynchronously, but callback 1 is still run immediately - Callback 1 causes URL 2 to start opening, but it must wait for the dismiss animation to finish, so callback 2 is stored in the BVC properties - Callback 2 eventually runs, and does two things: 1) open URL 3 2) append callback 3 to the BVC tab opening callbacks - Immediately after callback 2 finishes running, the BVC clears its callback property, erasing callback 3 that was just set by callback 2 - URL 3 is opened by callback 2, but callback 3 is never run because by the time the BVC tries to invoke the tab opening callback, it was already set to nil after running callback 2 The simplest fix I found is to simply clear the callback property before running the callback (the callback is stored to a local var before clearing the property). That way, callbacks can append new tab opening callbacks without fear of them being erased. Bug: 1124291 Change-Id: Ib3684f4ce93136c28bb55711b4a5742510082302 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2405897Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Reviewed-by:
Justin Cohen <justincohen@chromium.org> Commit-Queue: Guillaume Jenkins <gujen@google.com> Cr-Commit-Position: refs/heads/master@{#807487}
-
kylechar authored
Change ColorTransformPixelTest so that they run with Vulkan. Also merge together the list of color spaces to be tested for GLRenderer and SkiaRenderer. The only thing that doesn't work with SkiaRenderer is piecewise HDR transfer functions, so add a check and early return in the test body if using SkiaRenderer and the color space has a piecewise HDR transfer function. Also make sure VideoRendererPixelHiLoTest doesn't run if GL tests are turned off. The test only works with GL but it was still being run when BUILDFLAG(ENABLE_GL_BACKEND_TESTS) was false. Bug: 1117587 Change-Id: If24ec29ab36d7c20e93f5140786639fb99385ce3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2411700Reviewed-by:
Sean Gilhuly <sgilhuly@chromium.org> Commit-Queue: kylechar <kylechar@chromium.org> Cr-Commit-Position: refs/heads/master@{#807486}
-
Mohamed Amir Yosef authored
This is analogous to PasswordForm::in_store and reflects in which store those credentials are stored. Bug: 1119286 Change-Id: I18084d1af45f427346f0df4282a2505a1748c139 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2407900 Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#807485}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/08574aeb4454..fa54364cea0b 2020-09-16 jonaso@webrtc.org Update STUN attributes with latest IANA registrations 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: Ifa06912fbe794806fb81d78fe15c331f1ace5e24 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2413236Reviewed-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@{#807484}
-
danakj authored
The code has moved, so update the includes. R=avi@chromium.org Bug: 1091083 Change-Id: I2c9a5c20b78756fc0b460a85f00d8560f1883979 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412947 Commit-Queue: Avi Drissman <avi@chromium.org> Auto-Submit: danakj <danakj@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#807483}
-
Scott Violet authored
Couple of comments about 'overflow:clip': . Per spec it does not create a new formatting context. . Implementation will create a PaintLayer, but it won't create a scrollable area. . Can not be scrolled. This adds IsScrollContainer to ComputedStyle and LayoutObject. The name 'scroll container' comes from https://www.w3.org/TR/css-overflow-3/#scroll-container . As 'overflow: visible' may be set along one axis, and 'overflow: clip' another, this renames IsOverflowVisible() to IsOverflowVisibleAlongBothAxes() to make it clear it applies to both axes. BUG=1087667 TEST=this is mostly a rename, so no direct tests. Once OverflowClip is enabled, there are layout tests exercising this. Change-Id: I5a4647a19e31c7ad212929bb9e408708fd61d441 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2402303Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#807482}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/38f0f40310d6..d1382ae74fac 2020-09-16 primiano@google.com Merge "iorap API: fix thread race when initializing the consumer library" 2020-09-16 treehugger-gerrit@google.com Merge "Roll trace processor" 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: I3e6e58bed619a859b1cbc81fd370599ee7a209e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2413237Reviewed-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@{#807481}
-
Scott Violet authored
This makes all methods of Tab/Browser throw an exception if used after the Tab/Browser has been destroyed. Hopefully this will make it easier to identify bad usage rather than NPEs slightly deeper in the stack. BUG=none TEST=none Change-Id: Id42839c5aca2061f811b5ec07aefd1e0f187d55a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412550 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#807480}
-
Dan Harrington authored
* > autotest ... base/strings/strcat.cc will now run tests for base/strings/strcat_unittest.cc * > autotest ... strcat no longer complains "strcat" is ambiguous, since strcat_unittest.cc is the only matching test file. * Fixed rare case where caching could use the wrong target. Change-Id: Id028b8d5477687bef02b6cd94688f74839d08964 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2407857Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Commit-Queue: Dan H <harringtond@chromium.org> Cr-Commit-Position: refs/heads/master@{#807479}
-
Josh Nohle authored
If contacts are removed from the allowlist, only the selected-contacts visibility private certificates need to be recreated. If the contact list changes, only the all-contacts visibility private certificates need to be recreated. (Ideally, we would only recreate all-contacts visibility private certificates if contacts are removed from the contact list, but we do not have this information.) These revocations need to occur so that unauthorized contacts do not have access to valid certificates. Previously, all private certificates were being recreated in both scenarios, which is safe but overkill. Fixed: b/168022980, 1123134 Change-Id: I9263e02f2cca72c5b350368d42596c5f2d5655e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412890 Commit-Queue: Josh Nohle <nohle@chromium.org> Reviewed-by:
James Vecore <vecore@google.com> Cr-Commit-Position: refs/heads/master@{#807478}
-
Nina Satragno authored
Update obtaining a pinUvAuthToken using UV with the latest specification. * The error for a bad fingerprint read is now ERR_UV_INVALID. * ERR_UV_BLOCKED now may be returned both before or after the user taps the authenticator. Query the number of remaining attempts before trying to obtain a UV token to avoid a locked authenticator from blocking requests to other authenticators. Fixed: 1127062 Change-Id: I81a0eb7620213e1cbe0afbf63bf93c45fd4bea18 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412892 Commit-Queue: Nina Satragno <nsatragno@chromium.org> Reviewed-by:
Martin Kreichgauer <martinkr@google.com> Cr-Commit-Position: refs/heads/master@{#807477}
-
Daniele Castagna authored
When HardwareDisplayController get disabled we should disable all planes, including the primary, by setting the fb to 0. Not doing so can have privacy implications, where we might end up reading back the last fb when modesetting again and show to the user contents that should not be seen. In the past, when legacy modesetting was used, the fb associated with the crtc was set to 0 when calling drmModeSetCrtc. After that we'd disable the other planes but the primary. Since we moved to the atomic API for modesetting, we now need to explicitly set to 0 the fb of the primary plane along the other planes. Bug: 1102859 Test: Close/open the lid on krane, no old contents are visible. Change-Id: Ib93347b59cc847898c0713f81ad0903773699b07 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412974 Commit-Queue: Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Gil Dekel <gildekel@chromium.org> Cr-Commit-Position: refs/heads/master@{#807476}
-
Anthony Cui authored
Reland "Consolidate translation check and execution logic between TranslateManager and RenderViewContextMenu" This is a reland of 24c91f76 Original change's description: > Consolidate translation check and execution logic between TranslateManager and RenderViewContextMenu > > Consolidated logic by having RenderViewContextMenu::ExecTranslate call > TranslateManager::InitiateManualTranslation, and RenderViewContextMenu::AppendPageItems call > TranslateManager:CanManuallyTranslate. > > Bug: 898168 > Change-Id: If91cf98626f73d0c357522f8fdca840695185bc2 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2385636 > Reviewed-by: Avi Drissman <avi@chromium.org> > Reviewed-by: Megan Jablonski <megjablon@chromium.org> > Commit-Queue: Anthony Cui <cuianthony@chromium.org> > Cr-Commit-Position: refs/heads/master@{#806161} Bug: 898168 Change-Id: Iddd188034e471d3b4e8e09650d9f1e88cb2bc268 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2410645Reviewed-by:
Megan Jablonski <megjablon@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Anthony Cui <cuianthony@chromium.org> Cr-Commit-Position: refs/heads/master@{#807475}
-
chrome://versiondpapad authored
Using chrome://theme/current-channel-logo, which does not work on iOS and Android. Bug: 820678 Change-Id: I5b0028bd75dd52c55359264b7d72e8cd04d4be9d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1588799 Commit-Queue: Peter Boström <pbos@chromium.org> Auto-Submit: dpapad <dpapad@chromium.org> Reviewed-by:
Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#807474}
-
Mike Dougherty authored
Bug: 1128463 Change-Id: I32f05328e590ad7e8772cec8119e6d253d456853 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412448 Auto-Submit: Mike Dougherty <michaeldo@chromium.org> Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by:
Stepan Khapugin <stkhapugin@chromium.org> Cr-Commit-Position: refs/heads/master@{#807473}
-
Friedrich Horschig authored
This CL fixes the broken fade-in animation and adds an overshoot animation for the arriving suggestions (see linked bug for screenrecording). The animation is density-independent and works in RTL by mirroring the movement. Bug: 1124720 Change-Id: I3d93a610c66404ac0cfa6db5b277b0b210a62250 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2411938Reviewed-by:
Ioana Pandele <ioanap@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#807472}
-
Reda Tawfik authored
This CL adds a search view to the bottom sheet and filters the credentials list with every newly typed character. Screenshot: https://bugs.chromium.org/p/chromium/issues/detail?id=1104132#c18 Bug: 1104132 Change-Id: Ib9528abcc8ffa3981f37fe3d7334156aa71ff0fb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2391132 Commit-Queue: Reda Tawfik <redatawfik@google.com> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#807471}
-
arthursonzogni authored
Soon, the reports will contains URLS about the other document: - openeeURL - openerURL - otherDocumentURL - referrer - initialPopupURL Depending on whether they are same-origin or cross-origin with the reporter document, they might be reported or not. This patch add a same-origin/cross-origin variant for every type of accesses. Bug: 1090273 Change-Id: Ieeb396f6220877a2a61541cb33a964261e5f4070 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2407314Reviewed-by:
Pâris Meuleman <pmeuleman@chromium.org> Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Cr-Commit-Position: refs/heads/master@{#807470}
-
Jose Dapena Paz authored
GCC fails to build with this: ../../pdf/thumbnail.cc:116:1: error: function ‘chrome_pdf::Thumbnail::Thumbnail(chrome_pdf::Thumbnail&&)’ defaulted on its redeclaration with an exception-specification that differs from the implicit exception-specification ‘’ 116 | Thumbnail::Thumbnail(Thumbnail&& other) noexcept = default; | ^~~~~~~~~ ../../pdf/thumbnail.cc:118:12: error: function ‘chrome_pdf::Thumbnail& chrome_pdf::Thumbnail::operator=(chrome_pdf::Thumbnail&&)’ defaulted on its redeclaration with an exception-specification that differs from the implicit exception-specification ‘’ 118 | Thumbnail& Thumbnail::operator=(Thumbnail&& other) noexcept = default; | ^~~~~~~~~ Problem is that Skia SkBitmap move constructor and assignment operators are not noexcept (and after some discussion it is going to be kept this way). So the default implementation of move constructor and assignment operator of pdf::Thumbnail is noexcept, as SkBitmap is a member. The fix just removes noexcept declarations. Bug: 819294 Change-Id: I8e6d914e4850707fbfcb51200b2a80d185cd1c19 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2411656Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Commit-Queue: José Dapena Paz <jdapena@igalia.com> Cr-Commit-Position: refs/heads/master@{#807469}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/5453e7b7795f..3018d5e57091 2020-09-16 angle-autoroll@skia-public.iam.gserviceaccount.com Roll Vulkan-ValidationLayers from e59713d4edda to f105898a102f (6 revisions) 2020-09-16 angle-autoroll@skia-public.iam.gserviceaccount.com Roll SPIRV-Tools from 726af6f78f80 to e8ce4355ae1c (9 revisions) 2020-09-16 angle-autoroll@skia-public.iam.gserviceaccount.com Roll SwiftShader from dc552fcef1fe to b33e0b36a08c (6 revisions) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC jmadill@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:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win-asan;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: None Tbr: jmadill@google.com Change-Id: Id26dbefab6a7f2840d4e644edac3a93ca35c613e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412446Reviewed-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@{#807468}
-
Alex Ilin authored
Avatars displayed in webUI should have their size multiplied by the device scale factor. Otherwise, the avatars look blurry when Chrome is running with device scale factor > 1. Fixed: 1128102 Change-Id: I9a3f5e38b5bf9e8c14889bbef619f2a6eae565b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2413948 Commit-Queue: Alex Ilin <alexilin@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#807467}
-
Elly Fong-Jones authored
This reverts commit 8bb77f95. Reason for revert: not a proper fix per mark's comments Original change's description: > base: extend ImmediateCrashTest.ExpectedOpcodeSequence for mac arm64 > > This test needs a case for what clang does on Mac Arm64, so add that. > > Bug: 1128436,1121871 > Change-Id: I5d57f99f7d041bc91d9fc862c1d10ddc1b48f540 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2412552 > Commit-Queue: Nico Weber <thakis@chromium.org> > Reviewed-by: Nico Weber <thakis@chromium.org> > Cr-Commit-Position: refs/heads/master@{#807243} TBR=ellyjones@chromium.org,thakis@chromium.org Change-Id: I32e706cd2544cd91974a39dc37f54438fe6822c2 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1128436 Bug: 1121871 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2413562Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#807466}
-
Hazem Ashmawy authored
Add tests for the different testable scenarios where the dev UI is launched from a WebView package that is different from the current system provider. Fixed: 1128628 Test: run_webview_instrumentation_test_apk -f "*android_webview.devui*" Change-Id: I5c59d2351fbb1ec10fb11919fada1a953c39070a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2411984 Commit-Queue: Hazem Ashmawy <hazems@chromium.org> Reviewed-by:
Nate Fischer <ntfschr@chromium.org> Cr-Commit-Position: refs/heads/master@{#807465}
-
Stephen Nusko authored
This reverts commit f4886cba. Reason for revert: This causes unintended throttling of TouchMoves if the page is at the overflow already. Fix will land in next release. See crbug/1123304 for details. Original change's description: > Ensure that TouchEvents are always async after scrolling has started. > > Currently we can flip flop back and forth between TouchMoves being sent > async or sync which can cause some pretty noticeable cases of jank. This > ensures that we are always consistently scrolling after deciding > TouchMoves can be async. > > Bug: 1072364 > Change-Id: I3ef1ed6715d4b1fd2d3e1377ebd6b0c8bcc02872 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2182426 > Commit-Queue: Stephen Nusko <nuskos@chromium.org> > Auto-Submit: Stephen Nusko <nuskos@chromium.org> > Reviewed-by: David Bokan <bokan@chromium.org> > Cr-Commit-Position: refs/heads/master@{#776487} TBR=bokan@chromium.org,nuskos@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1072364,1123304 Change-Id: I7645aa130ddaf5214bde77a8cbcdd239e48f729c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2413491 Auto-Submit: Stephen Nusko <nuskos@chromium.org> Commit-Queue: David Bokan <bokan@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Reviewed-by:
Stephen Nusko <nuskos@chromium.org> Cr-Commit-Position: refs/heads/master@{#807464}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/b09e7729..49c0daeb 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: I10372035655d28c09fa64019c60725d1670adc0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2414012Reviewed-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@{#807463}
-
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: I3d05a5636400c1b1333ee8384403f80f1bbe4c5e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2414149Reviewed-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@{#807462}
-