- 26 Mar, 2020 40 commits
-
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/b1128ddc5b31..7e1439b863bd Created with: gclient setdep -r src-internal@7e1439b863bd 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: I05add8b578450054149350f9e8c75ed95c7e4132 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121956Reviewed-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@{#753589}
-
Bo Liu authored
Follow up to r654240 Apparently on apps that does not end up resizing the FullScreenView from the WebView, avoiding a resize cause odd problems such as FullScreenView not getting forwarded key events or not being on top of other views as it should be. Instead set the initial height to be arbitrarily 5 pixels less in height. This avoids affecting the compositing pipeline significantly, since height changes generally has small effects on layout. Bug: 1054286 Change-Id: Ia5bbb384a75736784e660567dd7e36bacbc4b5b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119802Reviewed-by:
Tobias Sargeant <tobiasjs@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#753588}
-
Alex Ilin authored
Reauth dialog uses SigninViewControllerDelegateViews as a webview container. Relevant navigation events are handled by ReauthTabHelper. SigninViewControllerDelegateViews required a slight change to display a close button in the top right corner. Bug: 1045515 Change-Id: I688b93eef8e8549aaf1c489c73655378e4e858ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111100 Commit-Queue: Alex Ilin <alexilin@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#753587}
-
Theodore Olsauskas-Warren authored
The new cookies page introduces a new control element for interacting with cookie settings. This CL adds the appropriate managed indicators to the element and disables the appropriate controls when cookie settings are managed or recommended. Bug: 1062649 Change-Id: Ifa2e6abc878a482e045f06fde35fd0723506e615 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107864 Commit-Queue: Theodore Olsauskas-Warren <sauski@google.com> Reviewed-by:
dpapad <dpapad@chromium.org> Reviewed-by:
Martin Šrámek <msramek@chromium.org> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Cr-Commit-Position: refs/heads/master@{#753586}
-
Clemens Arbesser authored
Bug: b/145043394 Change-Id: Id61c888bd0b2efb1b0232bb172b8e1613512714b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2115654 Commit-Queue: Clemens Arbesser <arbesser@google.com> Reviewed-by:
Marian Fechete <marianfe@google.com> Cr-Commit-Position: refs/heads/master@{#753585}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/ee2ec6de995b..12da77257204 git log ee2ec6de995b..12da77257204 --date=short --first-parent --format='%ad %ae %s' 2020-03-26 borenet@google.com [infra] Update infra dep again Created with: gclient setdep -r src/third_party/skia@12da77257204 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC robertphillips@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-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: robertphillips@google.com Change-Id: Idac3056eda4f09e3dd24ed877e2c8ac77b5d8bb0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121955Reviewed-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@{#753584}
-
Tim Schumann authored
This reverts commit e55b73bf. Reason for revert: TooltipControllerCaptureTest.Capture started failing on Chrome OS builds Original change's description: > Have DefaultScreenPositionClient() keep the Aura root window updated. > > It sets itself as the root window's client on construction and nulls > that out on destruction. DesktopScreenPositionClient already did this, > this just hoists that to the base class. > > This makes it harder to introduce bugs like "constructed the client > but forgot to set it on the window" or the more likely (and insidious) > "failed to null the client on shutdown, leaking state into another test > and triggering test flakiness". > > Bug: none > Change-Id: Ib8ddb193d78d7d7e5b0cf48683a7ef9653d31f7c > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117335 > Reviewed-by: Darren Shen <shend@chromium.org> > Reviewed-by: Yuchen Liu <yucliu@chromium.org> > Reviewed-by: Devlin <rdevlin.cronin@chromium.org> > Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org> > Commit-Queue: Darren Shen <shend@chromium.org> > Auto-Submit: Peter Kasting <pkasting@chromium.org> > Cr-Commit-Position: refs/heads/master@{#753540} TBR=sadrul@chromium.org,pkasting@chromium.org,rdevlin.cronin@chromium.org,shend@chromium.org,yucliu@chromium.org Change-Id: Ia758dbaa0441874217c62cd061cff21aae329740 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1064933 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121259Reviewed-by:
Tim Schumann <tschumann@chromium.org> Commit-Queue: Tim Schumann <tschumann@chromium.org> Cr-Commit-Position: refs/heads/master@{#753583}
-
Tim Schumann authored
It's flaky on many platforms. BUG: 1064434 TBR=ricea@chromium.org Change-Id: Ie379e3c39c694f5d4c00d6c8a311b5fa7831e530 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121378Reviewed-by:
-
Delan Azabani authored
This CL changes DeprecatedLower calls to LowerASCII where the result is only ever used in a comparison against one or more ASCII constants (including predicates like StartsWith) but is otherwise dead, and I believe it’s the last CL needed to finish our intent to remove Unicode case-insensitivity for ASCII constants [1]. The correctness of the image_encoder_utils.cc change isn’t obvious, but you can verify it by heading to the line in Chromium Code Search [2], hovering over lowercase_mime_type, and observing that the only reads are == MIME type literals and by two functions: • IsSupportedImageMIMETypeForEncoding (line 78), which does the same • ParseImageEncodingMimeType (line 80), which does the same These changes are potentially author-facing, except where the constant in question contains none of {s,k,S,K} [3], but I’ve optimistically made them without any additional tests based on Rick Byers’ reasoning on blink-dev [4]. Please let me know if that’s not sufficient here. [1] https://groups.google.com/a/chromium.org/d/topic/blink-dev/sFOpNuQ91UU [2] https://source.chromium.org/chromium/chromium/src/+/f7455871a37e53362183771af7de36844df73a38:third_party/blink/renderer/platform/image-encoders/image_encoder_utils.cc;l=36 [3] https://www.unicode.org/Public/UCD/latest/ucd/CaseFolding.txt [4] https://groups.google.com/a/chromium.org/d/msg/blink-dev/sFOpNuQ91UU/3u1HxbnQCQAJ Change-Id: I10e621bb2a6947bc3004d9a0c156dcae2e05a6e8 Bug: 627682 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2114510 Commit-Queue: Delan Azabani <dazabani@igalia.com> Reviewed-by:Frédéric Wang <fwang@igalia.com> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Darwin Huang <huangdarwin@chromium.org> Cr-Commit-Position: refs/heads/master@{#753581}
-
Andreas Haas authored
This CL was created automatically with tools/perf/update_wpr script R=crouleau@chromium.org BUG= chromium:1057645 Change-Id: I1b73fdee07197a178af5628531af6d5bf63e19d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2080251 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Reviewed-by:
John Chen <johnchen@chromium.org> Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Cr-Commit-Position: refs/heads/master@{#753580}
-
Jana Grill authored
Pass the feature flag's value to the UI so that the corresponding UI toggle can display the correct state (enabled/disabled). Additionally check the feature flag value when checking eligibility for enablind and disabling ADB sideloading. --enable-features=ArcManagedAdbSideloadingSupport parameter in /etc/chrome_dev.conf and check that in this case the toggle in settings is enabled and ADB sideloading can be activated. Test: Powerwash device and initialize with a managed account. Pass the Bug: 1058386 Change-Id: Ic172b8a8a90c485f86aa26d59f2710844a71253c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2084554Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Victor Hsieh <victorhsieh@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Joel Hockey <joelhockey@chromium.org> Commit-Queue: Jana Grill <janagrill@chromium.org> Cr-Commit-Position: refs/heads/master@{#753579}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/8e26c5f50e85..2df8c71258de git log 8e26c5f50e85..2df8c71258de --date=short --first-parent --format='%ad %ae %s' 2020-03-26 johnkslang@users.noreply.github.com Merge pull request #2154 from GMNGeoffrey/patch-1 Created with: gclient setdep -r src/third_party/glslang/src@2df8c71258de If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/glslang-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,cnorthrop@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_angle_vk32_deqp_rel_ng;luci.chromium.try:android_angle_vk32_rel_ng;luci.chromium.try:android_angle_vk64_deqp_rel_ng;luci.chromium.try:android_angle_vk64_rel_ng;luci.chromium.try:linux_angle_deqp_rel_ng;luci.chromium.try:linux-angle-rel;luci.chromium.try:win-angle-rel-32;luci.chromium.try:win-angle-rel-64;luci.chromium.try:win-angle-deqp-rel-32;luci.chromium.try:win-angle-deqp-rel-64 Bug: None Tbr: radial-bots+chrome-roll@google.com,cnorthrop@google.com Change-Id: I4f2691e737a276f232f8e1acd306962f8ad57e3a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121758Reviewed-by:
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/77090261dabc..e322072ab4fb git log 77090261dabc..e322072ab4fb --date=short --first-parent --format='%ad %ae %s' 2020-03-26 bclayton@google.com Merge "Initial drop of LLVM 10 to third_party/llvm-10.0" Created with: gclient setdep -r src/third_party/swiftshader@e322072ab4fb If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Please CC swiftshader-team+autoroll@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_chromium_msan_rel_ng;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: swiftshader-team+autoroll@google.com Change-Id: I1c7386fcd484f868000757ac1832e7d01a8c7044 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121552Reviewed-by:
-
Finnur Thorarinsson authored
The comparator is used for the priority queue and deserves a bit of testing for the different combinations of requests. bug: 1054520, 1054502, 656015 Change-Id: Ic8281b6b69d32e8d8e5d781527e6666ebeadd959 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119517Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> Cr-Commit-Position: refs/heads/master@{#753576}
-
Hajime Hoshi authored
This CL breaks down WebSchedulerTrackedFeature:: kOutstandingNetworkRequest into fetch, XHR, and the others. As one of the biggest reasons why the page is NOT cache is outstanding network requests so far, this breaking down would give us more insights. Bug: 1052793 Change-Id: I34a6a3fbea8f91f9b2977349b7daeb1a0c529c89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2104711Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Cr-Commit-Position: refs/heads/master@{#753575}
-
Mark Cogan authored
Removes use of TabModelList from the external file remover, replacing it with equivalent BrowserList code. The ComputeReferencedExternalFiles() helper function is updated to accept a single Browser parameter. Bug: 1058350 Change-Id: I258fa3c20cd932e9bd5ecdd6da538a5b68b915a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2116053Reviewed-by:
Mohammad Refaat <mrefaat@chromium.org> Commit-Queue: Mohammad Refaat <mrefaat@chromium.org> Cr-Commit-Position: refs/heads/master@{#753574}
-
Hiroshige Hayashizaki authored
The test seems no longer flaky. A couple of flakes on dashboard seem unrelated. (e.g. "with patch" runs with many other tests failing) Test: http/tests/images/png-partial-load-as-document.html Bug: 813704 Change-Id: Idc9629bb4cf9b2f80af15f96d5f1d97938548b7a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120997Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Cr-Commit-Position: refs/heads/master@{#753573}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/3714932e..36272def 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: Ia869d91c40af1d80293bcc210b5d609ab648214e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121281Reviewed-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@{#753572}
-
Oleg Davydov authored
This commit makes two changes for histograms related to policy-based force-installed extensions: * Move their expire data to 2020-11-01, since we still need these histograms for investigation. * Actualize owners for the histograms. Bug: 1063905, 1063906, 1063907, 1063908, 1063909 Bug: 1063910, 1063911, 1063912, 1063913, 1063914 Bug: 1063915, 1063916 Change-Id: Icebead17c11f40f8cee9c2a8878c1cb0daf3732b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120714Reviewed-by:
Mark Pearson <mpearson@chromium.org> Reviewed-by:
Askar Aitzhan <askaraitzhan@google.com> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Oleg Davydov <burunduk@chromium.org> Cr-Commit-Position: refs/heads/master@{#753571}
-
Marc Grimme authored
FakeSessionManagerClient stores its static global instance in g_instance. It is a predecessor of SessionManagerClient that already has its static global instance stored in g_instance. Therefore the g_instance in this class can be removed. This change removes the redundant static global instance. Bug: 1063850 Change-Id: I1f94b657d7478bd8cc3ba66dd1d28914fc703c05 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2115652 Commit-Queue: Marc Grimme <marcgrimme@chromium.org> Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Reviewed-by:
Oleg Davydov <burunduk@chromium.org> Cr-Commit-Position: refs/heads/master@{#753570}
-
Rouslan Solomakhin authored
Before this patch, the payment method manifest had to be hosted at a location that is pointed to from the payment method identifier URL via the HTTP Link rel=payment-method-manifest header. This complicated testing in Chrome and prevented hosting payment method manifests on popular hosting services, such as GitHub pages. This patch falls back to returning the content of the page if it does not have HTTP Link rel=payment-method-manifest header or that header is not valid. The downloader uses HTTP GET instead of HTTP HEAD for the payment method manifest, since the response body has to be considered. An empty response body now generates an error earlier: in the downloader instead of parser, because the parser always rejected empty input. After this patch, it's possible to host the payment method manifest directly at the URL of the payment method identifier, which makes testing Chrome and developing payment apps easier. Bug: 1035147 Change-Id: Id07c9023bac222ed45eecb8df91108f4df51e356 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2113735Reviewed-by:
Danyao Wang <danyao@chromium.org> Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#753569}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/22ff3cfaf62e..b1128ddc5b31 Created with: gclient setdep -r src-internal@b1128ddc5b31 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: I54c9b24296d86114c4492a1b778511768a11c1c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121551Reviewed-by:
-
Alex Ilin authored
This CL introduces a new class, ReauthTabHelper, that obsevers navigations within the reauth flow. It's done so the tab helper can be reused for reauth modal dialog. Bug: 1045515 Change-Id: I843f8846aefcc8d67ecc702ba385e166d6ad1af4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112910Reviewed-by:
-
Clemens Arbesser authored
This comprises style spans like <b> and <i>, as well as text links of the form <link1>. Also refactored testing util for style spans. Bug: b/145043394 Change-Id: I7bcd375db919ad050a4edaace00f2cff1cc61085 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2116233Reviewed-by:
-
Mark Brand authored
This CL disables fuzzing generation for the vulkan mojom target when enable_vulkan isn't set, since the generated MojoLPM fuzzer code will expect the vulkan headers to be available. This is a cherrypick/split from https://chromium-review.googlesource.com/c/chromium/src/+/2108507 Change-Id: Ia0e3d0f5d19994021bcbc94739ea51a802bd5877 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120476 Commit-Queue: Mark Brand <markbrand@google.com> Reviewed-by:
Ken Rockot <rockot@google.com> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#753565}
-
Armando Miraglia authored
Fixed: 1055399 Change-Id: Icb7cfbf3b15810f79db7b95cfe337b8345fcb828 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120582Reviewed-by:
-
Clemens Arbesser authored
All those parameters are in fact dependencies of the interaction handler. Thus, it makes sense to inject them on creation. This also cuts down on the number of parameters needed for subsequent calls. This CL also refactores the passing of the java context. Since the interaction handler needs to retain a reference, a global reference is passed. All other calls will continue to use local references. This is a refactoring only and should have no user-facing changes. Bug: b/145043394 Change-Id: Id2a62944a584cfbe3c089a5c7f76166fc67ec574 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120713Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/1346dd5de119..fd773eb50d62 git log 1346dd5de119..fd773eb50d62 --date=short --first-parent --format='%ad %ae %s' 2020-03-26 stevenperron@google.com Start SPIRV-Tools v2020.3 2020-03-26 stevenperron@google.com Finalize SPIRV-Tools v2020.2 2020-03-26 stevenperron@google.com Update CHANGES 2020-03-25 alanbaker@google.com Fix identification of Vulkan images and buffers (#3253) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@fd773eb50d62 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/spirv-tools-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,cnorthrop@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_angle_vk32_deqp_rel_ng;luci.chromium.try:android_angle_vk32_rel_ng;luci.chromium.try:android_angle_vk64_deqp_rel_ng;luci.chromium.try:android_angle_vk64_rel_ng;luci.chromium.try:linux_angle_deqp_rel_ng;luci.chromium.try:linux-angle-rel;luci.chromium.try:win-angle-rel-32;luci.chromium.try:win-angle-rel-64;luci.chromium.try:win-angle-deqp-rel-32;luci.chromium.try:win-angle-deqp-rel-64 Bug: None Tbr: radial-bots+chrome-roll@google.com,cnorthrop@google.com Change-Id: I9afc2f263fd7f43dafbbba407b48b11b95b14264 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121377Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/6719f54ac7ce..7a58e5b713c3 git log 6719f54ac7ce..7a58e5b713c3 --date=short --first-parent --format='%ad %ae %s' 2020-03-26 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/spirv-tools/src 1346dd5de119..fd773eb50d62 (4 commits) Created with: gclient setdep -r src/third_party/angle@7a58e5b713c3 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 cnorthrop@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: cnorthrop@google.com Change-Id: I4b0a55afd6154483eb5699326fcafffaee721e8c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121549Reviewed-by:
-
Omar Morsi authored
Bug: None Test: chromeos_unittests Change-Id: I2f41f49cce38b65db5cf5647be2f8f53689e18bb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119523Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Omar Morsi <omorsi@google.com> Cr-Commit-Position: refs/heads/master@{#753560}
-
Jan Wilken Dörrie authored
This change applies changes generated by running the PassedToMoveRewriter rewriter across a Linux build of the codebase. It changes no functionality, and only removes base::Passed invocations that can either be completely omitted, or rewritten in terms of std::move. TBR=dcheng Bug: 714018 Change-Id: Ie8af65a571271c2eb77f8f88c2678e8e8abbbb9d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120479 Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
-
Colin Blundell authored
This CL adds initial integration tests of external intent launching, covering the following cases: - That an intent is not launched for a vanilla browsing URL. - That an intent is blocked if navigated to directly. - That an intent is launched if navigated to via a redirect. Future CLs will add tests of more cases. Change-Id: I6141a6aff81acef40155dfaf74d4a11bc10c7a40 Bug: 1029710, 1031465 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119535 Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#753558}
-
Maxim Kolosovskiy authored
crrev/753172 removed public method |TabImpl.getProfile| and caused a compilation error. TODO: consider to add a CQ bot for the compilation flag TBR=selakovic@google.com,ioanap@chromium.org Bug: 1053522 Change-Id: I4a27681b83a70701c14b3dca3764105e3c1d8494 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120300Reviewed-by:
Maxim Kolosovskiy <kolos@chromium.org> Commit-Queue: Maxim Kolosovskiy <kolos@chromium.org> Cr-Commit-Position: refs/heads/master@{#753557}
-
Tim Schumann authored
This reverts commit 0bbf0f94. Reason for revert: Suspected to cause flaky test (see crbug/1064896) BUG=1064896,1058187 Original change's description: > [Media Feeds] Add feed items to WebUI > > This adds the feed items to the WebUI. You > can select a feed to show the content. > > BUG=1058187 > > Change-Id: I5051f77b68cb3da6710a547d50109cb95a7b890f > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2118758 > Commit-Queue: Becca Hughes <beccahughes@chromium.org> > Reviewed-by: Tommy Steimel <steimel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#753441} TBR=beccahughes@chromium.org,steimel@chromium.org Change-Id: I89b1447a80ccdeebbc93dd3b8c87cd8f72bbed0d No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1058187 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120301Reviewed-by:
-
Istiaque Ahmed authored
ServiceWorkerContextWrapper::StartWorkerForScope searches for registration with active or installing worker through DidFindRegistrationForFindImpl. For the case where there is an installingworker, DidFindRegistrationForFindImpl can see a registration.installing_version(), but the version can be gone by the time it PostTask-s to FoundRegistrationForStartWorker if the service worker script rejected the 'install' event. This CL removes the DCHECK that incorrectly assumes either active or installing version *must* be present in FoundRegistrationForStartWorker. The DCHECK failure is evident in the flaky extension test failure: ServiceWorkerBasedBackgroundTest.WorkerStartFailureClearsPendingTasks. The CL also enables the mentioned test. Bug: 1063476 Change-Id: I12dd3b5f2155bc8b8677f9734443832c1a96fe61 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119208Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Commit-Queue: Istiaque Ahmed <lazyboy@chromium.org> Cr-Commit-Position: refs/heads/master@{#753555}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/0e15e8064fd0..4f77e105e33d git log 0e15e8064fd0..4f77e105e33d --date=short --first-parent --format='%ad %ae %s' 2020-03-26 sigurds@chromium.org [issues] Add audits capability Created with: gclient setdep -r src/third_party/devtools-frontend/src@4f77e105e33d 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:1064587 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Ibcc1ab4c32ddea860df97f109a8d134613c890d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121548Reviewed-by:
-
Artur Kochowski authored
Bug: 1064209 Change-Id: I08ba090dfbb97efa097e8d574b99a66eaaf00f88 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117111Reviewed-by:
Allen Bauer <kylixrd@chromium.org> Commit-Queue: Artur Kochowski <akochowski@opera.com> Cr-Commit-Position: refs/heads/master@{#753553}
-
arthursonzogni authored
NavigationLoaderOnUI landed. Some comments about it are still there. This patch remove them. This disambiguate the fact this variable can be safely used: NavigationURLLoaderImpl::URLLoaderRequestController::browser_context_ Bug: 824840 Change-Id: Icb6dc5050a604878531958acbc0fdc2a7739d420 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120717 Auto-Submit: Arthur Sonzogni <arthursonzogni@chromium.org> Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
-
Mark Cogan authored
Removes TabModelList from IncognitoWebStateObserver. Instead the observer uses an AllWebStateListObservationRegistrar and an inner observer class to perform the same updates. Bug: 1050149 Change-Id: I70d77dcd228e75cb6769b44279cd2a604bc9b0a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117737Reviewed-by:
-
Rainhard Findling authored
Bug: 1015841 Change-Id: I03a5fff56c973663e307cd01fb75bbac1aff162e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117731Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Commit-Queue: Rainhard Findling <rainhard@chromium.org> Cr-Commit-Position: refs/heads/master@{#753550}
-
