- 16 Sep, 2019 40 commits
-
-
Yang Guo authored
This reverts commit fe3aa7f1. Reason for revert: Original culprit for the failing tests has been reverted: https://chromium-review.googlesource.com/c/chromium/src/+/1806414 Original change's description: > sheriff: Disable devtools accessibility tests > > Tests are failing on linux: > > https://ci.chromium.org/p/chromium/builders/ci/Linux%20Tests%20%28dbg%29%281%29/83157 > > TBR=pfeldman@chromium.org > > Bug: 1004150 > Change-Id: I4bf6d6a10d60c7659a23b6aba1884a5995957531 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1804709 > Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org> > Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> > Cr-Commit-Position: refs/heads/master@{#696729} TBR=pfeldman@chromium.org,ortuno@chromium.org Change-Id: I24340b26d513d1013077eca1189003e55824f7e5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1004150 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1804476Reviewed-by:
Yang Guo <yangguo@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#696748}
-
Mike Jackson authored
This reverts commit b1540bd6. Reason for revert: Breaks http/tests/devtools/a11y-axe-core/basic-a11y-test.js and 5 other(s), see issue 1004150 (on Linux) Original change's description: > Reland "Update axe-core to v3.3.2" > > This is a reland of 0f435b65 > > Original change's description: > > Update axe-core to v3.3.2 > > > > This change updates the version of axe-core to the latest released > > version. As part of picking up the latest version, the following > > additional changes were made: > > > > 1) axe-core removed the 'href-no-hash' rule, so that been removed > > from the test list. > > > > 2) Several 'link-in-text-block' tests started failing with > > axe-core v3.0.2 (and later). These tests do pass in v3.0.1, but > > are disabled as part of this change. > > - CrManagementA11yTest.SimpleTest_link_in_text_block > > - CrManagementA11yTestWithExtension. > > ExtensionSection_link_in_text_block > > - CrostiniAccessibilityTest.CROSTINI_link_in_text_block > > - CrExtensionsA11yTest.NoExtensions_link_in_text_block > > - MultideviceA11yTest.MULTIDEVICE_link_in_text_block > > - MultideviceFeaturesA11yTest. > > MULTIDEVICE_FEATURES_ACCESSIBILITY_link_in_text_block > > > > 3) One 'heading-order' test started failing in v3.0.0 (and later). > > This tests does pass in V2.4.2, but is disabled as part of this > > change. > > - CrExtensionsA11yTestWithMultipleExensions. > > WithExtensions_heading_order > > > > I spent some time investigating the test failures, and it looks > > like axe-core is highlighting legitimate product issues. > > > > Change-Id: I99c1f85087aea873e29b413dca967864fc93c786 > > Bug: 984817 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790125 > > Reviewed-by: Hector Carmona <hcarmona@chromium.org> > > Reviewed-by: Alice Boxhall <aboxhall@chromium.org> > > Commit-Queue: Mike Jackson <mjackson@microsoft.com> > > Cr-Commit-Position: refs/heads/master@{#695885} > > The original change caused failures in the Deterministic Linux builds. > This was due to devtools tests taking a dependency on axe.js, but not > declaring that dependency in the build.gn file. This has been fixed > in this change. > > Bug: 984817 > Change-Id: I0e55bae8014654ba92afb69bb6ff8d553c018126 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800003 > Reviewed-by: Alice Boxhall <aboxhall@chromium.org> > Reviewed-by: Pavel Feldman <pfeldman@chromium.org> > Reviewed-by: Hector Carmona <hcarmona@chromium.org> > Commit-Queue: Mike Jackson <mjackson@microsoft.com> > Cr-Commit-Position: refs/heads/master@{#696545} TBR=aboxhall@chromium.org,pfeldman@chromium.org,hcarmona@chromium.org,luoe@chromium.org,mjackson@microsoft.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 984817, 1004150 Change-Id: I5dc2d939ebcf338d8bd6a5ca95d41a25afc5d57f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1806414 Commit-Queue: Mike Jackson <mjackson@microsoft.com> Reviewed-by:
Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/master@{#696747}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/d32dae1bb37c..8f08fed925c5 git log d32dae1bb37c..8f08fed925c5 --date=short --no-merges --format='%ad %ae %s' 2019-09-16 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/spirv-tools/src ad7f2c5c4c7f..1e146e8a3485 (2 commits) Created with: gclient setdep -r src/third_party/angle@8f08fed925c5 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC geofflang@chromium.org 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.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=geofflang@chromium.org Bug: chromium:None Change-Id: I2ea016f72897bdd394f6b04b1bd4102cc13f178a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1805484Reviewed-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@{#696746}
-
Mugdha Lakhani authored
for one-off tasks, both for one-shot and periodic Background Sync. Update unit tests to verify that we pass the correct parameters to BackgroundTaskScheduler. Bug: 1000021, 996166 Change-Id: If47e77d5747c1e7d4a2aef1fc1c28c5e1064e58d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1782619Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Reviewed-by:
Rayan Kanso <rayankans@chromium.org> Commit-Queue: Mugdha Lakhani <nator@chromium.org> Cr-Commit-Position: refs/heads/master@{#696745}
-
Fredrik Söderquist authored
This eliminates implicit conversions to SMILTime. Adds FromSecondsD and FromMicroseconds to SMILTime. This is otherwise mostly s/0/SMILTime()/ and adding From{SecondsD,Microseconds} where needed. Bug: 1003338 Change-Id: I164c644a3b6161e9546af0049de9563d2c7beae2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803313 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#696744}
-
Pâris MEULEMAN authored
Avoid exposing getIdentityMutator in IdentityManager.java by passing the IdentityMutator reference directly in SigninManagerAndroid's and SigninManager.java's constructors. Bug: 934688 Change-Id: I02667bfe89fe2e3925363f2bf0075ef53b0bd8ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800405 Commit-Queue: Pâris Meuleman <pmeuleman@chromium.org> Auto-Submit: Pâris Meuleman <pmeuleman@chromium.org> Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#696743}
-
Fredrik Söderquist authored
'repeatCount' was previously shoehorned into a SMILTime since it was a reasonable match semantically. This makes less sense when storing SMILTimes as integers though, so rework it to use a small helper class that stores a double and exposes the various states that repeat count can have. Add a SMILTime::Repeat(SMILRepeatCount) helper to compute the repeated duration. SVGSMILElement::ParseCondition is reworked to use negation to add the sign to an offset value. With this operator* on SMILTime can be eliminated. operator-() (unary -) is added to be able to implement the negation. Bug: 1003338 Change-Id: I2827ff994b10194be8132e9db02dd7593b0a9d68 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803307 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:
-
Alex Clarke authored
We want to prioritize compositing and loading tasks until FCP which causes this test to flake. The problem is the test isn't waiting for the load event to fire whch means it's sometimes running the asset before the style sheet has loaded. This patch makes it wait. Bug: 971191 Change-Id: Ic7a53b2cbd52812e31de98c567d8d399c8b445b4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800771 Commit-Queue: Alex Clarke <alexclarke@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#696741}
-
Koji Ishii authored
This patch adds a DCHECK to detect when |NGLineBreaker| goes infinite loop by |HandleOverflow()|, |Rewind()|, but failed to find appropriate secondary break point and |Rewind()| to the same point again and again. Change-Id: I9963fb5252f61fcfd0e5d768da8a4968f1f7c9e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1801474Reviewed-by:
Emil A Eklund <eae@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#696740}
-
Emil A Eklund authored
Bug: 1002317 Change-Id: I14d4bde5f59897f5d5649ef37428a477a5e7eb37 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1804948 Commit-Queue: Emil A Eklund <eae@chromium.org> Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by:
Aleks Totic <atotic@chromium.org> Cr-Commit-Position: refs/heads/master@{#696739}
-
Marc Treib authored
Bug: none Change-Id: Iba2184b7e8cdf119aaac971fd5325ccee23a71d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803275 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Auto-Submit: Marc Treib <treib@chromium.org> Reviewed-by:
Jan Krcal <jkrcal@chromium.org> Cr-Commit-Position: refs/heads/master@{#696738}
-
Thomas Tangl authored
Account feature buttons are added at the bottom of the bordered box. Screenshot: https://drive.google.com/file/d/17tsNrV3HmOZD9yVNp6VwfbJ46GOAuvQu/view?usp=sharing Flag: profile-menu-revamp Bug: 995720 Change-Id: Icbd286cdf388742d710688e6c93cd44d1ad16e9a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803438 Commit-Queue: Thomas Tangl <tangltom@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#696737}
-
Omer Katz authored
Using atomic version of accessors whenever concurrent markers can be active. Bug: 986235 Change-Id: I111ee70e05419803a19ed119b30e1265dfa97d55 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803296Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#696736}
-
Victor Costan authored
IDBTransactionCommit has been "stable" since M76. At this point, it is a permanent part of the Web platform. Bug: 898257 Change-Id: I8fbb5574c01a5f36dc1d785451f5f4cf7b21510c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1806163 Auto-Submit: Victor Costan <pwnall@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#696735}
-
Ioana Pandele authored
This reverts commit 588d7e15. Reason for revert: This is likely causing TabletModeWindowmanagerWithClamshellSplitViewTest.ClamshellTabletTransitionTest to consistently fail on multiple ChromeOS bots. Bug:1004195 Original change's description: > split view: Remove window activation in SplitViewController's snap logic > > SplitViewController::OnWindowSnapped shall no longer activate the newly > snapped window. In many workflows, such window activation is redundant > or inappropriate (see the referenced issues on Monorail). Window > activation shall be done separately by code for workflows where the > snapped window actually does need to be activated. > > Bug: 982954, 983227, 991611, 995946 > Change-Id: Ie111d90786764f0612aea73ba9cfa4ce640fe1f2 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738338 > Reviewed-by: Xiaoqian Dai <xdai@chromium.org> > Reviewed-by: Mitsuru Oshima (slow - perfing & gardening) <oshima@chromium.org> > Reviewed-by: Scott Violet <sky@chromium.org> > Commit-Queue: Avery Musbach <amusbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#696599} TBR=sky@chromium.org,oshima@chromium.org,xdai@chromium.org,amusbach@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 982954, 983227, 991611, 995946 Change-Id: Ie6760645dd3ded55bf1190341155da63176d6f62 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803926Reviewed-by:
Ioana Pandele <ioanap@chromium.org> Commit-Queue: Ioana Pandele <ioanap@chromium.org> Cr-Commit-Position: refs/heads/master@{#696734}
-
Maksim Sisov authored
This CL removes the setup of WindowEventFilter in the DWTHX11 and uses the one already implemented in the DWTHPlatform. However, to make move/resize work with non-Ozone X11, the WMResizeHandler is subclassed by the X11Window instead of X11WindowOzone now. Bug: 990756 Change-Id: I9e95bf56adcf21267246c45264d8a801ee9f45cd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776027 Commit-Queue: Maksim Sisov <msisov@igalia.com> Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#696733}
-
Tanisha Mandre authored
- MobileAddCreditCard.AddPaymentMethodButton - Entry point from settings. - Modify ManualFallback_CreditCard_OpenAddPaymentMethod - Entry point from manual fallback. - MobileAddCreditCard.AddButton - Save a new card using the 'Add' button. Bug:1002512 Change-Id: I20ab8faf3b40345b21e00a37d801d6ae666b557b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796357Reviewed-by:
Javier Ernesto Flores Robles <javierrobles@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Commit-Queue: Tanisha Mandre <tanishamandre@google.com> Cr-Commit-Position: refs/heads/master@{#696732}
-
Jan Scheffler authored
This patch adds a min width to the debug panel in the sources panel to keep it from overflowing. Bug: chrome:841745 Change-Id: Id33fb70555a1d12a732c803fdf398a165a507ada Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786442 Commit-Queue: Jan Scheffler <janscheffler@google.com> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/ecd10922eec5..914862e8ec8a git log ecd10922eec5..914862e8ec8a --date=short --no-merges --format='%ad %ae %s' 2019-09-16 dberris@chromium.org PinpointScheduler: Port of find_isolate to Tasks Created with: gclient setdep -r src/third_party/catapult@914862e8ec8a If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/catapult-autoroll Please CC crouleau@chromium.org 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.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=crouleau@chromium.org Bug: chromium:988912 Change-Id: I3521c035e1e2327ef09d8cd8134ad4e584191322 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1806326Reviewed-by:
-
Giovanni Ortuño Urquidi authored
Tests are failing on linux: https://ci.chromium.org/p/chromium/builders/ci/Linux%20Tests%20%28dbg%29%281%29/83157 TBR=pfeldman@chromium.org Bug: 1004150 Change-Id: I4bf6d6a10d60c7659a23b6aba1884a5995957531 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1804709Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#696729}
-
Tanisha Mandre authored
- Test that if a card with an invalid expiry date is added, an alert is shown. - Test that the 'Cancel' button dismisses the screen. Bug:1000669 Change-Id: Ie5f22fbcb4e8aecb9955c21d35a0bffbb55522e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786646 Commit-Queue: Tanisha Mandre <tanishamandre@google.com> Reviewed-by:
-
Benedikt Meurer authored
The color selection for code coverage (red+green) is not ideal for developers that are impacted by color blindness. This changes the colors to a brighter red and some blueish green that, a combination that is known to work well enough for people impacted by different kinds of color blindness. https://pbs.twimg.com/media/EEU9fCzUUAItz4n?format=jpg shows an example of how these two colors look like now. Bug: chromium:1003671 Change-Id: I9c3e08fb350a6086624d63f44e069813c618588e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1801762 Auto-Submit: Benedikt Meurer <bmeurer@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/561b90bd1d76..86c48abc94f3 git log 561b90bd1d76..86c48abc94f3 --date=short --no-merges --format='%ad %ae %s' 2019-09-16 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 b1580a276713..d32dae1bb37c (5 commits) 2019-09-16 skia-recreate-skps@skia-swarming-bots.iam.gserviceaccount.com Update Go deps 2019-09-16 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader adb0d51ca672..30847688f059 (2 commits) Created with: gclient setdep -r src/third_party/skia@86c48abc94f3 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC brianosman@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.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 TBR=brianosman@google.com Bug: chromium:None,chromium:None Change-Id: I55ce7cc4f987c7ac46f7d1bdd8ff821a9b523a97 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1805719Reviewed-by:
-
Jason Lin authored
* Remove extra inset for a few crostini dialogs * Do not show the close button for CrostiniUpgradeContainerView so that it is consistent with other crostini dialogs. Bug: 965185 Change-Id: Idf3934116ea73ef9153e05b8370cd67f4d64a6f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803943Reviewed-by:
Nicholas Verne <nverne@chromium.org> Commit-Queue: Jason Lin <lxj@google.com> Cr-Commit-Position: refs/heads/master@{#696725}
-
Koji Ishii authored
When `word-break: break-all` or `keep-all` is specified, this patch fixes `overflow-/word-wrap: break-word` to be effective. Before this patch, it was forced to be `normal`. Bug: 1001378 Change-Id: I73f5956a6856f2cbd0944461a0aa7a2929e722cc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1806162 Commit-Queue: Koji Ishii <kojii@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/bdf64c9f6d8e..f02b13925471 Created with: gclient setdep -r src-internal@f02b13925471 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: I3efd12057c6ef0b6c30a0fba2a63014eff4e95d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1805904Reviewed-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@{#696723}
-
Renato Silva authored
ChromeOS update logo was chopped of on the tip of the arrow. This commit replaces the image with a proper one. Bug: 936616 Change-Id: I5943f187fb421b3ab70728344dc48590dda3e324 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803284Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Achuith Bhandarkar <achuith@chromium.org> Commit-Queue: Renato Silva <rrsilva@google.com> Cr-Commit-Position: refs/heads/master@{#696722}
-
Matt Giuca authored
Bug: 1001405 Change-Id: I918c57d6922915e624b731432b393311d8ab1a3b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1802780 Commit-Queue: Matt Giuca <mgiuca@chromium.org> Reviewed-by:
Eric Willigers <ericwilligers@chromium.org> Cr-Commit-Position: refs/heads/master@{#696721}
-
David Vallet authored
Bug: 937152 Change-Id: Ibf516ba18d596dd82f15cdcb060c7bd3e0345863 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803946Reviewed-by:
David Vallet <dvallet@chromium.org> Reviewed-by:
Darren Shen <shend@chromium.org> Auto-Submit: David Vallet <dvallet@chromium.org> Commit-Queue: Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#696720}
-
Keishi Hattori authored
This reverts commit 0d7047f8. Reason for revert: Caused regression in memory usage Bug: 1003130 Original change's description: > [Resolve Leak Error]Make ScriptValue Traceable > > We made ScriptValue Traceable to replace SharedPersistent > with TraceWrapperV8Reference in a follow-up CL. > > In this CL, most of the changes are just: > - Replace Vector<ScriptValue> with HeapVector<ScriptValue> because > ScriptValue has a Trace method. > - Add a trace method for ScriptValue and ScriptPromise since > they have objects to Trace. > > We recommend you to mainly check > - script_value.h where I add VectorTraits. > - script_promise.h where I add VectorTraits. > - v8_callback_function.py where I add a code to generate HeapVector. > - v8_types.py > * Add ScriptValue and ScriptPromise to "is_traceable==True" > which is used to generate Trace method & HeapVector. > > + I removed code to detect leak around ScriptPromise because > ScriptPromise no longer has persistent which may cause memory leak. > > Bug: 998994 > Change-Id: I63fcbf01de099577fe14045785b3072c615d78c4 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792583 > Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> > Reviewed-by: Keishi Hattori <keishi@chromium.org> > Reviewed-by: Yuki Shiino <yukishiino@chromium.org> > Reviewed-by: Kouhei Ueno <kouhei@chromium.org> > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Commit-Queue: Rika Fujimaki <rikaf@google.com> > Cr-Commit-Position: refs/heads/master@{#695505} TBR=peria@chromium.org,yukishiino@chromium.org,haraken@chromium.org,keishi@chromium.org,kouhei@chromium.org,rikaf@google.com Change-Id: I35a1c06c1e9079950e03023f7577b28ca0811bef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803942Reviewed-by:
Keishi Hattori <keishi@chromium.org> Reviewed-by:
-
Andrew Moylan authored
Bug: chromium:1000119 Change-Id: I2da457de5901a8771b249bb19c2f6edb6895e3b1 Tested: No Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1782020 Commit-Queue: Andrew Moylan <amoylan@chromium.org> Reviewed-by:
Tony Yeoman <tby@chromium.org> Cr-Commit-Position: refs/heads/master@{#696718}
-
Giovanni Ortuño Urquidi authored
TBR=afakhry@chromium.org Bug: 1003339 Change-Id: I49bdda14651699c562dbfc25be3ce434d570ab54 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803945Reviewed-by:
-
Jinsuk Kim authored
This CL makes half/full state height of BottomSheet dynamic so that the sheet can have both half/full state and keep the content wrapped. Gesture navigation shows how it can be used as an example. By overriding BottomSheetContent.getCustomized{Half,Full}Ratio: - The sheet expands to the half state with content wrapped. The height is maxed out at the half the container height. - If slided up, the content moves to the full state, the content still wrapped. The height is maxed out at the full container height, and the content becomes scrollable if necessary. Bug: 993051 Change-Id: I7e000716ee6b544b541d385e34cae026d64798d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1781123 Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#696716}
-
Maksim Sisov authored
To make ozone/linux really use the DesktopWindowTreeHostLinux, it is also needed to share the BrowserDesktopWindowTreeHost* impl between X11/linux and ozone/linux. This CL renames the BrowserDesktopWindowTreeHostX11 to BrowserDesktopWindowTreeHostLinux and adds some USE_X11 guards, which will be removed in the future. Bug: 990756 Change-Id: Id99d63a627257fbaca7e5de65ec4973d8396d4fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796547 Commit-Queue: Maksim Sisov <msisov@igalia.com> Reviewed-by:
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@chromium.org 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.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast TBR=cr-fuchsia+bot@chromium.org Change-Id: I82d3c60b12de9f9a0e5ceaf9fb1e960678c12e6c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1805482Reviewed-by:
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/cb52badca04a..561b90bd1d76 git log cb52badca04a..561b90bd1d76 --date=short --no-merges --format='%ad %ae %s' 2019-09-16 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ../src 46548997..48c8f915 (416 commits) Created with: gclient setdep -r src/third_party/skia@561b90bd1d76 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC brianosman@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.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 TBR=brianosman@google.com Bug: chromium:None Change-Id: Ifc855099d1236306a8a286dba2e706340bc10950 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1805481Reviewed-by:
-
Fergal Daly authored
This bug was introduced https://crrev.com/c/1772850. Set.add does not take multiple arguments, so most revealed elements were not added to the observed set. Bug: 1004102 Change-Id: If57b3125a54a52e1d0fd242c29b059b1cf69b2cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1804096 Commit-Queue: Fergal Daly <fergal@chromium.org> Reviewed-by:
Domenic Denicola <domenic@chromium.org> Cr-Commit-Position: refs/heads/master@{#696712}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/6fe107e0feca..b8a20f9d9f71 git log 6fe107e0feca..b8a20f9d9f71 --date=short --no-merges --format='%ad %ae %s' 2019-09-16 sidereal@google.com Update GUEST_IMAGES_PINS_PATH Created with: gclient setdep -r src/third_party/chromite@b8a20f9d9f71 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I93e870fa735a44d73d031a5aa8dde2a93d43b9ea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1805479Reviewed-by:
-
Jeevan Shikaram authored
This CL adds a no-search tag to the os-settings/app-management subpage. This prevents the search provider from loading the App Management subpages as this can lead to long search times and recursive behaviors. Bug: 1003277 Change-Id: Ie72a574bf0e040961fa6f79b6281af8fbedf6457 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1804697 Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org> Auto-Submit: Jeevan Shikaram <jshikaram@chromium.org> Reviewed-by:
Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#696710}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/fcbe4071ce46..7af1bb3f90b3 git log fcbe4071ce46..7af1bb3f90b3 --date=short --no-merges --format='%ad %ae %s' 2019-09-16 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 9f151687..1d12ff69 (696593:696696) Created with: gclient setdep -r src/third_party/webrtc@7af1bb3f90b3 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:None Change-Id: I97cb84522a0b5b125039d68c447f6680642a16a8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1805478Reviewed-by:
-
