- 08 Dec, 2020 40 commits
-
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/61355d416d9c..30805557e651 2020-12-08 enga@chromium.org Enable use_tint_generator w/ test suppressions for SPIR-V based backends 2020-12-08 rharrison@chromium.org Fix issues with roll into Skia 2020-12-08 cwallez@chromium.org Rename TryConvertAspect to better reflect what it does. 2020-12-08 dawn-autoroll@skia-public.iam.gserviceaccount.com Roll Tint from b8a3078bbec9 to 195917c0be32 (17 revisions) 2020-12-08 stha09@googlemail.com use cstring instead of string.h 2020-12-08 stha09@googlemail.com GCC: fix template specialization in ObjectContentHasher If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dawn-chromium-autoroll Please CC rharrison@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel Bug: None Tbr: rharrison@google.com Change-Id: I36597fcaff5c9fee9f830c70aee55f24652b3a50 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579765Reviewed-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@{#834852}
-
Aaron Leventhal authored
DCHECK Occurs when running test in a11y FYI bot, which uses --additional-driver-flags=--force-renderer-accessibility. Bug: None Change-Id: Ia7764492cb1aa171197b0f7e6ac7a6011aa15f0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577703Reviewed-by:
vmpstr <vmpstr@chromium.org> Reviewed-by:
Aaron Leventhal <aleventhal@chromium.org> Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Cr-Commit-Position: refs/heads/master@{#834851}
-
Yi Gu authored
The verification backend may be unavailable for some reasons. In most cases we will fall back to the user consent backend but occasionally we need to cancel the request. Regardless, we want to measure how often this happens and why. Bug: 1127397, 1141024 Change-Id: I7e8e92878c100d20df012705ef3b944e2ad3ae64 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2572956Reviewed-by:
Jesse Doherty <jwd@chromium.org> Reviewed-by:
Majid Valipour <majidvp@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#834850}
-
Josh Nohle authored
When the existing flag CryptAuthV2DeviceActivityStatus is enabled, we receive two piece of information from the server about each device: - the Bond last active time and - a connectivity status (online vs. offline). Previously, when the CryptAuthV2DeviceActivityStatus flag was enabled, the multi-device dropdown list was sorted with all online devices first (sorted by last activity time) then all offline devices (also sorted by last activity time). However, there are two issues with using the connectivity status to sort: (1) The connectivity statuses--as well as the last activity times--are only retrieved at sign-in or when new devices are synced down via a DeviceSync. A mildly stale last activity time is acceptable, but a stale offline status could be very misleading, especially because an "(offline)" label is displayed in the dropdown list. (2) During testing, even when forcing a connectivity status retrieval, we noticed that the connectivity status might not be accurate. In this CL, we put use of the connectivity status behind a flag: CryptAuthV2DeviceActivityStatusUseConnectivity. When enabled, online devices are sorted before offline devices, and offline devices are labeled with "(offline)" in the dropdown list. When disabled, the connectivity status is ignored during sorting, and no connectivity labels are applied in the UI. For example, say the Samsung Galaxy has a more recent Bond last active time than the Pixel, and the Pixel is online whereas the Galaxy is offline. The UI will appear as follows with CryptAuthV2DeviceActivityStatus enabled: CryptAuthV2DeviceActivityStatusUseConnectivity disabled (default): https://screenshot.googleplex.com/4cRYc5UXiAHMSyE.png CryptAuthV2DeviceActivityStatusUseConnectivity enabled: https://screenshot.googleplex.com/AeqyvBcA29meeyN.png This allows us to enable the CryptAuthV2DeviceActivityStatus flag to use the Bond last activity time to sort the dropdown list, while disabling the CryptAuthV2DeviceActivityStatusUseConnectivity flag. This immediately improves the dropdown experience without worrying about stale connectivity statuses. Manually tested with two eligible host phones, adjusting their last activity times and online/offline states, and going through multi-device setup. Bug: 923594 Change-Id: I19d08232d7d0f35b3b430793f22a4aac07d9b5c7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577632Reviewed-by:Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Josh Nohle <nohle@chromium.org> Cr-Commit-Position: refs/heads/master@{#834849}
-
Vincent Boisselle authored
Bug: 1156169 Change-Id: I9e8510b879663354b767e2a2b4c6c38b0f8f1499 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2576622Reviewed-by:
-
Michael Thiessen authored
Migrates all remaining ActivityTestRule usage to BaseActivityTestRule, which is more flexible and fixes various issues with ActivityTestRule. This change also bans the usage of ActivityTestRule so that more unintentional usage doesn't creep in. Bug: 1146574 Change-Id: I67f9427b8b637d36589f7d06e5cde51c8a50caa6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527770 Commit-Queue: Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Cr-Commit-Position: refs/heads/master@{#834847}
-
Moe Ahmadi authored
Fixed: 1078677 Change-Id: I4534a9a2834d8de4d7cb692e0d19ed648d686429 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2568354 Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#834846}
-
Ionel Popescu authored
In order to test multiple input events being fired, the input element should be disabled async. Bug: 1148986 Change-Id: Ia49f78c38ec459e6237a9e6171dcf64239363a1b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2578707Reviewed-by:
Mason Freed <masonfreed@chromium.org> Commit-Queue: Ionel Popescu <iopopesc@microsoft.com> Cr-Commit-Position: refs/heads/master@{#834845}
-
Rakib M. Hasan authored
Yesterday the refresh-weblayer-skew-tests recipe failed because of of error which was fixed in crrev.com/c/2577952. This CL updates the binaries skew tested. It also removes redundant test suites which skew test an older version of M88. Bug: 1147459, 1099017, 1041619 Change-Id: I1cc49a66ae457a3283bdc3439f4e7d1966ee1bfb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2580441Reviewed-by:
John Chen <johnchen@chromium.org> Commit-Queue: Rakib Hasan <rmhasan@google.com> Cr-Commit-Position: refs/heads/master@{#834844}
-
Becca Hughes authored
As part of our security review we were asked to split the KaleidoscopeDataProvider interface into two to reduce the attack surface from the NTP. BUG=1114862 Change-Id: Ie37472dfcda44cbea00b3e5b9b610b485d41d5fa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577899 Commit-Queue: Becca Hughes <beccahughes@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Tibor Goldschwendt <tiborg@chromium.org> Cr-Commit-Position: refs/heads/master@{#834843}
-
Gavin Mak authored
Generate DIR_METADATA files and remove metadata from OWNERS files for directories under //chrome/browser. Bug: 1113033 Change-Id: I998057216aba1f64121847848308f07b5b8e1e35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2578156Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Gavin Mak <gavinmak@google.com> Cr-Commit-Position: refs/heads/master@{#834842}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/1742f98791c9..c1fbb82f9e23 2020-12-08 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). 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 ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com Change-Id: Ic6acadb5db5c881d35f924877b900cddfbb0612b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579767Reviewed-by:
-
Gavin Mak authored
Generate DIR_METADATA files and remove metadata from OWNERS files for directories under //chrome. Bug: 1113033 Change-Id: I3f29622a021b2ffd73f91d81fa690553af87594e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577831Reviewed-by:
-
Will Harris authored
This reverts commit 6a60c787. Reason for revert: causing crashes BUG=1132179 Original change's description: > Reland "Add dbghelp.dll to delay load list" > > This is a reland of 4edc0503. With > the fix in clang for the delay load crash previously hit with this > change I am relanding this. > > Bug: 1132179 > > Original change's description: > > Add dbghelp.dll to delay load list > > > > Currently, many of the binaries have a static dependency on dbghelp.dll. > > This results in dbghelp getting loaded in all chrome processes. Upon > > inspection, dbghelp.dll is only necessary for stack sampling and mini > > dump handing in crashpad. Stack sampling is only enabled by default on > > unofficial builds of dev and canary. All other times we are needlessly > > loading this module which brings in 300k of image file refset and 60k > > of Copy on Write costs per process. This change adds it to the delay > > load list. > > > > Change-Id: I13da0ce0735431c5c51cdbc9ba2cfea0cc3cded7 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2336541 > > Commit-Queue: Chris Davis <chrdavis@microsoft.com> > > Reviewed-by: Cliff Smolinsky <cliffsmo@microsoft.com> > > Reviewed-by: Bruce Dawson <brucedawson@chromium.org> > > Reviewed-by: Nico Weber <thakis@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#796707} > > Change-Id: I5f946d76af853c2d277f670b99479e874e784f44 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523245 > Reviewed-by: Nico Weber <thakis@chromium.org> > Reviewed-by: Cliff Smolinsky <cliffsmo@microsoft.com> > Commit-Queue: Chris Davis <chrdavis@microsoft.com> > Cr-Commit-Position: refs/heads/master@{#832526} TBR=thakis@chromium.org,brucedawson@chromium.org,cliffsmo@microsoft.com,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com,chrdavis@microsoft.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1132179 Change-Id: I0d340b2853f5d67b8ddc080450a10c1f93e38b77 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579985Reviewed-by:
Will Harris <wfh@chromium.org> Commit-Queue: Will Harris <wfh@chromium.org> Cr-Commit-Position: refs/heads/master@{#834839}
-
Dominique Fauteux-Chapleau authored
TBR=jrw@chromium.org Bug: 1156767 Change-Id: I213f0d5776002d8ef6931ff564ad109e6e0bc052 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2580299Reviewed-by:
Dominique Fauteux-Chapleau <domfc@chromium.org> Commit-Queue: Dominique Fauteux-Chapleau <domfc@chromium.org> Cr-Commit-Position: refs/heads/master@{#834838}
-
chromium-autoroll authored
Roll Chrome Mac PGO profile from chrome-mac-master-1607428729-5d7fb4cc4f483e4100008e013fc190caf99a5621.profdata to chrome-mac-master-1607450253-216f09df6b61ffe8dbd79a2c1e7354b5e42173e1.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-mac-chromium Please CC pgo-profile-sheriffs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:mac-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: Ia124019516452d74a0afc378f5ddb3691453d5fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2580301Reviewed-by:
-
Dominic Farolino authored
kMBIMode is a feature that enables Multiple Blink Isolates work, which introduces a new scheduling boundary in the render process: AgentSchedulingGroup. What this CL sets out to solve: For MBI, we'd like to have some include-only bots that we can request to run "all of the tests" that a normal CQ bot would run, but in MBI mode (with our feature enabled). It's basically a substitute for us uploading a dummy CL that enables our feature and running the try bots to see what would fail. After speaking with hypan@ and I think indirectly, fergal@, the suggestion to use what bfcache is doing was made, that is, to set-up a recipe like [1]. However it appears that in order to set this kind of bot up, you would have to manually specify all of the test targets in the config. That might be worth doing, but since MBI makes some pretty fundamental changes, it would be best to run *all* the tests that a normal CQ bot would run on the waterfall. After speaking with hypan@ about this, he mentioned the simplest (only?) way to do this would be to make the feature controllable via gn args, and create an mb_config that applies the gn arg like: https://chromium-review.googlesource.com/c/chromium/src/+/2399129/4/tools/mb/mb_config.pyl#2772 Then we'll be able to reuse the default configs that the other waterfall bots use. For example, [2] uses [3]. [1]: https://source.chromium.org/chromium/chromium/src/+/master:testing/buildbot/chromium.linux.json;l=12199;drc=073d5574fa06b002a5a9754ff24c8e72cac779ea [2]: https://source.chromium.org/chromium/chromium/src/+/master:tools/mb/mb_config.pyl;l=1283-1285;drc=f64c223e2408d61250660c221d3f26ad28ec9060 [3]: https://source.chromium.org/chromium/chromium/src/+/master:tools/mb/mb_config.pyl;l=2792-2798;drc=f64c223e2408d61250660c221d3f26ad28ec9060 R=kinuko@chromium.org Bug: 1154360 Change-Id: I71eb06b9d63699e8774b5d8fc9abc75e89f05a3b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577593Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Reviewed-by:
Stephen Martinis <martiniss@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Tal Pressman <talp@google.com> Commit-Queue: Dominic Farolino <dom@chromium.org> Cr-Commit-Position: refs/heads/master@{#834836}
-
Alexander Surkov authored
File names has to match class names. Also see discussion at goolge accessibility mailing list, https://groups.google.com/u/1/a/chromium.org/g/chromium-accessibility/c/JvecMcj8thw Bug: 1133330 Change-Id: I8c09211b9705f56c3d4d29b6403cbde0e2f10c70 AX-Relnotes: n/a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2566633 Commit-Queue: Alexander Surkov <asurkov@igalia.com> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
-
Josh Karlin authored
Change-Id: I8d2eccaa5352291b589043a3fcb0575ea82bc242 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2580319Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Commit-Queue: Josh Karlin <jkarlin@chromium.org> Cr-Commit-Position: refs/heads/master@{#834834}
-
Dominic Mazzoni authored
The --pid argument is used for both pids and hwnds now, and the Chromium scripts should look for Chromium in the window name and not Google Chrome. Bug: none Change-Id: I36e41413684d9a93a0f2b7573172bbeef2e46f89 AX-Relnotes: N/A Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577925 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
-
Vladimir Levin authored
When we do a EnsureComputedStyle in a display none subtree, we are not in a style recalc but we still call SetRequestedState on a display lock context. This means that we should not dirty style, but rather rely on style adjustment for children to take the dirty bit information. The testcase for this is in https://chromium-review.googlesource.com/c/chromium/src/+/2575815 which DCHECKs without this patch. R=aleventhal@chromium.org Bug: 1147481 Change-Id: Iaa24945f4ad44e73d0bf79ec757ad79b7081af9c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579523Reviewed-by:
-
David Bertoni authored
These tests were previously disabled for flakiness, but they passed 3000 iterations on the bots with no flakes. Bug: 1132581 Change-Id: I954734c79a1b46829830257dadc04e7a8177ce2b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2568897Reviewed-by:
Kelvin Jiang <kelvinjiang@chromium.org> Commit-Queue: David Bertoni <dbertoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#834831}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/d9d73bd8981e..12b3841881bd 2020-12-08 changhaohan@chromium.org Fix flashing metrics widget when changing nodes 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/+doc/master/autoroll/README.md Bug: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Ieae68ab5ee6c29945aa143dfa4622015e6a0e4f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579025Reviewed-by:
-
Yulun Wu authored
Bug: 1156265 Change-Id: Ie422fd32cf651b6e0524425c7eae7f10f56271e7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577894Reviewed-by:
Toni Baržić <tbarzic@chromium.org> Commit-Queue: Yulun Wu <yulunwu@chromium.org> Cr-Commit-Position: refs/heads/master@{#834829}
-
Dominique Fauteux-Chapleau authored
TBR=jrw@chromium.org Bug: 1156754 Change-Id: I85af0e5b94b65be15392329d3fc11831530dac67 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579724Reviewed-by:
-
Jenny Zhang authored
Bug:1149452 Change-Id: If2defff6b2e1274a22b354e6d1563f77ec1c3004 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577996Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Jenny Zhang <jennyz@chromium.org> Cr-Commit-Position: refs/heads/master@{#834827}
-
arthursonzogni authored
I found something interesting. The initial empty document's javascript windows is reused for same-origin new document. We apply the sandbox flags to the window, instead of documents, which is very weird. My hypothesis: we are enforcing sandbox_flags twice into the same window. The bug might be caused by the remanence of the initial sandbox flags? This turned out to be the case. \o/ This patch adds a WPT test. It triggers a DCHECK in the browser process. Bug: 1041376 Change-Id: Ic7543148b0c2571e95c0c4dfe6df50563254981b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2578957 Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Ian Clelland <iclelland@chromium.org> Reviewed-by:
Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#834826}
-
Andre Le authored
Add MultiDevice.SecureChannel.Nearby.ConnectionResult Bug: 1106937, 1150634 Change-Id: I24a807a5de0e9322ae9e53ed6bf6290f0fa30861 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2575516 Commit-Queue: Andre Le <leandre@chromium.org> Reviewed-by:
-
Thomas Lukaszewicz authored
This change will enable WebUI bubbles used in top chrome to conform to the secondary UI spec on Mac platforms. Before: https://drive.google.com/file/d/12gCfewhEQrzLAPL2IkCYZy7P_NJNp_jp/view?usp=sharing After: https://drive.google.com/file/d/1mkz37-Vmpy6a8HGfvg_JMMiJXdirWtEn/view?usp=sharing Bug: 1156290 Change-Id: I06b3ef9090cfb8f08c0e9ba095bdeb9807ba107d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577939Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Thomas Lukaszewicz <tluk@chromium.org> Cr-Commit-Position: refs/heads/master@{#834824}
-
Aleks Totic authored
Fixed layout table should have definite length. Previously, we treated all !Auto width tables as fixed. Fixes external/wpt/css/css-tables/fixed-layout-2.html Bug: 958381 Change-Id: I98996d3f4760166f06bafb94995709658f8234f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2578705 Commit-Queue: Aleks Totic <atotic@chromium.org> Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Auto-Submit: Aleks Totic <atotic@chromium.org> Cr-Commit-Position: refs/heads/master@{#834823}
-
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 chrome-fuchsia-gardener@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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-deterministic-dbg;luci.chromium.try:fuchsia-x64-cast Tbr: chrome-fuchsia-gardener@grotations.appspotmail.com Change-Id: I7587157efbc862026de81ef9b8bc78d76fae6cc4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579021Reviewed-by:
-
Fredrik Söderqvist authored
This method is a bit special and only has a single user, so inline it into that user. Clean up the related expression to make it a bit easier to read and comprehend. Bug: 789511 Change-Id: Ib0299a47f17b8c4b7381005a20a9c8d57f676ea6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579783 Auto-Submit: Fredrik Söderquist <fs@opera.com> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Stephen Chenney <schenney@chromium.org> Commit-Queue: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#834821}
-
Weilun Shi authored
This is an unit test for https://chromium-review.googlesource.com/c/chromium/src/+/2464431. It makes sure that the patterned histograms should be generated before histogram suffixes so that the affected histograms in suffixes can find histograms generated from varaints. Change-Id: Id48924152076dcb7247104efd88f436ad4071776 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2576953 Auto-Submit: Weilun Shi <sweilun@chromium.org> Commit-Queue: Steven Holte <holte@chromium.org> Reviewed-by:
-
Mikel Astiz authored
Metric Sync.BookmarksModelMetadataCorruptionReason continues to be relevant and needs monitoring for a few more milestones. Change-Id: I9a0c5de1ecc4d846a0695a11af6ba7850a3be1f5 Bug: 1153773 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577684 Auto-Submit: Mikel Astiz <mastiz@chromium.org> Reviewed-by:
-
Marijn Kruisselbrink authored
https://github.com/whatwg/html/issues/3283 was resolved to change the spec to not allow access to session storage in opaque origins, matching our implementation. Bug: 677733 Change-Id: I23da98f166f0845312a0c8ec386d84ac2e3c74f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579023 Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org> Auto-Submit: Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
-
Austin Sullivan authored
GetDatabaseNames is no longer used as of https://crrev.com/c/2557571 This change removes the entire callpath, as well as IndexedDBCallbacks and WebIDBCallbacks specific to GetDatabaseNames. The change allows for the future removal of IDBAny::kDOMStringListType. Bug: 992958 Change-Id: Ide725ff2ba9143cc6a1d13123b7d9a624ac4a1cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2563872Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Emily Stark <estark@chromium.org> Commit-Queue: Austin Sullivan <asully@chromium.org> Cr-Commit-Position: refs/heads/master@{#834817}
-
Marc Treib authored
This matches the expiry of the related wallet-requires-first-sync-setup flag. Bug: none Change-Id: I29f8d54af8d23ea7142811c3614ba8d55931fe63 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579963 Auto-Submit: Marc Treib <treib@chromium.org> Reviewed-by:
Jared Saul <jsaul@google.com> Commit-Queue: Jared Saul <jsaul@google.com> Cr-Commit-Position: refs/heads/master@{#834816}
-
Andre Le authored
Set PhoneHubManager in all root window controllers so that Phone Hub UI can be shown in all displays. In onboarding view and continue browsing chip, use the controller in the current display (instead of the primary one). Bug: 1106937,1126208 Change-Id: I9e31f3140399d303861395ed52717f9a939bb9c0 Fixed: 1155971 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2574678 Commit-Queue: Andre Le <leandre@chromium.org> Reviewed-by:
Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#834815}
-
Dominic Mazzoni authored
We were exposing colindex:0;rowindex:0; on every table cell, and this was confusing NVDA and causing it to not output table cell info while navigating. Bug: 1078623 AX-Relnotes: improves screen reader feedback when navigating the Task Manager table on Windows Change-Id: I63776ee9c3211e9726e861a48ff5ed1a41c7121c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2578706Reviewed-by:
-
Javier Ernesto Flores Robles authored
As there are no exceptions at this moment, it only points to Google's style guide. Bug: N/A Change-Id: Ie5102b3152e38394d0482203dfeb57bfc94c648e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2555461Reviewed-by:
Mike Pinkerton <pinkerton@chromium.org> Commit-Queue: Javier Flores <javierrobles@chromium.org> Cr-Commit-Position: refs/heads/master@{#834813}
-
