- 19 Nov, 2020 40 commits
-
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/dad44f4d824e..1a14e7e02441 2020-11-19 dawn-autoroll@skia-public.iam.gserviceaccount.com Roll Tint from 46d9c7745efa to 36d9a061c847 (13 revisions) 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 kainino@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: kainino@google.com Change-Id: Idf57d4542ee09737c5cf255acdaf39b8adbdcd8b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550174Reviewed-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@{#829281}
-
Jack Franklin authored
https://crrev.com/c/2544131 renamed the method to remove the underscore but shipped both names in order to avoid layout test breakages. Once that's landed, we can land this change, and then remove the old `_linkInfo` method as it will then be entirely unused. Bug: 1149403 Change-Id: I21bcf71fb8e87fe4202d07ee83b3bbf481905f91 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2546760 Commit-Queue: Jack Franklin <jacktfranklin@chromium.org> Reviewed-by:
Tim van der Lippe <tvanderlippe@chromium.org> Cr-Commit-Position: refs/heads/master@{#829280}
-
Kelvin Jiang authored
This CL releases an extension's allocation when it is unloaded for the following reasons: - when it's blocklisted - when it's disabled by policy - when it's disabled for malware Rationale on why certain UnloadedExtensionReason/DisableReason will cause the allocation to be released (or not) are detailed in the doc (internal): docs.google.com/document/d/1oGpntLGf9myZ6lI4XWJNMPaZX1_zooJni6ROsqidC6A Bug: 983299 Change-Id: I2523ee9794a24bd1ffa320b04ff7941842781969 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2545735Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Kelvin Jiang <kelvinjiang@chromium.org> Cr-Commit-Position: refs/heads/master@{#829279}
-
Henrik Boström authored
This is part of investigating https://crbug.com/1148964. It may or may not cause Canary crashes again. We are monitoring crashes. TBR=handellm@google.com Bug: chromium:1148964 Change-Id: I5e778cd024f126811319ba750cb46adbdf3ce090 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550206Reviewed-by:
Henrik Boström <hbos@chromium.org> Commit-Queue: Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#829278}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/3a578ae35233..16973d613fed 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-win-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: I23624f575a9bfbcf6251d66dad0a52586f034abd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550361Reviewed-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@{#829277}
-
Ryan Daum authored
When obtaining initial focus when a webview becomes visible, ensure that the contained web contents within a webview are valid before attempting to focus it. Bug: internal b/173633818 Test: manual Change-Id: I7abec0ca2ccf4d4a804fbafc3c630d0def7ce242 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550161 Commit-Queue: Ryan Daum <rdaum@chromium.org> Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Cr-Commit-Position: refs/heads/master@{#829276}
-
Robert Sesek authored
Bug: 417532 Change-Id: Ie5973223b6f4434bb141e1d7f9e77d9a4f9704b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548342Reviewed-by:
Toni Baržić <tbarzic@chromium.org> Reviewed-by:
Derek Schuff <dschuff@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#829275}
-
Dave Tapuska authored
Use a common implementation in WebViewImpl for adjusting the page focus inside a local root. BUG=1097816 Change-Id: Ife309e622a301c1790f098b73166582593777137 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543834Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#829274}
-
Gang Wu authored
Bug: 1150497 Change-Id: I792c7831f860211a11dd548a04bd7f7ff5dce094 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548265Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Gang Wu <gangwu@chromium.org> Cr-Commit-Position: refs/heads/master@{#829273}
-
Benoit Lize authored
There is no reason to use FastMalloc() when PartitionAlloc is malloc(). Aside from increasing memory usage and heap fragmentation, this reduces the number of pointers allocated out of the GigaCage, which matters for CheckedPtr. To address that, make USING_FAST_MALLOC() a no-op in this case, and redirect the remaining callers of FastMalloc() to malloc(). Bug: 998048 Change-Id: I3bd04570cabc7c8e9d36908013bb1c82a65acddf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2310334 Commit-Queue: Benoit L <lizeb@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Bartek Nowierski <bartekn@chromium.org> Cr-Commit-Position: refs/heads/master@{#829272}
-
Martin Kreichgauer authored
This replaces the bits of GetAssertionRequestHandler that request a PIN/UV Auth Token by calling out to AuthTokenRequester instead. See CL:2469445 for the equivalent CL for MakeCredential. Bug: 1139111 Change-Id: I1669258ffabc679a5b6d0592ed14fbdaa710dc3f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2536930Reviewed-by:
Nina Satragno <nsatragno@chromium.org> Commit-Queue: Martin Kreichgauer <martinkr@google.com> Cr-Commit-Position: refs/heads/master@{#829271}
-
ckitagawa authored
The capture experiment was removed in: https://chromium-review.googlesource.com/c/chromium/src/+/2299184 However, some histogram deletions were missed. This CL makes those histograms obsolete. Bug: 1147100 Change-Id: Ie6d5f5d917ac1d7f14302648bbbc279ac745f263 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544046Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org> Cr-Commit-Position: refs/heads/master@{#829270}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/060020166679..de9e649438ed 2020-11-19 capn@google.com Fix GN build for aarch64 2020-11-19 zzyiwei@google.com AHB: minor clean up to avoid build failures on different platforms 2020-11-19 capn@google.com Add GN builds for Reactor unit tests 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/+doc/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;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: None Tbr: swiftshader-team+autoroll@google.com Change-Id: Ie91c87ffd94b4b2af5771df9e66b4ec29e0f1db1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550241Reviewed-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@{#829269}
-
Illia Klimov authored
This reverts commit 444c343b. Reason for revert: Flash is deprecated and should be removed. Original change's description: > Discard wildcard patterns for plugin enterprise policies > > This CL adds the possibility to throw away patterns with wildcards > defined in the enterprise policies for a content settings > > Bug: 1073883 > Change-Id: I99f2d742289e77f1d00081a7cf186d4b21a91f1f > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2207159 > Commit-Queue: Ravjit Singh Uppal <ravjit@chromium.org> > Reviewed-by: Balazs Engedy <engedy@chromium.org> > Cr-Commit-Position: refs/heads/master@{#770598} TBR=engedy@chromium.org,ravjit@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1149318 Change-Id: I1cc4691f41e5149a6419c94be756f681dcf9add9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2537842Reviewed-by:
Illia Klimov <elklm@google.com> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Ravjit Singh Uppal <ravjit@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Commit-Queue: Balazs Engedy <engedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#829268}
-
Rayan Kanso authored
The crash report seems to imply that this can be null, although I'm still unsure when that would happen. Bug: 1150628, 1143619 Change-Id: Ifceb52846deaab48ec749ee11992b3c586ecbcb5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550182 Commit-Queue: Rayan Kanso <rayankans@chromium.org> Reviewed-by:
Richard Knoll <knollr@chromium.org> Cr-Commit-Position: refs/heads/master@{#829267}
-
Leonid Baraz authored
The purpose is to make max_total_files_size, max_total_memory_size, single_file_size and max_record common parameters for all queues, and have them available in StorageQueue. The next step would be to manage queues based on them. Also, parameterize StorageTest (like StorageQueueTest was). Bug:b:157943192 Bug:b:159361496 Change-Id: I073ba6f3a4d3dd4e9c92ef263a3f569a9343c3c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548101Reviewed-by:
Zach Trudo <zatrudo@google.com> Commit-Queue: Leonid Baraz <lbaraz@chromium.org> Cr-Commit-Position: refs/heads/master@{#829266}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/fa9411e0a242..f577788041b6 2020-11-19 mmortensen@google.com Port FindSymbolFiles from scripts/upload_symbols.py, refactor for BuildAPI. 2020-11-19 vapier@chromium.org simplify gitiles links 2020-11-19 engeg@google.com api.payload: Add dlc support to api 2020-11-18 kimjae@chromium.org dlc_lib: Sort global variable names 2020-11-18 navil@google.com dependency: Only return changed packages. 2020-11-18 saklein@chromium.org scripts/OWNERS: Add autotest_quickmerge OWNERS. 2020-11-18 chrome-bot@chromium.org Update config settings by config-updater. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@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:chromeos-betty-pi-arc-chrome;luci.chromium.try:chromeos-kevin-rel Tbr: chrome-os-gardeners@google.com Change-Id: I7c8533dcb563beedcdb1cbe2208ad941ebbca111 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550166Reviewed-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@{#829265}
-
Amr Aboelkher authored
policy namespace Accessing the PSM UMA constants will be useful to be used in its corresponding testing file, instead of using a literal strings. As PrivateSetMembershipStatus is being used twice, once as the expected status of the request in AutoEnrollmentClientImpl, and another usage to indicate whether PSM is enabled or not in the test file. Thus, the one inside the test file has been renamed. BUG=chromium:1150248 Change-Id: I31944094c63e505592197400d3a8d63a3db3e852 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2545387 Commit-Queue: Amr Aboelkher <amraboelkher@google.com> Auto-Submit: Amr Aboelkher <amraboelkher@google.com> Reviewed-by:
Pavol Marko <pmarko@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Roland Bock <rbock@google.com> Cr-Commit-Position: refs/heads/master@{#829264}
-
Adam Langley authored
I tightened up a lot of visibility in f50ddd34 but, while this class isn't used directly from outside of the package, the Android system uses reflection in ways that require it to be public. BUG=1002262 Change-Id: Ie2c921e465e89de4f6ff650a240c0bebe799737c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548384 Auto-Submit: Adam Langley <agl@chromium.org> Commit-Queue: Martin Kreichgauer <martinkr@google.com> Reviewed-by:
Martin Kreichgauer <martinkr@google.com> Cr-Commit-Position: refs/heads/master@{#829263}
-
Xianzhu Wang authored
Change-Id: I6fdca5b9b5ebde0af04fb8dbdc6aa524ac8e5255 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548140Reviewed-by:
Stefan Zager <szager@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#829262}
-
Toni Barzic authored
The pref toggle change handler was referencing the toggle using this.$.<control-id>. This seems to be broken - possibly because the toggle is inside a template element. This changes the code to use this.$$('$<control-id>'). BUG=1150682 TEST=Toggle the setting to show shelf navigation buttons in tablet mode, and verify the shelf state (while in tablet mode) changes. Change-Id: I522a296320e886693dfd64688c51ab6a5030f9ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548189 Commit-Queue: Toni Baržić <tbarzic@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#829261}
-
Jack Franklin authored
Change-Id: Id80b71dbd68ed0cd390bdb804ed9378cc6ac3f3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550029Reviewed-by:
Tim van der Lippe <tvanderlippe@chromium.org> Commit-Queue: Jack Franklin <jacktfranklin@chromium.org> Cr-Commit-Position: refs/heads/master@{#829260}
-
Avi Drissman authored
On platforms where accelerators are not shown in menus, and in circumstances where the underline is not drawn, fully strip the CJK style of accelerators "(&x)". Note that this was already done for Mac native menus and associated Mac native code in FixUpWindowsStyleLabel(). Fixed: 974843 Change-Id: Ifc89fc574213cd350da8e992ac02683afec3718c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2546471 Commit-Queue: Robert Liao <robliao@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Auto-Submit: Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#829259}
-
Nate Fischer authored
No change to logic, this only changes OWNERS. This swaps out my LDAP for file:// URIs pointing to what I think are reasonable subfolders (of which I'm already an OWNER). I also added some new per-file rules for tests which fit in the same logical grouping (networking and metrics). Test: N/A Change-Id: Id55c72e29821255a2ab2375f64596ceef6c6f19f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548388 Commit-Queue: Richard Coles <torne@chromium.org> Auto-Submit: Nate Fischer <ntfschr@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#829258}
-
Sigurdur Asgeirsson authored
ScopedObserver is being deprecated in favor of two new classes: - base::ScopedObservation for observers that only ever observe a single source. - base::ScopedMultiSourceObservation for observers that do or may observe more than a single source. This CL was uploaded by git cl split. R=pkasting@chromium.org, tluk@chromium.org Bug: 1145565 Change-Id: Ifb06297c295ec0b76e7380bc3d3787bc72a4b411 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2547776 Auto-Submit: Sigurður Ásgeirsson <siggi@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#829257}
-
Anton Bikineev authored
If maybe_ptr is within super-page payload, but refers to not allocated memory, null is returned from GetSlotStartInSuperPage. The check for returned null was missing. Bug: 11297512 Change-Id: I4284b1c79f353c32c96e9cbc75d0200aa452bf30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2549921 Auto-Submit: Anton Bikineev <bikineev@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#829256}
-
David Grogan authored
These used some full-width characters that wpt doesn't have a font for. As a result, the sizes of the containing blocks that had been calculated to align with the text is misaligned on wpt. I replaced the characters with inline-blocks with fixed width and height. I tried using an mplus web font available in wpt but couldn't find a way to determine the size of the characters that I could use to derive the size of the blocks. Using fixed width/height blocks is more robust anyway. https://wpt.fyi/results/css/css-flexbox?label=experimental&label=master&aligned&q=css-flexbox-row Change-Id: I3e9484029c9e275c4dcc23dda967d2a166e44da3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548681Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: David Grogan <dgrogan@chromium.org> Cr-Commit-Position: refs/heads/master@{#829255}
-
David Tseng authored
- in SpeakOrEnqueue, for tts platforms that were not yet ready, we were enqueueing the utterance then immediately clearing it if the utterance was !CanBeEnqueued. We should clear the queue, then add the utterance to be spoken later. - cleanup for style/readability R=dmazzoni@chromium.org Change-Id: Ifb2311792c6dca042181871797de8aa0c67ba261 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543374 Commit-Queue: David Tseng <dtseng@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#829254}
-
Sigurdur Asgeirsson authored
ScopedObserver is being deprecated in favor of two new classes: - base::ScopedObservation for observers that only ever observe a single source. - base::ScopedMultiSourceObservation for observers that do or may observe more than a single source. This CL was uploaded by git cl split. R=estade@chromium.org Bug: 1145565 Change-Id: Icbd9700d976f131d4507a9601d7664894806da2a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2547961 Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Auto-Submit: Sigurður Ásgeirsson <siggi@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#829253}
-
chromium-internal-autoroll authored
Release_Notes: http://go/help_app-x20/relnotes/Main/help_app_nightly_202011190600_RC00.html https://chrome-infra-packages.appspot.com/p/chromeos_internal/apps/help_app/app/+/zwjjfdu5X9OPcMNJ2vD6-qpbr5JfXymOFoQz-LOYG6cC If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/help-app-chromium-autoroll Please CC help-app@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.chrome.try:linux-chromeos-chrome Bug: b/138961540 Tbr: help-app@grotations.appspotmail.com Change-Id: Ia9fff990bc4d8ccc7f02a1a8834a5474d3e3d3f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550164Reviewed-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@{#829252}
-
Peter Wen authored
Most of these suppressions no longer matter after switching to linting monochrome bundle only. Bug: 1116130 Change-Id: I0e675603ccb2ecdd056c6a3b8f9e995e1c0108d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2546994Reviewed-by:
Mohamed Heikal <mheikal@chromium.org> Commit-Queue: Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#829251}
-
Artem Sumaneev authored
The test was disabled with https://codereview.chromium.org/608153002 as flaky on Mac. It does not seem to be a case anymore. BUG=339240 TEST=unit_tests --gtest_filter="PolicyTest.FileURLBlacklist" Change-Id: I4f67e001ee73b75152895960e37af5e6c98ce65e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550028Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Commit-Queue: Artem Sumaneev <asumaneev@google.com> Cr-Commit-Position: refs/heads/master@{#829250}
-
Vasiliy Telezhnikov authored
Before this CL we would look just for substring in GL_EXTENSIONS string which is incorrect. In this particular case there is extension GL_ARM_shader_framebuffer_fetch_depth_stencil which extends base GL_ARM_shader_framebuffer_fetch to use for depth and stencil. Some devices (e.g Pixel 2) support it and don't support the base one. This CL switches over to gfx::ExtensionSet to avoid problems. Bug: 1150171 Change-Id: I4a2708bfeaf3ef54818ffe171fbeb7cfb2ff7452 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550223Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Vasiliy Telezhnikov <vasilyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#829249}
-
Ben Pastene authored
Got the buildergroup wrong. Whoops. Bug: 1146481 Change-Id: If00c61c494d9adc1970af948fb8b6433b241a65f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548124 Auto-Submit: Ben Pastene <bpastene@chromium.org> Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Reviewed-by:
Garrett Beaty <gbeaty@chromium.org> Cr-Commit-Position: refs/heads/master@{#829248}
-
Guido Trotter authored
Currently when the info dialog doesn't have the remove button, its text is not read on opening, and it is not accessible via ChromeVox controls. This patch fixes it, by reading it out by default, on opening. Signed-off-by:
Guido Trotter <ultrotter@chromium.org> Fixed: 901427 Change-Id: I091e9ff9c2e32a9fc1dc40ada25b2bccea8fe569 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544969Reviewed-by:
Thomas Tellier <tellier@google.com> Cr-Commit-Position: refs/heads/master@{#829247}
-
Willie Koomson authored
This change refactors GpuArcVideoServiceHost to split it into 2 services: GpuArcVideoServiceHost implements mojom::VideoHost, while GpuArcVideoKeyedService is an ARC KeyedService that registers GpuArcVideoServiceHost with the ARC bridge. This means that GpuArcVideoServiceHost can run before login. BUG=b:165826281 TEST=tast run <DUT> arc.VideoDecodeAccel.* Change-Id: I2cec09518f3a25d0c44420c058a36517e1f747b5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2538162Reviewed-by:
Hidehiko Abe <hidehiko@chromium.org> Reviewed-by:
Ryo Hashimoto <hashimoto@chromium.org> Commit-Queue: Willie Koomson <wvk@google.com> Cr-Commit-Position: refs/heads/master@{#829246}
-
Tanmoy Mollik authored
These test failures was caused by https://crrev.com/c/2432413 cl and was reverted. Enabling the tests again. Bug: 1145573 Change-Id: I8abe2e9b9bf999494719fa8136825695fb40cb27 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2549942Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org> Cr-Commit-Position: refs/heads/master@{#829245}
-
Lily Chen authored
This CL removes the code added in https://crrev.com/c/2243255. Bug: 1095192 Change-Id: I906b7c50e1e60184155ebabcd81dbef0798e9eb2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2547398Reviewed-by:
Maksim Orlovich <morlovich@chromium.org> Commit-Queue: Lily Chen <chlily@chromium.org> Cr-Commit-Position: refs/heads/master@{#829244}
-
Yifan Luo authored
Bug: 1116418 Change-Id: I7dca04f8b947bc88977691c5a6fb353a5484fb36 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2547702Reviewed-by:
Daniel Vogelheim <vogelheim@chromium.org> Commit-Queue: Yifan Luo <lyf@chromium.org> Cr-Commit-Position: refs/heads/master@{#829243}
-
Alex Turner authored
Currently, only typeface digests (i.e. the output) are included. We now include two likely lookup values: where the family name or the PostScript name were looked up with default FontSelectionRequests. While there will be many input digests that map to the same typeface, this should handle a good chunk. We also remove the duplicate signed integer copy of the typeface digest, leaving only the unsigned version. Bug: 1150480 Change-Id: I7f36061b5da4022984a908ba9988258094e9c5b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2548311Reviewed-by:
Asanka Herath <asanka@chromium.org> Commit-Queue: Alex Turner <alexmt@chromium.org> Cr-Commit-Position: refs/heads/master@{#829242}
-