- 03 May, 2019 40 commits
-
-
Alex Danilo authored
CL:1546804 added scrolling for when directory tree items get focus (to allow items that are tabbed with the keyboard to come into view). A side-effect of this caused selection and window focus to scroll the directory tree even if the item was within view, causing non-focussed items to be scrolled out of view. This change uses the non-standard scrollIntoViewIfNeeded() method to scroll the item only when necessary and then checks the element to see if it moved. If it has moved, the scrollIntoView() method is called to horizontally position the element since the latter method honours the 'inline' positioning property whilst the former method does not. Bug: 955172 Change-Id: I61e04cdd2529274ee19cfd9b8e7a0e387132af35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593885Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Commit-Queue: Alex Danilo <adanilo@chromium.org> Cr-Commit-Position: refs/heads/master@{#656302}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/d22de7578813..f6e10911735b Created with: gclient setdep -r src-internal@f6e10911735b 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=gogerald@chromium.org,caseq@chromium.org,ksakamoto@chromium.org,drott@chromium.org Change-Id: Icf1cbdd12697f0c92b9cf2efd7be5c14c13943f2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594245Reviewed-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@{#656301}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/bf6e6c9070d3..f616294c0b2a git log bf6e6c9070d3..f616294c0b2a --date=short --no-merges --format='%ad %ae %s' 2019-05-03 dberris@chromium.org SheriffRedux: Sheriff Config Service 2019-05-02 rmhasan@google.com [Typ] Use value of the all command line argument in should_skip 2019-05-02 rmhasan@google.com [Typ] Add validation for expectations globs that checks if they only have the '*' in the end Created with: gclient setdep -r src/third_party/catapult@f616294c0b2a 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:953181,chromium:945481,chromium:698902 TBR=zhanliang@chromium.org Change-Id: I772d3dd4fdd0d288b12e731976247662cd4afabb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594240Reviewed-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@{#656300}
-
Chris Hamilton authored
The overhead introduced by using an owned TLS pointer caused a performance regression of 0.5% at the median, as this is a hot code path executed for every single task that is evaluated. This CL reverts to using a plain TLS pointer as before, and repurposes the inline |ipc_program_counter| storage in PendingTask to store the current IPC message context. BUG=957693 Change-Id: I32ddf9a28b924b5a45301d4855dc08d8b914fbbd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1590247 Commit-Queue: Chris Hamilton <chrisha@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#656299}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/e99e2408e96d..f54bb68f47ec git log e99e2408e96d..f54bb68f47ec --date=short --no-merges --format='%ad %ae %s' 2019-05-03 rafael.cintron@microsoft.com Improve usage of static_casts over reinterpret_casts Created with: gclient setdep -r src/third_party/dawn@f54bb68f47ec The AutoRoll server is located here: https://autoroll.skia.org/r/dawn-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel TBR=cwallez@chromium.org Change-Id: Ib47dbf42b9ecb7d693f63b0fcdd1e4649d8b3335 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594244Reviewed-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@{#656298}
-
Toni Barzic authored
Tests existing user controller actions on: * TPM_ERROR * MISSING_CRYPTOHOME * OWNER_REQUIRED * wrong password * cryptohome service not available BUG=949231 Change-Id: I694bc364dfef3d1b655633b32910d858902a86f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592859Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Commit-Queue: Toni Baržić <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#656297}
-
Nico Weber authored
Using `!is_mac && !is_ios` instead of `is_win || is_fuchsia || is_android || is_linux` should be behavior-preserving and is simpler now that all the special cases we used to have have been fixed. Bug: none Change-Id: I78ec8f06bfdc2171deb9cdd060bb148a82926415 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593730 Commit-Queue: Nico Weber <thakis@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Auto-Submit: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#656296}
-
Alexander Alekseev authored
This CL adds OfflineGaiaLogin mixin. Bug: 950236 Change-Id: I704795a7877375f150aabe8a872b4e6c9c5530df Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1590898 Commit-Queue: Alexander Alekseev <alemate@chromium.org> Reviewed-by:
Toni Baržić <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#656295}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/c57ec6f0af33..10f7befc49a7 git log c57ec6f0af33..10f7befc49a7 --date=short --no-merges --format='%ad %ae %s' 2019-05-03 capn@google.com Fix overwriting of filter factor Created with: gclient setdep -r src/third_party/swiftshader@10f7befc49a7 The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel BUG=chromium:b/129523279 TBR=swiftshader-team+autoroll@chromium.org Change-Id: I8d76225ad3694719ec9ca0eb8254b83b1491ec00 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594243Reviewed-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@{#656294}
-
David Bokan authored
To understand why this is needed, note that the size of the layout viewport is calculated based on the minimum scale factor of the page. That is, we size the layout viewport to the size of the visual viewport when it's at the minimum allowable scale factor. |position: fixed| elements are sized relative to the layout viewport; therefore, their size depends on the minimum scale factor. When we override the minimum scale of a page, this can cause the layout viewport, and thus |position: fixed| elements, to increase in size. If we respect the initial-scale, this can cause fixed elements to appear cut-off by the viewport without a good visual indication to the user that zoom-out or horizontal scrolling is available. This is especially bad since the typical case here is a banner with a close [X] button at its far right. This causes those buttons to get cut out of view. Bug: 929709 Change-Id: Iacb6f6e92a5db11037710380e736f1f429575262 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1591888 Commit-Queue: David Bokan <bokan@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#656293}
-
Jeevan Shikaram authored
The app list in App Management would not render when navigating to it via a back button. This was caused by an onDomChange not being triggered by the page change while navigating within App Management. Bug: 948444 Change-Id: I9567a3baa6512550fdee221dfc062b5042925ec6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1560878Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Alan Cutter <alancutter@chromium.org> Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org> Cr-Commit-Position: refs/heads/master@{#656292}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/b9046c85c3ce..d22de7578813 Created with: gclient setdep -r src-internal@d22de7578813 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=gogerald@chromium.org,caseq@chromium.org,ksakamoto@chromium.org,drott@chromium.org Change-Id: I28250c5688b85ac0d7ff4804b4e10edaf119372b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594242Reviewed-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@{#656291}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=govind@chromium.org Change-Id: I589470df2c015be520f0a229b9434dd64dc2ca35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594334Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#656290}
-
Nico Weber authored
It's identical to gnuwin-9.zip but additonally contains tr.exe from UnxUpdates.zip. TBR=hans Bug: 958969 Change-Id: I6c2b49f57af8d85925750c03282ff6ae9dfd21b7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593366Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#656289}
-
Krishna Govind authored
TBR=meacer@chromium.org Change-Id: Ibd1e392c80d1d9786ccc0c300085a085f2d0f864 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594254 Commit-Queue: Krishna Govind <govind@chromium.org> Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#656288}
-
Evan Stade authored
TBR=tsepez@chromium.org Bug: 958154 Change-Id: I0d7773c19c5ebb10b18946d68d916af17f3553bb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592694Reviewed-by:
Evan Stade <estade@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#656287}
-
Kyle Milka authored
Previously requests for promos and the OGB were not sent until the NTP had almost finished loading. Instead start these requests when the request for the main HTML page is received. With this change promos and the OBG will appear along with search suggestions and the most visited tiles if they're available (which is now fairly likely). Otherwise they will appear once ready. Also removes a console error when the OGB failed to load. Bug: 770640 Change-Id: If041fd825dcc763a81b93337b85d4e939be7c7d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1518058 Commit-Queue: Kyle Milka <kmilka@chromium.org> Reviewed-by:
Kristi Park <kristipark@chromium.org> Cr-Commit-Position: refs/heads/master@{#656286}
-
Giovanni Panaro authored
In file manager, when the select all command was issued in a directory where there is only one file and the file was selected, checkSelect mode would not enable. CheckSelect mode now correctly enables. Bug: 938273 Change-Id: Ia44814582f5bf2c5e8e8028e77c35b6fdaf88ba3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576434 Commit-Queue: Giovanni Panaro <tsrwebgl@gmail.com> Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#656285}
-
Peter Kasting authored
* In-declaration initializers * =default * base::ClampToRange() * Use a default value to reduce to a single Reset() function * Call Stop() after resetting internal vars, not before, since it calls callbacks which could read these vars * Reduce redundancy * No || on function names * Shorten code * Rename ResetForSlide() to StopThrobbing() for parallel structure Bug: none Change-Id: I794e219401be3b7bcc5d09be65fe10daf072d537 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594061 Commit-Queue: Peter Kasting <pkasting@chromium.org> Auto-Submit: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#656284}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/719163224a6f..b9046c85c3ce Created with: gclient setdep -r src-internal@b9046c85c3ce 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=gogerald@chromium.org,caseq@chromium.org,ksakamoto@chromium.org,drott@chromium.org Change-Id: I37eb66aec3647614af550fe3d783b40428368271 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594236Reviewed-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@{#656283}
-
Jamie Walch authored
This CL renames LocalMouseInputMonitor to LocalPointerInputMonitor, plumbs the event type (ET_MOUSE_MOVED or ET_TOUCH_MOVED) through to the RemoteInputFilter, and adds touch input detection to ChromeOS. Change-Id: I1adb94df5d56ec3194e7ccd81b870d2c8e73efa6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1590449 Commit-Queue: Joe Downing <joedow@chromium.org> Reviewed-by:
Joe Downing <joedow@chromium.org> Cr-Commit-Position: refs/heads/master@{#656282}
-
Jimmy Gong authored
- Adds new max width constant for cr-searchable-down to shared_vars_css. - Update SmbShare page to use this new max width constant. - This constant will be used in other dialogs in future CL's. Bug: 955173 Test: End to End manual Change-Id: Icc68ca995c1bf0d8df81f16c954e2ae5a202a5b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584845 Commit-Queue: jimmy gong <jimmyxgong@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Bailey Berro <baileyberro@chromium.org> Cr-Commit-Position: refs/heads/master@{#656281}
-
James Cook authored
Use pageVisibility to hide: * internet * bluetooth * device Bug: 950007 Change-Id: I5fdfb525a33b19176d99bc6528bd060758fb2171 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1590604 Auto-Submit: James Cook <jamescook@chromium.org> Commit-Queue: Dan Beam <dbeam@chromium.org> Reviewed-by:
Dan Beam <dbeam@chromium.org> Cr-Commit-Position: refs/heads/master@{#656280}
-
Yuwei Huang authored
Change-Id: I20db073d1b987abf5e4e4c43267f2cbae9e7a70a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592619 Commit-Queue: Joe Downing <joedow@chromium.org> Reviewed-by:
Joe Downing <joedow@chromium.org> Cr-Commit-Position: refs/heads/master@{#656279}
-
Brian Sheedy authored
Ensures that the wrapper script for the XR browser tests properly exits with the browser tests' return code. Previously, it was doing nothing with the returncode, causing the tests to appear to always exit with 0. Bug: 954581 Change-Id: Iede65178478980a7e1770df9c4b20a38f2efa3b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592616 Auto-Submit: Brian Sheedy <bsheedy@chromium.org> Commit-Queue: Klaus Weidner <klausw@chromium.org> Reviewed-by:
Klaus Weidner <klausw@chromium.org> Cr-Commit-Position: refs/heads/master@{#656278}
-
Evan Stade authored
ImmersiveContext remains as an interface in //ash/public. Bug: 958248 Change-Id: Ie32c248e2683cf6957448b33436808394b551176 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593836Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#656277}
-
Daniel Cheng authored
Bug: 841899 Tbr: thestig@chromium.org Change-Id: I972785b8f6c189039c56049a8c132b7bf4c69876 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584531 Commit-Queue: Daniel Cheng <dcheng@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#656276}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/7fde8e172810..666a9f9f8bb9 git log 7fde8e172810..666a9f9f8bb9 --date=short --no-merges --format='%ad %ae %s' 2019-05-02 bungeman@google.com Update includes for Wuffs. 2019-05-02 bungeman@google.com IWYU for SkTLogic.h 2019-05-02 ethannicholas@google.com Child fragment processors are now written as separate functions 2019-05-02 skia-autoroll@skia-public.iam.gserviceaccount.com Roll skia/third_party/skcms 668026c511f3..853f66269c16 (2 commits) 2019-05-02 halcanary@google.com GMs: re-sort gni, script to regenerate gni, file name convention 2019-05-02 halcanary@google.com SkPaint: new constructor 2019-05-02 herb@google.com SkShaper JSON output with cluster visualization Created with: gclient setdep -r src/third_party/skia@666a9f9f8bb9 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=ethannicholas@chromium.org Change-Id: I420e09c36a614796d854398a1c7b319ad5194215 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593596Reviewed-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@{#656275}
-
Evan Stade authored
The part of the AshWindowManager mojom which was being used to preview and perform snaps (click on restore button and drag to left or right) is moved to a pure virtual interface in //ash/public/cpp, and implemented in //ash/frame. Bug: 958154 Change-Id: I1cb107784dfffe2209e27860d6735342913cc5d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592871 Auto-Submit: Evan Stade <estade@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#656274}
-
Evan Stade authored
This is mostly a revert of ddb9a768 and 08ebee42. It also deletes the tap visualizer app. You can test with --ash-touch-hud and ctrl-alt-i, or --show-taps. Bug: 958074 Change-Id: I7282603e02588ad4d92fb51181565e203ee12f06 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1589830Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Auto-Submit: Evan Stade <estade@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#656273}
-
Rakib M. Hasan authored
All GPU test suites have been moved onto the new expectations format. This CL also removes the generate_test_expectations.py script which was used to generate test expectations file for a test suite using the old test expectation python scripts. This CL also removes the --also-run-disabled-tests command line argument. This argument depended on the old format of expectations. The --all command line argument can be used instead to run disabled tests. It either by ignores the Skip expectation in the new format or the --skip command line globs. Bug: chromium:698902 Bug: chromium:937516 Change-Id: I5d866f0e2f21f2ada0ac94db45bf656ec11a9029 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594057 Commit-Queue: Rakib Hasan <rmhasan@google.com> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#656272}
-
John Williams authored
Bug: 954797 Change-Id: Ibc4db91c4ea76b109a6361aa6c269e99cce8480d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1577674 Commit-Queue: John Williams <jrw@chromium.org> Reviewed-by:
Takumi Fujimoto <takumif@chromium.org> Cr-Commit-Position: refs/heads/master@{#656271}
-
Dale Curtis authored
Since we allow duration to be infinite, a seek to 2**64 (kInfiniteDuration) and a subsequent currentTime retrieval will yield kInfiniteDuration. BUG=958324 Change-Id: Ia7e2beead415cb3cfca91672f9399d6fd09a0d89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592614Reviewed-by:
Dan Sanders <sandersd@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#656270}
-
Aleks Totic authored
Inline containing block dimensions were computed incorrectly if: - inline OOF container was split - OOF Element was inside non-root continuation This happened because container dimensions were computed by finding all fragments whose layout object root matched inline_container. If inline container was not a root, no fragments were found. Actual fix is simple: ensure that NGOutOfFlowPositionedDescendant always points to the root. While fixing this, I also did a little bit of cleanup. Main change was - change NGOutOfFlowPositionedDescendant.inline_container to LayoutInline* from LayoutObject* Bug: 931867 Change-Id: Ibd3c0e9fdeffaf2c63d3f59386bc2ea223ec726b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592528 Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#656269}
-
Raphael Kubo da Costa authored
This was removed in the spec in 2016 (commit 16cd4c74, "Spec update"), and there is no reason to keep it around at this point. Bug: 257511, 957953 Change-Id: I3d0d6400ec529891266633fba0f86d7df67a32c9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1588637Reviewed-by:
Robert Ma <robertma@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Ian Clelland <iclelland@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com> Cr-Commit-Position: refs/heads/master@{#656268}
-
Etienne Bergeron authored
The fallback fonts logic in VR was used to detect that every glyphs will exists when rendering. see: https://bugs.chromium.org/p/chromium/issues/detail?id=722861 This is roughly a revert of: https://chromium-review.googlesource.com/c/chromium/src/+/1345419 The Android fallback fonts is implemented here: https://chromium-review.googlesource.com/c/chromium/src/+/1589274 R=billorr@chromium.org BUG: 952963, 770893, 944227 Change-Id: Ia370f666369127f19bd7c950cbd32529e8927411 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1573652Reviewed-by:
Etienne Bergeron <etienneb@chromium.org> Reviewed-by:
Bill Orr <billorr@chromium.org> Commit-Queue: Etienne Bergeron <etienneb@chromium.org> Cr-Commit-Position: refs/heads/master@{#656267}
-
Pedro Amaral authored
Bug: 958769 Change-Id: I6001ac7307215bd947c21022d3cabc9bfbf37667 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594082Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Pedro Amaral <amaralp@chromium.org> Cr-Commit-Position: refs/heads/master@{#656266}
-
Ehsan Karamad authored
The tests started failing a while back; trying them locally they pass 100% of the time. Re-enabling the tests and monitoring dashboard. TBR=wjmaclean@chromium.org Bug: 949683 Change-Id: Ib96a0562492372e2bc66e52fb2c65465beec84ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593641Reviewed-by:
Ehsan Karamad <ekaramad@chromium.org> Commit-Queue: Ehsan Karamad <ekaramad@chromium.org> Cr-Commit-Position: refs/heads/master@{#656265}
-
Matt Reynolds authored
When PlatformEventController is initialized, it takes a handle to the current Document. In some navigation scenarios, the Document used to initialize the PlatformEventController is short-lived and may be shut down by the time the page has fully loaded. Attempting to use the shut down Document can lead to errors. NavigatorGamepad is a subclass of PlatformEventController. When NavigatorGamepad is constructed with a short-lived Document, it fails to properly initialize the PlatformEventDispatcher (GamepadDispatcher) if it passes the (null) LocalFrame associated with the shut down document. This CL modifies PlatformEventController::AddController to take a LocalFrame parameter to use instead of the frame associated with the possibly-stale Document. NavigatorGamepad is modified to pass the LocalFrame associated with the DOMWindow, which is not stale. BUG=949320 Change-Id: Ied380362eb26add5ef0c23195a9ba93bebb5f348 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1552410 Commit-Queue: Matt Reynolds <mattreynolds@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#656264}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/e2f2fee059f6..e207b5651204 git log e2f2fee059f6..e207b5651204 --date=short --no-merges --format='%ad %ae %s' 2019-05-02 manojgupta@google.com cros_fuzz: Update tool to handle msan. Created with: gclient setdep -r src/third_party/chromite@e207b5651204 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@chromium.org Change-Id: I2e344312c761b14c411dedc119f76433986663a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594092Reviewed-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@{#656263}
-