- 22 Jun, 2019 30 commits
-
-
Kenneth Russell authored
conformance2/textures/misc/ tex-base-level-bug.html tex-mipmap-levels.html are failing on 10.14 as well. Bug: 870856, 965842 Change-Id: I2dd15e3a3672579bd6557ef459564a5f508e4613 Tbr: ynovikov@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672565Reviewed-by:Kenneth Russell <kbr@chromium.org> Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#671497}
-
Mohamed Heikal authored
Using relative paths was causing build failures. To create this change: * find out/Debug -name *_jni.h -delete * find . -name BUILD.gn -exec sed -i '/jni_package =/d' {} \; * ninja $ALL_JNI_HEADERS minus third_party, chromecast, cronet, libassistant * refactor_jni_includes.py --out-dir out/Debug * fix DEPs files manually * split out services/, testing/, url/ changes into this cl See https://groups.google.com/a/chromium.org/forum/?#!topic/java/MEovGrAwbqI for discussion on naming scheme. Bug: 964169 Change-Id: If6cd9a19ccf2bf189aada4d165d89322ef429518 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1671669Reviewed-by:Ted Choc <tedchoc@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Mohamed Heikal <mheikal@chromium.org> Auto-Submit: Mohamed Heikal <mheikal@chromium.org> Cr-Commit-Position: refs/heads/master@{#671496}
-
Dominik Laskowski authored
This CL changes the type of OutputProtectionDelegate callbacks to OnceCallback, and fixes an uninitialized member variable. Bug: 929449 Test: Build Change-Id: If3bd6cc96fa2056ce59d044e68bc727166a5d720 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1597225 Commit-Queue: Dominik Laskowski <domlaskowski@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#671495}
-
Becca Hughes authored
Make the PictureInPictureOptions optional because we can infer everything from the element. BUG=953957 Change-Id: I85865a02a403d7694795d22d2fe49d030a5c2bdc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659151 Commit-Queue: Becca Hughes <beccahughes@chromium.org> Reviewed-by:
François Beaufort <beaufort.francois@gmail.com> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#671494}
-
Mohamed Heikal authored
Using relative paths was causing build failures. To create this change: * find out/Debug -name *_jni.h -delete * find . -name BUILD.gn -exec sed -i '/jni_package =/d' {} \; * ninja $ALL_JNI_HEADERS minus third_party, chromecast, cronet, libassistant * refactor_jni_includes.py --out-dir out/Debug * fix DEPs files manually * split out chrome/ and ui/ changes into this cl See https://groups.google.com/a/chromium.org/forum/?#!topic/java/MEovGrAwbqI for discussion on naming scheme. Bug: 964169 Change-Id: I1e3b455c0405d93d7f516796fa77b5d0679d9888 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1669073 Commit-Queue: Mohamed Heikal <mheikal@chromium.org> Reviewed-by:Mohamed Heikal <mheikal@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#671493}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/712f0a7bd31d..b4580351c794 git log 712f0a7bd31d..b4580351c794 --date=short --no-merges --format='%ad %ae %s' 2019-06-21 michaelludwig@google.com Fix subset behavior in makeWithFilter. 2019-06-21 herb@google.com Add API for glyph image data 2019-06-21 mtklein@google.com vpshufb 2019-06-21 michaelludwig@google.com Handle non-finite quads in attemptQuadOptimization 2019-06-21 mtklein@google.com vbroadcastss 2019-06-21 mtklein@google.com clarify can-we-call-JIT logic 2019-06-21 mtklein@google.com rearrange who mprotects Created with: gclient setdep -r src/third_party/skia@b4580351c794 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. 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=chromium:977315 TBR=bsalomon@google.com Change-Id: I1d5a3c1aa258038879549f918dcd05c475242fa7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672167Reviewed-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@{#671492}
-
Jacob DeWitt authored
This test verifies that XRInputSource objects are re-created when required by the SameObject attributes: when a gamepad is connected or disconnected from the input source, when handedness changes, and when target ray mode changes. The test also makes sure the input change events are sent correctly. This can't be an external WPT test because it uses MockXRInputSource and a cross-browser XR input test API is not available yet. Bug: 976978 Change-Id: I5fd5e90daf78c40fc8c57f70f0a389274222eb95 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672108 Commit-Queue: Jacob DeWitt <jacde@chromium.org> Reviewed-by:
Alexander Cooper <alcooper@chromium.org> Cr-Commit-Position: refs/heads/master@{#671491}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/ec9252e6d134..1c2b616bc420 git log ec9252e6d134..1c2b616bc420 --date=short --no-merges --format='%ad %ae %s' 2019-06-21 achuith@chromium.org device: Reduce dropped SSH connections. Created with: gclient setdep -r src/third_party/chromite@1c2b616bc420 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Change-Id: I9393f54503afbf02ec9b585b07c7e8411c2c3a73 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672505Reviewed-by:
-
Dana Fried authored
This allows proper UI update (color change) on tabs that have been added to a group. Had to remove the unit test ensuring contiguity across all update notifications, since it's no longer a valid assumption. Bug: 976502 Change-Id: I260eb4fb5128079068f7750b882dcb1d749eceed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1670281 Commit-Queue: Dana Fried <dfried@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Cr-Commit-Position: refs/heads/master@{#671489}
-
Jimmy Gong authored
- Fixes a bug where the default text for printer queue would appear grey, which made it seem inactive. Bug: 958740 Test: end to end manual, browsertests Change-Id: I18bdcc5a6984f737cabfaa7ab06a7714b5176194 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1663075Reviewed-by:
Bailey Berro <baileyberro@chromium.org> Reviewed-by:
Sean Kau <skau@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: jimmy gong <jimmyxgong@chromium.org> Cr-Commit-Position: refs/heads/master@{#671488}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/78e90396294f..13820f2a7ac9 Created with: gclient setdep -r src-internal@13820f2a7ac9 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=tzik@google.com,dbeam@google.com,huanzhong@google.com Change-Id: I5010e30f34435018a287cc64871b12d9479545f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672506Reviewed-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@{#671487}
-
Matt Mueller authored
Bug: 975173 Change-Id: I13507ea3eb6cf76a1cb2bc1134a6a255b55a650b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1666717 Commit-Queue: Matt Mueller <mattm@chromium.org> Reviewed-by:
David Benjamin <davidben@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#671486}
-
Christian Biesinger authored
...when computing the main-axis size of a flex item. As per this change: https://drafts.csswg.org/css-flexbox/#change-2018-used-cross-size R=dgrogan@chromium.org, eae@chromium.org Bug: 972576 Change-Id: I11a1ac14e2e0e44e39fcd45d4568874da2dcfce8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672014 Auto-Submit: Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
David Grogan <dgrogan@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#671485}
-
Eric Aleshire authored
Bug: 963613 Change-Id: I05a043cd5d1788b53e38a5b9a51bc5da08ba7290 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1668312Reviewed-by:
Eugene But <eugenebut@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#671484}
-
Xiaocheng Hu authored
This patch removes some redundant includes of the header file, and also clear one indirect include via css_primitive_value_mappings.h This is a preparation for augmenting calc() to support min() and max(). Bug: 825895 Change-Id: I4d607e501448015096fa98535e33a4505859509b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672465 Auto-Submit: Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#671483}
-
Ben Pastene authored
Bug: 932269, 977736, 977737, 977738, 977739, 977740, 977741, 977743, 977744, 977745, 977749, 977750 Change-Id: I79a8991e3a6db7602fc45aa1ff78ee2eecd2e584 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672568Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#671482}
-
Kenneth Russell authored
This reverts commit 8d4643f0. Reason for revert: caused some pixel and WebGL test failures per http://crbug.com/965842 . Original change's description: > Use high performance GPU by default in GPU tests. > > Upcoming CLs will change the heuristics for selecting the low-power > vs. high-performance GPUs on dual-GPU systems, macOS in particular. > The GPU tests in Chromium's hardware fleet expect to run on the > high-power GPU, so maintain this behavior by default. > > Bug: 965842 > Change-Id: I17707475a4505905ea95a8a57d06a1754fc101fc > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1665910 > Reviewed-by: ccameron <ccameron@chromium.org> > Commit-Queue: Kenneth Russell <kbr@chromium.org> > Cr-Commit-Position: refs/heads/master@{#670302} TBR=ccameron@chromium.org,kbr@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 965842 Change-Id: I6402b31efe05ebacb5a488c5d6cecba4f790c1db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1671863Reviewed-by:
-
Xianzhu Wang authored
Composited layer under multicol is still not fully correctly supported. Currently we don't allow fragmentation of composited layers, and skip fragment clip for them. However, if there are other clips below the fragment clip, we should not skip fragment clip to ensure the other clips are correctly applied. This is still not fully correct (see the comment in the code for details), but the result is less broken in many cases. Hopefully we'll fully fix the issue in CompositeAfterPaint+LayoutNG-box-fragments. Bug: 976187 Change-Id: Ica9acc71f0d1204d6380f1bcac3e2159631d63e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1670413 Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#671480}
-
Abhishek Bhardwaj authored
This change adds a separate class to do the dirty work of running and retrying a task if it fails. This will be used for all tasks that would need to be run on an update check. BUG=924762 TEST=Unit tests. Change-Id: I89cc65fc93631989857c3bd8dce6f53dca31fe2f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1661972Reviewed-by:
Pavol Marko <pmarko@chromium.org> Commit-Queue: Abhishek Bhardwaj <abhishekbh@chromium.org> Cr-Commit-Position: refs/heads/master@{#671479}
-
Matt Menke authored
And plan to remove it in Chrome 78. It was added about 2.5 years ago as a temporary measure to give Enterprises time to fix servers that were (likely accidentally) returning HTTP/0.9 responses. Usage appears to be low, allowing for relatively safe removal of the policy and the code backing it. Bug: 976751 Change-Id: I3d5c133cca77c1747bf52c905a5509e520f8626d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1666912Reviewed-by:
Eric Roman <eroman@chromium.org> Reviewed-by:
-
Yuly Novikov authored
With the exception of "GPU FYI XR Win Builder" Perf bot. Bug: 973992, 977735, 976471 Change-Id: I976a15c6d422981e402a1e8aa55bb710324fd875 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672177Reviewed-by:
-
Dan Beam authored
BUG=965811 Change-Id: I70024269a3239b1920d0cd73f79ab9242fde6ac6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1671467 Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Auto-Submit: Dan Beam <dbeam@chromium.org> Cr-Commit-Position: refs/heads/master@{#671476}
-
Mohamed Heikal authored
Using relative paths was causing build failures. To create this change: * find out/Debug -name *_jni.h -delete * find . -name BUILD.gn -exec sed -i '/jni_package =/d' {} \; * ninja $ALL_JNI_HEADERS minus third_party, chromecast, cronet, libassistant * refactor_jni_includes.py --out-dir out/Debug * fix DEPs files manually * split out net/, remoting/, printing/ changes into this cl See https://groups.google.com/a/chromium.org/forum/?#!topic/java/MEovGrAwbqI for discussion on naming scheme. Bug: 964169 Change-Id: I81f7ae06c3ccd0c9b7ac83df0159295804635374 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1671727Reviewed-by: -
Geoff Lang authored
Use the presence of the GL_CHROMIUM_compressed_texture_etc extension as an indicator that there is native support for GL_OES_compressed_ETC1_RGB8_texture in ANGLE. BUG=882580 Change-Id: Ibb0032052d971a19f00ebb2e5815da9185aa5148 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1666870Reviewed-by:
-
Mohamed Heikal authored
Using relative paths was causing build failures. To create this change: * find out/Debug -name *_jni.h -delete * find . -name BUILD.gn -exec sed -i '/jni_package =/d' {} \; * ninja $ALL_JNI_HEADERS minus third_party, chromecast, cronet, libassistant * refactor_jni_includes.py --out-dir out/Debug * fix DEPs files manually * split out components/ changes into this cl See https://groups.google.com/a/chromium.org/forum/?#!topic/java/MEovGrAwbqI for discussion on naming scheme. Bug: 964169 Change-Id: I4f6fa72e33092dc5f9a2ad78a4d410c881e244e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1669082Reviewed-by: -
Eric Roman authored
The NetLog is now an internal detail of the Network Service. In particular this CL: * Removes ChromeNetLog class * Removes ContentBrowserClient::GetNetLog() * Removes BrowserProcess::net_log() * Removes support for writing to a NetLog when Network Service is disabled. And exception is made for Android WebView, which preserves support for NetLog when Network Service is not enabled. Bug: 934009,767450 Change-Id: I95e2a6aae85ec9c7b90bce08c57e23cb688e5d02 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1663846Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Robbie McElrath <rmcelrath@chromium.org> Commit-Queue: Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#671472}
-
Matt Mueller authored
This is used to limit a CRL to a subset of the certs issued by a single issuer. (Does not support any other issuingDistributionPoints features.) Bug: 749276 Change-Id: Ifb5122bf7fdea17d8a884fe76896a65c570d2d31 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1671367 Commit-Queue: Matt Mueller <mattm@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/fd047540b0cd..24b441ff93e8 git log fd047540b0cd..24b441ff93e8 --date=short --no-merges --format='%ad %ae %s' 2019-06-21 rnasri@google.com Implement Visually Complete at 2s Metric Created with: gclient setdep -r src/third_party/catapult@24b441ff93e8 The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. 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_optional_gpu_tests_rel BUG=chromium:973838 TBR=sadrul@google.com Change-Id: I7f2da07e1b747768ab658ebab107cfb15cd1ac04 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1671907Reviewed-by:
-
Varun Khaneja authored
Here's how it works: 1. Client calls V4LDbM::CheckUrlForHighConfidenceAllowlist() 2. If the hash prefixes aren't ready or Safe Browsing is disabled, it returns NO_MATCH i.e. can't determine if the URL is safe. 3. If no match is found in the local database, it returns NO_MATCH to indicate the URL isn't on the allowlist. 4. If the full hash of the URL matches a full hash in the local database, MATCH is returned to indicate no further lookup is required. 5. If a hash prefix match is found, it initiates a full hash lookup and returns ASYNC to indicate that the client will be notified later. If the returned value is ASYNC, a full hash request is sent to the backend to check if the URL's full hash matches a full hash of a URL on the high confidence allowlist. Once the response is received, the client's |OnCheckUrlForHighConfidenceAllowlist| method is called with the final result, whenever it is available (or a timeout happens). This entire feature is behind a different Finch flag: SafeBrowsingRealTimeUrlLookupEnabled R=drubery Bug: 963165, 966646 Change-Id: Ifdc24d3f0b2e68013da5080febbe2aaf94edded4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1661288 Commit-Queue: Varun Khaneja <vakh@chromium.org> Reviewed-by:
Nathan Parker <nparker@chromium.org> Cr-Commit-Position: refs/heads/master@{#671469}
-
Ben Pastene authored
Fails on swarming due to missing size, eg: https://chrome-swarming.appspot.com/task?id=45909000afe02a10 R=erikchen, dpranke Bug: 961548 Change-Id: I2ce40de235243f21e4b52b02c2925efade230fcd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1669979Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#671468}
-
- 21 Jun, 2019 10 commits
-
-
Sofiya Semenova authored
Only prefetch the prefetch_url if the tab is in the foreground. Start a prefetch when a backgrounded tab is switched to the foreground. Only prefetch any URL once. Bug: 976071 Change-Id: If0a34ef5203794d34993c90dbe8db370b1780b2f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1667753 Commit-Queue: Sofiya Semenova <sofiyase@google.com> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Cr-Commit-Position: refs/heads/master@{#671467}
-
Kurt Horimoto authored
Observers who had started UI because of overlay presentation previously would have had to store state to track whether the UI has been started so that it could be stopped when the corresponding DidHideOverlay() signal was received. This CL updates DidHideOverlay() to include the request corresponding with the overlay that was stopped so observers could detect whether UI needs to be updated based off of the request instead of storing this state. Implementing this callback required updating OverlayRequestQueueImpl to return unique pointers of popped requests so that the request lifetime can be extended beyond the observer callback. Bug: 941745 Change-Id: I256fa21766148ea4b8856c86e3d3e99517b48be4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659232 Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> Reviewed-by:
Mike Dougherty <michaeldo@chromium.org> Auto-Submit: Kurt Horimoto <kkhorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#671466}
-
Scott Little authored
This CL marks "loading" attribute support as experimental, so that tests run with this attribute supported. Note that this only turns on lazy loading of images and iframes marked with "loading=lazy", and does not turn on any automatic lazy loading of images/iframes that aren't marked "loading=lazy". Bug: 937986 Change-Id: Iccf1c4614f648f0596198e9c76eafac818daac23 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1636639 Commit-Queue: Scott Little <sclittle@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#671465}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/6caa1897fd2c..9d1193b591c5 git log 6caa1897fd2c..9d1193b591c5 --date=short --no-merges --format='%ad %ae %s' 2019-06-21 thestig@chromium.org Clean up image_diff_png.cpp. 2019-06-21 thestig@chromium.org Change PNG functions to return std::vector. 2019-06-21 tsepez@chromium.org Ensure CPDF_ColorSpace classes are only created via MakeRetain<>. 2019-06-21 thestig@chromium.org Check encoding results in DiffImages(). 2019-06-21 chinsenj@google.com Move PNG encoding logic to helper in pdfium_test_write_helper.cc 2019-06-21 tsepez@chromium.org Make SharedCopyOnWrite() invoke class-specific Clone() method. Created with: gclient setdep -r src/third_party/pdfium@9d1193b591c5 The AutoRoll server is located here: https://autoroll.skia.org/r/pdfium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=pdfium-deps-rolls@chromium.org Change-Id: Ie9762abb8a8978523d98502d4d34c396979e0724 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672168Reviewed-by:
-
Dirk Pranke authored
It turns out isolates built on Mac builds that enable_dsyms didn't include the symbols in the .dSYM/ directories for frameworks, apps, and plugins. This CL fixes that. R=bpastene@chromium.org, rsesek@chromium.org BUG=961548 Change-Id: I60f27e7e51235cac6d4fa3c5b7c6de11c0b97819 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1671667Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Ben Pastene <bpastene@chromium.org> Commit-Queue: Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#671463}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/067687f467dd..647f200016ab git log 067687f467dd..647f200016ab --date=short --no-merges --format='%ad %ae %s' 2019-06-21 geofflang@chromium.org Limit max texture size and max MSAA samples on Android. 2019-06-21 geofflang@chromium.org Always scalarize mat and vec constructor arguments. 2019-06-21 tobine@google.com Print perf results to stdout on Android 2019-06-21 syoussefi@chromium.org Vulkan: Handle 0-sized viewports Created with: gclient setdep -r src/third_party/angle@647f200016ab The AutoRoll server is located here: https://autoroll.skia.org/r/angle-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. 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_optional_gpu_tests_rel BUG=chromium:882580,chromium:882580,chromium:977330 TBR=jonahr@google.com Change-Id: I33951ca3853d2818eabc7154456bcad3d69dd1d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1671909Reviewed-by:
-
Findit authored
This reverts commit 07f70727. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 671446 as the culprit for failures in the build cycles as shown on: https://analysis.chromium.org/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzA3ZjcwNzI3ODc5Yjg5MGJjZGI4MTgzMDM2ZTg5NWRlMzc1Y2E2OTUM Sample Failed Build: https://ci.chromium.org/buildbot/chromium/android-archive-rel/1542 Sample Failed Step: compile Original change's description: > [build] Cleanup for specifying [min|target|max]SdkVersion in GN > > crrev/c/1650265 enforced setting [min|target|max]SdkVersion in GN. > However, this is to strict for some dependencies, e.g. WebRTC. > Therefore, relax rules so that you can specify the SDK versions in both > GN and the AndroidManifest.xml. And the build system will assert that > these values match. > > + Update some SDK versions that had been changed inadvertently in > crrev/c/1650265. > > TBR=tedchoc@chromium.org,torne@chromium.org > > Bug: 891996 > Bug: 977417 > Bug: 977328 > Change-Id: I89b944c1f73a35ba8fcdafdf4bdf62508759811e > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1660361 > Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org> > Reviewed-by: Andrew Grieve <agrieve@chromium.org> > Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org> > Reviewed-by: Eric Stevenson <estevenson@chromium.org> > Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org> > Auto-Submit: Tibor Goldschwendt <tiborg@chromium.org> > Cr-Commit-Position: refs/heads/master@{#671446} Change-Id: I06e05d04cf5e8cbc4c663bd6a5ebfda1ba0b2b3c No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 977328 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1672178 Cr-Commit-Position: refs/heads/master@{#671461}
-
Koji Ishii authored
This patch fixes |ComputeMinMaxSize| not to clear |NeedsLayout| when all of following conditions are met: 1. Needs |MinMaxSize| 2. Is an orthogonal flow root. 3. Is legacy layout. 4. Change is applied that dirties the layout. In this case, |NGBlockNode::ComputeMinMaxSize()| calls |Layout()|, which goes to |RunLegacyLayout()|, which calls |LayoutObject::LayoutIfNeeded()|. However, its constraint space is intermediate that it does not update |CachedLayoutResult|. Then when laying it out, because |NeedsLayout()| is cleared, existing out-of-date |CachedLayoutResult| is used. This patch fixes: 1. Not to pre-layout orthogonal roots if it has |CachedLayoutResult|. 2. Set |NeedsLayout()| if |RunLegacyLayout()| clears it, but |CachedLayoutResult| was not updated. Bug: 976859 Change-Id: Ic2d979a3c4f9479040bfd0e54cf4283b60ed52b5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1670146Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
-
Jared Saul authored
This CL removes the following histograms, which essentially reverts https://codereview.chromium.org/2864363004/ and https://codereview.chromium.org/2866993002/ from 2017: Autofill.DaysSincePreviousUseAtSubmission.Profile Autofill.HasModifiedProfile.CreditCardFormSubmission Bug: 975959, 975904 Change-Id: I7fac9ca5851767e6f595a8943ca0d5bea3480030 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1669994 Commit-Queue: Jared Saul <jsaul@google.com> Reviewed-by:
Robert Kaplow (slow) <rkaplow@chromium.org> Reviewed-by:
Parastoo Geranmayeh <parastoog@google.com> Cr-Commit-Position: refs/heads/master@{#671459}
-
Jordy Greenblatt authored
The color contrast between minimize+close button and the background in the Wallpaper modal[1] is low enough to cause accessibility issues for vision impaired users at a ratio of 3.61 (compare to the minimal acceptable ratio of 4.5 to meet W3C guidelines). The contrast provided by the new color for both of the buttons, i.e. Google Grey 700 aka #5F6368 aka rgb(95, 99, 104), provides an acceptable ratio of 6.05. ------ Result of this change: http://screen/d6eeOZx9ted Note: newly colored buttons in red rectangles. Bug: 910362 Change-Id: I1193b43675b81d0492dd16d4115ade48c0512f7f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1671728Reviewed-by:
May Lippert <maybelle@chromium.org> Commit-Queue: Jordy Greenblatt <jordynass@chromium.org> Cr-Commit-Position: refs/heads/master@{#671458}
-
