- 07 May, 2019 40 commits
-
-
Hector Carmona authored
Bug: 956673 Change-Id: I8ec2ddfbc5ee09a09a8737fafb5a8fd3ded02e1c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592801 Commit-Queue: Hector Carmona <hcarmona@chromium.org> Reviewed-by:
Tommy Martino <tmartino@chromium.org> Reviewed-by:
Esmael El-Moslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#657284}
-
Becca Hughes authored
If we clone the web contents we do not set the audio focus id map and therefore we hit a DCHECK. BUG=959308 Change-Id: Ib2395c56af4b51b6e9b6fbe5fb0e64a51f0ff296 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594256Reviewed-by:
Alexey Baskakov <loyso@chromium.org> Commit-Queue: Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#657283}
-
Mihai Sardarescu authored
A few unit tests were failing when UnfiedConsent feature is enabled by default. This CL updates them to pass when the UnifiedConsent feature is enabled or disabled by default. Bug: 960260 Change-Id: I47c08697d595378edc129352579dc8b858e6558a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598792Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Commit-Queue: Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#657282}
-
Xiaohan Wang authored
This reverts commit c8e149a5 with more deps fix. Note that any target that includes base/logging.h but misses the dependency to //base could cause a compile error. Original CL description: Currently NOTREACHED() uses LogErrorNotReached() on ChromeOS for release builds, and DCHECK(false) otherwise. However, there are other projects depending on Chromium that would prefer using DCHECK(false) in all cases, and currently there's no easy way to do so. This CL adds an |enable_log_error_not_reached| gn arg to control whether to use LogErrorNotReached() for NOTREACHED() to solve this problem. It is enabled by default only for ChromeOS release builds to keep the current behavior. Tbr: thakis@chromium.org Bug: 760393 Change-Id: Ied42ceb2193f2946568051397597dd68809c4c61 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1595298 Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#657281}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/b0119a63642e..808c6984e664 git log b0119a63642e..808c6984e664 --date=short --no-merges --format='%ad %ae %s' 2019-05-07 lalitm@google.com Merge "trace_processor: bump min chunk size in trace processor" 2019-05-07 lalitm@google.com Merge "trace_processor: add heap profiling tables" 2019-05-07 ilkos@google.com Merge "Correct ion values" Created with: gclient setdep -r src/third_party/perfetto@808c6984e664 The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-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. TBR=perfetto-bugs@google.com Change-Id: I01a4072560186d8f7c32723d263d45a3376a7c28 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598283Reviewed-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@{#657280}
-
Dan Harrington authored
* Use GetPagesTask and DeletePageTask routines to remove duplicate queries. * Using PageCriteria replaces multiple queries (one for each namespace) with a single one, so this should be more efficient. Change-Id: I7057d35e6e5296fddf6ad3e6b2ef63c1179044c3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592038 Commit-Queue: Dan H <harringtond@chromium.org> Reviewed-by:
Carlos Knippschild <carlosk@chromium.org> Cr-Commit-Position: refs/heads/master@{#657279}
-
Chris Lu authored
Fixes bug where badge was being displayed defocusing the omnibox regardless if there was an infobar. Sets the leadingButton to be hidden before focus animation. Then restores the displayed state of the leadingButton after the defocus animation. Adds a property in LocationBarViewController to keep track of that displayed state. Video: https://drive.google.com/open?id=1OrzvgnQez4d54EWVDf9RkBvLtwtkX7sC Bug: 954624 Change-Id: I0aaa24437fefaf572285a7470c975fa720d04c52 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1590605Reviewed-by:
Stepan Khapugin <stkhapugin@chromium.org> Commit-Queue: Chris Lu <thegreenfrog@chromium.org> Cr-Commit-Position: refs/heads/master@{#657278}
-
Ian Clelland authored
TBR=rdevlin.cronin@chromium.org Bug: 960363 Change-Id: Id9f274bc7c6ece5dd8d5af8f89d851f1b77dc7a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598418Reviewed-by:
Ian Clelland <iclelland@chromium.org> Commit-Queue: Ian Clelland <iclelland@chromium.org> Cr-Commit-Position: refs/heads/master@{#657277}
-
Xianzhu Wang authored
After https://chromium-review.googlesource.com/c/chromium/src/+/1594083 which caches cc node ids in blink property tree nodes, PropertyTreeManager no longer needs to be long lived. Change-Id: I489cb1b0cf67503b87d6498dbe95614286be3059 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1597752Reviewed-by:
Philip Rogers <pdr@chromium.org> Reviewed-by:
vmpstr <vmpstr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#657276}
-
Toni Barzic authored
BUG=959237 Change-Id: I94483503fdc8c662ca5b479ea15a42c1de68f1e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1595984 Commit-Queue: Toni Baržić <tbarzic@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Cr-Commit-Position: refs/heads/master@{#657275}
-
Kyle Spiers authored
Bug: 953008 Change-Id: I97b1ff90562c24e57fde43e0e1a35e36583c9b42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1595031 Commit-Queue: Kyle Spiers <ksspiers@google.com> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Cr-Commit-Position: refs/heads/master@{#657274}
-
Abhijeet Kandalkar authored
Bug: 891908 Change-Id: Iaf30794eead97c2a4575f3ad3ac59e0c206172a3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1595198 Commit-Queue: Abhijeet Kandalkar <abhijeet@igalia.com> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#657273}
-
Richard Townsend authored
Bug: 893460 Change-Id: I9d440e160aff9f5c4a26b1881d9db8bdf675bdf9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593363 Commit-Queue: Richard Townsend <richard.townsend@arm.com> Reviewed-by:
François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#657272}
-
Adithya Srinivasan authored
Uses portal postMessage APIs instead of broadcast channel in the following tests: - portal-activate-data.html - portals-adopt-predecessor.html - portals-nested.html Change-Id: I9a0bff6504c1cb6d8972e988d1614652212d303f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1591803Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Reviewed-by:
Lucas Gadani <lfg@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#657271}
-
Friedrich Horschig authored
Since we launch the keyboard accessory in M74, set the feature to enabled by default with M76 (to keep as kill switch). With M77, drop it then - this is already reflected in the flag-metdata.json. Automatic password generation is already enabled and also targeted to removed with M77. Bug: 853794 Change-Id: Iaf0dfe51753445eb868c27d5401636d4f0b1c42d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598821Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#657270}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/f6cbe44a8d88..b62fce1587b9 git log f6cbe44a8d88..b62fce1587b9 --date=short --no-merges --format='%ad %ae %s' 2019-05-07 jonahr@google.com Use VPython for dependency management of scripts/generate_stats.py Created with: gclient setdep -r src/third_party/angle@b62fce1587b9 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 TBR=jonahr@chromium.org Change-Id: Ic8c57f99e3b2aa95fa00b1fa5ab0dfc2716d707b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598282Reviewed-by:
-
Leonard Grey authored
Seems like in practice, when the system is under load, BEST_EFFORT priority tasks can have *significant* leeway. This means that just when the system is most likely to be using a lot of power, we are sampling at a lower frequency, which skews the Power.Mac.* stats low. This change moves recording up to the next highest priority level. Bug: 956241 Change-Id: If46b3bf91ef6cef6b52370195aef51bd8357567a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593400 Commit-Queue: Leonard Grey <lgrey@chromium.org> Reviewed-by:
Sidney San Martín <sdy@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#657268}
-
A Olsen authored
During the SAML flow, information about the password - such as when it may expire - can be provided by the SAML IdP. This is extracted using javascript during the SAML signin process. This adds a C++ class to matches the JS information - this class is tested but not yet used. In a follow up CL, this will be wired up so that the information extracted during login is saved to a users's prefs, and a BrowserTest will be added to test the whole flow. Bug: 930109 Change-Id: I0fde7e5808c0c3995ffed765457e788b9e701876 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594586Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Lutz Justen <ljusten@chromium.org> Commit-Queue: A Olsen <olsen@chromium.org> Cr-Commit-Position: refs/heads/master@{#657267}
-
Ben Pastene authored
Gonna see how the bots do at 100% load. Bug: 960100 Change-Id: Ieacafba9b1e8ce0f26001ffbe7862fa2d6b7129b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598422Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#657266}
-
Varun Khaneja authored
This came up in crrev.com/i/1254979 R=drubery Bug: 907280, 954255 Change-Id: Ie0e05df6a9229796d112acdc790f2d2bc02736e7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1597976 Commit-Queue: Daniel Rubery <drubery@chromium.org> Reviewed-by:
Daniel Rubery <drubery@chromium.org> Auto-Submit: Varun Khaneja <vakh@chromium.org> Cr-Commit-Position: refs/heads/master@{#657265}
-
Sam Maier authored
Since Webview is only shipped on L+, it shouldn't matter which dex file webview is found in. These keeps prevent optimizations happening across the main dex boundary, so it is preferred to remove. Bug: 887942 Change-Id: I1b40c4537b0d5246ec4b3de3f622637fe688900b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598316 Auto-Submit: Sam Maier <smaier@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#657264}
-
Fernando Serboncini authored
Following: https://www.w3.org/TR/input-events-2/ Bug: 892360 Change-Id: I735bb22dd366df3f75854fbda5fb80c9caae4050 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1585167Reviewed-by:
Changwan Ryu <changwan@chromium.org> Reviewed-by:
sebsg <sebsg@chromium.org> Commit-Queue: Fernando Serboncini <fserb@chromium.org> Cr-Commit-Position: refs/heads/master@{#657263}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 973f1c0b. With Chromium commits locally applied on WPT: e28269b6 "[LayoutNG/PE] Clear visual overflow for LayoutText before setting." aff3c8fe "Add tests for crbug.com/363609 and crbug.com/954591" c4dc3441 "[WPT] Reland Upload a test for sending mouse events with key pressed" 9492aa14 "Add touchTapInTarget function for wpt touch event tests" eb4c1182 "Add pointerDragInTarget function for wpt touch event tests" fec827fb "Added wpt tests for overscroll customization." a554ea37 "Reland "[EventTiming] Round instead of ceil the |duration|"" a06bcc90 "WebXR: Rename requestSession parameter variables to match use" Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: rego@igalia.com: external/wpt/css/css-contain NOAUTOREVERT=true TBR=lpz No-Export: true Change-Id: I564b7588517d97a7540130204a365f656df00319 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598209Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#657262}
-
Uzair Jaleel authored
Bug: None Change-Id: Iab6ada7d4b6a72bb050c2674e85721719b6a97c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1589086Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Yuichiro Hanada <yhanada@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Samuel Huang <huangs@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#657261}
-
Scott Violet authored
BUG=958121 TEST=covered by tests Change-Id: Ibfe6adc213d9d9c46a9fe3b250a4f6c296a8728a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598329Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/08905df5b4a3..80dc010a4be6 Created with: gclient setdep -r src-internal@80dc010a4be6 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=nhiroki@chromium.org,iclelland@chromium.org,dullweber@chromium.org,dalecurtis@chromium.org,jrummell@chromium.org Change-Id: I6a3dc4edbb2702b78efd03f8308ee0b2d0e61e40 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598158Reviewed-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@{#657259}
-
François Beaufort authored
This CL makes sure Picture-in-Picture icon sizes are the same as native control icons. TBR: mlamouri@chromium.org Bug: 960164 Change-Id: I77010d64a6500c2cb996f1baa80fed1913f26784 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598091Reviewed-by:
François Beaufort <beaufort.francois@gmail.com> Commit-Queue: François Beaufort <beaufort.francois@gmail.com> Cr-Commit-Position: refs/heads/master@{#657258}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/1133c0c5d7c5..8b7597e64f17 git log 1133c0c5d7c5..8b7597e64f17 --date=short --no-merges --format='%ad %ae %s' 2019-05-07 sugoi@google.com Change decompressed type of compressed RGB formats Created with: gclient setdep -r src/third_party/swiftshader@8b7597e64f17 The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-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 TBR=swiftshader-team+autoroll@chromium.org Change-Id: I52b23d888efa83f0e42e50ed0feca1ede783bd58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598157Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/94bcf1e49bec..396ab3c6c165 git log 94bcf1e49bec..396ab3c6c165 --date=short --no-merges --format='%ad %ae %s' 2019-05-07 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@396ab3c6c165 The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-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. TBR=agable@chromium.org Change-Id: I9434fbbc2385f07e5df4771d351f2934a2d3274e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598281Reviewed-by:
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/b0d8b76c4f62..0362dc268a6f git log b0d8b76c4f62..0362dc268a6f --date=short --no-merges --format='%ad %ae %s' 2019-05-07 raul@tambre.ee Fix use of 'is' operator for comparison Created with: gclient setdep -r src/third_party/skia@0362dc268a6f 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:958874 TBR=robertphillips@chromium.org Change-Id: I474eeb0c650d4b10b9eb6ff49954d1a717157ef4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598156Reviewed-by:
-
Kevin McNee authored
If a user visits an extension page in a tab and zooms it, it will save the zoom level on a per-origin basis. If a user then opens a browser action popup for the same extension, the popup will use the saved zoom level, even though the popup itself cannot be zoomed by the user. We now have the popup always use the default zoom level. Bug: 957115 Change-Id: I6824013951581c80396d0d0e9fdece6647242729 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592720Reviewed-by:
James MacLean <wjmaclean@chromium.org> Reviewed-by:
-
Dan Zhang authored
https://quiche.googlesource.com/quiche.git/+log/5de87051f54d..73e2413865fc $ git log 5de87051f..73e241386 --date=short --no-merges --format='%ad %ae %s' 2019-05-04 bnc Use QuicMakeUnique and QuicWrapUnique in quartc_bidi_test.cc. 2019-05-03 nharper gfe-relnote: Modify ParsedQuicVersion::HasHeaderProtection to support chromium merge. Not flag protected; currently unused 2019-05-03 ianswett gfe-relnote: Remove the SEND_CONNECTION_CLOSE_PACKET_WITH_NO_ACK enum value because it was only used by Quartc and PACKET_TOO_BIG. No functional GFE change, not flag protected. 2019-05-03 quiche-dev Replace random delay test case with a competing flow case in QuartcBidiTest. Created with: roll-dep src/net/third_party/quiche/src src/third_party/quic_trace/src R=rch@chromium.org Change-Id: I15ff9129115c728aeab7ae561da42bbb7f02afdb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594169Reviewed-by:
Bence Béky <bnc@chromium.org> Commit-Queue: Dan Zhang <danzh@chromium.org> Cr-Commit-Position: refs/heads/master@{#657253}
-
Etienne Pierre-doray authored
Reuse TaskSourceExecutionMode as a replacement of test::ExecutionMode. Change-Id: I9fedf58e2c53e97a52fd5fc612ebc39c65733f80 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593918 Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org> Reviewed-by:
-
Vladimir Levin authored
In display locking, we don't process subtrees of locked elements. In PositionAndLayoutOnceIfNeeded, we can use this information to skip recursing into layout code, which would abort child layout anyway. This takes the referenced bug case from 250ms per unlock to 110ms per unlock. R=mstensho@chromium.org, ikilpatrick@chromium.org Bug: 958804 Change-Id: Ied76301cb006894f3160f8243c58f9a753ed8fd2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1596837 Commit-Queue: vmpstr <vmpstr@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#657251}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/ef719ba0..3a702546 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,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ie144f5af7e1bafbf8b1e49427fbf314d824e2f35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598154Reviewed-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@{#657250}
-
kylechar authored
LayerTreeFrameSinkHolder doesn't handle losing LayerTreeFrameSink for all situations today. LayerTreeFrameSinkHolder assumes that the only time the LayerTreeFrameSink is lost is during destruction which is *mostly* true without OOP-D. With OOP-D the LayerTreeFrameSink holds a message pipe to the GPU process, so anytime the GPU process crashes the LayerTreeFrameSink will be lost and needs to be recreated. Have SurfaceTreeHost replace the LayerTreeFrameSinkHolder and LayerTreeFrameSink. Also change WindowPortLocal assumption that LayerTreeFrameSinks never get recreated for a window. Bug: 954151 Change-Id: I30a76d615b187086fcecfaf34c9c36e4d3424d4d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594478Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Commit-Queue: kylechar <kylechar@chromium.org> Cr-Commit-Position: refs/heads/master@{#657249}
-
Ben Kelly authored
Bug: 960372 Change-Id: Id8861f021ccfbde22c97be68f0085934fac0008c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1597669 Commit-Queue: Ben Kelly <wanderview@chromium.org> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#657248}
-
Fabrice de Gans-Riberi authored
The underlying issue was fixed in Fuchsia. Bug: 851747 Change-Id: I6db7eb475e8e348cd4e505c8182367d986c4b821 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1579772 Auto-Submit: Fabrice de Gans-Riberi <fdegans@chromium.org> Commit-Queue: danakj <danakj@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#657247}
-
Robbie Gibson authored
Bug: 960311 Change-Id: Ieb7c692893b0c68094326d0918f0404528b1420a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598699Reviewed-by:
-
Sam Maier authored
Bug: 668211 Change-Id: I77bdd5ac4caeb3b9ffb175aa3e2f29eb908803e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1557858 Commit-Queue: Sam Maier <smaier@chromium.org> Reviewed-by:
Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#657245}
-
