- 26 Mar, 2020 40 commits
-
-
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:
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@{#753577}
-
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:
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@{#753568}
-
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:
David Roger <droger@chromium.org> Commit-Queue: Alex Ilin <alexilin@chromium.org> Cr-Commit-Position: refs/heads/master@{#753567}
-
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:
Marian Fechete <marianfe@google.com> Commit-Queue: Clemens Arbesser <arbesser@google.com> Cr-Commit-Position: refs/heads/master@{#753566}
-
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:
Tim Schumann <tschumann@chromium.org> Commit-Queue: Tim Schumann <tschumann@chromium.org> Cr-Commit-Position: refs/heads/master@{#753556}
-
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}
-
Rohit Agarwal authored
The method testAllowPermissionDoNotLeakFromIncognitoToIncognito failed when CustomTabIncognitoManager#hasIsolatedProfile returns true. This happened because the test was missing a method call to request permission. This CL adds the method invocation and updates comments. Bug: 1035770 Change-Id: I4ca71b082a6140d79c2e4835d3e803099fb26232 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120415Reviewed-by:
Ramin Halavati <rhalavati@chromium.org> Reviewed-by:
Peter Conn <peconn@chromium.org> Reviewed-by:
Rohit Agarwal <roagarwal@chromium.org> Commit-Queue: Rohit Agarwal <roagarwal@chromium.org> Cr-Commit-Position: refs/heads/master@{#753549}
-
Anastasiia Nikolaienko authored
Add component and strings for parent info screen. Move edu-login-button position CSS to shared CSS file. Update template to allow scrollable content. Screenshot: http://screen/ty41uTYuCQO Bug: 1043108 Change-Id: Iea78f81e8469a41ca17357e891355676bf57d8a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2010963Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Kush Sinha <sinhak@chromium.org> Commit-Queue: Anastasiia Nikolaienko <anastasiian@chromium.org> Cr-Commit-Position: refs/heads/master@{#753548}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/d8c6c0e0f3a6..ee2ec6de995b git log d8c6c0e0f3a6..ee2ec6de995b --date=short --first-parent --format='%ad %ae %s' 2020-03-26 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 107e4e989e94..6719f54ac7ce (6 commits) 2020-03-26 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ../src 772d2084..16171704 (389 commits) Created with: gclient setdep -r src/third_party/skia@ee2ec6de995b 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: I837d2815d36b92c241a094e0b286a16c3ca93f7e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121761Reviewed-by:
-
David Jean authored
Use AppGround to open "learn how" link. Bug: 1045455 Change-Id: Ic9c8bb4c12a51804f2d95510d401777fa576da87 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2115543 Commit-Queue: David Jean <djean@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Reviewed-by:
Javier Ernesto Flores Robles <javierrobles@chromium.org> Cr-Commit-Position: refs/heads/master@{#753546}
-
Trent Apted authored
To fix [..:INFO:test_launcher.cc(1498)] The following parameterized test case is not instantiated: UninstantiatedParamaterizedTestSuite< BrowserNonClientFrameViewAshTestNoWebUiTabStrip> UninstantiatedParamaterizedTestSuite< ImmersiveModeBrowserViewTestNoWebUiTabStrip> These test suite aliases were added in r748410. Bug: 1059514 Change-Id: I6a60a2d862a08e0e8f3cad8083de7966cdf19823 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121520 Auto-Submit: Trent Apted <tapted@chromium.org> Reviewed-by:Peter Kasting <pkasting@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Commit-Queue: Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#753545}
-
Eric Willigers authored
We make the kWebAppFileHandlers key available on stable rather than trunk. The ExtensionFromWebApp unit tests now use a test fixture that sets up a testing environment (by inheriting from ExtensionServiceTestBase). This is a speculative fix for a bug (crbug.com/1062239) that seems to have been introduced by a change to ConvertWebAppToExtension (crrev.com/c/2087327). The test WebAppFileHandlersAreCorrectlyConverted is re-enabled. This CL replaces https://chromium-review.googlesource.com/c/chromium/src/+/2108208 Bug: 1062239 Change-Id: I107a0731ab5b17faa51660384c10cab400e3cb7c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120676 Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Reviewed-by:
Alexey Baskakov <loyso@chromium.org> Reviewed-by:
-
Matthias Körber authored
Currently, either the country field in an address form is used to determine the address requirements for autofill profile creation, or the the app_locale is used as a fallback option. This change adds the country code supplied from Chrome Variations as a primary fallback before the app_locale is used to deduce the country of a user. Also the country code is used to as a source for the default country for address creation. Change-Id: Ia7d28c66a31775b26d7d33153670b1db6eade74c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107981 Commit-Queue: Matthias Körber <koerber@google.com> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Vadym Doroshenko <dvadym@chromium.org> Cr-Commit-Position: refs/heads/master@{#753543}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/d5a3755f19aa..0e15e8064fd0 git log d5a3755f19aa..0e15e8064fd0 --date=short --first-parent --format='%ad %ae %s' 2020-03-26 sigurds@chromium.org [coverage] Disable save button if there is nothing to save Created with: gclient setdep -r src/third_party/devtools-frontend/src@0e15e8064fd0 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: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Id0f793f72de9b90406ad02a88f794a1da189dba3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121760Reviewed-by:
-
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: I5fc94e77c5636e33c8cce31006c21534ccdd7011 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121759Reviewed-by:
-
Peter Kasting authored
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/+/2117335Reviewed-by:
Darren Shen <shend@chromium.org> Reviewed-by:
Yuchen Liu <yucliu@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}
-
Trent Apted authored
MediaAppIntegrationTest.MediaAppLaunchWithFile and MediaAppIntegrationWithFilesAppTest.FileOpenCanTraverseDirectory have been flaking at low rates, but got flakier in r753464. Haven't been able to reproduce locally. https://crrev.com/c/2108400 adds some robustness, but without a local repro I haven't been able to verify this. Disabling to investigate further. TBR=zafzal@google.com Bug: 1064863 Change-Id: I1b27f9b9b89f3ad72d45b2548c2115b8361620f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121119Reviewed-by:
Trent Apted <tapted@chromium.org> Commit-Queue: Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#753539}
-
chrome://media-appTrent Apted authored
This adds a new IDR resource id, to decouple the shown-icon from any icon asset that gets brought in via DEPS. Bug: b/150637102 Change-Id: I55b98e9fe7cf0d3fde4a87661d9dfbd39dc12ea7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120661 Commit-Queue: Trent Apted <tapted@chromium.org> Reviewed-by:
Patti <patricialor@chromium.org> Cr-Commit-Position: refs/heads/master@{#753538}
-
