- 10 Oct, 2019 40 commits
-
-
Sertay Sener authored
This CL ports the Blob related IndexedDB Web Tests to WPT tests. Bug: 866164 Change-Id: Ie2b1b6ddc0c1ddbbb92478a2d5c81bd669c30e45 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807963Reviewed-by:
Joshua Bell <jsbell@chromium.org> Commit-Queue: Sertay Sener <sesener@microsoft.com> Cr-Commit-Position: refs/heads/master@{#704755}
-
Shakti Sahu authored
Change-Id: I6d6fa7aed8164314611e547ad4f96891db259933 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852175Reviewed-by:
Min Qin <qinmin@chromium.org> Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Cr-Commit-Position: refs/heads/master@{#704754}
-
Aaron Krajeski authored
We use both throughout the code base but heavily lean towards 2D. Make it 2D throughout. Change-Id: Ia9f3af6592ffade92c80c6dc04749a915dd7b1ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847893 Commit-Queue: Aaron Krajeski <aaronhk@chromium.org> Reviewed-by:
Juanmi Huertas <juanmihd@google.com> Reviewed-by:
Fernando Serboncini <fserb@chromium.org> Cr-Commit-Position: refs/heads/master@{#704753}
-
danakj authored
This value is part of ScreenInfo which will be once again locally scoped to each local root RenderFrame/RenderWidget. A RenderView without a local main frame won't know the DeviceScaleFactor. R=avi@chromium.org TBR=thestig, rdevlin.cronin Bug: 419087 Change-Id: Ife3bf17d3d80a8f19e817ed270c9dbfe954a8a65 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852949Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#704752}
-
Emircan Uysaler authored
This CL checks the current state before creating SkSurfaceCharacterization to properly set protected param. Bug: fuchsia:38284 Change-Id: Ie570cff987427f4667289a60b858c48ad307cd0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849149Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Peng Huang <penghuang@chromium.org> Commit-Queue: Emircan Uysaler <emircan@chromium.org> Cr-Commit-Position: refs/heads/master@{#704751}
-
Yun Liu authored
Also fixed a bug to create directory if host directory doesn't exist, otherwise pull command will pull file to "host directory" as a file. Bug: 985574 Change-Id: I158acdf39a90a3e014a88ffb42d520fc6abf8d67 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1839351 Commit-Queue: Yun Liu <yliuyliu@google.com> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#704750}
-
Ken Rockot authored
ScopedFeatureList is unsafe to use after browser threads have been started. This constraint will imminently be enforced by DCHECK to prevent further erroneous usage from landing. This CL corrects usage within miscellaneous Content browser tests as well as Chrome tests related to site islation. This is split from a larger CL where in some rare cases, correction was too complex to resolve before landing the DCHECK, so corresponding test(s) may be disabled instead of fixed. Bug: 846380 Change-Id: Idf6fcaabc37d09b49acb70ad3bd177985bed91cc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850736 Commit-Queue: Ken Rockot <rockot@google.com> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#704749}
-
Maks Orlovich authored
It should be treated as a 1st party context, not 3rd party one. Bug: 1012742 Change-Id: Ic8ceb0c7f2e60e15b4517550c50ab67c637b492c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1853044 Commit-Queue: Maksim Orlovich <morlovich@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#704748}
-
Chris Mumford authored
The Android device serial number, specified by the --device-serial command-line option, was unused by the android_webview wptrunner. This resulted in adb failing with the following error: adb wait-for-device error: more than one device/emulator Bug: None Change-Id: I85e236297d7e83950ac26a9283bdad4ef642d62c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1845486 Commit-Queue: Chris Mumford <cmumford@google.com> Reviewed-by:
Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#704747}
-
Lei Zhang authored
Use more C++11. Fix nits and lint errors along the way. Change-Id: I8d905472cbae1568be1f3ce5b8866b0cb564335a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1843313Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#704746}
-
danakj authored
These widgets are not in use, and VisualProperties will be sent when they want to be revived by a new local main frame. R=avi@chromium.org Bug: 419087 Change-Id: I4bcdd8a505c4e5c18a3d9e7b6c3b91f444774e2a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850759 Commit-Queue: Avi Drissman <avi@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#704745}
-
Shakti Sahu authored
This CL disables notifications suggested pages for all kinds of contents : offline pages and content indexing. Bug: 1011415 Change-Id: I4555f6f189cbc763379a52d7b163464ff6d8c8c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1853111Reviewed-by:
Rayan Kanso <rayankans@chromium.org> Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Cr-Commit-Position: refs/heads/master@{#704744}
-
John Lee authored
This CL also adds differentiation between a loading and a waiting state, each displaying a spinner in their own colors and directions, similar to how the native tab strip works. This CL also updates types to allow for the new data. Bug: 1004985 Change-Id: I1b9361b3796e369d683b8f63c6a2026364d439f6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1848573Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: John Lee <johntlee@chromium.org> Cr-Commit-Position: refs/heads/master@{#704743}
-
Dominic Mazzoni authored
Bug: none Change-Id: Ifee79eca593238e0f85cd2d5d4b07ea98c9e256b Tbr: aleventhal@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852419Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#704742}
-
Erik Chen authored
This CL is a refactor with no behavior change. In the future, undead RenderWidgets won't exist. By returning nullptr, we prevent accidentally uses from cropping up. Bug: 419087 Change-Id: Ie71c7c818921f1d0a464003e69d6cf267f8fe3fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850575Reviewed-by:
danakj <danakj@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Auto-Submit: Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#704741}
-
Steven Bennetts authored
This CL uses a unuion for type specific properties in ConfigProperties. Since the config properties are typically generated on the fly and passed from UI -> the CrosNetworkConfig implementation, the union type is used to specify the network type (i.e. no explicit type is required). Bug: 853953 Change-Id: I08ae054f8d5554727c4d9422ca325b284c3927e8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850724 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#704740}
-
David Benjamin authored
It was only used by tests at this point. Bug: 340460 Change-Id: I1a52c17d439bfb5a198a908aee0fe7600bb7845d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846468Reviewed-by:
Matt Mueller <mattm@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Auto-Submit: David Benjamin <davidben@chromium.org> Cr-Commit-Position: refs/heads/master@{#704739}
-
Aaron Krajeski authored
This mostly means that we need to be more explicit about when we finalize frame. This is covered in prior CLs: https://chromium-review.googlesource.com/c/chromium/src/+/1848934 https://chromium-review.googlesource.com/c/chromium/src/+/1814457 Next step is to create a CanvasDeferral class that mixes in to the generic CanvasRenderingContext class and manages deferral for all types of canvases. Also, finding a way to get write pixels within deferral by using putImageData would be nice. Bug: 1002523 Change-Id: Ia5aa50e3082177df1f422d14fc7899f8d173a874 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762467 Commit-Queue: Aaron Krajeski <aaronhk@chromium.org> Commit-Queue: Fernando Serboncini <fserb@chromium.org> Reviewed-by:
Fernando Serboncini <fserb@chromium.org> Reviewed-by:
Kai Ninomiya <kainino@chromium.org> Cr-Commit-Position: refs/heads/master@{#704738}
-
Yuwei Huang authored
Chromoting is now fully on FTL signaling, so this CL removes LCS support in SignalingAddress and cleans up the code. We can't remove XMPP yet since IT2ME still uses it to communicate with the website. Bug: 983282 Change-Id: I6d1a6d110a7f04ad28e963e7ab7c31c8bcc707be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1851226Reviewed-by:
Jamie Walch <jamiewalch@chromium.org> Commit-Queue: Yuwei Huang <yuweih@chromium.org> Cr-Commit-Position: refs/heads/master@{#704737}
-
Ian Wells authored
Bug: 1001506 Change-Id: Ice31c3588d42038ebe7728739189e69710557dc9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1815084Reviewed-by:
Peter Williamson <petewil@chromium.org> Commit-Queue: Ian Wells <iwells@chromium.org> Cr-Commit-Position: refs/heads/master@{#704736}
-
Yun Liu authored
Bug: 985581 Change-Id: I53edb1be6c0761e0f52868119576c7dff6b2a3c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850737Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Yun Liu <yliuyliu@google.com> Cr-Commit-Position: refs/heads/master@{#704735}
-
Adrienne Walker authored
This code is currently in content/browser/loader but is useful to other features. Bug: 1012869 Change-Id: Ie129d975d0d381f2ec8e98670efc8ed52c39d40a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1848055Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: enne <enne@chromium.org> Auto-Submit: enne <enne@chromium.org> Cr-Commit-Position: refs/heads/master@{#704734}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/01cfdcb21734..b21973d2c0c9 Created with: gclient setdep -r src-internal@b21973d2c0c9 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@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/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=jbudorick@google.com Bug: chromium:None Change-Id: I656518a05a1558d434755887f419e751eb56daf6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852195Reviewed-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@{#704733}
-
Jack Lynch authored
This adds an axe tests for the Sources panel global listeners pane. Bug: 963183 Change-Id: Iba5ef2bdc927439604f01d91c1e8b9966c43fdf8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1666329 Commit-Queue: Jack Lynch <jalyn@microsoft.com> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#704732}
-
Antonio Gomes authored
It is now possible after mock_peer_connection_dependency_factory.h|cc move [1]. [1] https://crrev.com/c/1816789 BUG=787254 R=guidou@chromium.org, haraken@chromium.org Change-Id: I76ea8c3477467f5b4956af64cc4b76cc28c20775 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816792 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#704731}
-
Jack Lynch authored
This CL adds context menu items to each listener to reveal its source code, remove it, or toggle whether it is passive or blocking (when applicable). This improves the accessibility of the event listeners view in the Elements panel as well as the global listeners view in the Sources panel, as previously the passive toggle and delete buttons were available only on hover. Previously reviewed at https://chromium-review.googlesource.com/c/chromium/src/+/1665298, split into this CL as it impacts multiple parts of the UI. Screenshot: https://gyazo.com/dc1b54b5e0ae1f9b67f89b0400a168a8 Bug: 963183 Change-Id: I497e96ee3eb5d58358f904dd7e9ac92b15840baf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1716462 Commit-Queue: Jack Lynch <jalyn@microsoft.com> Reviewed-by:
Lorne Mitchell <lomitch@microsoft.com> Cr-Commit-Position: refs/heads/master@{#704730}
-
Xida Chen authored
Bug: 900581 Change-Id: Iafa4c8d546e1de8eb540deb17d82b7c055854cf8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1853132 Commit-Queue: Xida Chen <xidachen@chromium.org> Reviewed-by:
Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#704729}
-
Marc Treib authored
article_entry.h/cc contained a bunch of code for converting from Sync types (SyncData/SyncChange/EntitySpecifics) to dom_distiller types. After recent changes, all that code isn't used anymore and can go. This also lets us remove special workarounds for the deprecated ARTICLES data type that we had in sync code. Bug: 1007942 Change-Id: Ic9e4d243dd7a4276973398149fd14da61bf5fe05 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1835367 Commit-Queue: Marc Treib <treib@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Reviewed-by:
Jan Krcal <jkrcal@chromium.org> Auto-Submit: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#704728}
-
Stephen Chenney authored
The name of Paint Holding has replaced the original name of AvoidFlashBetweenNavigation. Renaming it allows a forthcoming finch study to use the current name. Also update the description etc. Bug: 1012732 Change-Id: Ia5994207e5d8e8b1f6a136ae709c1c8c773d418d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849145Reviewed-by:
Peter Boström <pbos@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#704727}
-
Collin Baker authored
This implements triggering the native tab context menu from the WebUI tab strip. To do so, it adds an interface that TabStripUI requires and WebUITabStripContainer implements. Design doc: https://docs.google.com/document/d/1KIasUON6Jd2gRiQ6gDXiOWVqBL4vE5vu4I6Vb61BAp8 Bug: 997887 Change-Id: Id9dc48709250545dcc20f3fc7acdf17a11ed6446 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1838678Reviewed-by:
Peter Boström <pbos@chromium.org> Reviewed-by:
John Lee <johntlee@chromium.org> Commit-Queue: Collin Baker <collinbaker@chromium.org> Cr-Commit-Position: refs/heads/master@{#704726}
-
Ken Rockot authored
ScopedFeatureList is unsafe to use after browser threads have been started. This constraint will imminently be enforced by DCHECK to prevent further erroneous usage from landing. This CL corrects usage within browser tests for predictors and various features related to data reduction. This is split from a larger CL where in some rare cases, correction was too complex to resolve before landing the DCHECK, so corresponding test(s) may be disabled instead of fixed. Bug: 846380 Change-Id: I95e319ed19778bfd563bf3a0f65b88ae7420452c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849980 Commit-Queue: Ken Rockot <rockot@google.com> Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Reviewed-by:
Egor Pasko <pasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#704725}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/cd0eedb2483b..f17976d01993 git log cd0eedb2483b..f17976d01993 --date=short --no-merges --format='%ad %ae %s' 2019-10-10 kcwu@google.com Use single thread vp9 decoder for fuzzing 2019-10-10 kwiberg@webrtc.org Remove the unused `receive_timestamp` arg to NetEq::InsertPacket 2019-10-10 kcwu@google.com Cap vp9 fuzzer frame size to prevent OOM Created with: gclient setdep -r src/third_party/webrtc@f17976d01993 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@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/+/master/autoroll/README.md TBR=webrtc-chromium-sheriffs-robots@google.com Bug: chromium:1009073,chromium:1009073 Change-Id: I4db6bc3540865fd49b13a46845f66f6288ae7387 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1851688Reviewed-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@{#704724}
-
Greg Daniel authored
We are trying to remove ComputeTextureSize in Skia so this shifts the current uses of it to the new API. Change-Id: Ia2fa7598c5a36bf4c674efc5da8f134fcda5de61 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1853106Reviewed-by:
Robert Phillips <robertphillips@google.com> Reviewed-by:
Khushal <khushalsagar@chromium.org> Commit-Queue: Greg Daniel <egdaniel@chromium.org> Cr-Commit-Position: refs/heads/master@{#704723}
-
Aga Wronska authored
This reverts commit 479f1be6. Reason for revert: The issue that caused flakiness is fixed (https://crbug.com/1005723). Original change's description: > Disable flaky/failing ScreenTimeControllerTests. > > TBR: michaelpg > Bug: 1007954, 1008289, 1008288, 1008286, 1008284, 1008282, 1008280, 1008273, 1008272, 1008269 > Change-Id: Ic611a4f597d2ddf9aaac66ff77fda64e3e4da8a4 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1827277 > Reviewed-by: Finnur Thorarinsson <finnur@chromium.org> > Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> > Cr-Commit-Position: refs/heads/master@{#700186} TBR=finnur@chromium.org,michaelpg@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1007954, 1008289, 1008288, 1008286, 1008284, 1008282, 1008280, 1008273, 1008272, 1008269 Change-Id: I2f01ef970dc13c08820ea7baff3e6fd950da91af Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852417Reviewed-by:
Aga Wronska <agawronska@chromium.org> Commit-Queue: Aga Wronska <agawronska@chromium.org> Cr-Commit-Position: refs/heads/master@{#704722}
-
Miriam Zimmerman authored
BUG=None TEST=tast -verbose run 100.127.92.23 arc.AppCrash Change-Id: I04b8f0f91230802861c14cb67d2a16c901550db8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1848552Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Ian Barkley-Yeung <iby@chromium.org> Commit-Queue: Miriam Zimmerman <mutexlox@chromium.org> Cr-Commit-Position: refs/heads/master@{#704721}
-
Nicolas Arciniega authored
While testing the related bug on a debug version of the auditor, I found that a DCHECK is missing a case. It is possible to have an AuditorResult with an empty message of type ERROR_MUTABLE_TAG. An AnnotationInstance will specifically create an AuditorResult of type ERROR_MUTABLE_TAG without a message. The detailed text provided by the AuditorResult itself should give sufficient context. Bug: 1005537 Change-Id: I39d66e758ed5b847430a7385e1d15bd33470d0ec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849990Reviewed-by:
Ramin Halavati <rhalavati@chromium.org> Reviewed-by:
Nicolas Ouellet-Payeur <nicolaso@chromium.org> Commit-Queue: Nicolas Arciniega <niarci@microsoft.com> Cr-Commit-Position: refs/heads/master@{#704720}
-
Xiaohui Chen authored
Bug: b:141751193 Test: locally build and unittests Change-Id: I1eb5ccfecb83723e54f67997283885c7f85c0fc1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1845795Reviewed-by:
Tao Wu <wutao@chromium.org> Commit-Queue: Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#704719}
-
Xida Chen authored
The test is actually using width interpolation to test that the 'vh' value is computed correctly. Bug: 900581 Change-Id: Icfffb6bd06e1d89a25110727fab8b3107f2c5613 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852580 Commit-Queue: Xida Chen <xidachen@chromium.org> Reviewed-by:
Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#704718}
-
rbpotter authored
If more than one page was visible in the viewport, getPageAtY_(y) fell through to returning 0 when y was the bottom of the document, because in the last iteration of the loop, where max = min = page = N-1 (for an N page document), the bottom of the page was exactly equal to y, rather than strictly > y. This resulted in the most visible page being set to N-1 rather than N when the document was scrolled to the bottom in this case. Bug: 1000121 Change-Id: I541204141d1fb7891f1df1bc95d00be28d39bbfd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1851086 Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#704717}
-
Yuta Kitamura authored
This CL adds more tests for Agent generation. They test the following cases: * HTTP frame loading a same-origin HTTPS child frame * iframe loaded as data: URL * iframe loaded with srcdoc attribute Bug: 961186 Change-Id: I21eae63c9b6e3d5a2be911cb4cce1eb975e48cb8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852325 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Auto-Submit: Yuta Kitamura <yutak@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#704716}
-