- 18 Oct, 2019 40 commits
-
-
Noel Gordon authored
All directory items now derive from TreeItem: this constant is unused, and can be removed. Also use this.rowElement for the labelElement in class TreeItem, so it works in the FILE_NG_ENABLED case as well. Bug: 992819 Change-Id: I1997e7884adf3a956294acdcf043dec487160e8f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868355 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Reviewed-by:
Alex Danilo <adanilo@chromium.org> Cr-Commit-Position: refs/heads/master@{#707323}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/e99b9182214f..3cdd6444cf91 git log e99b9182214f..3cdd6444cf91 --date=short --no-merges --format='%ad %ae %s' 2019-10-17 rharrison@google.com Check text->str before destroying (#2981) 2019-10-17 kubak@google.com Disallow use of OpCompositeExtract/OpCompositeInsert with no indices (#2980) 2019-10-17 rharrison@google.com Add fuzzer for spirv-dis call path (#2977) 2019-10-17 rharrison@google.com Check binary->code existence before destroying (#2979) 2019-10-17 akb825@gmail.com Improved CMake install step. (#2963) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@3cdd6444cf91 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/spirv-tools-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,jmadill@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_angle_vk32_deqp_rel_ng;luci.chromium.try:android_angle_vk32_rel_ng;luci.chromium.try:android_angle_vk64_deqp_rel_ng;luci.chromium.try:android_angle_vk64_rel_ng;luci.chromium.try:linux_angle_deqp_rel_ng;luci.chromium.try:linux-angle-rel;luci.chromium.try:win-angle-rel-32;luci.chromium.try:win-angle-rel-64;luci.chromium.try:win-angle-deqp-rel-32;luci.chromium.try:win-angle-deqp-rel-64 TBR=radial-bots+chrome-roll@google.com,jmadill@google.com Bug: None Change-Id: I94bcdd8a19fd9d1cf4e4f2b6a087adc88d272abe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1869009Reviewed-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@{#707322}
-
sandromaggi authored
Add a test for selecting an option. Add a test for getting outer HTML. Bug: b/141853667 Change-Id: If420e913d4be4e3eb43614aa9faddcb7524399e3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1863019 Commit-Queue: Sandro Maggi <sandromaggi@google.com> Reviewed-by:
Mathias Carlen <mcarlen@chromium.org> Cr-Commit-Position: refs/heads/master@{#707321}
-
Antonio Gomes authored
BUG=787254, 919392 R=guidou@chromium.org, haraken@chromium.org Change-Id: I9ec5df81e870536e4b9a97d9e4f21678c14da57b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1867151Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#707320}
-
Alan Cutter authored
This CL moves the "last update check time" from RAM to disk via the HostContentSettingsMap which is backed by the PrefService. This persists the timestamp between browser restarts and also allows the user to clear the timestamp when clearing browsing history for the origin. Update throttling is tested by: ManifestUpdateManagerBrowserTest.CheckIsThrottled Bug: 926083 Change-Id: I4f4ebb3c54de302610c40f65a7dfbf9622754dd0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1828785 Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Eric Willigers <ericwilligers@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Auto-Submit: Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#707319}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/c336dd1cb69c..119e2197b75e git log c336dd1cb69c..119e2197b75e --date=short --no-merges --format='%ad %ae %s' 2019-10-18 peah@webrtc.org AEC3: General cleanup after multichannel changes 2019-10-18 phoglund@webrtc.org Rewrite the lib link test to just be a binary. 2019-10-18 saza@webrtc.org AEC3: Update SpectrumBuffer API Created with: gclient setdep -r src/third_party/webrtc@119e2197b75e 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: Ibb989f72e91b4104932062ecbc61abe9cd749a4d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868503Reviewed-by:
-
Jan Wilken Dörrie authored
This change adds a missing label to the ManagePasswordsReferrer entry in enums.xml and adds a required comment to the C++ enum definition. Bug: 1013257 Change-Id: I62da6edd7a7161bc2b2b5c127d0232fce027df49 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868994 Auto-Submit: Jan Wilken Dörrie <jdoerrie@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#707317}
-
Michael van Ouwerkerk authored
Also: * Refactor SharingBrowserTest to be explicit that there are always two sync clients, and set features for both "devices" so that default values don't matter * Clean up context menu helper method Bug: 1006264,989875 Change-Id: Ifc845eae13cef1c54bf982f5df7c9a3ea8f5f63b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1866512 Commit-Queue: Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Reviewed-by:
Richard Knoll <knollr@chromium.org> Cr-Commit-Position: refs/heads/master@{#707316}
-
Mikhail Khokhlov authored
This CL enables processing of all supported formats by Results Processor instead of Telemetry. Bug: 981349 Change-Id: I5bbbb09002babbe9122f55914922ef0340a36bd9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1864677 Commit-Queue: Mikhail Khokhlov <khokhlov@google.com> Reviewed-by:
Juan Antonio Navarro Pérez <perezju@chromium.org> Cr-Commit-Position: refs/heads/master@{#707315}
-
Niels Möller authored
Previous cl in this series: https://chromium-review.googlesource.com/c/chromium/src/+/1836496 Bug: webrtc:9378 Change-Id: Ib44219631559dd702c10675bfc41a198e822c893 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1862449 Auto-Submit: Niels Möller <nisse@chromium.org> Reviewed-by:
Florent Castelli <orphis@chromium.org> Commit-Queue: Niels Möller <nisse@chromium.org> Cr-Commit-Position: refs/heads/master@{#707314}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/12fa2ef9c6d1..72d92573ab57 Created with: gclient setdep -r src-internal@72d92573ab57 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: Icffc6bd28059ecf9ae1f7444d7597102ba3de91e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868510Reviewed-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@{#707313}
-
Friedrich Horschig authored
The experiments for label disambiguation and substring matching autofill suggestions are still ongoing. Therefore, update owners and extend the expiry milestone. Bug: None Change-Id: I61dfbaf0b281e71dd6c0a6e908ddf2677047bcfd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868992Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#707312}
-
Miyoung Shin authored
This CL converts CameraHalDispatcherRequest and BindingSet<CameraHalDispatcher> in media to the new Mojo type. Bug: 955171 Change-Id: I2345178fc387d0db1b2b3b43d787d439ac517404 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1863090 Commit-Queue: Miyoung Shin <myid.shin@igalia.com> Reviewed-by:
Ken Rockot <rockot@google.com> Reviewed-by:
Ricky Liang <jcliang@chromium.org> Cr-Commit-Position: refs/heads/master@{#707311}
-
Lowell Manners authored
Rather than synchronously deleting all pages in the BackForwardCache when there is a conflicting BrowsingInstance, call EvictFramesInBrowsingInstance which asynchronously deletes only the conflicting pages. Calling evict is safer than immediately deleting frames, which can lead to UAF bugs. Change-Id: I4e453c3c1dacc5ed764cc344efbc26f24d7fe115 Bug: 993337 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1844964 Commit-Queue: Lowell Manners <lowell@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Cr-Commit-Position: refs/heads/master@{#707310}
-
Yoshifumi Inoue authored
This patch Introduces |NGInlineCursor| member functions to utilize in |NGCartPosition|[1]: - MoveToFirstLogicalLeaf() - MoveToPreviousLine() - Property getter * HasSoftWrapToNextLine() * IsGeneratedText() * IsText() * CurrentNode() [1] http://crrev.com/c/1866101 Utilize NGInlineCursor in NGCaretPosition Bug: 982194 Change-Id: I9f261ebe0fe680ddc98f39a44f3c1c9dd26ef3a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868358 Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Auto-Submit: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#707309}
-
Christian Dullweber authored
Most requests/callers have a NetworkIsolationKey to provide the top_level_origin but a few are still missing. Remove the DCHECK to avoid crashes in Canary. Callers that don't supply a top_level_origin will fall back to site_for_cookies, which is more restrictive. Bug: 1015548 Change-Id: Ic2b7e08e56427f1e4a6600cd813948c2d462b396 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868991 Commit-Queue: Christian Dullweber <dullweber@chromium.org> Commit-Queue: Balazs Engedy <engedy@chromium.org> Auto-Submit: Christian Dullweber <dullweber@chromium.org> Reviewed-by:
-
Raphael Kubo da Costa authored
The old name made more sense before commit 87be7af4 ("wake lock: Implement new proposed API"), when we had a clear separation of concern between WakeLock, WakeLockController and WakeLockStateRecord. After that commit, WakeLockController was merged into WakeLock, and WakeLockStateRecord became a manager of WakeLockSentinel instances, so rename the class to make the name more descriptive. Bug: 257511 Change-Id: Ic6d5f6059cc8fbd315eaf5f158d822cb1d487cdc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1864661 Commit-Queue: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com> Commit-Queue: Kentaro Hara <haraken@chromium.org> Auto-Submit: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#707307}
-
Wei Lee authored
format We use {width: data_size, height: 1} as size of gbm buffer to store data for BLOB format. Since height might be aligned and makes plane.size bigger than the actual data_size, we change to use plane.stride to represent the data size. Doing so, it won't fail for NativePixmapHandle validation. Bug: b/142105578 Test: ./jpeg_encode_accelerator_unittest Change-Id: Iaf93b0ae7834d2310e1bdfe9cf76be489a3747f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868931Reviewed-by: -
Yuta Kitamura authored
They are very flaky on Fuchsia. Bug: 1014916 Change-Id: Id848846490fd3763fa4538c4ffb25463b6b17804 Tbr: parastoog@google.com Tbr: mmenke@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1869051 Commit-Queue: Yuta Kitamura <yutak@chromium.org> Reviewed-by:
Yuta Kitamura <yutak@chromium.org> Cr-Commit-Position: refs/heads/master@{#707305}
-
Antonio Gomes authored
BUG=787254, 919392 R=guidou@chromium.org, haraken@chromium.org Change-Id: Idb9ef67a8a97181eccf9406b46434e30cf2f72ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1867489Reviewed-by:
-
Yutaka Hirano authored
This is the first CL to implement QuicTransport. This CL - creates a directory in blink, - introduces a new blink runtime feature flag, and - defines QuicTransport IDL interface behind the flag, Bug: 1011392 Change-Id: Id432cee2b6d5df6ab4a4c3e8aa6828d5a7937dd9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868569Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Adam Rice <ricea@chromium.org> Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#707303}
-
Marc Treib authored
Introduces a GetMutableEntityForSyncId helper, the code for which was duplicated many times before, and updates a few comments. Bug: none Change-Id: I775a5912990be1d80d7476ff2af8177b05928f14 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1866633Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#707302}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 6426ba4c. With Chromium commits locally applied on WPT: 6386d9d8 "Update service_worker_unregister_and_register()" 1d5b803f "Add WebVTT support for inline styling - Web Platform Tests" ba49a3d6 "autofocus: Change the timing of 'flush autofocus candidates'" 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 NOAUTOREVERT=true TBR=lpz No-Export: true Change-Id: If392822cca320841b4af57d361784ec6f5a4126f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868889Reviewed-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@{#707301}
-
Antonio Gomes authored
BUG=787254, 919392 R=guidou@chromium.org, haraken@chromium.org Change-Id: Ia9e4ca1ea07c0386972a5f113972f3e2be3bb27e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1867449Reviewed-by:
-
Noel Gordon authored
TreeItem handles the item content creation, label, tree parent_ etc so remove local FakeItem code related to that. Bug: 992819 Change-Id: Ice70bf57e9e739a4dc806c5e3973f893be5e7bd7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868212Reviewed-by:
Austin Tankiang <austinct@chromium.org> Reviewed-by:
-
Pâris MEULEMAN authored
This adds a failing assert when SigninManager.Signin is triggered while the primary account is already set in IdentityManager. Bug: 995201 Change-Id: Ib404f930694ea82aada3bac5a144dac096fa1541 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1824267Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Pâris Meuleman <pmeuleman@chromium.org> Cr-Commit-Position: refs/heads/master@{#707298}
-
Huanzhong Huang authored
Reserve space for (lazy-loaded) cookie controls toggle Bug: 967668 Change-Id: Ia6fa8e84590c8512fcd6cb84f72f7db339b69e56 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1865222 Commit-Queue: Huanzhong Huang <huanzhong@chromium.org> Reviewed-by:
Dan Beam <dbeam@chromium.org> Cr-Commit-Position: refs/heads/master@{#707297}
-
Friedrich Horschig authored
With this CL, the peeking height is set depending on whether a second suggestion will be half visible. Additionally, various paddings and margins are adjusted to match the specs. The see the differences with layout bounds, find screenshots in the linked bug. Bug: 1009331 Change-Id: Ib522938becca2012efbd30200f8a19e6c56c5380 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1866629 Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Reviewed-by:
-
Roman Aleksandrov authored
This is a reland of 71c00bee Original change's description: > Welcome screen: Fix language list initialization > > Change ResolveLanguageListInThreadPool so that it uses AppLocale instead > of default locale if exists. > > Bug: 955397 > Change-Id: I6947dd89e297adea3fd019e38d083e45419ea282 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1704098 > Reviewed-by: Alexander Alekseev <alemate@chromium.org> > Reviewed-by: Denis Kuznetsov <antrim@chromium.org> > Reviewed-by: Roman Sorokin [CET] <rsorokin@chromium.org> > Commit-Queue: Roman Aleksandrov <raleksandrov@google.com> > Cr-Commit-Position: refs/heads/master@{#706436} Bug: 955397 Change-Id: I9b6bc855f0a87eadab42af14d5e3d797f77d6a57 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1865227 Commit-Queue: Roman Aleksandrov <raleksandrov@google.com> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Cr-Commit-Position: refs/heads/master@{#707295}
-
Michael Lippautz authored
This unblocks fully deprecating the methods that take v8::Value. Switching Blink to fully use references based on v8::Data is future work. Bug: v8:9841 Change-Id: I263ae03e9c6057b84fe84bbeff59a0bfbba01c6e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868949 Auto-Submit: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Omer Katz <omerkatz@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#707294}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/6dee176c8d7d..c6db4a035a17 git log 6dee176c8d7d..c6db4a035a17 --date=short --no-merges --format='%ad %ae %s' 2019-10-18 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/SwiftShader 6652f0b64287..a68a80a4dbf9 (1 commits) Created with: gclient setdep -r src/third_party/angle@c6db4a035a17 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 jmadill@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=jmadill@google.com Bug: chromium:None Change-Id: Ibf16bda63c21dafa64ded15626905c649dcecb15 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868969Reviewed-by:
-
Tim van der Lippe authored
With the removal of the terminal, we no longer need this experiment. BUG=1011466 Change-Id: I0c097d33d09329eae41079504d825041f082e1fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1866930 Commit-Queue: Yang Guo <yangguo@chromium.org> Reviewed-by:
Paul Irish <paulirish@chromium.org> Reviewed-by:
Yang Guo <yangguo@chromium.org> Auto-Submit: Tim van der Lippe <tvanderlippe@chromium.org> Cr-Commit-Position: refs/heads/master@{#707292}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/7bf8699dd1ce..c336dd1cb69c git log 7bf8699dd1ce..c336dd1cb69c --date=short --no-merges --format='%ad %ae %s' 2019-10-18 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 8a67b116..f29003dd (707122:707259) Created with: gclient setdep -r src/third_party/webrtc@c336dd1cb69c 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: Id55eb5681de8dd6f4336b87758b1fd2791817aaf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868910Reviewed-by:
-
Miyoung Shin authored
This CL converts AudioDecoderClient{AssociatedPtr, AssociatedPtrInfo} and AssociatedReceiver<AudioDecoderClient> in media to the new Mojo type, and uses pending_associated_remote<AudioDecoderClient> in audio_decoder.mojom. Bug: 955171 Change-Id: I0639e2bbbd97b0f5aebc958a739dae4e1769240f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1865162 Commit-Queue: Miyoung Shin <myid.shin@igalia.com> Reviewed-by:Sam McNally <sammc@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
-
CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-betty-chrome CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-betty-pi-arc-chrome CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-eve-compile-chrome CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-kevin-compile-chrome BUG=762641 TBR=chrome-os-gardeners@google.com Change-Id: I897849bfeace4d4c0396261e1e92d370a872d15b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868511Reviewed-by:
ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> Commit-Queue: ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#707289}
-
Leo Zhang authored
IME decoder shared library has its own threads. In order to make sure all the child theads post message via mojo safely, we must post all of these tasks to MainTaskRunner. Bug: 837156 Change-Id: I564662bfae078844da47628ad95560bdc160e6a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868532 Commit-Queue: Leo Zhang <googleo@chromium.org> Auto-Submit: Leo Zhang <googleo@chromium.org> Reviewed-by:
Marti Wong <martiw@chromium.org> Cr-Commit-Position: refs/heads/master@{#707288}
-
Yuta Kitamura authored
This reverts commit f2baca71. Reason for revert: This CL is triggering a CHECK failure in a few tests in ash_unittests. Failed tests: ShelfApplicationMenuModelTest.VerifyContentsWithMenuItems ShelfApplicationMenuModelTest.VerifyContentsWithNoMenuItems ShelfApplicationMenuModelTest.VerifyHistogramBuckets ShelfApplicationMenuModelTest.VerifyHistogramOnExecute ShelfViewInkDropTest.ShelfButtonWithMenuPressRelease Started to fail since: https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-dbg/14866 (Just look at ash_unittests, other failures are irrelevant) Crash log sample: [ RUN ] ShelfViewInkDropTest.ShelfButtonWithMenuPressRelease [15893:15893:1017/174420.948496:6297578335:FATAL:simple_menu_model.cc(561)] Check failed: item.command_id >= 0 (-2 vs. 0) Original change's description: > Menu Model: fix bug where menu item title was greyed out because it is disabled. > > The old approach of adding title as a disabled item meant that this CL: > > https://chromium-review.googlesource.com/c/chromium/src/+/1788198 > > causes the menu title to be displayed in grey text instead of solid black text. This CL introduces a new menu type that, while still disabled and non-reactive, be displayed in solid black text. > > Bug: 1006203 > Change-Id: I9fa38132160228ebde5bd43d8db9c2f60d5f001c > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1841583 > Reviewed-by: Scott Violet <sky@chromium.org> > Reviewed-by: Toni Baržić <tbarzic@chromium.org> > Commit-Queue: Yulun Wu <yulunwu@chromium.org> > Auto-Submit: Yulun Wu <yulunwu@chromium.org> > Cr-Commit-Position: refs/heads/master@{#707147} TBR=tbarzic@chromium.org,sky@chromium.org,msw@chromium.org,newcomer@chromium.org,yulunwu@chromium.org Change-Id: I39a3bebb393d70e1dbeb6da61908f7ec2e3e9b2e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1006203 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1869050Reviewed-by:
-
Victor-Gabriel Savu authored
Remove WilcoDtcSupportdClient from DBusThreadManager and use a feature check to determine if initialization is required. Moving the files is required to be able to access ::features::kWilcoDtc. Bug: 1014062 Test: unit_tests --gtest_filter="WilcoDtcSupport*" Test: on device with diagnostics_dpsl_demo_vm Change-Id: Ia83b91495cfab32a05a9642a871a4cdd8b13392e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1865156 Commit-Queue: Victor-Gabriel Savu <vsavu@google.com> Reviewed-by:
Oleh Lamzin <lamzin@google.com> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Polina Bondarenko <pbond@chromium.org> Cr-Commit-Position: refs/heads/master@{#707286}
-
Miyoung Shin authored
This CL converts CameraHalClient{Ptr, Request} in media and components to the new Mojo type, and uses pending_remote<CameraHalClient> in cros_camera_service.mojom and camera.mojom. Bug: 955171 Change-Id: I33fc6c386f1fe8dc205a8ea76d5afc0782c7996b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1862924 Commit-Queue: Miyoung Shin <myid.shin@igalia.com> Reviewed-by: -
Chih-Yu Huang authored
We plan to create another class to replace VideoFrame usage inside the CrOS VideoDecoder implementations. In order to output another type to the client, the current CrOS VideoDecoder implementations have to inherit a new interface. This CL adds a new interface VideoDecoderPipeline::DecoderInterface, and changes CrOS VideoDecoder implementations to inherit this. BUG=chromium:998413 TEST=run video_decode_accelerator_tests on Eve and Kevin with dcheck_always_on = true Change-Id: I822014474e2cc708fdfc96e30ec4be99c0b4ac8e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1777446 Commit-Queue: Chih-Yu Huang <akahuang@chromium.org> Reviewed-by:Hirokazu Honda <hiroh@chromium.org> Reviewed-by:
Alexandre Courbot <acourbot@chromium.org> Cr-Commit-Position: refs/heads/master@{#707284}
-
