- 11 May, 2019 26 commits
-
-
Kristi Park authored
Bug: 851335 Change-Id: Ia5e1ca86f1940851337009b923c6259fb5a05676 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606766 Commit-Queue: Kristi Park <kristipark@chromium.org> Reviewed-by:
Kyle Milka <kmilka@chromium.org> Cr-Commit-Position: refs/heads/master@{#658850}
-
Sergey Ulanov authored
1. Update StructTraits for NativePixmapHandle to allow null vmo handle. VMO handle is set only for NativePixmap instances that can be mapped to memory. 2. Updated the test to set buffer_collection_id as it is required for all NativePixmap's on Fuchsia. 3. Removed gfx_unittests test filter as we no longer need to filter any tests. Bug: 952652 Change-Id: I9882849a302700b411ded0a9e9f4a805d8425c63 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1603542 Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#658849}
-
Sergey Ulanov authored
Some targets were already enabled on Fuchsia, but were not in gn_all. This moves them to gn_all Change-Id: I707812fa1394f87fd48e4078fccbef5fc749d028 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1603882Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#658848}
-
Zentaro Kavanagh authored
- No change to behavior BUG=None TEST=compiles Change-Id: I55d53415dbce1b004fdda015695d3a10b7eb118c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1607121Reviewed-by:
Sean Kau <skau@chromium.org> Commit-Queue: Zentaro Kavanagh <zentaro@chromium.org> Cr-Commit-Position: refs/heads/master@{#658847}
-
Ilia Samsonov authored
Fail TestLauncher if disabled tests shares a name with another test. Added unit tests for TestLauncher disabled tests logic. Changed TestResultTracker unit test to better reflect disabled test tags. Changed problematic disabled tests name that caused CQ to fail. DISABLED_BrowserCloseInfiniteBeforeUnload was removed since it is a copy of BrowserCloseInfiniteBeforeUnload. Bug: 961424,960619 Change-Id: I064a6d5c06e5dfccab2e54d2d5e54ff3903e81f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1603483 Commit-Queue: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#658846}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/fd99d6f2..e7fe3e3c Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I2280b7dbf4ecb518cce5ac6141e94c63e0b5496b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606632Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#658845}
-
Kyle Milka authored
This should make things a bit more clear. Choose to use richerPicker_fooBar instead of richerPickerFooBar to stand out more. Bug: 937570 Change-Id: I2db7d65c6d7aaaea424a95511b57bef8b7849ba7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1607220 Commit-Queue: Kyle Milka <kmilka@chromium.org> Auto-Submit: Kyle Milka <kmilka@chromium.org> Reviewed-by:
Kristi Park <kristipark@chromium.org> Cr-Commit-Position: refs/heads/master@{#658844}
-
Jerome Jiang authored
https://chromium.googlesource.com/webm/libvpx.git/+log/1cbcb820ace4..4d0fe85c1957 $ git log 1cbcb820a..4d0fe85c1 --date=short --no-merges --format='%ad %ae %s' 2019-05-10 jianj Revert "vp9: Enable ml based partition for speed>=8 low res." 2019-05-10 jingning Assign perceptual AQ mode as 5 2019-05-10 jianj Cast buffer offset to int64_t 2019-05-10 jianj Reland "vp9-rtc: tx_size selection for intra mode in nonrd" 2019-05-09 jianj Fix tsan failure in webrtc test. 2019-05-09 johannkoenig disable row mt test 2019-05-08 jingning Fix key frame detection 2019-05-03 jianj vp8: Remove duplicated code in datarate tests. 2019-05-02 angiebird Add mismatch_debug tool 2019-05-06 jingning Cap arf boost in perceptual quality mode 2019-05-06 jingning Increase min arf boost from 240 to 250 2019-05-06 johannkoenig android: clarify RTCD usage 2019-04-29 deepa.kg Avoid two GF only groups just before a kf Created with: roll-dep src/third_party/libvpx/source/libvpx R=johannkoenig@google.com BUG=chromium:946409 Change-Id: Ibe74bc4de06f2a1a79427695829867216ce3bb9a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1607184Reviewed-by:
Johann Koenig <johannkoenig@google.com> Commit-Queue: Jerome Jiang <jianj@google.com> Cr-Commit-Position: refs/heads/master@{#658843}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/29f4ba7d6eb8..3dbe480b559e git log 29f4ba7d6eb8..3dbe480b559e --date=short --no-merges --format='%ad %ae %s' 2019-05-10 shrekshao@google.com Validating draw buffers now also considers color mask settings Created with: gclient setdep -r src/third_party/angle@3dbe480b559e The AutoRoll server is located here: https://autoroll.skia.org/r/angle-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:958374 TBR=jonahr@google.com Change-Id: Ic017b464a656b8508baf37b2f599abc337138b03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1607030Reviewed-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@{#658842}
-
Kyle Milka authored
Add the selected state for the default tile, using a placeholder icon for now. Make clicking done rest the background to default. Screencast: http://go/scrcast/NTg4MzA0MjMyMTEzNzY2NHwxYTFhMDM1YS0zNA Bug: 937570 Change-Id: Iab0b09c16124333caf9513126ee94e7d403d15f2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606577 Commit-Queue: Kyle Milka <kmilka@chromium.org> Reviewed-by:
Gayane Petrosyan <gayane@chromium.org> Cr-Commit-Position: refs/heads/master@{#658841}
-
Robert Ogden authored
See bug for more details. TBR=tbansal@chromium.org Bug: 961934 Change-Id: I7cca3920fa8c8a1402a7c2189a5e8775de58bd30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1607182Reviewed-by:
Robert Ogden <robertogden@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Robert Ogden <robertogden@chromium.org> Cr-Commit-Position: refs/heads/master@{#658840}
-
Matt Wolenetz authored
With recent removal of LegacyByDts buffering logic, SourceBufferRange(ByPts)::Get{Buffered}EndTimestamp methods return values in PTS form, and there are no longer any ByDts overrides of those methods. This change removes the now-unnecessary test-only "GetRangeEndTimesForTesting" method and modifies the tests to use the SBRByPts methods instead. BUG=771349 Change-Id: I26e557a6628c0ba42ce279d340d12a4fe10bfdd9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593928Reviewed-by:Dan Sanders <sandersd@chromium.org> Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org> Cr-Commit-Position: refs/heads/master@{#658839}
-
Matt Wolenetz authored
With recent removal of LegacyByDts buffering logic, this change removes obsolete code in SourceBufferStream that detects and logs to media-internals when a non-keyframe is followed in append sequence by a keyframe with the same decode timestamp. With DTS timelines of two distinct GOPs now made immaterial in the ByPts buffering logic (below FrameProcessor), this log has even less value than the marginal value it had when we still had LegacyByDts. BUG=771349 Change-Id: I21c84fb1df4e314719bc8b0ae7809f2c9cbdc44f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1595172Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/1643e6b61293..d4bdb8b0e03e Created with: gclient setdep -r src-internal@d4bdb8b0e03e 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=anthonyvd@google.com,alph@google.com,aboxhall@google.com,battre@google.com Change-Id: Ibe6f52fd67ad07e6bd5dab6d1241fc5c951603ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1607027Reviewed-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@{#658837}
-
Ahmed Fakhry authored
This CL adds mirroring of the desk contents as well as the wallpaper into the desk's mini_view. Demo: https://bugs.chromium.org/p/chromium/issues/detail?id=866622#c28 BUG=866622 TEST=Mostly manual, minor tests modification. Change-Id: Ia558a35e7a5b5e04020f669830010c55279c5361 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1602815 Commit-Queue: Ahmed Fakhry <afakhry@chromium.org> Reviewed-by:
Mitsuru Oshima (slow - gardening) <oshima@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Sammie Quon <sammiequon@chromium.org> Cr-Commit-Position: refs/heads/master@{#658836}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 48b83dda. With Chromium commits locally applied on WPT: c4dc3441 "[WPT] Reland Upload a test for sending mouse events with key pressed" fec827fb "Added wpt tests for overscroll customization." c9077e39 "Restore original tail-processing for ScriptProcessor and AudioWorklet" faed29aa "[EventTiming] Remove unneeded prefix from WPT tests" 9d5bc5ec "Portals: Add support for frame-src CSP directive." d814fafc "Check for detached ArrayBuffers in Web Bluetooth writeValue functions" 82728505 "[Animation Worklet] Do not allow undefined/NaN local time" 12376bce "Correctly invalidate snap container data when relevant css props change" 01d571da "Rename touch tap function in pointerevent_support.js" 5e3c6036 "[sms] Restrict SMS Receiver API to Top Level Frames" Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: jsbell@chromium.org: external/wpt/resources NOAUTOREVERT=true TBR=lpz No-Export: true Change-Id: I27ea38d7c87bcb84d873180ed3dd7033f1741f94 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606575Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#658835}
-
Alexei Filippov authored
This reverts commit 3bd9684c. Reason for revert: the test is still flaky https://ci.chromium.org/p/chromium/builders/ci/Mac10.13%20Tests/13241 Original change's description: > [EventTiming] Fix programmatic click test > > Commits aee8357d and > faed29aa landed almost at the same time, > so we accidentally landed a test with the wrong script src for the > helper functions. > > Bug: 961547 > Change-Id: Id8a615477a54e56139d9d90fc4feb5f780b729a2 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606225 > Reviewed-by: Liquan (Max) Gu <maxlg@chromium.org> > Commit-Queue: Nicolás Peña Moreno <npm@chromium.org> > Cr-Commit-Position: refs/heads/master@{#658761} TBR=npm@chromium.org,maxlg@chromium.org Change-Id: I5612c80e93779288839660d8336e6b7ca741732b No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 961547 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606823Reviewed-by:
Alexei Filippov <alph@chromium.org> Commit-Queue: Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#658834}
-
Daniel McArdle authored
We cannot delete the base::MD5 implementation until NaCl is removed. However, this CL effectively unships the code from binaries where NaCl is not enabled. Bug: 755368 Change-Id: I1a0b3a1f71a2798af26d2819a7112e01fb8f5781 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1596358 Commit-Queue: Daniel McArdle <dmcardle@chromium.org> Auto-Submit: Daniel McArdle <dmcardle@chromium.org> Reviewed-by:
Ryan Sleevi <rsleevi@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Sam McNally <sammc@chromium.org> Cr-Commit-Position: refs/heads/master@{#658833}
-
Adam Langley authored
This change is the result of UI review for resident keys: • Update privacy string. • Update account selection title and remove description. • Merge missing capability messages for user-verification and resident keys into a single, generic message. • Tweak “storage full” message. Change-Id: Id3614020ff691c74f56118476fcb5fdb898ff44d Bug: 941120 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606243 Auto-Submit: Adam Langley <agl@chromium.org> Commit-Queue: Martin Kreichgauer <martinkr@google.com> Reviewed-by:Martin Kreichgauer <martinkr@google.com> Cr-Commit-Position: refs/heads/master@{#658832}
-
Dan Sanders authored
This CL adds two new features (both enabled by default) for testing HLS configuration: - CanPlayHls: Enables CanPlayType() and other queries for HLS MIME types. Note that disabling this feature also disables playback by navigating to .m3u8 files. - HlsPlayer: Enables WebMediaPlayerImpl to use MediaPlayerRenderer to play HLS manifests. These features can be disabled to test site behavior when HLS is not supported. Change-Id: Iba16d8d60a991ba36d88c0f5188c6189308ab5e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1601973Reviewed-by:Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Dan Sanders <sandersd@chromium.org> Cr-Commit-Position: refs/heads/master@{#658831}
-
Sebastien Marchand authored
Bug: 961336 Change-Id: Ib6d553da8733e54eabf98049be85812a10cdc83c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1602596Reviewed-by:
Chris Hamilton <chrisha@chromium.org> Reviewed-by:
-
Chisoon Jeong authored
Reduce copies of PipelineMetada (which contains AudioDecoderConfig and VideoDecoderConfig) by using pass by reference. This was originally authored by Andre Rosa<andre.rosa@lge.com> Change-Id: I9ddf137d5d35154862501edcbbfe5b26c1dacbf0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1601613 Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
John Rummell <jrummell@chromium.org> Cr-Commit-Position: refs/heads/master@{#658829}
-
Ahmed Mehfooz authored
Will be used to display small buttons below feature pods to indicate current page. Can be clicked to switch to corresponding page. Bug: 914077 Change-Id: Ida92a7bd7ddb0cf8601e468500155822fc6f1424 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576005Reviewed-by:
Jenny Zhang <jennyz@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Tim Song <tengs@chromium.org> Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org> Cr-Commit-Position: refs/heads/master@{#658828}
-
Kristi Park authored
Hide the new shortcut layout behind a flag and re-add the old flexbox layout. Bug: 851335 Change-Id: Ic10b20f5a77874138cbd356aac98f55aa7109c06 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606583 Commit-Queue: Kristi Park <kristipark@chromium.org> Reviewed-by:
-
Alexei Filippov authored
BUG=961938 TBR=msw@chromium.org,etienneb@chromium.org Change-Id: Ic4ed7e67cdee714b46c06b913e1fee28fc2985e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606773Reviewed-by:
-
Kevin Babbitt authored
The omnibox sets up a "Controls" relationship with its popup. Our UIA implementation maps this to the ControllerFor property. The OS would discover this relationship and ask for the popup's fragment root. But when the popup is closed, it has no associated AcceleratedWidget, which is the key we use to look up the fragment root. The fix is in two parts. First, for UIA properties that return one or more related elements, we validate that those related elements will not immediately fail when queried by the OS for additional information. Even with that validation, it's still possible for us to return a related element that's valid at the time we check but later becomes invalid due to the widget being closed. Thus the second part of the fix is to convert the DCHECK that was hitting to an `if`. Bug: 959033 Change-Id: Ic8cf47229e45cfe1650806e6365b17aa67a614c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600408Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Kevin Babbitt <kbabbitt@microsoft.com> Cr-Commit-Position: refs/heads/master@{#658825}
-
- 10 May, 2019 14 commits
-
-
Felicia Lim authored
https://chromium.googlesource.com/chromium/third_party/ffmpeg.git/+log/90cf969d61d6..68f1932090d5 $ git log 90cf969d6..68f193209 --date=short --no-merges --format='%ad %ae %s' 2019-05-09 flim Revert "Revert "libopus: Add channel mapping 2 support in libopusdec"" 2019-05-06 msalama Default stack frames are needed on Android for the Orderfile. Created with: roll-dep src/third_party/ffmpeg Bug: chromium:958645 Change-Id: I1e59b53c569de513cd2b097021e69937aba41924 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1607081Reviewed-by:
-
John Budorick authored
Bug: 861983 Change-Id: Ic78e461b1ec9b7d7d66f1c87e4c7842fab77f915 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606767 Commit-Queue: John Budorick <jbudorick@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Auto-Submit: John Budorick <jbudorick@chromium.org> Reviewed-by:
Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#658823}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/2568cf76ee2c..8ab4053f00b0 git log 2568cf76ee2c..8ab4053f00b0 --date=short --no-merges --format='%ad %ae %s' 2019-05-10 bclayton@google.com Implement Float16 formats and additional blend modes. Created with: gclient setdep -r src/third_party/swiftshader@8ab4053f00b0 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/132434966,chromium:b/132433217 TBR=swiftshader-team+autoroll@google.com Change-Id: I1c157e2d1e1daf72ba68a348dde50b75a2765998 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606622Reviewed-by:
-
Yue Ru Sun authored
Bug: 959378 Change-Id: I727cd1464c8e4c72461184871e989eec299729d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598337Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Dominic Battré <battre@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Commit-Queue: Yue Ru Sun <yrsun@chromium.org> Cr-Commit-Position: refs/heads/master@{#658821}
-
Yining Wang authored
Toolbar is not showing correct content for Focus Mode Window when the site is not secure. This cl resolve this issue by processing an UI update after tab inserted to browser. Change-Id: I35c4baf20ba6c90c85a03f471c51356853e8f165 Bug: 961820 Change-Id: I35c4baf20ba6c90c85a03f471c51356853e8f165 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1603768 Commit-Queue: Yining Wang <yiningwang@google.com> Reviewed-by:
Dana Fried <dfried@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#658820}
-
Matt Wolenetz authored
MseBufferByPts was enabled by default in M75. This series of changes (in M76) removes: 1) (previous CL) the tests for the old "LegacyByDts" MSE buffering logic, and 2) (this change) the old "LegacyByDts" MSE buffering logic, the templating of SourceBufferStream by the buffering implementation, and the associated feature gating Later changes will remove residual complications such as the split SourceBufferRange and SourceBufferRangeByPts types and clean up confusing usage of DecodeTimestamp where it really contains a presentation timestamp within and below FrameProcessor. BUG=771349,760264,718641,398141 Change-Id: I93491bd47e8cd498b4452f9e5109eb9a7c7b9852 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1586461Reviewed-by:
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/ab32f656538d..5c7b54112bca git log ab32f656538d..5c7b54112bca --date=short --no-merges --format='%ad %ae %s' 2019-05-10 egdaniel@google.com Fix compatible format check for vkCmdCopyImage. 2019-05-10 bungeman@google.com Remove SK_IGNORE_LINEAR_METRICS_FIX. 2019-05-10 bsalomon@google.com Fix build for GCC. Cast enum to int when using with %d format specifier Created with: gclient setdep -r src/third_party/skia@5c7b54112bca 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=robertphillips@google.com Change-Id: I5810257ced8af126bc92def20a970746cc716a15 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606983Reviewed-by:
-
Sammie Quon authored
This is a reland of 09566165. Fixed and ran locally. TBR=oshima@chromium.org Original change's description: > Reland "cros: Add tablet <-> clamshell performance test." > > This is a reland of 963bfd6e > > Could not reproduce locally with the new changes. Suspected causes for > flakiness before. > 1) Rounding down because of int means we never waited for browser to > startup. > 2) Disabling animations too early means the creation of the browser window > has an animation which may have interfered with the tablet window animation. > > Original change's description: > > cros: Add tablet <-> clamshell performance test. > > > > Test: interactive_ui_tests TabletModeTransitionTest.EnterExit > > Bug: 953896 > > Change-Id: Ie2332df2bea2f7d2a63b39d8b390fcf4b28999f8 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1580583 > > Commit-Queue: Sammie Quon <sammiequon@chromium.org> > > Reviewed-by: Mitsuru Oshima <oshima@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#656911} > > Bug: 953896 > Change-Id: I214d12fc24f16423f323c87fa7a182eebb0ee745 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598784 > Commit-Queue: Sammie Quon <sammiequon@chromium.org> > Reviewed-by: Mitsuru Oshima (slow - gardening) <oshima@chromium.org> > Cr-Commit-Position: refs/heads/master@{#658294} Bug: 953896 Change-Id: Ifce43db563c6901a7095d4b8137e041f74cca8e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606761Reviewed-by:
-
Alexei Filippov authored
This reverts commit d7e1ad36. Reason for revert: Likely the cause of a broken headless_browsertests https://ci.chromium.org/p/chromium/builders/ci/Mac10.10%20Tests/42776 The crash callstack mentions pdf. Original change's description: > Roll src/third_party/pdfium 3fa919347bc7..6f8388801226 (2 commits) > > https://pdfium.googlesource.com/pdfium.git/+log/3fa919347bc7..6f8388801226 > > > git log 3fa919347bc7..6f8388801226 --date=short --no-merges --format='%ad %ae %s' > 2019-05-10 thestig@chromium.org Remove impossible code related to CPDF_Type3Font. > 2019-05-10 thestig@chromium.org Fix nits in CPDF_Pattern and related code. > > > Created with: > gclient setdep -r src/third_party/pdfium@6f8388801226 > > The AutoRoll server is located here: https://autoroll.skia.org/r/pdfium-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. > > > TBR=pdfium-deps-rolls@chromium.org > > Change-Id: I5a98794e854c9082d9ddecd36f0ce6ce29193eb8 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606621 > Reviewed-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@{#658792} TBR=chromium-autoroll@skia-public.iam.gserviceaccount.com,pdfium-deps-rolls@chromium.org Change-Id: I0a97472a23ee78d4faa959658176425c05b4dc8b No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1607146Reviewed-by:
-
Dale Curtis authored
1ad19fb1 continued the trend of incorrect statements about dav1d thread counts :/ The actual total is (t + 1) * f when configuring tilethreads=t and framethreads=f. I've triple checked this, so this will hopefully be the last update to the calculations and comments. If a system has the cores for it, we'll end up using the following: <300p: 2 tile threads, 2 frame threads = 2 * 2 + 2 = 6 total threads. <700p: 3 tile threads, 2 frame threads = 3 * 2 + 2 = 8 total threads. For higher resolutions we hit limits::kMaxVideoThreads (16): >700p: 4 tile threads, 3 frame threads = 4 * 3 + 3 = 15 total threads. Because of this correct total exceeds limits::kMaxVideoFrames for 720p content we've had to reduce the tile thread count to 4 to ensure smooth playback. Otherwise 5 tile threads and 2 frame threads is insufficient for smooth playback on my Z840. This is handled by a new combined GetDecoderThreadCounts() which outputs both tile and frame thread counts concurrently for clarity. BUG=954659,957511 TEST=printf() on pthread_create, python test program to be sure: def count_threads(f,t): threads = 0 for i in xrange(0, f): for j in xrange(0, t): if t > 1: threads += 1 if f > 1: threads += 1 return threads R=chcunningham Change-Id: Ifcf1069ee9b1a8cede82c102fddbc3c57c2036f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606192 Auto-Submit: Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Reviewed-by:
Chrome Cunningham <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#658815}
-
Brad Hall authored
Tests are consistently timing out and the low shard timeout of 960 isn't helping here Bug: 957194 Change-Id: Ibacccb7b82a2d139747e9288ff2078810024c4d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1603493Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Commit-Queue: Brad Hall <bradhall@google.com> Cr-Commit-Position: refs/heads/master@{#658814}
-
Kenneth Russell authored
It's been promoted to community approved in Khronos' repo. Bug: 849576 Change-Id: I4212df3f5ae226fdb363882aaf1bc70089ccd983 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606584 Commit-Queue: Kenneth Russell <kbr@chromium.org> Commit-Queue: Kai Ninomiya <kainino@chromium.org> Auto-Submit: Kenneth Russell <kbr@chromium.org> Reviewed-by:
Kai Ninomiya <kainino@chromium.org> Cr-Commit-Position: refs/heads/master@{#658813}
-
Daniel Libby authored
In order to quantify the impact of moving scrollbar scrolls to the compositor thread, we want end to end latency metrics that we can look at during experimentation. Add a new SCROLLBAR type to LatencyInfo to facilitate this, this new type will be set on scrollbar-generated gestures so that they'll flow to the end of the latency pipeline and be logged separately, instead of as MOUSE latency as they are today. The Scrollbar latency metrics are the same as the ones that are currently gathered for wheel and touch scrolling, both for ScrollBegin (the first scroll update in a scroll gesture sequence) and for ScrollUpdate. Bug: 954007 Change-Id: I362486be76a4427df271f1cf31e4d23c12857b40 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1595296 Commit-Queue: Daniel Libby <dlibby@microsoft.com> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Reviewed-by:
Timothy Dresser <tdresser@chromium.org> Cr-Commit-Position: refs/heads/master@{#658812}
-
Kenneth Russell authored
Firefox already allows Uint8ClampedArray in the texImage entry points, so this is a behavioral difference that needs to be clarified. Tested with new WebGL conformance test in https://github.com/KhronosGroup/WebGL/pull/2887 . Bug: 961658 Change-Id: I5e2b1591b7994a3271ead7dda2ca0aebd9364526 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606249Reviewed-by:
-
