- 27 Aug, 2020 40 commits
-
-
Randy Rossi authored
Chromecast implementation of ExtensionsBrowserClient can return null for GetExtensionResourceManager. Added a check to return early if null. Bug: internal b/166307109 Test: Local display assistant build with screen reader extension Change-Id: I61e3a1a55026d531e1947cd0d80e59e57d386a67 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2372890 Commit-Queue: Randy Rossi <rmrossi@chromium.org> Reviewed-by:
Ryan Daum <rdaum@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Cr-Commit-Position: refs/heads/master@{#802374}
-
zhaoyangli authored
The pool is going to be removed. Bug: 1119885 Change-Id: I5dc5511a8078e305a139c07fd572ab2fac373fe6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2378777Reviewed-by:
Garrett Beaty <gbeaty@chromium.org> Commit-Queue: Zhaoyang Li <zhaoyangli@chromium.org> Cr-Commit-Position: refs/heads/master@{#802373}
-
Sahel Sharify authored
This cl downloads and encodes instrument icon for SPC before it is written to db. Bug: 1121021, 1121331 Change-Id: Ifa875ad6a42380fb5f7a7790022f4dd17b835122 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380142 Commit-Queue: Sahel Sharify <sahel@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#802372}
-
Tom Anderson authored
When running under a VNC session, the connection setup data sometimes has duplicated depths, like so: depth=24: visuals={123,456} depth=24: visuals={789} Previously, we were searching for the unique depth that matches the root depth, then searching for the visual within the list corresponding to that depth. However, since the depths may not be unique, search through all depths until we find a matching visual ID, and then use that to set both the default depth and default visual. R=sky Change-Id: I51f75260f584f85f2bdc93048e6c4848cf00997a Fixes: 1121438, 1115929 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2375888Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Auto-Submit: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#802371}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/80848e3b..81f3d945 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,vahl@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ide9420d14b8d9271c462bd9f9065050aa735dfe2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2379633Reviewed-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@{#802370}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/faa844b3686c..532a264c141d 2020-08-27 herb@google.com use bit 15 to encode page information in glyph UV 2020-08-27 adlai@google.com Add GrDirectContext arg to SkImage::readPixels 2020-08-27 csmartdalton@google.com Don't pre-scale tessellated strokes by the matrix scale 2020-08-27 brianosman@google.com Run glnarrow/glesnarrow on GPU perf bots 2020-08-27 reed@google.com use pathbuilder 2020-08-27 brianosman@google.com Make SkRuntimeShaderBuilder safe for reuse 2020-08-27 johnstiles@google.com Improve control-flow support in early-return detection. 2020-08-27 bsalomon@google.com New helpers for SkYUVAInfo when used with pixmaps. 2020-08-27 bsalomon@google.com Remove do-nothing MPD benchmarking from nanobench 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 mtklein@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/+doc/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 Cq-Do-Not-Cancel-Tryjobs: true Bug: None Tbr: mtklein@google.com Change-Id: Ie546955976a0d4665307f374713fe276c3c9aa3b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380378Reviewed-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@{#802369}
-
Steven Bennetts authored
This is a fix for logic introduced in: https://chromium-review.googlesource.com/c/chromium/src/+/2350315 It also includes a unit test that would have caught the regression. BUG=1112814 Change-Id: I057f01f29892278231f36e6abb5334692e8ea401 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2378459Reviewed-by:
Azeem Arshad <azeemarshad@chromium.org> Reviewed-by:
Jon Mann <jonmann@chromium.org> Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#802368}
-
Mugdha Lakhani authored
Currently, the network prediction user preference allows link rel="prerender" to continue because disabling it doesn't really prevent web developers from fetching these resources by other means, like a regular fetch. The same argument should apply to link rel="next", which is what this CL does. This also affects the handling of the WebLayer network prediction setting in the same way. Bug: 1122563 Change-Id: I54517c9f12ae11f56f43a0e6c42f586492d89579 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2379890Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Commit-Queue: Mugdha Lakhani <nator@chromium.org> Cr-Commit-Position: refs/heads/master@{#802367}
-
Ben Mason authored
TBR=govind@chromium.org Change-Id: Ib8d61a2f38dc6a083bd6510475d2bf4cdefd18fb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380077Reviewed-by:
Ben Mason <benmason@chromium.org> Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#802366}
-
Kalvin Lee authored
base::StringPiece erased support for its starts_with() method in https://crrev.com/c/2358745. This change modifies PpdMetadataManager to use the base::StartsWith() function instead. Bug: chromium:888189 Test: chromeos_unittests --gtest_filter='PpdMetadataManagerTest.*' Change-Id: If8fef9b2e066f852168cadcf8a02c445033e5028 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2373387Reviewed-by:
Sean Kau <skau@chromium.org> Commit-Queue: Kalvin Lee <kdlee@chromium.org> Cr-Commit-Position: refs/heads/master@{#802365}
-
Mohamed Mansour authored
While sorting the TableView the focus ring was in the incorrect spot. Upon investigation, the position was always the model_index instead of the view_index. Added a unit test to make sure that the active row index matches the view index being shown meanwhile the model index differs (which tells us it has been sorted). Previously this test will fail since the focus ring is being rendered in a different position. AX-Relnotes: A focus ring now appears around the correct cell in tables that are found in e.g. Chromium's Task Manager. Bug: 1121394 Change-Id: I6921268a7274d5f41eaec8de013e2ea90e3abdb1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2373236 Commit-Queue: Mohamed Mansour <mmansour@microsoft.com> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Peter Boström <pbos@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#802364}
-
Tom Anderson authored
These were removed in [1] because the message loop was changed to TYPE_UI, which automatically flushes. However, if there are a lot of tasks in the queue, the flush may happen later than we'd like, and this increases latency between frames. This CL adds the flushes back in an effort to fix perf regressions. [1] https://chromium-review.googlesource.com/c/chromium/src/+/2343314 R=sky BUG=1119639 Change-Id: I04d6854fce7df69830f421dd81d734a14ae6d5a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2377609 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Auto-Submit: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#802363}
-
chromium-autoroll authored
Roll ChromeOS Broadwell AFDO profile from 86-4183.59-1598262024-benchmark-86.0.4240.8-r1 to 86-4183.59-1598262024-benchmark-86.0.4240.11-r1 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/cros-afdo-broadwell-chromium Please CC c-compiler-chrome@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/+doc/master/autoroll/README.md Tbr: c-compiler-chrome@google.com Change-Id: I6c228452555f76d4e082e79097961eb03ed90163 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380047Reviewed-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@{#802362}
-
Mugdha Lakhani authored
Bug: 1122177 Change-Id: Iffcd5094ab99f403f1abcde908f417efaad9695d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2379830Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Commit-Queue: Mugdha Lakhani <nator@chromium.org> Cr-Commit-Position: refs/heads/master@{#802361}
-
Mike Dougherty authored
Bug: 891834 Change-Id: I2f7247085324fc1c31bd7092b51d181fbe4e2328 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2377512 Commit-Queue: Mike Dougherty <michaeldo@chromium.org> Auto-Submit: Mike Dougherty <michaeldo@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#802360}
-
Scott Little authored
This test has been consistently timing out on Mac, so this CL marks that in the TestExpectations. https://ci.chromium.org/p/chromium/builders/ci/Mac10.13%20Tests%20%28dbg%29/19844 TBR=gogerald@chromium.org Bug: 1111287 Change-Id: Ia0aeaaa6f09cae3e27544200a444e5eb6611bbc6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380424Reviewed-by:
Scott Little <sclittle@chromium.org> Commit-Queue: Scott Little <sclittle@chromium.org> Cr-Commit-Position: refs/heads/master@{#802359}
-
Tom Anderson authored
This is needed so we can reland [1] and [2]. [1] https://chromium-review.googlesource.com/c/chromium/src/+/2379674 [2] https://chromium-review.googlesource.com/c/chromium/src/+/2380370 R=thestig Change-Id: If2dbda4d38e7df58905718525918135a617e7f76 Bug: 1122702 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380572 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Auto-Submit: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#802358}
-
Michael Moss authored
This is primarily needed to bring in symupload native symbols support on Mac. 9c38ab7 Add native symbol uploads to Mac OS symupload tool. 62d9272 linux: Fix segfault when there is no section header in ELF file 087795c processor: subtract 1 from return pointers while scanning 014e842 Escape more characters in Mac OS sym-upload-v2 debug_file strings. BUG=1021742 R=nbilling@google.com, rohitrao@chromium.org Change-Id: I7bc63acfb5fb94e1c9b9f6d38801886a6225c2f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380139Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Commit-Queue: Rohit Rao <rohitrao@chromium.org> Auto-Submit: Michael Moss <mmoss@chromium.org> Cr-Commit-Position: refs/heads/master@{#802357}
-
Mike Wasserman authored
Only fire screenschange for frames with the Window Placement permission. Fire isMultiScreen changes for all visible frames (e.g. 1 <-> 2 screens) (obviates pollling isMultiScreen(), which is exposed without permission) Add a TODO to postpone events instead of dropping, and refine criteria. Update test expectations and add TODO to test events with permission. Bug: 1080690, 1109989 Test: event only fires for permissioned sites or isMultiScreen changes. Change-Id: I5b46bd6944bc4b923b6357e9b370c80ce14a09b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2375671Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#802356}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/c08c71bedfbb..7d98e228f836 2020-08-27 glenrob@chromium.org Allow 'main' as a local main branch name. 2020-08-27 sweilun@chromium.org Also Pretty print the split histogram_suffixes_list If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@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/+doc/master/autoroll/README.md Bug: chromium:1121852 Tbr: ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com Change-Id: Ifb50a0b64aa1c8594a4ad9755baed24ec08fae4d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380530Reviewed-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@{#802355}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/1f580a97e55f..3cdde9771d8f 2020-08-27 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 824b4e64..55f2a783 (802144:802244) 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/+doc/master/autoroll/README.md Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: I8bee58ed2e937bf4303176b1773a4fd8bf987059 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380336Reviewed-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@{#802354}
-
Nnamdi Theodore Johnson-Kanu authored
- Add news strings for eSim UI - Adds new UI elements to esim_flow_ui.html - Current UI: https://imgur.com/zdfcmbx - UI spec https://docs.google.com/presentation/d/1Mfk5ZMEmeHwSbadUbyRU0Si5k6XDIvw-j4u7TUaCIGY/edit#slide=id.g921c12802d_0_1 Bug: 1093185 Change-Id: Iadf0b8a411d93d59c1782d103f34c96442092481 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2351392 Commit-Queue: Nnamdi Theodore Johnson-kanu <tjohnsonkanu@google.com> Reviewed-by:
Azeem Arshad <azeemarshad@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#802353}
-
Jonathan Backer authored
There are crash reports and performance issues that remain uninvestigated. Disable for now until we can fix that. Bug: 1122650 Change-Id: I09fb05de39b5e3b5e3c26a8e0dbb90cbcffea4ae Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380374 Auto-Submit: Jonathan Backer <backer@chromium.org> Reviewed-by:
Peng Huang <penghuang@chromium.org> Commit-Queue: Jonathan Backer <backer@chromium.org> Cr-Commit-Position: refs/heads/master@{#802352}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/cc61e5a98324..ed2e4739280f 2020-08-27 treehugger-gerrit@google.com Merge "Fix missing include." 2020-08-27 oksamyt@google.com Merge "[trace processor] Add core type distribution to WV power metric" 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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:linux-perfetto-rel Bug: None Tbr: perfetto-bugs@google.com Change-Id: I0f5260bda2fb427909033989f1df4156d5dcef4a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380373Reviewed-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@{#802351}
-
Xianzhu Wang authored
This is because this blink runtime feature doesn't work by itself. It's controlled by the corresponding Chromium feature which needs to be enabled to make the whole feature work. We'll continue to use the virtual test suites to test the feature. Bug: 1121839 Change-Id: I902cf87e8fba8e1123dd4a6f5e6fe8c7f821f524 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2378631Reviewed-by:
Alison Maher <almaher@microsoft.com> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#802350}
-
Rouslan Solomakhin authored
Before this patch, if WebAuthn returned an error when creating an assertion, Web Payment called into ShowErrorMessage(), which called into OnCancel(), which rejected the PaymentRequest.show() promise, but kept the secure payment confirmation UI open. This patch makes sure that the dialog is closed in OnCancel(). After this patch, if WebAuthn returns an error, then the secure payment confirmation UI is closed. Bug: 1122696 Change-Id: I99fae1d7b0b903338fca8cd03ecf57646dd78699 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380550 Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Auto-Submit: Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/master@{#802349}
-
Sebastien Marchand authored
This ensures that child processes don't die before flushing their profile data to disk. This is supposed to happen automatically at shutdown but it doesn't always have a chance to complete (probably because we tend to fast kill processes). This affects mostly the utility processes, and as they end up writing their data into the same file as the renderer they can sometime end up corrupting these files and make us lose some precious profiling data. This only affects profile instrumented builds (PGO profiling builds and coverage builds). Bug: 1119887 Change-Id: Ib50e876e7637a670ad05982c5ff03a898dc84ff5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2378272Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Yuke Liao <liaoyuke@chromium.org> Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org> Cr-Commit-Position: refs/heads/master@{#802348}
-
Torne (Richard Coles) authored
Some apps appear to pass null for one or more of the header names/values in WebView.loadUrl(url, extra_headers). Check for null before trying to validate the header format as Matcher doesn't permit null arguments. Fixed: 1122574 Bug: 1108405, 1105745 Change-Id: Ib2675efa78266d97b5e2c5c63b4b7f6182e089d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380269 Auto-Submit: Richard Coles <torne@chromium.org> Reviewed-by:
Nate Fischer <ntfschr@chromium.org> Commit-Queue: Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#802347}
-
Dave Tapuska authored
The "prefer compositing to LCD text" moves into the callback notification where other screen properties adjust the settings. This is ok because after the screen bounds are updating a Resize of the Widget will occur. BUG=1097816 Change-Id: I9b27af51386cb85ab8f0ecc87f60cf5ad5dd7226 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2366669 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#802346}
-
Richard Knoll authored
This is a reland of fb578abb Original CL (in PS1) had a conflict with crrev.com/c/2375250. The fix is in PS2 of this CL which was to update the API of FileAttachment to use only the |file_path| as input. Original change's description: > [Nearby] Refactor NearbySharingService Send* methods > > Instead of SendText and SendFiles methods this now has one unified > SendAttachments methods to make it easier to handle Attachments coming > in from the UI. Also hooks it up through the DiscoveryManager all the > way to NearbySharingService. File URLs coming from the ShareSheet on > ChromeOS are converted to FileAttachments. > > Bug: 1085067 > Change-Id: I61ce93842920ffa303ee71f4373c237b113948bc > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2378026 > Reviewed-by: Alex Chau <alexchau@chromium.org> > Reviewed-by: James Vecore <vecore@google.com> > Commit-Queue: Richard Knoll <knollr@chromium.org> > Cr-Commit-Position: refs/heads/master@{#802242} Bug: 1085067 Change-Id: Ia0ecb446bf926ecf1f8a497cb0e50504f5eff899 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2379862Reviewed-by:
Richard Knoll <knollr@chromium.org> Reviewed-by:
Himanshu Jaju <himanshujaju@chromium.org> Reviewed-by:
James Vecore <vecore@google.com> Commit-Queue: Richard Knoll <knollr@chromium.org> Cr-Commit-Position: refs/heads/master@{#802345}
-
chromium-autoroll authored
Roll Chrome Mac PGO profile from chrome-mac-master-1598529464-36870a872fe98f9e50202a2a325506fd4bcc24c9.profdata to chrome-mac-master-1598550957-d21440687343c81b6a2048a74a0e183994baca81.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-mac-chromium Please CC sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@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/+doc/master/autoroll/README.md Cq-Include-Trybots: chrome/try:mac-chrome Tbr: sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com Change-Id: I5e2f34c02e0112b3cce135883194de748bfe78e7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380379Reviewed-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@{#802344}
-
Wez authored
There is no longer a need to use E1000 under x64, and Fuchsia defaults to virtio-net-pic for both architectures. Bug: fuchsia:59041 Change-Id: I817bc69e84e4b024db6b9e3d8ee8fabe2502a552 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2379718Reviewed-by:
David Dorwin <ddorwin@chromium.org> Auto-Submit: Wez <wez@chromium.org> Commit-Queue: David Dorwin <ddorwin@chromium.org> Cr-Commit-Position: refs/heads/master@{#802343}
-
Ben Pastene authored
This: - Moves the dict-building bits of CmdExport() into a func called _ToJsonish(), which can be used for crbug.com/1117577. - Includes args_files in the dict that gets built. - Uses 'seperators=' arg in json.dumps() to avoid trailing whitespaces. Bug: 1117577 Change-Id: I7c6f501d75969b76cc0e684417befb41e1f40a15 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2378759Reviewed-by:
Garrett Beaty <gbeaty@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@google.com> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#802342}
-
Michael Crouse authored
The browser test times outs and causes flakes. There are 2 wait events that could cause timeouts: blocklist loaded and a histogram check. This CL uses the updated blocklist loaded provided by the delegate which which more clearly marks when the blocklist is loaded. This CL also shifts the tab closing later to remove potential race of the hint check logic with the tab closing. A future change will remove the tab closing logic that may improve this further. Bug: 1122603 Change-Id: I898096f3350c057a14661163a8b99ac04fedd299 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380153Reviewed-by:
Sophie Chang <sophiechang@chromium.org> Commit-Queue: Michael Crouse <mcrouse@chromium.org> Cr-Commit-Position: refs/heads/master@{#802341}
-
James Vecore authored
This namespace rewrite got missed in the move of nearby_share_page to be CrOS only. Change-Id: I394bbed07d9515f6c63f0468201168250e397ef9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380257 Auto-Submit: James Vecore <vecore@google.com> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: James Vecore <vecore@google.com> Cr-Commit-Position: refs/heads/master@{#802340}
-
Alice Wang authored
This CL handles the account picker bottom sheet backpress - When account list is expanded, backpress leads to collapsed account list; - In other cases, backpress dismisses the bottom sheet. Bug: 1113653 Change-Id: I4d380cf7ae27b3da03a828960224efde72857f7d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2374650Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Commit-Queue: Alice Wang <aliceywang@chromium.org> Cr-Commit-Position: refs/heads/master@{#802339}
-
Joshua Pawlicki authored
It was accidentally added the default archive in https://chromium-review.googlesource.com/c/chromium/src/+/2374513 Bug: 1058102, 1122538 Fixed: 1122538 Change-Id: I18ba7213aa72800a234a29576c5163820890c2b6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380133 Commit-Queue: Michael Moss <mmoss@chromium.org> Auto-Submit: Joshua Pawlicki <waffles@chromium.org> Reviewed-by:
Michael Moss <mmoss@chromium.org> Cr-Commit-Position: refs/heads/master@{#802338}
-
chromium-autoroll authored
Roll ChromeOS Airmont AFDO profile from 86-4183.74-1598264968-benchmark-86.0.4240.8-r1 to 86-4183.74-1598264968-benchmark-86.0.4240.11-r1 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/cros-afdo-airmont-chromium Please CC c-compiler-chrome@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/+doc/master/autoroll/README.md Tbr: c-compiler-chrome@google.com Change-Id: I5f00d9f761a3f18cb997d3c0834bda3d9261c2b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380202Reviewed-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@{#802337}
-
Xianzhu Wang authored
Previously composited scrollbars were always placed above all descnedants if they were not reordered, causing the scrollbars to cover non-scrolling stacked descendants. Now place non-overlay scrollbar layers at the bottom of the child layers, which is the same as how we paint non-composited scrollbars in the background paint phase. Bug: 984167 Change-Id: I1cd9b0bcd1559ad0811bc83af8959b33b3739b4c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2370716Reviewed-by:
Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#802336}
-
Ana Salazar authored
Holding space keyed service now observes downloads. As new downloads are completed, they are added to the holding space model. Bug: 1111995 Change-Id: Id45b276a3ada06d94f7efd42bc35cf89cc3fe070 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2378648 Commit-Queue: Ana Salazar <anasalazar@chromium.org> Reviewed-by:
Toni Baržić <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#802335}
-