- 27 May, 2020 40 commits
-
-
Sandeep Vijayasekar authored
Revert of https://chromium-review.googlesource.com/c/chromium/src/+/2145870 Bug: internal 154522981 Test: build cast Change-Id: I4821a920f40e468e28e556985108412199ea0fbd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218670Reviewed-by:
Yuchen Liu <yucliu@chromium.org> Commit-Queue: Sandeep Vijayasekar <sandv@google.com> Auto-Submit: Sandeep Vijayasekar <sandv@google.com> Cr-Commit-Position: refs/heads/master@{#772365}
-
Dominic Mazzoni authored
The goal is that when we enable kEnableAccessibilityExposeHTMLElement, relatively few tests will be affected. In this tests, there was an assumption about nodes being an immediate child of the root element. Bug: 1063155 Change-Id: I4b6388e5529e1b39cf67881b4830192363106b77 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2206338 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#772364}
-
Peter Wen authored
Ignore setText not being translated for test code. Tbr: Suppressing existing errors to fix build. Notry: true Bug: 1082743 Change-Id: I7de995b0d6cdc0bba867d9064d7daf245d7837bb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218910Reviewed-by:
Peter Wen <wnwen@chromium.org> Commit-Queue: Peter Wen <wnwen@chromium.org> Auto-Submit: Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#772363}
-
Tommy Nyquist authored
This reverts commit 9a7c97d5. Reason for revert: Now failing lint because of https://chromium.googlesource.com/chromium/src/+/7b39bed1b5c6d5adc3e470f0d936079cf15680b0 Original change's description: > Add nickname field to local card editor and new instrumentation tests. > > The field is flag protected and this is only a UI change, so setting the > nickname won't set the nickname in the database. Will follow up with a CL > that updates it in the database. > > Screencast: https://drive.google.com/file/d/1aIMedpnwlWtJ1YKNT0cpRCowuhDOnNRq/view?usp=sharing > (Googlers only) > > Bug: 1082013 > Change-Id: Ib26a67c858da4388563e4ba241604517a5e7fde8 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2213341 > Reviewed-by: Jared Saul <jsaul@google.com> > Reviewed-by: sebsg <sebsg@chromium.org> > Reviewed-by: Theresa <twellington@chromium.org> > Commit-Queue: Siddharth Shah <siashah@chromium.org> > Cr-Commit-Position: refs/heads/master@{#772310} TBR=twellington@chromium.org,sebsg@chromium.org,jsaul@google.com,siashah@chromium.org Change-Id: Ie04f8b35826b4cc9c92ec07156d5c893bcf3e970 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1082013 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218722Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Commit-Queue: Tommy Nyquist <nyquist@chromium.org> Cr-Commit-Position: refs/heads/master@{#772362}
-
Leonard Grey authored
This reverts commit 8f6be972. Reason for revert: [Sheriff] Failing on several bots: https://ci.chromium.org/p/chromium/builders/ci/Win7%20Tests%20%281%29/103319 https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-rel/38321 Original change's description: > Add LayoutShiftTrackerSimTest.OOPIFSubframeWeighting test. > > > BUG=943668 > > Change-Id: I0157278fa1b7f8f5a0358e8811f21d4be1dd990c > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208026 > Reviewed-by: Steve Kobes <skobes@chromium.org> > Commit-Queue: Annie Sullivan <sullivan@chromium.org> > Cr-Commit-Position: refs/heads/master@{#772334} TBR=sullivan@chromium.org,skobes@chromium.org Change-Id: I27217d484c55a52e9aab52c95dbd053f44fb12b1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 943668 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218908Reviewed-by:
Leonard Grey <lgrey@chromium.org> Commit-Queue: Leonard Grey <lgrey@chromium.org> Cr-Commit-Position: refs/heads/master@{#772361}
-
Joel Fernandes authored
The coresched prctl interface is subject to change based on upstream discussions. Add a TODO as a code comment. Signed-off-by:
Joel Fernandes <joelaf@google.com> Change-Id: I664d0b5815dc4d7fe3bed259474b76635aa992af Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2216799Reviewed-by:
Brian Geffon <bgeffon@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#772360}
-
liberato@chromium.org authored
Bug: 1072667 Change-Id: I4d7fa88313f452c4f1720ba235916f1c54829354 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2207480 Auto-Submit: Frank Liberato <liberato@chromium.org> Commit-Queue: Steven Holte <holte@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#772359}
-
Brian Geffon authored
We shouldn't log errors for CrOS specific behavior when run on linux-chromeos. Bug: 1085453 Change-Id: Ibc0bf32a2fd6488eee224a9efd796a7114d51bdc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2216378Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Brian Geffon <bgeffon@chromium.org> Cr-Commit-Position: refs/heads/master@{#772358}
-
Mike Wittman authored
This is a reland of 01b9624c This is now safe to reland with the fix in https://crrev.com/772300. Original change's description: > [Sampling profiler] Provide unwinder in stack_unwinder DFM > > Implements the Android native/Java stack unwinder within the stack_unwinder > dynamic feature module. > > Adds visibility/assert_no_deps constraints to //base/BUILD.gn to avoid > leaking module implementation types into general Chrome code, which > would defeat the purpose of using the module to avoid Chrome binary size > increase. > > Adds required stub declarations/implementation for MemoryRegionsMap. > > Bug: 1083530, 1004855 > Change-Id: If9d31c4e1b9d214bb0baabe3afa8475ba7adef4a > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209297 > Commit-Queue: kylechar <kylechar@chromium.org> > Reviewed-by: kylechar <kylechar@chromium.org> > Reviewed-by: Etienne Pierre-Doray <etiennep@chromium.org> > Auto-Submit: Mike Wittman <wittman@chromium.org> > Cr-Commit-Position: refs/heads/master@{#771018} TBR=etiennep@chromium.org,kylechar@chromium.org Bug: 1083530, 1004855 Change-Id: I7c889d39cd7ae08c2dec9128a31742c2ff9525f8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218669Reviewed-by:
Mike Wittman <wittman@chromium.org> Reviewed-by:
Etienne Pierre-Doray <etiennep@chromium.org> Reviewed-by:
kylechar <kylechar@chromium.org> Commit-Queue: Mike Wittman <wittman@chromium.org> Cr-Commit-Position: refs/heads/master@{#772357}
-
John Williams authored
Fuzz testing now covers all of cast_framer.cc and cast_message_util.cc, and part of cast_auth_until.cc. Additional work is needed for functions in cast_auth_util.cc. Bug: 796717 Change-Id: I732175645feae2ccd5c9d0c8e0f19f287eb1d6d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1680610Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Commit-Queue: John Williams <jrw@chromium.org> Cr-Commit-Position: refs/heads/master@{#772356}
-
Alex Gough authored
Provides new tracing category "sandbox" and emits tracing once process launch has been attempted. Tracing includes the same information as chrome://sandbox's detailed json. Refactors type-to-string lookup from chrome://sandbox handler. Underlying functions already have tests. Bug: 1085213 Change-Id: I5c5a876102aba8803401a85a42a8acb2126d4c3c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212798 Commit-Queue: Alex Gough <ajgo@chromium.org> Reviewed-by:
Will Harris <wfh@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Cr-Commit-Position: refs/heads/master@{#772355}
-
yilkal authored
This cl removes the pause badge from the notification icon for per app time limits on chrome os. Bug: 1076207 Change-Id: Ic4a779006b7edcc4ecf0b0b3ec46b12572030d81 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218553 Commit-Queue: Yilkal Abe <yilkal@chromium.org> Reviewed-by:
Aga Wronska <agawronska@chromium.org> Reviewed-by:
Nancy Wang <nancylingwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#772354}
-
Fabrice de Gans-Riberi authored
* Move the existing Cast Streaming browser test into a separate file. * Add more Cast Streaming browser tests. Bug: 1042501 Change-Id: I34752670b58a8b54b3eccdf6fb17f9d7f2c5c11c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2194790Reviewed-by:
David Dorwin <ddorwin@chromium.org> Reviewed-by:
Thomas Guilbert <tguilbert@chromium.org> Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org> Cr-Commit-Position: refs/heads/master@{#772353}
-
Khushal authored
R=holte@chromium.org Bug: 976583 Change-Id: I7169cd35f84fee9b286e1357155fe357405b4ea9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208651 Auto-Submit: Khushal <khushalsagar@chromium.org> Commit-Queue: Steven Holte <holte@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#772352}
-
Nohemi Fernandez authored
Removes references in Identity code to: - addAccountWithCompletion - reauthenticateUserWithID:email:completion - cancelAndDismissAnimated Renames AddAccountSigninMediator to AddAccountSigninManager. Bug: 1078815 Change-Id: Ief8c868096c58009e94aee0c26d42e848f1edb99 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2207178 Commit-Queue: Nohemi Fernandez <fernandex@chromium.org> Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Cr-Commit-Position: refs/heads/master@{#772351}
-
Dale Curtis authored
Prep work for enabling these experiments on the beta channel. Bug: 982799, 1062849 Test: Passes CQ. Change-Id: Ibe880a566f741e695d9cf416def2180e822eaec8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208082 Commit-Queue: Steven Holte <holte@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Auto-Submit: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#772350}
-
Leonard Grey authored
TBR=rohitrao@chromium.org Bug: 1084280 Change-Id: I220f5ba04f8c974947764cdb883a9f78af14cd57 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218613Reviewed-by:
Leonard Grey <lgrey@chromium.org> Commit-Queue: Leonard Grey <lgrey@chromium.org> Cr-Commit-Position: refs/heads/master@{#772349}
-
Stephan Hartmann authored
The move assignment operator in PaintPreviewClient::PaintPreviewData is noexcept. Because this class has a member of type base::FilePath, the move assignment operator of base::FilePath must be noexcept too. Otherwise GCC fails like this: ../../components/paint_preview/browser/paint_preview_client.cc:107:1: error: function 'paint_preview::PaintPreviewClient::PaintPreviewData& paint_preview::PaintPreviewClient::PaintPreviewData::operator= (paint_preview::PaintPreviewClient::PaintPreviewData&&)' defaulted on its redeclaration with an exception-specification that differs from the implicit exception-specification '' 107 | PaintPreviewClient::PaintPreviewData::operator=( | ^~~~~~~~~~~~~~~~~~ Bug: 819294 Change-Id: I87b88a81e8af6b7e61f0d1a8c8444fd6707e50d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218168 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#772348}
-
Andrew Xu authored
A shelf arrow button should not get the accessibility focus ring. Because when spoken feedback is enabled, if a hidden shelf icon receives the accessibility focus, scrollable shelf will ensure the icon to be visible by scroll animation. So ignore the arrow button in spoken feedback. Bug: 1086943 Change-Id: Ibe82d0315bcd71b01255fe496ccba875f38e7fe3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218447 Commit-Queue: Andrew Xu <andrewxu@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#772347}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/c95dd320931c..1443c9df589b 2020-05-27 csmartdalton@google.com Create indirect index buffer for tessellation during onPrepare 2020-05-27 rmistry@google.com [Infra] Create AMD Rome test bots 2020-05-27 mtklein@google.com shader dither in SkRPBlitter 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 westont@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 Bug: None Tbr: westont@google.com Change-Id: I619c29596fc07b402c80bbc0d9f1d29dbfa90501 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218731Reviewed-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@{#772346}
-
Nate Chapin authored
These were regressions tests added to chromium based on recent crashes. Bug: 1083437, 1083721, 1083793 Change-Id: I2fe8ec74957539ba4c0ac587b4829ca44124e65e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2216512 Commit-Queue: Nate Chapin <japhet@chromium.org> Reviewed-by:
Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#772345}
-
Anastasiia Nikolaienko authored
Rename account_migration_browser_proxy to account_manager_browser_proxy to use it on all account_manager_* pages. Bug: 1083147 Change-Id: Ib2058fd25ef905b1920d3aafade1247601738a1d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2210231Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Kush Sinha <sinhak@chromium.org> Commit-Queue: Anastasiia N <anastasiian@chromium.org> Cr-Commit-Position: refs/heads/master@{#772344}
-
Gayane Petrosyan authored
Bug: 1086565 Change-Id: I7f13079b352c09cab272566aaf498d58d72d5540 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218527Reviewed-by:
Travis Skare <skare@chromium.org> Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Cr-Commit-Position: refs/heads/master@{#772343}
-
chromium-autoroll authored
Roll ChromeOS Orderfiles from 84-4136.1-1589189756-benchmark-84.0.4142.0-r1 to 84-4136.1-1589799722-benchmark-84.0.4147.17-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-orderfile-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: I2259bad5a60c4599d3947cb1111ca9972542adc2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218736Reviewed-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@{#772342}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/212c527f90e9..800891e5ad60 2020-05-27 tvanderlippe@chromium.org Use ESM build for acorn 2020-05-27 tvanderlippe@chromium.org Fix node_modules/OWNERS file after updating a dependency If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.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:1011811 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I4c097d5f9bd9d74882ff0c952aebfb35ff95dbc6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218727Reviewed-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@{#772341}
-
Wei Li authored
Refactor the time view to use two subviews to manage different layouts. This will allow us to remove the usage of set_owned_by_client() since all the labels will always owned by the views hierarchy. In addition, this will simplify the process of updating layouts and re-layout. BUG=1044687 Change-Id: I234fd2315d73d2508dc60f3544a04a12fbd170d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211085Reviewed-by:
Jenny Zhang <jennyz@chromium.org> Commit-Queue: Wei Li <weili@chromium.org> Cr-Commit-Position: refs/heads/master@{#772340}
-
Ben Joyce authored
Need to add espresso_web, idling, and contrib to migrate androidx espresso tests. espresso_web has tagsoup is a dependency. Should only need license review for tagsoup as others are covered when espresso_core was added. Bug: 1005891,1054152 Change-Id: Ie8937b7083d6b8141122001fd114f65601d495f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2115942 Commit-Queue: benjamin joyce <bjoyce@chromium.org> Reviewed-by:
Peter Wen <wnwen@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#772339}
-
Roman Aleksandrov authored
Add handling new status code 906 from DMServer on an erollment request. It occurs when user tries to enroll device while the administrator has not accepted enterprise ToS. Bug: 1053831 Change-Id: I2984ae671727771b1c6e8bf538a6fd8a37ee7df4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2213528 Commit-Queue: Roman Aleksandrov <raleksandrov@google.com> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#772338}
-
Mounir Lamouri authored
These videos are, as far as the user is concerned, non-existent. This is the first part into strengthening the conditions in which the video element takes a screen lock. Bug: 1086776 Change-Id: Ife302518d45986a75fbb27a91fc5dc77d5f07d03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2213933Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#772337}
-
Fredrik Söderqvist authored
The argument is the list to copy from, marking it const makes that clear and reduces the risk for accidental mutations. Change-Id: I38fbcc64dc9dd6d2a54444ccd8c76c1959ccda7f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218272 Commit-Queue: Fredrik Söderquist <fs@opera.com> Commit-Queue: Stephen Chenney <schenney@chromium.org> Auto-Submit: Fredrik Söderquist <fs@opera.com> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#772336}
-
Gang Wu authored
SearchActivity did not handle image search data since image search need post data, so the fix is add the post data into intent to pass to chrome. Bug: 1078773 Change-Id: I04efdad0bba315508e0b05d3d9e444d8c1a5d955 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2187191 Commit-Queue: Gang Wu <gangwu@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Ender <ender@google.com> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#772335}
-
Annie Sullivan authored
BUG=943668 Change-Id: I0157278fa1b7f8f5a0358e8811f21d4be1dd990c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208026Reviewed-by:
Steve Kobes <skobes@chromium.org> Commit-Queue: Annie Sullivan <sullivan@chromium.org> Cr-Commit-Position: refs/heads/master@{#772334}
-
Sophey Dong authored
This fixes the bug where the STTS icon overlaps with the bookmark icon when the animation gets interrupted by reloading the page. Bug: 1085708 Change-Id: I1296d282afb2a8e5e6ec1500db9828856701be57 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2217768Reviewed-by:
Kristi Park <kristipark@chromium.org> Commit-Queue: Sophey Dong <sophey@chromium.org> Cr-Commit-Position: refs/heads/master@{#772333}
-
Alexander Surkov authored
Regressed by 487bd8ec change-id when class attribute serialization was switched from html to string attribute serialization type. Bug: 1086941 Change-Id: I1fc17f1ce05589e6005d3bf233944e2bbfe68fff AX-Relnotes: n/a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218467Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Alexander Surkov <asurkov@igalia.com> Cr-Commit-Position: refs/heads/master@{#772332}
-
Angela Yoeurng authored
When a button on the button row is highlighted (by pressing tab), the focus ring was rectangular shape. No highlight path generator was specified so the default path generator with a corner radius of 3 was used. This was fixed by specifying a pill shaped path generator. Bug: 1084320 Change-Id: I203ed3cdb47904c8d414842b3e4ea531ccfad1ab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211183Reviewed-by:
manuk hovanesian <manukh@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Commit-Queue: Angela Yoeurng <yoangela@chromium.org> Cr-Commit-Position: refs/heads/master@{#772331}
-
Steven Bennetts authored
This includes a few minor fixes to the network config UI: * Eliminates an error log when AutoConnect has not been set for a network by initializing it to a default (false) value. * Fixes an unlikely but possible edge case in CrosNetworkConfig::GetManagedPropertiesSuccess found while testing. * Uses network-config-toggle instead of cr-toggle in network_config.html for 'Share' for consistency. * Fixes network-config-toggle so accept clicks anywhere on the toggle for consistent behavior with other Settings UI. Bug: 971704 Change-Id: I0fb5a33fa7ca5ec42241fb634eca8f697d02c279 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212977Reviewed-by:
James Vecore <vecore@google.com> Reviewed-by:
Azeem Arshad <azeemarshad@chromium.org> Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#772330}
-
Liviu Tinta authored
Move mouse pointerevents compat events to wpt Change-Id: I5b64c7108222d68f52711c2a19069ccb5cd3f379 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211913Reviewed-by:
Lan Wei <lanwei@chromium.org> Reviewed-by:
Navid Zolghadr <nzolghadr@chromium.org> Commit-Queue: Liviu Tinta <liviutinta@chromium.org> Cr-Commit-Position: refs/heads/master@{#772329}
-
Gauthier Ambard authored
This CL fixes the display when showing the NTP on a cold start. It prevents the animation by directly adding the BVC to the view hierarchy when the animations are disabled. It is also making sure that the location bar isn't shown on the NTP when doing a cold start. Fixed: 1082726 Change-Id: I02658837b177bbe07d58b507311657d09107fde0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2216250Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#772328}
-
Reilly Grant authored
This change replaces the CHECKs added in r770407 with logic to reject invalid lengths from IOCTL_USB_GET_DESCRIPTOR_FROM_NODE_CONNECTION. Crash reports from when the original DCHECKs were replaced with CHECKs indicate that the hub driver will sometimes return a response which is shorter than the struct originally passed to it. This is invalid and so is treated as a failure. Bug: 1084316 Change-Id: I8462a5d535935e139f6cd511c26a0ab79db5950e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2216548 Commit-Queue: Reilly Grant <reillyg@chromium.org> Auto-Submit: Reilly Grant <reillyg@chromium.org> Reviewed-by:
James Hollyer <jameshollyer@chromium.org> Cr-Commit-Position: refs/heads/master@{#772327}
-
chrome-untrusted://resourcesTommy Steimel authored
This CL fixes a bug in chrome-untrusted://resources where the Access-Control-Allow-Origin policy would only allow for chrome:// origins. This changes it so that chrome-untrusted://resources only allows for chrome-untrusted:// origins and chrome://resources only allows for chrome:// origins. Bug: 1086327 Change-Id: Icbf05aa66e587ed3cb5d5f26ef2b46c0628641c1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2216282 Commit-Queue: Tommy Steimel <steimel@chromium.org> Reviewed-by:
dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#772326}
-