- 05 Jun, 2019 40 commits
-
-
Etienne Bergeron authored
This CL is removing useless forward declaration and unused include file. There is no behavior change. R=robliao@chromium.org, asvitkine@chromium.org CC=benck@google.com Bug: 944227 Change-Id: Ifcda56f000119dd3682eff2e7665f59a49c84ed1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1643381Reviewed-by:
Robert Liao <robliao@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#666282}
-
Yeol authored
When calling AddView from AddLabelledRowToGridLayout, released unique_ptr to pass to argument. Bug: 948287 Change-Id: I71373a27ee7b8c7ae3bf8c1fdb8eed86b806eac2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644889 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#666281}
-
Liquan(Max) Gu authored
ImagePaintTimingDetector has an overflow issue in multiplying two integers (height*width). In this patch we will use rect.Size().Area() to compute the size. As the function has handled the overflown issue properly, we can use it to fix the bug. The fix has been verified with a local run of the Cluster Fuzz. Bug: 969295 Change-Id: I67fda6339bbcbc8346b02397031d9a73c3c61f4b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1643091Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Liquan (Max) Gu <maxlg@chromium.org> Cr-Commit-Position: refs/heads/master@{#666280}
-
Jose Dapena Paz authored
When RenderProcessUserData is destroyed from the destructor of RenderProcessHostImpl, it is done in the destructor of RenderProcessHost. At this point RemoveObserver override is already freed, so RenderProcessHost is pure virtual. This crash happens at least building with GCC: at /usr/include/c++/8/ext/new_allocator.h:140 (this=0x7fffffffcb50, __in_chrg=<optimized out>) at /usr/include/c++/8/bits/stl_tree.h:964 We need to destroy RenderProcessUserData before that happens. To do that we can just override RenderProcessHostDestroyed. Bug: 910288 Change-Id: I38107b178829b0cb7494f5333b765e5b087d82cd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645366 Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Reviewed-by:
Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#666279}
-
Xida Chen authored
TBR=gayane@chromium.org NOTRY=true Bug: 971252 Change-Id: I1b93944ca76b422c7b8c4972e161326717a612c3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645475Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#666278}
-
Xida Chen authored
TBR=gayane@chromium.org NOTRY=true Bug: 971241 Change-Id: Ib0e2541601ee7eef544ff5d5a0a9adc46f54762f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645793Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#666277}
-
Robert Ma authored
Also add Fuschia to the port list (should have been done in https://crrev.com/c/1640796). Bug: 958017, 567947 Change-Id: I872e95652477f472fc1f54421b08425900dbc283 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1643840Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Robert Ma <robertma@chromium.org> Auto-Submit: Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#666276}
-
Xida Chen authored
TBR=gayane@chromium.org NOTRY=true Bug: 971247 Change-Id: If2170d68287e5021e25986cc4dd875ca864c53fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645812Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#666275}
-
Vladislav Kuzkokov authored
Everywhere where needed the copy in PrintedDocument is available. Bug: 964948 Change-Id: I34a7e1eaddb884f5e0bff9710767ed03eda96ce0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1631598Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Vladislav Kuzkokov <vkuzkokov@chromium.org> Cr-Commit-Position: refs/heads/master@{#666274}
-
Xida Chen authored
TBR=gayane@chromium.org NOTRY=true Bug: 971241 Change-Id: I373aa360faa185ff15ff940fea5dd622b376d86d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645160Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#666273}
-
kylechar authored
This CL adds temporary detailed logging for all the failures cases in the CopyOutputResult deserialization code. https://crbug.com/967856 is the result of a deserialization error and I haven't been able to reproduce it on a dev machine. Add logging so it gets picked up in canary or dev release to give more insight. Bug: 967856 Change-Id: I3b390c3ba9800f5c7f19d75a3cc0b9457c39d2c5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1643495 Commit-Queue: kylechar <kylechar@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Cr-Commit-Position: refs/heads/master@{#666272}
-
Yuwei Huang authored
Bug: 965746 Change-Id: I0d9b31fea543ee0b9e661f2025bc9133863f9dce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1643786Reviewed-by:
Joe Downing <joedow@chromium.org> Commit-Queue: Joe Downing <joedow@chromium.org> Cr-Commit-Position: refs/heads/master@{#666271}
-
Mark Cogan authored
After the switch to use WebStateListObserver instead of TabModelObserver, BrowserViewWrangler will get a notification that the active webState is changing when the app quits (the active webState is changing to nullptr). This in turn causes a call to -updateDeviceSharingManager, which explodes when there's no active WebState. The fix is to not call -updateDeviceSharingManager with a nullptr new active web state. Bug: 968475 Change-Id: I78b25c627d311d2e2a3f152e8cd8f663161962bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645230 Auto-Submit: Mark Cogan <marq@chromium.org> Reviewed-by:
David Jean <djean@chromium.org> Commit-Queue: Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#666270}
-
Ehsan Karamad authored
TestNavigationThrottle::Status seems unused. Removing with this CL. Bug: None Change-Id: I69ba98e5b9eaa976861ada7dc772ca7f794d4115 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642019Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Commit-Queue: Ehsan Karamad <ekaramad@chromium.org> Cr-Commit-Position: refs/heads/master@{#666269}
-
Xiyuan Xia authored
Bug: 958229 Change-Id: I444c8a563c328efcbfebe1426ddec692d41a171a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1643757Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#666268}
-
Xida Chen authored
TBR=gayane@chromium.org NOTRY=true Bug: 922951 Change-Id: Icdf09cc7c60f19ef7aaf8fa32503445f5c3c617c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645202Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#666267}
-
Pavel Shmakov authored
Based on flakiness dashboard: https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=network_service_out_of_process_chrome_public_test_apk&tests=org.chromium.chrome.browser.profiling_host.ProfilingProcessHostAndroidTest TBR=erikchen@chromium.org,tedchoc@chromium.org Bug: 970205 Change-Id: Ia2164945ceecab937403e4c69e0d407b454e61d3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645228 Commit-Queue: Pavel Shmakov <pshmakov@chromium.org> Reviewed-by:
Pavel Shmakov <pshmakov@chromium.org> Cr-Commit-Position: refs/heads/master@{#666266}
-
Mario Sanchez Prada authored
This is no longer used now that we have removed PushMessagingClient from //content/renderer, so we can safely remove it. Bug: 939943 Change-Id: If7074fb428b25f16482d91940d84c48fd71bc3bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1631403 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#666265}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/f6dc12884b53..bc5e8283ebb0 Created with: gclient setdep -r src-internal@bc5e8283ebb0 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=xidachen@google.com,gayane@google.com,calamity@google.com,kinuko@google.com Change-Id: I78823431426d4de688a5c4c1adde19a018626765 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645176Reviewed-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@{#666264}
-
Elly Fong-Jones authored
On the slower macOS bots, this test sometimes takes ~150ms longer than kLargeResponseTimeoutMs for some of the timeout tests, so bump up the threshold for success. Bug: 905265 Change-Id: I0a968a0075d5ea7448f327a07ec36389de25fe7a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642737 Auto-Submit: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Joe Downing <joedow@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#666263}
-
Tanmoy Mollik authored
This cl is part of the effort to use CoreAccountId instead of std::string to denote account_id. Bug: 959157 Change-Id: Ib0b41739551932824fc383dc869a3f9f5ab7dcf7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642637 Commit-Queue: Tanmoy Mollik <triploblastic@google.com> Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#666262}
-
Jit Yao Yap authored
Change-Id: I41a7558c411870a106a7f5766f68c91a136ad579 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645354Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Commit-Queue: Jit Yao Yap <jityao@google.com> Cr-Commit-Position: refs/heads/master@{#666261}
-
Pavel Shmakov authored
Disable flaky ServicificationBackgroundServiceTest#testHistogramsPersistedWithServiceManagerOnlyStart TBR=hanxi@chromium.org,tedchoc@chromium.org Bug: 970190 Change-Id: Iddad9df7ad507ede7f164e6702a535fd0ae2bcbf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645227Reviewed-by:
Pavel Shmakov <pshmakov@chromium.org> Commit-Queue: Pavel Shmakov <pshmakov@chromium.org> Cr-Commit-Position: refs/heads/master@{#666260}
-
arthursonzogni authored
Previously, navigation to a MHTML subframe was handled by the renderer process. This CL makes it use the normal path (e.g. driven by the browser process). After this and two other edge cases (about:blank, about:srcdoc) will be removed, then every navigation to a new document will be driven by the browser process. * Stricter enforcement could be added for https://crbug.com/943610. * It unblocks RenderDocument https://crbug.com/936696. Note: Some error pages will still be able to be driven by the renderer process. Others related CLs: * https://chromium-review.googlesource.com/c/chromium/src/+/1578544 * https://chromium-review.googlesource.com/c/chromium/src/+/1547522 Bug: 936696 Change-Id: If030dcc43dcc038a588285a639ea08b755310094 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547521 Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#666259}
-
Xida Chen authored
TBR=gayane@chromium.org NOTRY=true Bug: 856601 Change-Id: I03801abfea02047bbe200958d7a8bc7a560c7710 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644994Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#666258}
-
Denis Kuznetsov authored
Bug=b:133461735 Change-Id: Ib75f9977dd9b5aa044fa9844dfcf35f27e0fd548 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1628718Reviewed-by:
Sergey Poromov <poromov@chromium.org> Commit-Queue: Denis Kuznetsov <antrim@chromium.org> Cr-Commit-Position: refs/heads/master@{#666257}
-
Xida Chen authored
This is MSAN only. TBR=gayane@chromium.org NOTRY=true Bug: 856601 Change-Id: I623d7f5833d92aa5d9227c623069e7c53810cda3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644993Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#666256}
-
Mario Sanchez Prada authored
Now that we have blink::PushMessagingClient, which is also frame-bound, we no longer need this helper class since we'll communicate over mojo right from Blink, without needing to interact with //content at all. Thus, we can remove this class, as it's no longer used anywhere. Bug: 939943 Change-Id: I41528d3c7048040b2cf854ee7991f0a21e8b6e1b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1631595 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#666255}
-
Mario Sanchez Prada authored
We now use DocumentInterfaceBroker to communicate with the RenderFrame in the browser process, so we don't need to pass a routing ID for the render frame anymore. This CL removes the parameter and all its references from callsites, which were already passing -1 (invalid ID) anyway. Bug: 939943 Change-Id: I3cecd93de28feff24722a7870673225512dfd8ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1638545 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#666254}
-
Antonio Gomes authored
This CL removes the redundant enum MediaStreamRequestResult, defined in [1], in favor of the mojom existing variant in [2]. [1] third_party/blink/public/common/mediastream/media_stream_request.h [2] third_party/blink/public/mojom/mediastream/media_stream.mojom The associated typemap [3] and traits declarations [4][5] are also removed. [3] third_party/blink/public/common/mediastream/media_stream.typemap [4] third_party/blink/common/mediastream/media_stream_mojom_traits.cc [5] hird_party/blink/public/common/mediastream/media_stream_mojom_traits.h TBR=haraken@chromium.org BUG=923394,919392 This is a reland of https://crrev.com/c/1639237. Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1639237 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#665431} Change-Id: I7af5c650648c53f1ef9763fb5a12b32fa0c0544c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642734 Cr-Commit-Position: refs/heads/master@{#666253}
-
Pavel Shmakov authored
TBR=szermatt@chromium.org Bug: 970219 Change-Id: I00ac5f6d684a6be77eb692f901fafd323227aebd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645226Reviewed-by:
Pavel Shmakov <pshmakov@chromium.org> Reviewed-by:
Stephane Zermatten <szermatt@chromium.org> Commit-Queue: Pavel Shmakov <pshmakov@chromium.org> Cr-Commit-Position: refs/heads/master@{#666252}
-
Xida Chen authored
The specific one is virtual/blink-cors/http/tests/security/contentSecurityPolicy/require-sri-for/require-sri-for-script-preload-allowed.php The original test is already marked flaky. TBR=gayane@chromium.org NOTRY=true Bug: 678499 Change-Id: I7d220e6aa12bd4079d392021e387e7cac1422fb1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644935Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#666251}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/082a93c5faa3..a6e239978796 git log 082a93c5faa3..a6e239978796 --date=short --no-merges --format='%ad %ae %s' 2019-06-05 perezju@chromium.org [Telemetry] Add story_run.AsDict method Created with: gclient setdep -r src/third_party/catapult@a6e239978796 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:966835 TBR=skyostil@google.com Change-Id: I3b917cc089811a3c77ce590a645c14c2129f1fb3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645125Reviewed-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@{#666250}
-
Xida Chen authored
The specific test is virtual/blink-cors/http/tests/security/offscreencanvas-placeholder-read-blocked-no-crossorigin.html is comes from a virtual test suite. The original version of this is already skipped and has a bug associated with it. TBR=gayane@chromium.org NOTRY=true Bug: 922951 Change-Id: I091ddd40fa96f73ac4d832920cdcf2b002d978bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645374Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#666249}
-
Roman Sorokin authored
Both oobe_dialog_host.css and screen_gaia_signin.css had :host rules. Removed one from screen_gaia_signin.css Also removed v2 class - introduced saml-interstitial class. Alternative would be to keep saml-interstitial with the same size and not shirk it Bug: 971172 Change-Id: Ib9e93c4bc347910286812114df29579467c0b2cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645360Reviewed-by:
Denis Kuznetsov <antrim@chromium.org> Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#666248}
-
Stephen Chenney authored
Drawing a 100px x 100px div with border radius 50px is not the same as with border radius calc(100% - 1px) = 99px. R=foolip@chromium.org BUG=968430 Change-Id: I953c26b72d55d930cdbbcfa969f38b892e3151f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1639566 Auto-Submit: Stephen Chenney <schenney@chromium.org> Commit-Queue: Philip Jägenstedt <foolip@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Cr-Commit-Position: refs/heads/master@{#666247}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/80255d87a852..f6dc12884b53 Created with: gclient setdep -r src-internal@f6dc12884b53 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=xidachen@google.com,gayane@google.com,calamity@google.com,kinuko@google.com Change-Id: I88f5e59de71b08e8f1394e82eca55a053e42884b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645175Reviewed-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@{#666246}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/82e7e7a3..13da560b 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: Ic7cb18c61ebee265facd13e6bd32db77a56e3516 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645372Reviewed-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@{#666245}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 40bc2a0e. With Chromium commits locally applied on WPT: e90c6ef0 "NG/DL: Implement size containment (and display lock) for NG fieldset algo" abf2ac5e "webrtc wpt: avoid race conditions" 1bbbd106 "Add feature policy for client hints, used for third-party subresources." ab94a58b "Add the remaining Client Hints to Feature Policy." a5b31124 "DetachLayoutTree clear hover/active state on dom removal." 04c3df15 "Canvas::TestMetrics: Measure the TextMetrics info after loading the font" 2d4812d4 "[css-contain] Fix <select> dimensions with size containment" 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: igrigorik@chromium.org, yoavweiss@chromium.org: external/wpt/preload NOAUTOREVERT=true TBR=foolip No-Export: true Change-Id: Ie398bf1d5ba104960265ed368d0bf31dfb9e032d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645492Reviewed-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@{#666244}
-
Mythri A authored
We would like to run a finch trial on this feature. Bug: v8:8394 Change-Id: Iecac081955574c4add95627e10c3dbd5eda547c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642640Reviewed-by:
Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Mythri Alle <mythria@chromium.org> Cr-Commit-Position: refs/heads/master@{#666243}
-