- 12 Oct, 2020 40 commits
-
-
Yifan Luo authored
Drop `config_` and use HashSet for config values instead. Bug: 1116418 Change-Id: I7581614ad2fd0e750b5bf428679242efe362566f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461336 Commit-Queue: Yifan Luo <lyf@chromium.org> Reviewed-by:
Daniel Vogelheim <vogelheim@chromium.org> Cr-Commit-Position: refs/heads/master@{#816278}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/896ca12d..ba88e59b 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: Ib89c2d178816166b44bba0f3a3c4e2b4ee5e8b03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466436Reviewed-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@{#816277}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/109c355900d1..7c7b98f06001 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-mac-chromium Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: I00e9f11cda48c06191483103f79c30f2a40f00de Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466376Reviewed-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@{#816276}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/9213e610ed92..4aa4c6014d56 2020-10-12 adlai@google.com Migrate gpu resource methods to GrDirectContext 2020-10-12 johnstiles@google.com Rename SkTArray::allocCntForTest to capacity. 2020-10-12 adlai@google.com Migrate resource cache methods to GrDirectContext 2020-10-12 bsalomon@google.com Remove SkSurface::MakeFromBackendTextureAsRenderTarget 2020-10-12 adlai@google.com Migrate releaseResourcesAndAbandonContext to GrDirectContext 2020-10-12 johnstiles@google.com Add test for disabling the inliner. 2020-10-12 johnstiles@google.com Clean up SkSL inliner and allow it to be disabled. 2020-10-12 johnstiles@google.com Reland "Remove inliner from IR generation stage." 2020-10-12 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ANGLE from 3df629c55d89 to 0f55e3e5d266 (62 revisions) 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/+doc/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 Cq-Do-Not-Cancel-Tryjobs: true Bug: None Tbr: robertphillips@google.com Test: Test: Test: CQTest: Test: GLES1ConformanceTest.SpotExpDir/ES1_VulkanTest: Test: TransformFeedbackTest.RecordAndDrawWithScissorTestTest: Test: angle_end2end_tests on OSX 10.15 Change-Id: Ifd3b407f7d0aa50fab30040c80a3fc77c1ad9784 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466400Reviewed-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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast Tbr: cr-fuchsia+bot@chromium.org Change-Id: I7f36bd11efe6121efb8bde066607a8e70bea3ca4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465868Reviewed-by:
-
Erik Chen authored
Ash now sends, at lacros startup, a map of all crosapi interfaces and their versions to Lacros. Lacros can use this map to synchronously query version info. Bug: 1136652 Change-Id: I14be7f1c9c39709e342c6ab8045587f7d1245e74 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2454629 Commit-Queue: Erik Chen <erikchen@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Greg Kerr <kerrnel@chromium.org> Auto-Submit: Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#816273}
-
Tommy Li authored
Currently, when the user has chosen about:blank as their homepage, and Chromium starts up, the location bar is focused, but the URL "about:blank" is NOT selected. This leads to the cursor being at index 0, and the user starts typing BEFORE the existing "about:blank" URL, so typing "foo" leads to: "fooabout:blank". This is surprising behavior, and this CL fixes that, as well as adds a test. We originally tried to fix this in 2019, but this effort had to be reverted because we caused a bug on the NTP: https://chromium-review.googlesource.com/c/chromium/src/+/1559115 This CL preserves the fixed NTP behavior (validated by a test), and also fixes the about:blank behavior, also validated by a new test. Bug: 45260 Change-Id: I8ca57ea10215c297aabc22e51138af7ea21ef677 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464202Reviewed-by:
Peter Kasting <pkasting@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#816272}
-
Steven Bennetts authored
This API is unused and is being deprecated. See issue for details. BUG=1124419 Change-Id: I4083be1d76c61a95f76c744c26621b14eb9ee651 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2423749 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Erik Staab <estaab@chromium.org> Reviewed-by:
Azeem Arshad <azeemarshad@chromium.org> Reviewed-by:
Ben Wells <benwells@chromium.org> Cr-Commit-Position: refs/heads/master@{#816271}
-
dpapad authored
- Update "on-tap=" to "on-click=" in remaining elements. - Update addEventListener('tap', ...) to addEventListener('click', ...) - Address TODO in JSChecker::TapOnPath() to no longer fire 'tap' events - Remove JSChecker::set_polymer_ui() which becomes obsolete. - Remove EulaTest::NonPolymerOobeJS() which becomes obsolete. - Address TODOs in enable_debugging_browsertest.cc Bug: 812035,949377 Change-Id: Ia57b80f3ac7573fc7f505edfc26c967f2944b4f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463546Reviewed-by:Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#816270}
-
Austin Eng authored
Bug: none Change-Id: I6a46219a111f9522183c3f496651a0772229d5d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464427 Commit-Queue: Austin Eng <enga@chromium.org> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Reviewed-by:
Corentin Wallez <cwallez@chromium.org> Cr-Commit-Position: refs/heads/master@{#816269}
-
Alexander Cooper authored
Moves ArCoreInstallHelper to components, including supporting classes ArCoreShim(Impl) and ArCoreInstallUtils and XrInstallInfoBar. In order to fully move ArCoreInstallUtils, a ArCoreDeviceUtils class was created to manage the last "init" that comes from the ChromeActivity. Note that this is a remnant of the AR DFM which other embedders likely won't need, and could theoretically be removed from //chrome. In moving the "Install" and "Update" strings, they are made a bit more generic, since nothing in them is unique to ArCore. The ic_ar_core_install install graphic is an identical SVG version of the icon currently being used. As there is a TODO to replace it, a new graphic is used to allow for easier replacing. Bug: 1064073,960542 Change-Id: I96c2be6182a5e6d9f09d268e98dbaef94ce68275 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2453873 Commit-Queue: Alexander Cooper <alcooper@chromium.org> Reviewed-by:
Clark DuVall <cduvall@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Piotr Bialecki <bialpio@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Auto-Submit: Alexander Cooper <alcooper@chromium.org> Cr-Commit-Position: refs/heads/master@{#816268}
-
Nico Weber authored
This is a reland of 174d9faa For the reland, tolerate explicitly set number of concurrent links in thinlto builds as long as it's explicitly set to 1. This is needed for some cros bots. Original change's description: > build: In LTO links, run just one link at once, but give it all cores > > Normally, we allow running several links in parallel if there's enough > memory, since links tend to be memory- and io-bound, not compute bound. > However, in LTO builds the linker does codegen for every linked > translation unit, which needs a lot of CPU. > > Previously, we used to hardcode 8 threads per link for thinlto builds, > which makes builds needlessly slow on machines with many cores. > > Instead, automatically use all cores for a thinlto link, but only run > one link in parallel at once. > > chromeos-chrome-9999.ebuild in the Chrome OS build and link_jobs_32 in > mb_config.pyl currently manually set the number of threads per lto link > to the number of cores. Once this change here is in, we should stop > explicitly setting the arg there and then we can turn this from a gn arg > to a regular variable that's not settable in args.gn > > Bug: 1132930 > Change-Id: If2fda3e38b2b1827beb2da844f024f52211210db > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464782 > Reviewed-by: Hans Wennborg <hans@chromium.org> > Commit-Queue: Nico Weber <thakis@chromium.org> > Auto-Submit: Nico Weber <thakis@chromium.org> > Cr-Commit-Position: refs/heads/master@{#816129} Bug: 1132930 Change-Id: I40840f971e2a87afe33e77f537776ba78cf027a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464913Reviewed-by:
Hans Wennborg <hans@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#816267}
-
Tal Pressman authored
This mirrors the same APIs on Receiver, that should also exist for associated interfaces. Change-Id: Ie2feef11f86b2597a8b431b2bbbbb8110b54b0f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461981 Commit-Queue: Ken Rockot <rockot@google.com> Reviewed-by:
Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#816266}
-
Adlai Holler authored
Change-Id: Iebefeb039768f6d15bc0055ca515defebec7e3ae Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464099 Auto-Submit: Adlai Holler <adlai@google.com> Commit-Queue: Khushal <khushalsagar@chromium.org> Reviewed-by:
Khushal <khushalsagar@chromium.org> Cr-Commit-Position: refs/heads/master@{#816265}
-
Adam Langley authored
Phone-as-a-security-key uses top-level Activities in two ways: 1) When receiving a cloud message that says that an authentication is being attempted, an Android notification is shown that triggers a specified Activity. 2) When using USB fallback, desktop Chrome will use the AOA protocol over USB to attempt to communicate with Chrome on the phone. For the Android system to wire things up correctly, Chrome needs an Intent filter on a top-level Activity to catch the USB device. Currently SettingsActivity (which allows for an arbitrary Fragment to be specified via arguments) is used for case one, and case two isn't functional. This change wires up case two via a new top-level Activity, designed for the purpose, and also routes case one via that Activity too. BUG=1002262 Change-Id: Id2246eddc20073d0695c9f7f7f2abc63b53b48a3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2454273 Auto-Submit: Adam Langley <agl@chromium.org> Reviewed-by:Martin Kreichgauer <martinkr@google.com> Commit-Queue: Ted Choc <tedchoc@chromium.org> Commit-Queue: Adam Langley <agl@chromium.org> Cr-Commit-Position: refs/heads/master@{#816264}
-
yilkal authored
This cl launches InlineLoginDialog from OOBE after the family link notice screen exits and before the sync screen is shown. The dialog is a windowless dialog, is the exact same shape of the oobe screens and is modal to the oobe screens. Bug: 1128620 Change-Id: Id8d76e54bfa85b2819d8f80f8ed2801d1f63d48e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2410813 Commit-Queue: Yilkal Abe <yilkal@chromium.org> Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Weilun Shi <sweilun@chromium.org> Reviewed-by:
Anastasiia N <anastasiian@chromium.org> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Kush Sinha <sinhak@chromium.org> Reviewed-by:
Dan S <danan@chromium.org> Cr-Commit-Position: refs/heads/master@{#816263}
-
Kartik Hegde authored
Switch "of" to "or". BUG=chromium:956783 TEST=unit_tests --gtest_filter=GatewayCanBePingedRoutineTest.* Change-Id: I25a8fdfd7bf168a304a6ee97e285e2b81513a1fb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461484Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Kartik Hegde <khegde@chromium.org> Cr-Commit-Position: refs/heads/master@{#816262}
-
George Burgess IV authored
This new clang-tidy check complains when param names /*in_comments=*/ don't match up with the actual parameter name they're being passed to. Sometimes this is benign, other times, actual bugs are found. Bug: 1136209 Change-Id: I1fd87d09c51889669dc1fe0e6d95508fcc28f643 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2458890Reviewed-by:
Rushan Suleymanov <rushans@google.com> Commit-Queue: George Burgess <gbiv@chromium.org> Cr-Commit-Position: refs/heads/master@{#816261}
-
Morten Stenshorne authored
There should be no need to walk all documents in an inner loop while already doing it in the outer loop. The O(n^2) performance complexity is probably negligible, but what I'm actually trying to fix here is problems when compiling with -Wshadow (the problem here being the various |document| variables). Bug: 294205 Change-Id: Ide1ab566e3b9ddb53d4b4f44c9c03633c90348a1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460745Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#816260}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/69d0c1acc28a..f4f1d8a352ca 2020-10-12 hwguy.siplus@gmail.com SPIR-V: Remove SpvTools.h include from disassemble.cpp (#2417) 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 on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_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 Tbr: radial-bots+chrome-roll@google.com Change-Id: I1151e0ddf3a4c2c8b1cc87b8aa52d608883450e7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466322Reviewed-by:
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/7c7b98f06001..a4b8d27d96b7 2020-10-12 treehugger-gerrit@google.com Merge "tp: implement async track set tracker" If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:linux-perfetto-rel Bug: None Tbr: perfetto-bugs@google.com Change-Id: I1f5b35689586f207e007095f8ef79983dc56e8a8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465866Reviewed-by:
-
Mathias Bynens authored
Context: go/navigator-webdriver-legacyunforgeable Change-Id: I5adcc41f519f6b97a88f2f9c74fe6db094b60b81 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460744 Commit-Queue: Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Mathias Bynens <mathias@chromium.org> Cr-Commit-Position: refs/heads/master@{#816257}
-
Rune Lillesveen authored
Bug: 1110330, 1137184 Change-Id: I056150482128f72cd8b882ad5953db08106275b4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463610Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#816256}
-
Allen Bauer authored
Bug: 772945 Change-Id: Idb01c79d10872f91aa83c915e16034e727d70d30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466387 Commit-Queue: Allen Bauer <kylixrd@chromium.org> Commit-Queue: Xiyuan Xia <xiyuan@chromium.org> Auto-Submit: Allen Bauer <kylixrd@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#816255}
-
Moe Ahmadi authored
This reverts commit 4bf05b95. Reason for revert: Causing failures in ArcVmClientAdapterTest See: https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-rel/41873 Original change's description: > arc: Only call StopVm() if ARCVM is running > > This change adds a check in ArcVmClientAdapter::StopArcInstance() to see > if a VM is running (current_cid_ is set). If not, skip calling > ConciergeClient::StopVm() and notify observers that ARCVM is stopped > right away. > > BUG=b:169893632 > TEST=arc.AuthPerf.unmanaged_vm > > Change-Id: Ic927f122cacea0e89cacffbbcd67a217772dba43 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462573 > Commit-Queue: Willie Koomson <wvk@google.com> > Reviewed-by: Yusuke Sato <yusukes@chromium.org> > Cr-Commit-Position: refs/heads/master@{#816221} TBR=yusukes@chromium.org,wvk@google.com Change-Id: Ia248a8ca9bda2eee2a24bf8d23c8549fa0ae7ce8 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: b:169893632 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466501Reviewed-by:
Moe Ahmadi <mahmadi@chromium.org> Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Cr-Commit-Position: refs/heads/master@{#816254}
-
Akihiro Ota authored
This change adds a special case to the voice switching code to suppress announcements when switching from 'en-us' to 'en', or vice versa. This was done for two main reasons: 1. This case frequently appears in the real-world, e.g. Docs, which was leading to overly verbose announcements. 2. This logic shouldn't be generalized to other languages. For example, 'pt' -> 'pt-br' or 'pt-br' -> 'pt' still warrants an announcement since Brazilian Portuguese is quite different from European Portuguese. Tests are included to confirm new behavior. switching is active. Bug: 1098548 Change-Id: Ifcfdcd53e12863fffd25ffb2dfafe8d91a718e4c AX-Relnotes: ChromeVox gives less frequent announcements when voice Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2402152 Commit-Queue: Akihiro Ota <akihiroota@chromium.org> Reviewed-by:
David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#816253}
-
Fredrik Söderqvist authored
We should no longer call the ResourceBoundingBox() method on the above classes during layout, so make the checking stricter by removing the SelfNeedsLayout() check and adding a DCHECK on NeedsLayout(). Bug: 1028061 Change-Id: I00f406e760f5a15a70575e24f6232da4389b7f92 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466240Reviewed-by:
-
Jamie Madill authored
This allows other projects like ANGLE to import protobuf and use the GN files without importing the JS dependencies. Bug: angleproject:5161 Change-Id: I71b08c7c03a63a186312edee0a4b55e95e75cf69 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464426 Commit-Queue: Peter Kasting <pkasting@chromium.org> Reviewed-by:
-
Yuki Shiino authored
Fixes a busy lock of PartitionAlloc initialization. compare_exchange_strong may store true to |expected|, but we always expect false. Bug: 1121427 Change-Id: I8d6513a4ec20fd7c1e9efe997d048f9e4f42c1e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464520Reviewed-by:
Benoit L <lizeb@chromium.org> Reviewed-by:
Bartek Nowierski <bartekn@chromium.org> Reviewed-by:
Will Harris <wfh@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#816250}
-
Rob Buis authored
mtable sets displaystyle to false [1] so add an appropriate stylesheet entry for it, i.e. math-style: compact. This makes displaystyle-011.html pass. [1] https://mathml-refresh.github.io/mathml-core/#table-or-matrix-mtable Bug: 6606 Change-Id: I376698e43153d6893b5bd826bb7625563cbab030 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464269Reviewed-by:
Frédéric Wang <fwang@igalia.com> Commit-Queue: Rob Buis <rbuis@igalia.com> Cr-Commit-Position: refs/heads/master@{#816249}
-
Ben Mason authored
TBR=govind@chromium.org Change-Id: Ie506eb048249e2e1977e066966b97c47dde27d93 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466139Reviewed-by:
Ben Mason <benmason@chromium.org> Reviewed-by:
Krishna Govind <govind@chromium.org> Commit-Queue: Ben Mason <benmason@chromium.org> Cr-Commit-Position: refs/heads/master@{#816248}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/d185008d04b3..716915427088 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 mahmadi@google.com,gab@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:linux-chromeos-chrome Bug: None Tbr: mahmadi@google.com,gab@google.com Change-Id: I2e5c8984f00b56f1c2567a9b9a659578b7e410ac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466358Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Reviewed-by:
-
Xiaohan Wang authored
This is now defined the same as `is_linux` and is no longer needed. Bug: 1132395 Change-Id: Ibbdc6666c52c38e4765d4792a7a10e0c1e8c6747 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2443496Reviewed-by:
Dirk Pranke <dpranke@google.com> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#816246}
-
Yuwei Huang authored
This is a reland of dbe36b16 Original change's description: > Add remoting Linux debug symbols to the upload list > > This CL attempts to make the official build bot upload remoting debug > symbols, so that we can use them to decipher stack traces coming from an > official build remoting Linux host. > > Bug: 1136663 > Change-Id: I5669140b3b63399d521644de3b8524def6092560 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462547 > Reviewed-by: Jamie Walch <jamiewalch@chromium.org> > Reviewed-by: Michael Moss <mmoss@chromium.org> > Commit-Queue: Yuwei Huang <yuweih@chromium.org> > Cr-Commit-Position: refs/heads/master@{#815701} Bug: 1136663 Change-Id: I9dce6bc57a57c7729365045fb0d99cf2e50cba37 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466498Reviewed-by:
Michael Moss <mmoss@chromium.org> Commit-Queue: Yuwei Huang <yuweih@chromium.org> Cr-Commit-Position: refs/heads/master@{#816245}
-
Eric Willigers authored
Various chrome/browser chromeos and ash files were needlessly including chrome_features.h and scoped_feature_list.h Bug: 1137231 Change-Id: I92c88eecb9d15da80481375dd4a9d0827df8b9b8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463091 Auto-Submit: Eric Willigers <ericwilligers@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Reviewed-by:
-
Chris Lu authored
Change-Id: Iff472835b3cd4a444d6fd5c25adc431cc71f4cf7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462887 Commit-Queue: Chris Lu <thegreenfrog@chromium.org> Auto-Submit: Chris Lu <thegreenfrog@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#816243}
-
Alex Gough authored
Also fix a typo. These are opened with FLAG_READ over here https://source.chromium.org/chromium/chromium/src/+/master:content/browser/hyphenation/hyphenation_impl.cc;drc=da8e399b1f509d29565224190204860b22c9062b;l=51 Bug: 1130762 Change-Id: I6840ea5399ca7d7d75592865a02a491cff229199 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462555Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Alex Gough <ajgo@chromium.org> Cr-Commit-Position: refs/heads/master@{#816242}
-
Victor Hugo Vianna Silva authored
No more TextInputServerCreate crashes are seen on Windows following crrev.com/c/2454311 and all the previously disabled tests now have score < 300, so let's re-enable them. Fixed: 1111227 Change-Id: I13ca59f1596826696438f8b84a3a9ba1771c6b14 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461335Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Victor Vianna <victorvianna@google.com> Cr-Commit-Position: refs/heads/master@{#816241}
-
dpapad authored
Bug: 1132403 Change-Id: Iab13dc6afca566f5056f37a25849ac907f0efb2d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466336Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#816240}
-
Mark Mentovai authored
The bucket index is needed internally, but it can be computed from the threshold size rather than being hard-coded. This was partially fixed by 94069728 (https://chromium-review.googlesource.com/c/2428896) which added a constant for the threshold size which can be checked by PA_CHECK. Unfortunately, that new CHECK was violated on mac-arm64, where the environment reports that the system allocator guarantees a minimum 8-byte alignment, thus the bucket corresponding to allocation size 512 occurs at a higher index than other 64-bit platforms, where the environment reports a minimum 16-byte alignment. It’s possible that the alignment reported on mac-arm64 is a bug, because the system allocator does in fact impose a 16-byte minimum alignment, but perhaps it believes that while this is true, it’s only guaranteeing 8-byte alignment. Regardless, the new PA_CHECK did catch that the thread cache threshold was not tuned as expected on mac-arm64. It seems to be intended to have used 512 everywhere, but 256 was being used on mac-arm64 instead. Bug: 1135726 Change-Id: Ic820842d2a6eb669a0a65afb68c34638adf598d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463445 Commit-Queue: Mark Mentovai <mark@chromium.org> Reviewed-by:
-
