- 27 Aug, 2019 40 commits
-
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/f01008997555..c445c7d5c87b git log f01008997555..c445c7d5c87b --date=short --no-merges --format='%ad %ae %s' 2019-08-27 senorblanco@chromium.org Dawn: Remove spirv-tools and spirv-headers rules from Dawn build files. 2019-08-27 mtklein@google.com remove unused sk_sp comparison operators 2019-08-27 michaelludwig@google.com Fix long line Created with: gclient setdep -r src/third_party/skia@c445c7d5c87b 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 stani@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=stani@google.com Bug: None Change-Id: Iccdeaf45fad85d62e97d26091b70257e965dbe37 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773356Reviewed-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@{#690806}
-
David Bokan authored
Normally, scrolls started from a position: fixed element will chain up to the "viewport", reglardless of DOM ancestry. This is so that scrolling over a banner/menu scrolls the page. The situation is a little different in the root scroller case. There, the "viewport" is composed of the visual viewport, and a non-default layout viewport. e.g. <body> <div id='myrootscroller'>...</div> <div id='fixed'></div> </body> In this case, "viewport" scrolling will scroll |myrootscroller|. In the above case, scrolling over |fixed| shouldn't cause scrolling in |myrootscroller| since that would change semantics on the page which isn't acceptable for an implicitly applied feature (e.g. this is an interop problem). However, in BGPT the scroll_tree_index on a fixed cc::Layer is made to point to the ScrollNode for the nearest ScrollTranslationTransformNode which will always be the visual viewport's, since the Fixed layer translates with the visual viewport. In the compositor, we unconditionally treat "inner/visual viewport" scrolls as scrolling using cc::Viewport which will have the behavior of scrolling both visual and layout viewports (it will scroll |myrootscroller| above). This CL makes two conceptual changes: -The compositor no longer unconditionally scrolls the inner ScrollNode using cc::Viewport. Whether or not the inner ScrollNode should scroll with the cc::Viewport now depends on a Blink-set bit |prevent_viewport_scrolling|. We scroll using cc::Viewport only when this bit is false. The change in LayerTreeHostImpl::TryScroll to scroll the outer viewport even if it has no extent is to compensate for the fact that we no longer unconditionally fallback to viewport scrolling. -Set this bit in Blink during pre-paint based on whether or not we have a non-default root scroller. That is, we only want to provide full viewport scrolling in this case if the root scroller hasn't been changed. This also required some small fixes to tests, in particular, some tests use an awkward layer tree where the outer viewport has no srolling and the visual viewport is scrollable. Inner/Outer scrolling is unusual in these cases because we typically want to scroll both at the same time using cc::Viewport. Changing to explicitly scroll inner only meant these tests break. I've changed them by adding a real scrolling layer as a child of the outer. For more details see: https://docs.google.com/document/d/1bp7nQq76I1HgtWW6fjfTFo__7Sy5XTUsigS-WHpC3B0/edit Note: This is still an issue in main thread scrolling (added tests fail there). I don't believe this is worth fixing there since it would require major changes to allow adding a non-DOM element to the scroll chain and the provided value is low (root scroller is currently only active on Android where main thread scrolling is rare). This should be fixed by scroll unification. Bug: 977954 Change-Id: I644e0d618a21baced50b7b99d543bd517adbd91a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1752866Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#690805}
-
Xida Chen authored
Right now all the FrameSequenceTrackers are owned by LayerTreeHostImpl, moreover, LayerTreeHostImpl also owns a FrameSequenceTrackerCollection. This CL makes FrameSequenceTrackerCollection owns all the trackers, any operation on the trackers can be done throughput FrameSequenceTrackerCollection. This CL should not cause any behavior change. Bug: None Change-Id: If8b15990ed6c1664af8eaa9c81c1a2888cf40ee7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769025Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#690804}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/605f863173b6..a9da772f0587 git log 605f863173b6..a9da772f0587 --date=short --no-merges --format='%ad %ae %s' 2019-08-27 chrisforbes@google.com Pass draw-to-draw varying state directly to Renderer::draw 2019-08-27 chrisforbes@google.com Do not expose extensions for KHR_shader_draw_parameters and KHR_variable_pointers Created with: gclient setdep -r src/third_party/swiftshader@a9da772f0587 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Please CC swiftshader-team+autoroll@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=swiftshader-team+autoroll@google.com Bug: chromium:b/132280877 Change-Id: I5b0f1ded5be153b46704afe4630a948c8aed9334 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773339Reviewed-by:
-
Min Qin authored
On Q, we never call addCompletedDownload. As a result, there is no download ID, and querying the downloadmanager will always fail. This cause clicking the notification to always open download home, instead of the download itself. BUG=997346 Change-Id: I99ca3d02ce918281ea823785a783c88f3c8df366 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769630 Commit-Queue: Min Qin <qinmin@chromium.org> Reviewed-by:
Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#690802}
-
Peter Kasting authored
* Use constexpr * Use in-declaration initializers * Factor SetSize() out of ProcessInfoHeader(), to facilitate subsequent change that adds color profile decoding between the two. * Factor out a new helper, InitFrame(), to make DecodeBMP() more readable * Reorder DecodePixelData() to follow the class convention of "declare and define methods roughly in called order" * Be consistent about updating |decoded_size_| at the end of helpers, to make it harder to add changes mid-helper that intend to be run repeatedly but aren't (ran into this while adding color profile support) * Functional change: in theory an .ico could contain an OS/2 2.1x bitmap; change from DCHECK() failure on this to setFailed() * Detect top-down BMPs at the same time we're doing other things with the height field * Added some SCOPED_TRACEs to the test code to help track down failures I was getting while writing this Bug: none Change-Id: I5d9a97cdeda66a0236c557cf6345ab8bbea55de6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772282Reviewed-by:
Leon Scroggins <scroggo@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Auto-Submit: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#690801}
-
Kevin Qin authored
This is the second part of bug 976446 where first part only updates eye and headset related parameters, this cl updates stage related parameters Bug: 976446 Change-Id: I6f18a522a2f929e6c0f1e871eaa7bee85265fd75 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762587 Commit-Queue: Zheng Qin <zheqi@microsoft.com> Reviewed-by:
Alexander Cooper <alcooper@chromium.org> Cr-Commit-Position: refs/heads/master@{#690800}
-
Sam Maier authored
cpu_features GN target moved from android_sdk to android_ndk TBR: minor rename of target Bug: 990407 Change-Id: I82e25a24716d73e17f2e19dd5e83e94b636b8b0f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1756203 Auto-Submit: Sam Maier <smaier@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Sam Maier <smaier@chromium.org> Cr-Commit-Position: refs/heads/master@{#690799}
-
Naoki Fukino authored
This CL depends on issue http://crrev.com/c/1763553 to have a preference prefs::kArcVisibleExternalStorages. (will rebase later) prefs::kArcVisibleExternalStorages remembers a list of visible external storage's UUIDs. "Visible" means that the storage is visible to Android apps. (i.e. Android apps can use the external storage to read/write files.) With this CL, Chrome checks visibility of newly mounted storages by reading the preference, and tell the visibility to ARC container using extended MountPointInfo. In addition, when the remembered UUID list changes, all external mount points will be remounted in ARC container to make sure the latest visibility is applied. Bug: 954228 Change-Id: Ief8bd50881462f7e5ae0677883ac119eacf33f63 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765099 Commit-Queue: Naoki Fukino <fukino@chromium.org> Reviewed-by:
Jorge Lucangeli Obes <jorgelo@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Cr-Commit-Position: refs/heads/master@{#690798}
-
Sadrul Habib Chowdhury authored
BUG=986679 Change-Id: I60a17ce118f64291d9bc263ad587c87a651b075a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764560Reviewed-by:
Juan Antonio Navarro Pérez <perezju@chromium.org> Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org> Cr-Commit-Position: refs/heads/master@{#690797}
-
Xing Liu authored
This CL adds a few metrics to track the behavior of background tasks for notification scheduling system. TBR=nikunjb@chromium.org Bug: 963298 Change-Id: Ic6107269db6eb5fbd70bef2836a18336daef55be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762954Reviewed-by:
Hesen Zhang <hesen@chromium.org> Commit-Queue: Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#690796}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/76a93c3325f5..5c81d94fa362 git log 76a93c3325f5..5c81d94fa362 --date=short --no-merges --format='%ad %ae %s' 2019-08-27 geofflang@chromium.org Fix a typo in the GL_NUM_REQUESTABLE_EXTENSIONS_ANGLE enum value. Created with: gclient setdep -r src/third_party/angle@5c81d94fa362 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 cwallez@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=cwallez@chromium.org Bug: None Change-Id: I5ba651da99ae31a43b9971e3573b94d1641f433c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773338Reviewed-by:
-
Christopher Cameron authored
AppShimHostManager doesn't actually manage AppShimHosts (the ExtensionAppShimHandler does, confusingly). Rather, it listens for incoming connections from app shim processes. Rename the files and structures to reflect this. Bug: 982024 Change-Id: I0c1e7031da57157da7dfc5bd5dc0c6743a6a89bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772356Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: ccameron <ccameron@chromium.org> Cr-Commit-Position: refs/heads/master@{#690794}
-
Ayu Ishii authored
This change fixes flaky failures for SMS browsertests. Tests were previously failing due to the race condition of tests ending before the UKM had been recorded. Verified by running the browsertests consecutively 100 times without any failures. Command: testing/xvfb.py out/ChromeOs/content_browsertests --gtest_filter="SmsBrowserTest.*" --gtest_repeat=100 Bug: 997549 Change-Id: Ic15025412bf6aa766e9443a07a70c2b489a07e93 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772451 Commit-Queue: Ayu Ishii <ayui@chromium.org> Reviewed-by:
Jun Cai <juncai@chromium.org> Cr-Commit-Position: refs/heads/master@{#690793}
-
David Maunder authored
This is the first CL containing our work to assist the user in closing stale/unused tabs. Bug: 993826 Change-Id: Ib73d7a18ea9f5415289ef3c73cd79cc80f97c3be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1753689Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Commit-Queue: David Maunder <davidjm@chromium.org> Cr-Commit-Position: refs/heads/master@{#690792}
-
Daniel Rubery authored
If the BinaryUploadService attempts to read the file before the download code has renamed it to its final location, we can end up having a race condition between the renaming and reading the file contents. So the BinaryUploadService will now wait for the file to be renamed before reading the contents. We'll have to revisit this decision when launching a blocking mode, since then the file isn't renamed until after the upload. Bug: 980784 Change-Id: I5228e8610df9d5db7e84fbfbc2bb53bea79a9e9a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769677Reviewed-by:
Varun Khaneja <vakh@chromium.org> Commit-Queue: Daniel Rubery <drubery@chromium.org> Cr-Commit-Position: refs/heads/master@{#690791}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/9d5b5c3f49f3..a203388eab27 git log 9d5b5c3f49f3..a203388eab27 --date=short --no-merges --format='%ad %ae %s' 2019-08-27 lalitm@google.com Merge "trace_processor: move more files to the sqlite folder" Created with: gclient setdep -r src/third_party/perfetto@a203388eab27 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@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=perfetto-bugs@google.com Bug: None Change-Id: Ia88663610582d682fc6114f4672742ad2f8dc55b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773341Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/ede169e1..d7cac7cb 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: I610cc31afb0088129233a94379e74e0d212c056b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772553Reviewed-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@{#690789}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/c74d7abaa6bd..9f4081b4ef9b Created with: gclient setdep -r src-internal@9f4081b4ef9b 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,chromium:None Change-Id: I72977643b356e0633552a85c7e44a5f7c2adafd1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773358Reviewed-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@{#690788}
-
Sergey Ulanov authored
Previously all web_engine processes had the same name, which made it hard to distinguish between processes. Now each process started by web_engine will have the name suffixed with process type, e.g. web_engine_exe:renderer. Change-Id: Ic1fa0f4f7bc87f4789d0d6d85d71d836d89df99f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769996 Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Auto-Submit: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#690787}
-
Kush Sinha authored
|AccountManagerMigrator| uses |AccountMigrationRunner| for actually running its migrations. A given instance of |AccountMigrationRunner| silently ignores multiple calls to its |Run| API, which makes it impossible to run migrations multiple times. Fix this by re-creating the |AccountMigrationRunner| object when a migration re-run is required. Bug: 998111 Change-Id: Iead9ce954a5488c5ed3844bbe1077a92c9d80b73 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769453 Commit-Queue: Kush Sinha <sinhak@chromium.org> Reviewed-by:
Alexander Hendrich <hendrich@chromium.org> Cr-Commit-Position: refs/heads/master@{#690786}
-
Stephen McGruer authored
Bug: 900581 Change-Id: Ieda118b47ebb266e7e52db20055de6664ad171bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773183Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Stephen McGruer <smcgruer@chromium.org> Cr-Commit-Position: refs/heads/master@{#690785}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/4271afbc301e..44bd29a3b068 git log 4271afbc301e..44bd29a3b068 --date=short --no-merges --format='%ad %ae %s' 2019-08-27 sakal@webrtc.org Detect leaks of TextureBufferImpl objects. 2019-08-27 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 925c16d3..abb1ee24 (690586:690691) Created with: gclient setdep -r src/third_party/webrtc@44bd29a3b068 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:b/139745386,chromium:None Change-Id: I0ff411e2f9bdffcb7fae6b0ef8d52a2d5c6fa7ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769340Reviewed-by:
-
Allen Bauer authored
Bug: 651648 Change-Id: Ice5dac45a04acfd7e69232e5114c8cc17c302e71 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773321 Commit-Queue: Allen Bauer <kylixrd@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Auto-Submit: Allen Bauer <kylixrd@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#690783}
-
Mike Jackson authored
In the event there are multiple targets on page, clearing the storage via the applications tab can result in: console.assert(this._databaseNamesBySecurityOrigin[securityOrigin]); firing in Resources.IndexedDBModel.clearForOrigin because only the top level security origin is passed to each target, and the security origins might not match. Repro steps: 1) Load http://nparashuram.com/trialtool/#example=/IndexedDB/trialtool/index.html&selected=#saveData 2) Run the following examples: "Pre Requisities", "Open Database", "Create Object Store", "Add Data" 3) Load the devtools, and clear all site data from the application tab Change-Id: Iccad7f70c61eb6593058280f971849f139f473a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1716445 Commit-Queue: Mike Jackson <mjackson@microsoft.com> Reviewed-by:
Erik Luo <luoe@chromium.org> Reviewed-by:
Jeff Fisher <jeffish@microsoft.com> Cr-Commit-Position: refs/heads/master@{#690782}
-
Doug Arnett authored
Attempting a closer histogram to wait on. Was able to run locally 2000 times without failure whereas previous histogram did fail locally after a couple hundred attempts. Bug: 997697 Change-Id: I1f826945091f6e2fce035755fe62ba781f219adf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1771045Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Doug Arnett <dougarnett@chromium.org> Cr-Commit-Position: refs/heads/master@{#690781}
-
Kush Sinha authored
Chrome OS Account Manager migrator currently does not expose the results of its migration runs. Add a |AccountManagerMigrator::GetLastMigrationRunResult|. This will be used to retry migrations in ARC. Bug: 998111 Change-Id: I528c9fe04d144d3c883aebc45f1cd30da1e52bdb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773218Reviewed-by:
-
Erik Chen authored
All calls to GetWebWidget() should either DCHECK that the WebWidget exists, or else have a conditional with appropriate behavior if the WebWidget does not. Bug: 995981 Change-Id: I32e148efe04c5a889b0b26ad1d7a6ca9eba8752d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1771052 Auto-Submit: Erik Chen <erikchen@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#690779}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast TBR=cr-fuchsia+bot@chromium.org Change-Id: I57e2099af02b1ba20075d05def0fed08abf73843 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773317Reviewed-by:
-
Clifford Cheng authored
Bug: 997947 Change-Id: I26f2be79db4137686a37ec4968269c6ec4b72a18 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769849Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Clifford Cheng <cliffordcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#690777}
-
Adam Bujalski authored
This patch fixes one of three memory leaks found when page reloads NaCl module(s). In mentioned scenario HTMLPlugin element is added to LocalFrameView in LoadPlugin method, however there is no way to remove plugin from LocalFrameView. Bug: 922925 Change-Id: I1bf01f417f24bcbd82d5289a2138e3a69c31109f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1680175 Commit-Queue: Nate Chapin <japhet@chromium.org> Reviewed-by:
Nate Chapin <japhet@chromium.org> Cr-Commit-Position: refs/heads/master@{#690776}
-
Jenny Blessing authored
This specifically reduces duplication between ClearSiteDataHandlerBrowserTest and SameSiteDataRemoverBrowserTest related to creating and retrieving service workers. Bug: 987177 Change-Id: Iaa5f772e61e0ca73cceaff56fdaf9c370390dfeb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1751202 Commit-Queue: Jenny Blessing <jblessing@google.com> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Reviewed-by:
Martin Šrámek <msramek@chromium.org> Cr-Commit-Position: refs/heads/master@{#690775}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 31d22c06. With Chromium commits locally applied on WPT: 1194775a "Add ability to block same-origin access via document-access feature policy" 5546163a "Temporarily send `Sec-Fetch-*` headers via non-secure transport." d3b97055 "Update simulateUserActivation to match webxr-test-api spec" 2d090a27 "[WPT/referrer-policy] Call getRequestURLs() for each subtest" 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=robertma No-Export: true Change-Id: I6fe5717614005263ad95fd96522b53240751dd62 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772548Reviewed-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@{#690774}
-
Toby Huang authored
ExtensionWebstorePrivateApiTestChild in webstore_private_apitest.cc has been using legacy switches to log in as supervised user for the InstallBlocked test. This CL converts the test class to use child accounts instead. Bug: 990542 Change-Id: Ib704b601e23a9f815328c8c5cfbe621f7bf62ba5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1752528Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Reviewed-by:
Toni Baržić <tbarzic@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Toby Huang <tobyhuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#690773}
-
Anirudh Tadakamala authored
Add a retry logic with number of retries set to 1 for AudioTrack creation. This is to prevent the intermittent issue of alarm sound not going off from occuring. From stack traces it looked like getOutputForDevice in AudioPolicyManager couldn't find an output at that time. We couldn`t repro this consistently and there was no pattern leading to the conclusion that it`s transient in use case. In the cases we saw, it seemed to happen during some race with other output. From digging in to the Android source this is totally possible from the HW layer. Adding a single retry to try to break this case so the user experience is improved. There are other (leak) cases with the number of system audio tracks possible as well, but that wasn't this use case (and owners releasing shared resources correctly is the only true fix there anyhow). Bug: 130373632 Change-Id: If5f3a2fcd2f3e50dd86d864638546e839913f6d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1747028Reviewed-by:
Kenneth MacKay <kmackay@chromium.org> Commit-Queue: Anirudh Tadakamala <antadaka@google.com> Cr-Commit-Position: refs/heads/master@{#690772}
-
Piotr Pawliczek authored
The libipp is a new library to parse & build IPP frames. IPP = Internet Printing Protocol. BUG=chromium:902085 TEST=on local chrome build Change-Id: I2952870b332cc1ce456eb418bc1f8e5104fc6364 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769161Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Sean Kau <skau@chromium.org> Commit-Queue: Piotr Pawliczek <pawliczek@chromium.org> Auto-Submit: Piotr Pawliczek <pawliczek@chromium.org> Cr-Commit-Position: refs/heads/master@{#690771}
-
Lambros Lambrou authored
In ClientSession, lossless-color defaulted to true. But the actual encoders defaulted this setting to false. Now that the lossless flags have been plumbed through to the encoders, the default setting has been changed from false to true. This CL restores the default setting to false, since lossless-color is not yet working for VP9. VP8 ignores the flags anyway, so is not affected. The command-line option is also removed. The lossless flags can be controlled via VideoControl messages if needed. Change-Id: I14637cdff74be5671aaa53ffd9f00b70302b55cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1768879 Auto-Submit: Lambros Lambrou <lambroslambrou@chromium.org> Commit-Queue: Jamie Walch <jamiewalch@chromium.org> Reviewed-by:
Jamie Walch <jamiewalch@chromium.org> Cr-Commit-Position: refs/heads/master@{#690770}
-
Tommy Steimel authored
While working on crrev.com/c/1771937 to split by profile, it was clear that MediaToolbarButtonController was going to have to be refactored to be an AudioFocusObserver and would essentially have to do the same tracking that MediaDialogController currently does. Instead of having this tracking in two places, I've instead refactored the two controllers into one. Bug: 992463 Change-Id: Ie424044e8ee678effe869a131347ba7869157624 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772421Reviewed-by:
Becca Hughes <beccahughes@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#690769}
-
Becca Hughes authored
This crash is still appearing and I think |owner_| might be going away and cause the seg fault. This removes |owner_| from MediaNotificationBackground. BUG=994395 Change-Id: I59364ea14e9e7614728dac1e227133bcf76733c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772375Reviewed-by:
Tommy Steimel <steimel@chromium.org> Commit-Queue: Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#690768}
-
Vasiliy Telezhnikov authored
This CL fixes inconsistency between GMB (and hence ShareImage) format for video frame resources and TrasferableResource format. Bug: 988120 Change-Id: I77305a8bb5bcf1cefd6a5696d6f4b5e41e5311e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1768936Reviewed-by:
Frank Liberato <liberato@chromium.org> Commit-Queue: Vasiliy Telezhnikov <vasilyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#690767}
-
