- 10 Sep, 2019 40 commits
-
-
Mitsuru Oshima authored
* Added OnMouseEnter to PlatformWindow{Delegate} * Updated DRM/X11 impl to call it when the mouse location enters it. and chrome for chromeos on linux desktop. Bug: 990589 Test: X11WindowOzoneTest.MouseEnterAndDelete, manually tested on device Change-Id: I452c34ef0019f10d20d1096ef608cf84dd3a73db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761772 Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Reviewed-by:Scott Violet <sky@chromium.org> Reviewed-by:
Michael Spang <spang@chromium.org> Cr-Commit-Position: refs/heads/master@{#695264}
-
Yi Gu authored
This function is unused. Bug: None Change-Id: Ic0c39655a6ea8087dbfbf2150e4d1478163b06e5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795756Reviewed-by:
kylechar <kylechar@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#695263}
-
Yue Li authored
Move the status sync to prevent timing issues. Bug: None Test: Manual Test Change-Id: I2e2847c07a4125d830a423ab9e5c4e454f891de3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788176 Commit-Queue: Yue Li <updowndota@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#695262}
-
Dana Fried authored
Details are in the new, improved documentation. But MoveWebContentsAt() will now never leave a group in a non-contiguous state. Also, did some fixup in the unit tests to support testing of the new functionality. Bug: 915956 Change-Id: Ibd83249ebbaeb74e50f6805dc626c26c0291f86e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793544Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Commit-Queue: Dana Fried <dfried@chromium.org> Cr-Commit-Position: refs/heads/master@{#695261}
-
Eric Karl authored
This reverts commit 797ebfdc. Reason for revert: Fixed minor build issue causing initial revert. Original change's description: > Revert "Cleanup Android video targets" > > This reverts commit 9c44fedd. > > Reason for revert: broke https://ci.chromium.org/p/chromium/builders/ci/android-jumbo-rel/13515 > > Original change's description: > > Cleanup Android video targets > > > > Fixes WebView component build issues. > > > > Previous CLs introduced a number of new targets including > > texture_owner_android and shared_image_video. These were causing build > > problems due to duplicate file compilation which led to DCHECKs. > > > > Moved most of these files into gpu/command_buffer/service:gles2, as > > they all depended on this target (so no new transitive deps), and it's > > probably a good place for them for now and simplifies dependency > > management. > > > > Bug: 996285 > > Change-Id: Ic5a9cced7e67500cfd5fff8809d4ee5e17bbab8b > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787165 > > Reviewed-by: Frank Liberato <liberato@chromium.org> > > Reviewed-by: Nasko Oskov <nasko@chromium.org> > > Reviewed-by: vikas soni <vikassoni@chromium.org> > > Auto-Submit: Eric Karl <ericrk@chromium.org> > > Commit-Queue: Eric Karl <ericrk@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#695006} > > TBR=kinuko@chromium.org,nasko@chromium.org,liberato@chromium.org,ericrk@chromium.org,vikassoni@chromium.org > > Change-Id: Ib51ec09046ecfa51543836755b6b830d76fbce94 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 996285 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793931 > Reviewed-by: Tao Bai <michaelbai@chromium.org> > Commit-Queue: Tao Bai <michaelbai@chromium.org> > Cr-Commit-Position: refs/heads/master@{#695062} TBR=michaelbai@chromium.org,kinuko@chromium.org,nasko@chromium.org,liberato@chromium.org,ericrk@chromium.org,vikassoni@chromium.org Change-Id: I35661eb7691dbbacb063e308513daca68efcff75 Bug: 996285 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796110Reviewed-by:
vikas soni <vikassoni@chromium.org> Reviewed-by:
Tao Bai <michaelbai@chromium.org> Reviewed-by:
Frank Liberato <liberato@chromium.org> Reviewed-by:
Eric Karl <ericrk@chromium.org> Commit-Queue: Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#695260}
-
Andrew Xu authored
In ShelfLayoutManagerTest::ShelfUpdatedWhenStatusAreaChangesSize, it assumes that the shelf view fills the display space. It will be incorrect under new specs. In ShelfLayoutManagerTest::RtlPlacement, the test fails to reset the RTL setting. This CL fixes these two tests. Bug: 1002576 Change-Id: I9dc2a82f5dab18c33e6947e21112af5aa4913f0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796118 Commit-Queue: Andrew Xu <andrewxu@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#695259}
-
Scott Violet authored
We can't use base in the public API, so this makes a copy of ObserverList. BUG=none TEST=none Change-Id: Ia346c8a5c11204c9ee8e77a2a0c2ca861f25543f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793782Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#695258}
-
Tommy Steimel authored
This CL adds a highlight to the GMC toolbar button when the in-product help is shown. Bug: 991585 Change-Id: I6911ba92d323e859bfb95e7d43173ef53a669d71 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790044Reviewed-by:
Collin Baker <collinbaker@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#695257}
-
Ilia Samsonov authored
The purpose of this cl is to prevent the tests from crashing on all platforms when running in a batch. The Task Runner outlives the PooledTaskRunnerDelegate, which it holds a reference too. This causes failure in batch test run. Bug: 1002573 Change-Id: I5fca10116e7b40a2b70bf867d55165b8b5825708 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1794303Reviewed-by:
Derek Schuff <dschuff@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#695256}
-
Vadym Doroshenko authored
This CL removes some leftovers of the old parser. Bug: 949519 Change-Id: I8f94cc9367752da83a20906b238f68eb5ea0a784 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1789539 Commit-Queue: Vadym Doroshenko <dvadym@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Auto-Submit: Vadym Doroshenko <dvadym@chromium.org> Cr-Commit-Position: refs/heads/master@{#695255}
-
David Black authored
The current value is 15ms. We may need to modify this behavior during experimentation if we find that this threshold is insufficient to give the user time to digest the event. Early feedback suggests that may be the case. Bug: b:140765359 Change-Id: I81586347b374342ef836980a69d86e820b1a0886 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793750 Commit-Queue: David Black <dmblack@google.com> Reviewed-by:
Tao Wu <wutao@chromium.org> Cr-Commit-Position: refs/heads/master@{#695254}
-
Kevin Marshall authored
If the HARDWARE_VIDEO_DECODER_ONLY Context flag is set, then only the video codecs which are decodable in hardware are reported as playable. * Adds "enable_software_video_decoder" GN flag for Cast Runner. * Defines new command line switch --disable-software-video-decoders for propagating the flag across ContextProvider/Browser and Browser/Renderer process boundaries. * Bypasses software codec factory registration if the switch is set. * Uses hardcoded temporary logic for defining which codecs are hardware accelerated; will be replaced with a check against the Fuchsia media codec service later. * Adds new browser tests to verify the effect of --disable-software-video-decoders. Bug: 1000858 Change-Id: I780bd38eacf7537bfb1ceb52db2bad5a4ed8a879 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786079 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Auto-Submit: Kevin Marshall <kmarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#695253}
-
Wez authored
Reduce the default Run() timeout set by TaskEnvironment from TestTimeouts::action_max_timeout() to action_timeout(). This typically reduces the timeout from 30s (45s in Debug) down to 10s. Sheriffs! This may result in: - Failures on slow builds, which are not on CQ. Please consider adding an #ifdef to restore those builds to action_max_timeout(), rather than simply reverting. - Flakiness in tests which are too close to the new timeout. Please consider whether such tests should really be taking so long, and whether they can be fixed-forward with a ScopedRunTimeoutForTest rather than reverting (see ThroughputAnalyzerTest in this CL). Bug: 922098 Change-Id: Icd75af79f6d9d3555c91b33ea2566fe7c56cd2f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548382 Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Reviewed-by:
Maks Orlovich <morlovich@chromium.org> Auto-Submit: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#695252}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/884c81e1703f..1f562b7ad68c git log 884c81e1703f..1f562b7ad68c --date=short --no-merges --format='%ad %ae %s' 2019-09-10 perezju@chromium.org [Telemetry] Rename story_runner.Run to StorySetRun Created with: gclient setdep -r src/third_party/catapult@1f562b7ad68c 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 zhanliang@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_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=zhanliang@google.com Bug: chromium:866458 Change-Id: I5de6f69bea96d124e27ea1e1968894e353bf70e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795985Reviewed-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@{#695251}
-
Piotr Bialecki authored
Bug: 992033 Change-Id: I5d79658062e574b7c910af03b0ec0c838c169f05 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1774659 Auto-Submit: Piotr Bialecki <bialpio@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Klaus Weidner <klausw@chromium.org> Commit-Queue: Piotr Bialecki <bialpio@chromium.org> Cr-Commit-Position: refs/heads/master@{#695250}
-
David Black authored
Previously the close button in the proactive suggestions view was 16dip as the view bounds matched the icon bounds. Now, the icon is still 16dip but the view bounds are increased to 32dip which is also the standard size across Assistant for buttons. Bug: b:140669269 Change-Id: I89e8fb9e2027553cec8fb6a8d605ce10ec4cdd67 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792979Reviewed-by:
-
Chris Blume authored
The SkiaRenderer GL bot was accidentally added outside of alphabetical order. This CL rearranges the order to be alphabetical. Bug: 969855 Change-Id: I3e1183483ba7dcca1f4cccf084b062f41d3c6fe9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795668Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Chris Blume <cblume@chromium.org> Cr-Commit-Position: refs/heads/master@{#695248}
-
chrome-bot authored
Same change as https://crrev.com/c/1793050. This break affects all our UITests: https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=celab_tests Validation: https://cr-buildbucket.appspot.com/build/8902773145814860448 Change-Id: I3dd0efbc1dc797cd51bf587e9fd217385beba06c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791795Reviewed-by:
Owen Min <zmin@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Auto-Submit: Mathieu Binette <mbinette@google.com> Cr-Commit-Position: refs/heads/master@{#695247}
-
John Rummell authored
Moving the code that handles the CDM manifest into a separate file in chrome/common to support manifest parsing outside of the component updater. Bug: 889561,971433 Test: new unit_tests pass Change-Id: I1d6591834f01f1fd01daf5450f5cad08f4af2e26 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764249Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: John Rummell <jrummell@chromium.org> Cr-Commit-Position: refs/heads/master@{#695246}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/c1ec0cbb921b..c9800e5adb7a Created with: gclient setdep -r src-internal@c9800e5adb7a 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: I11f333c4ae254ae9af70628c5e52127afcd699cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796109Reviewed-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@{#695245}
-
Alex Clarke authored
We want to prioritize loading and compositing tasks until first contentful paint. That wac casusing fast/replaced/replaced-breaking.html to flakily have image diffs because the test harness is taking screen shots before the plugins have finished loading. This patch makes the test wait until they have both loaded before finishing which fixes the flakes we'd have when turning on the new scheduler policy. Bug: 971191 Change-Id: I08f34cfefa5ffdec14741b7ab1cf8d83fc3e33b6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795789Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Commit-Queue: Alex Clarke <alexclarke@chromium.org> Cr-Commit-Position: refs/heads/master@{#695244}
-
Gabriel Charette authored
These tests were previously migrated from single-threaded MessageLoop to a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as part of crbug.com/891670. //base OWNERS decided in retrospect that it was better to keep a single-threaded option for TaskEnvironment and introduced SingleThreadTaskEnvironment. This CL retrofits that decision for /services/device/public/cpp/power_monitor. This CL is a no-op if it passes CQ. This CL was uploaded by git cl split. R=bajones@chromium.org Bug: 891670 Change-Id: Ibb48e3b6a6c219f1c781e719f033d0ee88c33f1c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787099 Auto-Submit: Gabriel Charette <gab@chromium.org> Commit-Queue: Brandon Jones <bajones@chromium.org> Reviewed-by:
Brandon Jones <bajones@chromium.org> Cr-Commit-Position: refs/heads/master@{#695243}
-
dpapad authored
The code was adding a trailing space when the URL ends with "\", to work around https://bugs.webkit.org/show_bug.cgi?id=28885 which no longer seems to be an issue. Bug: None Test: Run https://jsfiddle.net/m7brn0q4/, verify that a network request to "foo/bar/?\" is sent. Change-Id: I351d60350ab43b99270edaef063da76b35955a1f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793789 Commit-Queue: Esmael Elmoslimany <aee@chromium.org> Auto-Submit: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#695242}
-
rbpotter authored
- Autogenerate Polymer 3 cr-toolbar, cr-toolbar-search-field, and cr-toolbar-selection-overlay - Autogenerate Polymer 3 cr-toolbar-search-field test - Add a cr-toolbar to the demo page Bug: 965770 Change-Id: Id51ce14f0a8812be7a858d5362c42f212d199f41 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790435 Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
-
Christy Chen authored
Two strings for the checkboxes in Layers panel are not localized. This CL wrap them with ls, and include the strings in the GRDP. https://imgur.com/fbrbvKY Bug: 941561 Change-Id: Iab4e3bfaa791dbb9542133109367284465b59d98 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1779170Reviewed-by:
Erik Luo <luoe@chromium.org> Commit-Queue: Christy Chen <chrche@microsoft.com> Cr-Commit-Position: refs/heads/master@{#695240}
-
Adithya Srinivasan authored
Test flakily fails on TSAN, see bug for more details. Bug: 1002602 Change-Id: I0b68f636f601d2a138297ce0b2da49dc2883304a Tbr: peter@chromium.org No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796385Reviewed-by:
Adithya Srinivasan <adithyas@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#695239}
-
Jenny Zhang authored
Bug: 1001708 Change-Id: I036e5546d84a60fc9248f785c1801bc8d88eef57 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793945 Commit-Queue: Jenny Zhang <jennyz@chromium.org> Reviewed-by:
Tony Yeoman <tby@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/e5641be5fe30..d9c1c856c64c git log e5641be5fe30..d9c1c856c64c --date=short --no-merges --format='%ad %ae %s' 2019-09-10 raul@tambre.ee fix_encoding: Decode bytestrings on Python 3 Created with: gclient setdep -r src/third_party/depot_tools@d9c1c856c64c If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC agable@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 TBR=agable@chromium.org Bug: chromium:984182 Change-Id: I7518842e1bdcf77ce98f9087790a06e121238964 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796107Reviewed-by:
-
Nico Weber authored
There are a few existing violations. Cleaning these up doesn't look super involved; that's tracked at crbug.com/1002509 Bug: 912946, 1002509 Change-Id: I58d3334040e90967c6614bf17f705cd43d5c5b0b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1780884 Auto-Submit: Nico Weber <thakis@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#695236}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/507f43465b33..29ab487ea7c1 git log 507f43465b33..29ab487ea7c1 --date=short --no-merges --format='%ad %ae %s' 2019-09-10 philipel@webrtc.org Revert "Removes string support in field trial parser." Created with: gclient setdep -r src/third_party/webrtc@29ab487ea7c1 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: None Change-Id: I804344bc4330d9a6226ec1b7c4fc01e7cd6d3f0d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795565Reviewed-by:
-
Himanshu Jaju authored
For users who have sync turned off on their current device, we show a button to direct them to the settings page to enable sync. Light mode - https://drive.google.com/open?id=1hYpPBCBQdAfxTsw0a13lS2aBSsvTrqk6 Dark mode - https://drive.google.com/open?id=1FYts7p-9W7duyv9Gqs5XEReJT8j3yYnd Bug: 1001102 Change-Id: I0ad3de82739b74541a06249447ad5c7c72b3ae23 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795426 Commit-Queue: Himanshu Jaju <himanshujaju@chromium.org> Reviewed-by:
Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Cr-Commit-Position: refs/heads/master@{#695234}
-
David Black authored
HTTP headers from proactive suggestion responses may be unescaped. These strings can be shown in both the proactive suggestions UI as well as in Assistant UI. We need to unescape these strings before displaying them to the user. Bug: b:140645078 Change-Id: I967c85a67e230bb86f029f517147b4b1b1fbd3ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790479Reviewed-by:
Asanka Herath <asanka@chromium.org> Reviewed-by:
-
Brian Sheedy authored
Adds the option to run tests using Python3 instead of Python2 to allow for tests to be migrated over in preparation for the Python2 deprecation. This can be achieved by adding a "'use_python3': True" entry to the test's gn_isolate_map.pyl entry. Bug: 898348 Change-Id: I675ec068e18cec6bf59ea9d19802bd712f18e87e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769719Reviewed-by:
Marc-Antoine Ruel <maruel@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
-
Erik Språng authored
Remove ios, add android webview. Bug: 1000023 Change-Id: Iace7f258104df5d46f51c20e3ceb19f43120e14f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795908 Commit-Queue: Erik Språng <sprang@chromium.org> Commit-Queue: Alexei Svitkine <asvitkine@chromium.org> Auto-Submit: Erik Språng <sprang@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#695231}
-
Toni Barzic authored
Allow more aggressive apps container UI scaling down when dealing with extremely restricted vertical space (e.g. full zoom and docked magnifier): * reduce the minimum scale value when scaling the tile size * if minimum scale is not enough (or barely enough), remove item icon top padding. * scale down fade out area * if screen size is < 600, reduce the search box size, and the margin between the search box and suggestion chips container BUG=1001989 Change-Id: I887ba805c99e5882f3f9bbc5a6f48064bff53819 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793926 Commit-Queue: Toni Baržić <tbarzic@chromium.org> Reviewed-by: -
Andrew Xu authored
ShelfViewTest::ShouldHideTooltipTest and ShelfViewTest:: ShouldHideTooltipWithAppListWindowTest assumes that home button is still the child view of shelf view. It is incorrect now. This CL fixes these two tests. Bug: 1002576 Change-Id: I8350fa9fab8c21497f3ac24f9bf0a32bde1aaf30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795759Reviewed-by:
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/b2d5d40a3393..2ab2ac03d8ee git log b2d5d40a3393..2ab2ac03d8ee --date=short --no-merges --format='%ad %ae %s' 2019-09-10 brianosman@google.com GL: Don't leak shader objects when program linking fails Created with: gclient setdep -r src/third_party/skia@2ab2ac03d8ee 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 mtklein@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=mtklein@google.com Bug: None Change-Id: Ie716422697a87d63b73393c9140214b076ae419b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796106Reviewed-by:
-
Chrome Metrics Logs authored
Updates the expires_after attribute for 70 histograms that show frequent access in the past 90 days. These are the 80% most frequently used histograms over that time that do not already have a date later than or within 60 days of 2020-03-08. Change-Id: I0e3f3d4df5a91effbd04caa8427f403d2820afe9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795804 Commit-Queue: Chrome Metrics Logs <chrome-metrics-team+robot@google.com> Commit-Queue: Brian White <bcwhite@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#695227}
-
Andrew Xu authored
Scrollable shelf should place shelf icons in the center of display when there is sufficient space. That is also the UI behavior of the current shelf view. This CL achieves the UI consistency for scrollable shelf. Bug: 973481 Change-Id: Ifc50e6d62f5508dc84578519c4f6c6522fb845e1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1789677 Commit-Queue: Andrew Xu <andrewxu@chromium.org> Reviewed-by:
-
Gabriel Charette authored
These tests were previously migrated from single-threaded MessageLoop to a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as part of crbug.com/891670. //base OWNERS decided in retrospect that it was better to keep a single-threaded option for TaskEnvironment and introduced SingleThreadTaskEnvironment. This CL retrofits that decision for /chrome/browser/android/download. This CL is a no-op if it passes CQ. This CL was uploaded by git cl split. R=qinmin@chromium.org Bug: 891670 Change-Id: Ifeb9efb0ae6973e97ef9107881387ae2e20cd789 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786684 Auto-Submit: Gabriel Charette <gab@chromium.org> Commit-Queue: Min Qin <qinmin@chromium.org> Reviewed-by:
Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#695225}
-
