- 20 Mar, 2020 40 commits
-
-
Peter Wen authored
This clarifies the intent of the previous _java_header_patterns as well as includes more targets that ultimately uses java_library_impl. Also makes use of filter_exclude for header generation. Bug: 1056299 Change-Id: Ia241a8c7256a567d2d41cbb0901d90cb0c05a29d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2110686 Commit-Queue: Peter Wen <wnwen@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Auto-Submit: Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#752012}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/a0199a91..65a94404 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: Ia236ea464304831e945d269983f4da1987459a59 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111576Reviewed-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@{#752011}
-
Dan Harrington authored
And forward updates to FeedStream surfaces. Bug: 1044139 Change-Id: I3fada9c5054b2cfa8c3d412bca8a655ac38a2501 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107706 Commit-Queue: Dan H <harringtond@chromium.org> Reviewed-by:
Ian Wells <iwells@chromium.org> Cr-Commit-Position: refs/heads/master@{#752010}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/464e9ff4f368..44baaea9783e git log 464e9ff4f368..44baaea9783e --date=short --first-parent --format='%ad %ae %s' 2020-03-20 Richard.Townsend@arm.com fix: use x86 Python 3 binary where available Created with: gclient setdep -r src/third_party/depot_tools@44baaea9783e If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC agable@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/+/master/autoroll/README.md Bug: None Tbr: agable@chromium.org Change-Id: I12013be12f833dd927ed4cf4b683f5f1950b5f97 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112470Reviewed-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@{#752009}
-
Clemens Arbesser authored
Bug: b/145043394 Change-Id: I32068d58638d7dfaf94f1112d622118b6bdfafaa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107993 Commit-Queue: Clemens Arbesser <arbesser@google.com> Reviewed-by:
Mathias Carlen <mcarlen@chromium.org> Cr-Commit-Position: refs/heads/master@{#752008}
-
Fredrik Söderqvist authored
An <svg> element can have both a 'mask' and a '-webkit-mask' property (both yielding a Mask paint property), so we need to (re)check the existence of the masker LayoutObject before creating the SVGMaskPainter. Bug: 1063166 Change-Id: I55ce2b22185275b4dd23aefa65316230d98d960b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111316Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#752007}
-
Liquan (Max) Gu authored
Context: * WebPaymentIntentHelper#createPayIntent * WebPaymentIntentHelper#createIsReadyToPayIntent Change: * throws IllegalArgumentException for these inputs - null or empty packageName - null or empty activityName - null or empty id - null merchantName - null or empty schemelessOrigin - null or empty schemelessIframeOrigin - invalid certificateChain - invalid methodDataMap - null total - invalid modifiers * test all these above Bug: 1057488 Change-Id: I5affd2503551f3976287d3f83c07a126bc8a933c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103863 Commit-Queue: Liquan (Max) Gu <maxlg@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#752006}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/4432f93ff194..8d03f6fcf682 Created with: gclient setdep -r src-internal@8d03f6fcf682 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@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/+/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: None Tbr: jbudorick@google.com Change-Id: Ibf94e53eb941fc7e4e2464aa6d06eabb9a259e79 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112404Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#752005}
-
Mathias Bynens authored
DevTools front-end CL: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2112361 Bug: chromium:1056588 Change-Id: Id9b97397d114a1fe2fb57ca10d0aad3131a7ae44 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111179 Commit-Queue: Mathias Bynens <mathias@chromium.org> Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Auto-Submit: Mathias Bynens <mathias@chromium.org> Reviewed-by:
Sigurd Schneider <sigurds@chromium.org> Cr-Commit-Position: refs/heads/master@{#752004}
-
Sami Kyostila authored
Story filters are now required for pinpoint job configs as of https://chromium.googlesource.com/catapult.git/+/41a6191ed80e747e0924c4f9286597f84492b530 Change-Id: I5f3ec92bd6f335ff683dcd002cc90487fce0bfdb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112468 Commit-Queue: Sami Kyöstilä <skyostil@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#752003}
-
Gauthier Ambard authored
The |_documentController| wasn't reset between two files, leading to an issue where it wasn't pointing to the correct file. Fixed: 1062392 Change-Id: I5c4214c793c0e4e10f2c16af189c899683ba95b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111092 Commit-Queue: Mark Cogan <marq@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Auto-Submit: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#752002}
-
Mario Sanchez Prada authored
Create a dedicated method in content::mojom::RenderAccessibility to perform hit testing over an accessibility object that expects a reply that might include the required information to perform another hit testing, if a child frame was found to exist at the especified point. By doing this we can remove the HandleAXChildFrameHitTestResult() method, previously added to content::mojom::RenderAccessibilityHost, and fix the current situation where a message from an untrusted renderer would be sent to the browser when a child frame is found, since now such information would be shared as a reply to the original HitTest() method invoked from the browser process. Bug: 1010831, 1060223 Change-Id: I5b60d8a5e4b9f7fb2fc658c4f9156d7721b38db9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105217Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Cr-Commit-Position: refs/heads/master@{#752001}
-
Elly Fong-Jones authored
This change: 1) Moves the bulk of CocoaTest to a separate class, CocoaTestHelper, that can be held as a member 2) Removes CocoaTest::Init, which was misnamed and mostly unused, and replaces it with MarkCurrentWindowsAsInitial for the one use case that does need it 3) Removes a lot of testing macros from the CocoaTest header since they are either unused or only used in a single place 4) Also creates a //chrome CocoaTestHelper analog that bootstraps the Cocoa app bundle, for use in tests that need Cocoa but don't inherit from CocoaTest (eg because they inherit from another fixture) Bug: 39725 Change-Id: I463480fbdbe4942561269478dc2f396db8ea8fb0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107766Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#752000}
-
Denis Kuznetsov authored
Bug: 955194 Change-Id: I39b6c4c28ce1ee69f078b3875308b6ca486f749e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2110871Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Commit-Queue: Denis Kuznetsov [CET] <antrim@chromium.org> Cr-Commit-Position: refs/heads/master@{#751999}
-
Mikhail Khokhlov authored
Change-Id: Ia2333c6856de7e9bb8ff1ddd26fba3d0af62d6f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112358Reviewed-by:
Stephen Nusko <nuskos@chromium.org> Commit-Queue: Mikhail Khokhlov <khokhlov@google.com> Cr-Commit-Position: refs/heads/master@{#751998}
-
Luciano Pacheco authored
Use requestAnimationFrame() to change the CSS to collapsed the search box to give the chance for the button to process the collapse at the click. Add a test so we don't regress on this again. Test: browser_tests --gtest_filter="*searchButtonToggles*" Bug: 1062148 Change-Id: I628fb0b22f0a0223d659743618e7077ff0a7007d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112171 Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#751997}
-
Elly Fong-Jones authored
This change removes the last BubbleDelegate subclass in //chrome. Specifically: 1) ChooserBubbleUiViewDelegate is now a subclass of LocationBarBubbleDelegateView, so that it will close on cross-origin navigations and fullscreen transitions; this behavior was previously provided by ChromeBubbleManager 2) ChooserBubbleUi is now removed, along with all references to it The next steps in this refactor are: 1) Removing the now-dead ChromeBubbleManager and all references to it 2) Removing BubbleUi itself, since ChromeBubbleManager is the last reference to it Bug: 496955 Change-Id: I558cecf89dee6c1dce987a678e2f25805f6f1f82 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2109972 Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#751996}
-
Mugdha Lakhani authored
A delegate interface PageInfoDelegate has been introduced to allow the embedder to customize PageInfo. ChromePageInfoDelegate is Chrome's implementation of it. There will be follow up work to clean up chrome dependencies from PageInfo, after which, PageInfo can be moved to components. Bug: 1052375 Change-Id: I9b2ac5811ede505171787f23120cc46927e90f4d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2089710 Commit-Queue: Mugdha Lakhani <nator@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#751995}
-
Ben Joyce authored
This is a reland of: https://chromium-review.googlesource.com/c/chromium/src/+/2064823 After it was reverted: https://chromium-review.googlesource.com/c/chromium/src/+/2079147 This was caused by needing a new .info file because of breakage on https://ci.chromium.org/p/chromium/builders/ci/android-archive-rel/9669 . Apparently the uploaded .info file had an extra newline character. Bug: 1005891,1054152 Change-Id: Ibe273671811a7e772b4a6a5dbafcf5ec97c0b820 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2079436Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Commit-Queue: Yaron Friedman <yfriedman@chromium.org> Auto-Submit: benjamin joyce <bjoyce@google.com> Cr-Commit-Position: refs/heads/master@{#751994}
-
Brian White authored
Also converted all other calls from macros to functions which is the recommended method for any histogram emitted more than 1 per second or so. Bug: 1053105 Change-Id: Id9af6e33098de961f5e8cebf334615a12a0602e7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108802 Commit-Queue: Brian White <bcwhite@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#751993}
-
chromium-autoroll authored
This CL may cause a small binary size increase, roughly proportional to how long it's been since our last AFDO profile roll. For larger increases (around or exceeding 100KB), please file a bug against gbiv@chromium.org. Additional context: https://crbug.com/805539 Please note that, despite rolling to chrome/android, this profile is used for both Linux and Android. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/afdo-chromium-autoroll Please CC gbiv@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/+/master/autoroll/README.md Tbr: gbiv@chromium.org Change-Id: I1bc03528cd22546c780b61002c290f23028b89aa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112511Reviewed-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@{#751992}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/03867b5533b7..e3bc20cd8816 git log 03867b5533b7..e3bc20cd8816 --date=short --first-parent --format='%ad %ae %s' 2020-03-20 tvanderlippe@chromium.org Typecheck workspace/ with TypeScript Created with: gclient setdep -r src/third_party/devtools-frontend/src@e3bc20cd8816 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.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/+/master/autoroll/README.md Bug: chromium:1061125 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Ib8374b36058e90db87b36ccaf49fb3c3a1ace14b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112405Reviewed-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@{#751991}
-
Stephen McGruer authored
According to FlakePortal[0] and the legacy dashboard[1] this hasn't flaked since 2019/09/09. It seems worth attempting to renable it. [0]: https://analysis.chromium.org/p/chromium/flake-portal/flakes/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVybwsSBUZsYWtlImRjaHJvbWl1bUBibGlua193ZWJfdGVzdHNfZXhwYXJjaGl2ZUBleHRlcm5hbC93cHQvY3NzL2Nzcy1tYXNraW5nL2NsaXAtcGF0aC9jbGlwLXBhdGgtaW5saW5lLTAwMS5odG1sDA [1]: https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests%20(with%20patch)&tests=external/wpt/css/css-masking/clip-path/clip-path-inline-001.html Bug: 1001936 Change-Id: I85bbf8ec315c708244d42f9793d598865ad196f6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2110218Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Stephen McGruer <smcgruer@chromium.org> Cr-Commit-Position: refs/heads/master@{#751990}
-
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/+/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: Ic6127a4255dbb7852234f90a28c102f537a60cd5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112403Reviewed-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@{#751989}
-
Demetrios Papadopoulos authored
It was previously failing because it was asserting on the value of a cr-link-row's sublabel before it was actually updated in the prod code. Introducing a 'site-settings-list-labels-updated-for-testing' event to give a singal on the test code on when to assert fixes the issue. Bug: 1061249 Change-Id: Ia9c31229da79ac83acb38254ccee053b26f3ec1e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111778 Commit-Queue: Sean Harrison <harrisonsean@chromium.org> Auto-Submit: dpapad <dpapad@chromium.org> Reviewed-by:
Sean Harrison <harrisonsean@chromium.org> Cr-Commit-Position: refs/heads/master@{#751988}
-
Anatoliy Potapchuk authored
This reverts commit cf2d38fa. Reason for revert: let's have it in. Original change's description: > Revert "Reland "[Ash] Simplify AccessibilityControllerImpl to remove duplications"" > > This reverts commit 181c9ba9. > > Reason for revert: reland by mistake, will reland it together with the fix > > Original change's description: > > Reland "[Ash] Simplify AccessibilityControllerImpl to remove duplications" > > > > This reverts commit 812f314f. > > > > Reason for revert: culprit found > > > > Original change's description: > > > Revert "[Ash] Simplify AccessibilityControllerImpl to remove duplications" > > > > > > This reverts commit 2f2a33d5. > > > > > > Reason for revert: <INSERT REASONING HERE> > > > It looks like this CL breaks some test case > > > https://ci.chromium.org/p/chromium/builders/ci/Linux%20ChromiumOS%20MSan%20Tests/18262?blamelist=1#blamelist-tab > > > https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8885542762262570176/+/steps/exo_unittests/0/logs/Deterministic_failure:_All__x2f_SurfaceTest.AcquireFence__x2f_0__status_CRASH_/0 > > > > > > > > > Original change's description: > > > > [Ash] Simplify AccessibilityControllerImpl to remove duplications > > > > > > > > AccessibilityControllerImpl had a lot of duplications. Let's fix that. > > > > Also, now it controls fullscreen and docked magnifiers. Before that, > > > > we have to go AccessibilityDelegate and MagnificationManager to do > > > > this. > > > > > > > > Change-Id: I6a5d827ded2a88a56eb173fed67e010aa002070d > > > > Bug: 1051892 > > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2050688 > > > > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> > > > > Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org> > > > > Commit-Queue: Anatoliy Potapchuk <apotapchuk@chromium.org> > > > > Cr-Commit-Position: refs/heads/master@{#751086} > > > > > > TBR=xiyuan@chromium.org,dmazzoni@chromium.org,nektar@chromium.org,apotapchuk@chromium.org,burunduk@chromium.org,amraboelkher@google.com > > > > > > Change-Id: Ifbd8e25594790d7417af06c84651d3837ad17b5d > > > No-Presubmit: true > > > No-Tree-Checks: true > > > No-Try: true > > > Bug: 1051892 > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107028 > > > Reviewed-by: Nancy Wang <nancylingwang@chromium.org> > > > Commit-Queue: Nancy Wang <nancylingwang@chromium.org> > > > Cr-Commit-Position: refs/heads/master@{#751181} > > > > TBR=xiyuan@chromium.org,dmazzoni@chromium.org,nektar@chromium.org,apotapchuk@chromium.org,burunduk@chromium.org,amraboelkher@google.com,nancylingwang@chromium.org > > > > Change-Id: If1d69e1f28c1e850c55c038ea7138c75da9ea885 > > No-Presubmit: true > > No-Tree-Checks: true > > No-Try: true > > Bug: 1051892 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107545 > > Reviewed-by: Anatoliy Potapchuk <apotapchuk@chromium.org> > > Commit-Queue: Anatoliy Potapchuk <apotapchuk@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#751188} > > TBR=xiyuan@chromium.org,dmazzoni@chromium.org,nektar@chromium.org,apotapchuk@chromium.org,burunduk@chromium.org,amraboelkher@google.com,nancylingwang@chromium.org > > Change-Id: I382410f4fabc53c4ae5045310b22b729d5ec6cc7 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 1051892 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107546 > Reviewed-by: Anatoliy Potapchuk <apotapchuk@chromium.org> > Commit-Queue: Anatoliy Potapchuk <apotapchuk@chromium.org> > Cr-Commit-Position: refs/heads/master@{#751189} TBR=xiyuan@chromium.org,dmazzoni@chromium.org,nektar@chromium.org,apotapchuk@chromium.org,burunduk@chromium.org,amraboelkher@google.com,nancylingwang@chromium.org Change-Id: I3143d661b24e4995161cd358b0a778562b292539 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1051892 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108544Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Reviewed-by:
Anatoliy Potapchuk <apotapchuk@chromium.org> Commit-Queue: Anatoliy Potapchuk <apotapchuk@chromium.org> Cr-Commit-Position: refs/heads/master@{#751987}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/02c1ddad..a0199a91 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: Ica0b20de175c18f1d3e0608b2594ea5734bcfd29 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111574Reviewed-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@{#751986}
-
Noel Gordon authored
I believe "gear-menu" becomes before "install_linux_package_dialog" so be consistent with alpha-ordering in this BUILD.gn file. I was about to put "focus_visibilty.js" in this file, until I realized I don't need that file anymore, but it would have been adjacent to the gear_menu rule, from which I noted this minor alpha-order-disorder. Tbr: alex, lucmult Bug: Consult a dictionary. Change-Id: I84404d9c20054634552f1b9ed03e866594e27652 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111514Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#751985}
-
Mark Brand authored
This adds the standalone code for MojoLPM; it's not functional by itself but it is a dependency for the generated code. This is a cherrypick/split from https://chromium-review.googlesource.com/c/chromium/src/+/2108507 Change-Id: I71deba7182adc31acde2546b42808a959a87b8a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2109698Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Mark Brand <markbrand@google.com> Cr-Commit-Position: refs/heads/master@{#751984}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/005d0fff3b9c..835dde17a0c9 git log 005d0fff3b9c..835dde17a0c9 --date=short --first-parent --format='%ad %ae %s' 2020-03-20 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 578730be..2a41e24b (751777:751948) Created with: gclient setdep -r src/third_party/webrtc@835dde17a0c9 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/+/master/autoroll/README.md Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: Ib84096f51a79c6626942ac79d5aea4ae21c51085 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112398Reviewed-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@{#751983}
-
Stepan Khapugin authored
Instead of using deferred initialization, show the account selector when a scene becomes active. This means if there are multiple scenes, the user will always have to go through this dialog even if they switch to another window. Also completely get rid of AppNavigation, as it's not used anymore. Bug: 1045663 Change-Id: Idbbd19f1e188b033e39ce318def6b1338ad3ac6a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107581 Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#751982}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/bf995d913927..4432f93ff194 Created with: gclient setdep -r src-internal@4432f93ff194 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@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/+/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: chromium:995903 Tbr: jbudorick@google.com Change-Id: I92854a65d4fcb54abddc6f00ab1e62d86135f4b6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112392Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#751981}
-
Karandeep Bhatia authored
Currently the ruleset checksum for each extension using the declarativeNetRequest API is stored as an entry corresponding to the "dnr_ruleset_checksum" or "dnr_dynamic_ruleset_checksum" key. This CL replaces them with the "dnr_static_ruleset" and "dnr_dynamic_ruleset" keys respectively, corresponding to which we store a dictionary mapping ruleset IDs to their prefs. The prefs for each ruleset is a dictionary storing just the ruleset checksum for now. This CL helps move the code away from the assumption of a single static ruleset and will aid in the transition towards multiple static rulesets. A side effect of this change is that existing extensions using declarativeNetRequest will fail to load their rulesets with an install warning. Users will have to reinstall the extension. However that's ok given that the API is only in beta and has seen limited usage as of now. BUG=754526 Change-Id: Iaa77fca18322320dea4c7b02b0b9d6edea123975 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103547 Commit-Queue: Sergey Poromov <poromov@chromium.org> Auto-Submit: Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#751980}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/0e069b42b334..12a1c373822e git log 0e069b42b334..12a1c373822e --date=short --first-parent --format='%ad %ae %s' 2020-03-20 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/spirv-tools/src 18d3896a15fc..60104cd97446 (2 commits) 2020-03-20 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/SwiftShader bc98fbeec99b..1c9f24263e6c (2 commits) Created with: gclient setdep -r src/third_party/angle@12a1c373822e 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 courtneygo@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/+/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 Bug: None Tbr: courtneygo@google.com Change-Id: I8c62e29a53fece57d300cf9e765399459c635ffd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112394Reviewed-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@{#751979}
-
Nico Weber authored
(But not in use_libfuzzer builds.) Suppress the last few issues. This doesn't always go for the nicest fix for now, but instead for the simplest behavior-preserving change. The hope is to get this in while the codebase builds with the warning on, to make sure that no new regressions land. I'll then clean things up a bit on linux (e.g. make nice conversion functions for skia to libyuv and libpng color formats), and gradually enable on androidchromeosfuchsiaiosmacwindows in follow-ups. Bug: 346399 Change-Id: Ic920d3b63ac83aa59c2bf862f60662196d0b28e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2093537 Auto-Submit: Nico Weber <thakis@chromium.org> Commit-Queue: Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Cr-Commit-Position: refs/heads/master@{#751978}
-
Jinsuk Kim authored
For some of the callsites of TabImpl.getActivity(), what they actually need is not ChromeActivity, but either Context or Activity. This CL replaces them with Tab.getContext(). TabImpl.getActivity() and Tab.getContext() is different in that the former returns null if the tab is not attached to ChromeActivity (or not attached at all) while the latter returns the application context for such case. The callsites updated in this CL are not affected by the difference due to one of following reasons: - a null check to see if the tab is detached is already in place (ModuleInstallUi) - it is okay to proceed with the application context (ContextualSearchTabHelper - works for ChromeActivity only. For non-ChromeActivity, its scaleFactor could be set wrongly but CS won't be activated anyway) - getActivity() is always returning a non-null activity i.e. Tab.getContext() returns activity contenxt (LayerTitleCache, ExploreSitesPage, SurveyInfoBar, PictureInPictureActivity, TabParentIntent, ArConteInstallUtils) - a null check was added newly to check if the tab is in the detached state (AddToHomescreenInstaller - it is used by ChromeActivity-inherited activity only, therefore TabImpl.getActivity() returning null due to the tab being attached to non-ChromeActivity won't happen). Bug: 995903 Change-Id: I763787c57db62e3af61ee7f88bf172b1b93c45ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2109392 Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#751977}
-
Jan Wilken Dörrie authored
This change modifies StartPasswordCheck to wait for the initialization of the PasswordCheckDelegate, which requires waiting for the initial response from the PasswordStore. In order to support this async API, the method now takes a callback, rather than synchronously returning a value. Bug: 1062985 Change-Id: I878bade15d06d1a564836566898a3d748fe82590 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2110230 Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#751976}
-
Side Yilmaz authored
|Profile#getLastUsedProfile| is deprecated and replaced with |Profile#getLastUsedRegularProfile|. This CL updates ConsentAuditorBridge class. Note: This change is only code clean-up, does not change any behavior. Bug: 1041781 Change-Id: I5b4908a03d576ebfccfeb83bae4ac84baf455daa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107488Reviewed-by:
Martin Šrámek <msramek@chromium.org> Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Reviewed-by:
Ramin Halavati <rhalavati@chromium.org> Commit-Queue: Side YILMAZ <sideyilmaz@chromium.org> Cr-Commit-Position: refs/heads/master@{#751975}
-
Fredrik Söderqvist authored
* Const qualify referenced objects (SVGPreserveAspectRatio, TreeScope). * Have ReferencedLayoutObject() return a const LayoutObject. * Rearrange the code in FEImage::MapInputs() and CreateImageFilter() to get the same general structure (element reference, external image, failure). * Reorder PaintRecord recording in CreateImageFilterForLayoutObject(). * Mark local helper MakeMapBetweenRects() static. Split out "magic viewport"-mapping into a helper function. * Use DynamicTo<>() where applicable. Change-Id: If9fe1cf076257cd680c05bb1c97e14ab2fe7c1c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2110013 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#751974}
-
David Jean authored
Bug: 1045455 Change-Id: I8e72c9466ae00d87141c4fcea30ee817a1fbfb55 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106166 Commit-Queue: David Jean <djean@chromium.org> Reviewed-by:
Javier Ernesto Flores Robles <javierrobles@chromium.org> Cr-Commit-Position: refs/heads/master@{#751973}
-