- 27 Aug, 2019 40 commits
-
-
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:
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@{#690795}
-
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}
-
Ioana Stefan authored
BackgroundTaskScheduler currently supports scheduling of one-off tasks and periodic tasks. The scheduling of tasks at exact moments in time will be introduced. This requires a new type of of information to be stored and a different flow and logic for scheduling. This will require AlarmManager, which will be added along the back-end logic in a future CL. This CL makes the changes related to the new information type. A new TimingInfo implementation is added, in order to store exact data. The TimingInfoVisitor implementations are updated. Changes are also done to the tests. The Design Doc for this change can be found here: https://docs.google.com/document/d/1o9BS5qwv10QVN3Df6W2m4QKjSo1DXdGIgLf8p2l0PYc/edit?usp=sharing. Bug: 970160 Change-Id: Ifd6cf89edeb2d9380b96bc095969d5875e8d8133 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1751190Reviewed-by:
Mark Pearson <mpearson@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Reviewed-by:
Mugdha Lakhani <nator@chromium.org> Commit-Queue: Ioana Stefan <ioanastefan@chromium.org> Cr-Commit-Position: refs/heads/master@{#690766}
-
Himanshu Jaju authored
Updates the omnibox icon behaviour when sending SharingMessage Screenshot of text for translation : https://drive.google.com/file/d/1KmamVMMuhhX5skrxZvM1AhQshVK-ETxM/view?usp=sharing Video of animation : https://drive.google.com/file/d/1T9zrhBG-kMmZ6igkYKU-MybW9mjPlihh/view?usp=sharing Bug: 995694 Change-Id: I96af8098e70cd92867e1aa114f6617ec20b88581 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765749 Commit-Queue: Himanshu Jaju <himanshujaju@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Alex Chau <alexchau@chromium.org> Cr-Commit-Position: refs/heads/master@{#690765}
-
Steven Bennetts authored
The remaining instances of CrNetworkPolicyBehavior either use the 'Mojo' versions of the behavior, or are incorrect. This replaces them with CrNetworkPolicyBehaviorMojo. A follow-up could rename CrNetworkPolicyBehaviorMojo to just CrNetworkPolicyBehavior, but we should hold off until the mojo migration is complete before doing so. Bug: 853953 Change-Id: I9265eef9bde070d603ce6b2f88e6cc4d1b91fd5f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769596 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#690764}
-
Jan Krcal authored
The test is disabled due to high flakiness. TBR: treib@chromium.org Bug: 997912 Change-Id: Ifa5e1cc89354163622a6a6ca593498c6fe3a6028 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1771974 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Reviewed-by:
Jan Krcal <jkrcal@chromium.org> Cr-Commit-Position: refs/heads/master@{#690763}
-
Alex Turner authored
DoneHeadersAddToEntryComplete assumes that the previous result is OK. However, it may be ERR_CACHE_RACE in some situations, resulting in a crash. Converting to a CHECK would verify that this is the cause of some crashes observed in the wild. Bug: 959194 Change-Id: Ib989e85b5334667875e8669f442f32c274b0d23c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773163Reviewed-by:
Josh Karlin <jkarlin@chromium.org> Commit-Queue: Alex Turner <alexmt@chromium.org> Cr-Commit-Position: refs/heads/master@{#690762}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/a7c5d0734c18..9d5b5c3f49f3 git log a7c5d0734c18..9d5b5c3f49f3 --date=short --no-merges --format='%ad %ae %s' 2019-08-27 primiano@google.com Merge "Minor cleanups to build files" Created with: gclient setdep -r src/third_party/perfetto@9d5b5c3f49f3 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: Ib8ecce1ec6b127865d68e4fcdfcd4e244db3adb2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773276Reviewed-by:
-
Joe Laughlin authored
startup BUG: 992003 Change-Id: Idac6ddfc0870fbe8f7a2bc7211b87b0ee5bb9c1f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1741782 Commit-Queue: Joe Laughlin <joel@microsoft.com> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Reviewed-by:
Alex Ilin <alexilin@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#690760}
-
Sammie Quon authored
Fix issue with scrolling not working after rotating. Add keyboard shortcut to scroll - values will need to match webview keyboard values later on. Fix issue with flings not working on overview items. Fix issue with flings bouncing the grid occasionally. Test: manual, added tests Bug: 978112 Change-Id: I6ff805c321f3d59506e239a28f198211d94cde56 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762575Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Commit-Queue: Sammie Quon <sammiequon@chromium.org> Cr-Commit-Position: refs/heads/master@{#690759}
-
David Bokan authored
These inject fling events directly into Blink. Since we've moved fling into the browser this is no longer how Chrome works. The tests that used this are all marked Skip so we'll remove these methods to avoid confusion. The (skipped) tests still using this API are: fast/events/touch/gesture/pad-gesture-cancel.html fast/events/touch/gesture/pad-gesture-fling.html fast/events/touch/gesture/touch-gesture-fling-with-page-scale.html fast/events/wheel/mainthread-touchpad-fling-latching.html fast/events/wheel/wheel-fling-cancel.html Bug: 950314,857490 Change-Id: I7fe94e5654cc3d7c0769102989919d5a441d4f5a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773196 Commit-Queue: David Bokan <bokan@chromium.org> Auto-Submit: David Bokan <bokan@chromium.org> Reviewed-by:
-
