- 07 Feb, 2020 40 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/463c318ab759..e02f4996c6e9 git log 463c318ab759..e02f4996c6e9 --date=short --first-parent --format='%ad %ae %s' 2020-02-06 cnorthrop@google.com Vulkan: Pass correct level count in getLevelDrawImageView 2020-02-06 spang@chromium.org Fuchsia: Fix size & position of test window on Scenic 2020-02-06 jmadill@chromium.org Make TransformFeedbackTest deterministic. Created with: gclient setdep -r src/third_party/angle@e02f4996c6e9 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 cnorthrop@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-asan;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: cnorthrop@google.com Change-Id: If7e2ce521e15c1b0ef612fb6f86e777dffd0cbb5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2043052Reviewed-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@{#739260}
-
rbpotter authored
- Autogenerate Polymer 3 versions for files in on_startup_page/* - Update closure compiler and polymer_modulizer/js_modulizer targets for these files - Autogenerate tests for the Polymer 3 files Bug: 1026426 Change-Id: I122bcb77d366b7fc7fd979b235d13d283faf8d43 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040626 Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#739259}
-
Isabella Scalzi authored
Create new private helper in |quick_view_controller| called |canDeleteEntry_|, so we can create the promise from |quick_view_controller| rather than |file_manager_commands|. No-try: true Bug: 803259 Change-Id: I829a798e3d19989b880df1e554b23e2ffd67c987 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2041834 Commit-Queue: Isabella Scalzi <isabellascalzi@google.com> Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#739258}
-
Wei Lee authored
Adds 'resizeMode: none' in constraints to avoid being cropped unexpectedly by Chrome. Bug: b/148045035 Test: Manually Change-Id: I4e2062cc75b438b1b64e7aa467b7c794a67cce95 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2041213Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Wei Lee <wtlee@chromium.org> Auto-Submit: Wei Lee <wtlee@chromium.org> Cr-Commit-Position: refs/heads/master@{#739257}
-
Maks Orlovich authored
No need for services/network/, since it doesn't use the concrete implementation types (any more?). Change-Id: I242540cd2bea4d135934ee0a2fa5b84a3e22f195 Bug: 1049894 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042250 Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#739256}
-
Hiroki Sato authored
Rename API name to match what this is really doing. This is a follow up of http://crrev/c/2018624. Bug: None Change-Id: Iae0cf9019c03d08d078f3181929146d94602563d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2035507 Commit-Queue: Hiroki Sato <hirokisato@chromium.org> Reviewed-by:
David Tseng <dtseng@chromium.org> Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Cr-Commit-Position: refs/heads/master@{#739255}
-
Filip Filmar authored
To unblock Chromium rolls into Fuchsia. Until https://bugs.fuchsia.dev/p/fuchsia/issues/detail?id=45491 is fixed. The current tests do not reinitialize the internal state of `icu_util` correctly, leading to spurious test errors. The fix should be straightforward, but takes some time. Disabling these tests will allow the rolls to continue while we fix the issue in the background. Bug: fuchsia:45491 Change-Id: Ifeabd973970cad70d7604da76f0948dbbb8b63ab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042148 Commit-Queue: Filip Filmar <fmil@google.com> Reviewed-by:
Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
David Dorwin <ddorwin@chromium.org> Cr-Commit-Position: refs/heads/master@{#739254}
-
Jason Chase authored
In crbug.com/1049126, we discovered that origin trials are not enabled when running in headless mode. This was not intentional, as headless should behave the same as regular Chrome with respect to origin trials. The root cause is that origin trials require the embedder (i.e. Chrome) to explicitly opt-in to enable trials. The opt-in is done by providing an implementation of OriginTrialPolicy [1]. It turns out that headless is actually a separate embedder than Chrome, which was missed in the initial implementation. A workaround for the problem has been landed in [2], and merged back to M80. This CL is intended to be merged back to M81. This CL implements the opt-in for the headless embedder, by copying the Chrome embedder implementation. Ideally, headless should re-use the Chrome implementation, but that would require larger changes. The duplication will be addressed in M82 in crbug.com/1049317. [1] https://source.chromium.org/chromium/chromium/src/+/master:third_party/blink/public/common/origin_trials/origin_trial_policy.h [2] https://chromium-review.googlesource.com/c/chromium/src/+/2039601 Bug: 1049126 Change-Id: Ide596c100baae4b69eab2149169eb92353a7bac7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042382 Commit-Queue: Jason Chase <chasej@chromium.org> Reviewed-by:
Mason Freed <masonfreed@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
Johannes Henkel <johannes@chromium.org> Cr-Commit-Position: refs/heads/master@{#739253}
-
Rahul Chaturvedi authored
This CL adds code to collect metrics regarding the processor that the user is running on. R=rockot@chromium.org Bug: 1032858 Change-Id: I38628c722849071c8d0219ce942a257fc0623424 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1982237 Commit-Queue: Rahul Chaturvedi <rkc@chromium.org> Auto-Submit: Rahul Chaturvedi <rkc@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Sébastien Marchand <sebmarchand@chromium.org> Reviewed-by:
Will Harris <wfh@chromium.org> Reviewed-by:
Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#739252}
-
Kenichi Ishibashi authored
Resource ID allocation will become async (see the planning doc[1]). As a preparation, this CL moves resource ID allocation from ServiceWorkerNewScriptLoader to ServiceWorkerScriptLoaderFactory. Once resource ID allocation becomes async the factory will create ServiceWorkerNewScriptLoader asynchronously. [1] https://docs.google.com/document/d/1j0WP5wsenJISViJzhngSJG3hKcIf39KbfRgCbzgWt5A/edit?usp=sharing Bug: 1046335 Change-Id: Ia6618085ec4e9413f670e75fe2d074736906a623 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2041677Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Cr-Commit-Position: refs/heads/master@{#739251}
-
Yusuke Sato authored
On some boards like nami, CrOS rootfs' build.prop doesn't have the complete ro.product.board entry. It's sometimes a template. To always get the board name on all boards, ArcDefaultAppList needs to wait for ArcSessionManager to generate the board's own build.prop file in its stateful partition. This is the same as what arc-setup does today. This CL also stops special casing ARCVM and does the same for ARC too. Also, when calling GetBoardName(), this CL uses MayBlock() to make it debug build compatible. BUG=b:144199481 TEST=try, ARCVM still starts Change-Id: Ie878d9ffff98ecf3b9dccb837fd6dce1a382a4be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2026668 Commit-Queue: Yusuke Sato <yusukes@chromium.org> Reviewed-by:
Yury Khmel <khmel@chromium.org> Cr-Commit-Position: refs/heads/master@{#739250}
-
Jay Harris authored
Previously, file handlers were registered once, when the app was installed. This makes it possible to update registrations, which is a requirement for the origin trial. Bug: 1028448 Change-Id: Id1e05f6efdc69da601573349de6ab6932a7901c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2032700 Auto-Submit: Jay Harris <harrisjay@chromium.org> Reviewed-by:
Alexey Baskakov <loyso@chromium.org> Commit-Queue: Jay Harris <harrisjay@chromium.org> Cr-Commit-Position: refs/heads/master@{#739249}
-
Chris Lu authored
This reverts commit ac791f4c. Reason for revert: Compile build fails. Original change's description: > [ios] Update Firebase library to 6.16 > > This brings in the new library files as a CIPD package and update > DEPS to refer to this updated library. > > Fixed: 1049042 > Change-Id: I874f1a148bd7c47a4756d25f5edb590a39709b0d > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042215 > Reviewed-by: Eugene But <eugenebut@chromium.org> > Commit-Queue: Chris Lu <thegreenfrog@chromium.org> > Cr-Commit-Position: refs/heads/master@{#739162} TBR=eugenebut@chromium.org,pkl@chromium.org,thegreenfrog@chromium.org Change-Id: I225f141037073ac8d7926bd2c96825b94b74a0da No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2043081Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Commit-Queue: Chris Lu <thegreenfrog@chromium.org> Cr-Commit-Position: refs/heads/master@{#739248}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/6c6827cbf27d..73c76703bee6 git log 6c6827cbf27d..73c76703bee6 --date=short --first-parent --format='%ad %ae %s' 2020-02-06 ehmaldonado@google.com git-cl: Don't use assertRegexpMatches on tests. Created with: gclient setdep -r src/third_party/depot_tools@73c76703bee6 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 agable@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 Bug: None Tbr: agable@chromium.org Change-Id: I3512bcf721de2a061a15c13a479fda45af0ae77d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2043055Reviewed-by:
-
shrekshao authored
Add new expectations for Android Nvidia devices Bug: 1013369, 1049733 Change-Id: I9162775442db52e0c3187b68b4670bfc743232b8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042754 Commit-Queue: Shrek Shao <shrekshao@google.com> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#739246}
-
Eric Willigers authored
Various shortcut tests are extracted from hosted_app_browsertest.cc They now also run with BMO, i.e. with no dependencies on Extensions. Bug: 1026080 Change-Id: I6438bcbf38028b700b5f451dd4d781cedaf96d2b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040593 Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Reviewed-by:
Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#739245}
-
Kenichi Ishibashi authored
This enum is going to be used as status code of operations performed by the Storage Service. Convert it to a mojo enum so that we can define mojo interfaces for service worker storage. Bug: 1046335 Change-Id: Ie4cf45a5dc119cc6b9e454d31e680dce32084986 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040790Reviewed-by:
Victor Costan <pwnall@chromium.org> Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Cr-Commit-Position: refs/heads/master@{#739244}
-
Nicholas Verne authored
We restart vm when upgrading, if it wasn't running already. CrostiniRestarter now calls the final callback with RESTART_ABORTED when aborting. Call sites and test expectations have been changed to match this new behavior. Bug: 1024693 Change-Id: I9a24577eb6a8514e9eb8a8941f5c59d114ee673b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040501Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Nicholas Verne <nverne@chromium.org> Cr-Commit-Position: refs/heads/master@{#739243}
-
Christopher Cameron authored
This applies to viz::OutputSurface::Reshape and viz::SkiaOutputDevice::Reshape, their many many many sub-classes, and to viz::BufferQueue::Reshape. This value is still effectively treated as a bool -- we convert from a bool to this value in viz::DirectRenderer::DrawFrame by selecting either RGBA_8888 or RGBX_8888, and we convert from this back to a bool by the new gfx::AlphaBitsForBufferFormat. The next steps for this include - Make viz::BufferQueue::Reshape use the format value from Reshape, and remove the initialization parameter - Plumb format to through glResizeCHROMIUM to gl::GLSurface, where it can be used by gl::DirectCompositionChildSurfaceWin. Bug: 1049334 Change-Id: I3f1d1402cac76bb7b399c247a84d2fe2b1cd6b84 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040446 Commit-Queue: ccameron <ccameron@chromium.org> Reviewed-by:
Khushal <khushalsagar@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#739242}
-
tby authored
This re-enables the launcher collection that we disabled while moving from the hashed logging prototype to the structured metrics framework. All logging is the same, with one exception: we previously had two metrics aimed at recording which provider a search result comes from, with each determining about half of the providers. I've simplified this by combining them into one metric, which records the value of RankingItemType, the enum we use for type identification for all on-device models. To do this I've cleaned up the RankingItemType enum, removing item types that we no longer use and adding a "don't change this" header. Bug: 1016655 Change-Id: Id04156a4818f842af6489aea23d4048d7aeb4824 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2038292Reviewed-by:
Charles . <charleszhao@chromium.org> Reviewed-by:
Jenny Zhang <jennyz@chromium.org> Commit-Queue: Tony Yeoman <tby@chromium.org> Cr-Commit-Position: refs/heads/master@{#739241}
-
Omer Katz authored
This is a followup to CL 2012030: https://chromium-review.googlesource.com/c/chromium/src/+/2012961 Loading whole keys atomically resulted in data races involving the fields prev_ and next_ (which were not accessed concurrently before). This CL takes the same approach as was used for ListHashSetNode (https://chromium-review.googlesource.com/c/chromium/src/+/1984308) and wraps prev_ and next_ with atomic writes. This CL already includes the optimizations added to ListHashSetNode in https://chromium-review.googlesource.com/c/chromium/src/+/1995175. Bug: 986235 Change-Id: I4ef881ce181af25e53ccb51169b199c3315c7132 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2012345Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#739240}
-
Omer Katz authored
This is part of an ongoing effort to simplify heap collections and limit the types they can support (specifically, this CL removes the only existing non-member usage of HeapHashSet). Bug: 1047147 Change-Id: I2d408a5bbace633002c354c17b13d892e1938dec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2025427Reviewed-by:
-
Steven Bennetts authored
Bug: 1047812 Change-Id: I68513f609e42db54fa0d76c7a127970f4b123e00 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042316 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Jon Mann <jonmann@chromium.org> Reviewed-by:
Azeem Arshad <azeemarshad@chromium.org> Cr-Commit-Position: refs/heads/master@{#739238}
-
Robert Liao authored
The IncreasedContrastThemeSupplier previously had these colors: Frame Tab Bkgnd Contrast Color Text Ratio Light Mode Inactive Gray Black 5.92 Light Mode Active Dark Gray Black 2.15 Dark Mode Inactive Gray White 3.54 Dark Mode Active Dark Gray White 9.73 This doesn't really make sense as the frame color doesn't change with light/dark mode. Previously, this worked accidentally as BlendForMinContrast would save us. After http://crrev.com/687315 , the IncreasedContrastThemeSupplier was no longer blended, revealing the bug. Since the frame color is invariant with respect to light/dark mode, the new colors are as follows: Frame Tab Bkgnd Contrast Color Text Ratio Light Mode Inactive Gray Black 5.92 Light Mode Active Dark Gray White 9.73 Dark Mode Inactive Gray Black 5.92 Dark Mode Active Dark Gray White 9.73 BUG=1046818 Change-Id: Ic9452154f833f471a51e2ea6eacfb49f10cf1232 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042144Reviewed-by:Peter Kasting <pkasting@chromium.org> Commit-Queue: Robert Liao <robliao@chromium.org> Auto-Submit: Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#739237}
-
Jiewei Qian authored
This CL is a preparation to use SystemWebAppManager in WebAppTabHelper. Bug: 1024655 Change-Id: Id2a4e71e852696bfbf8386ffdd221d523ab55161 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040755 Commit-Queue: Jiewei Qian <qjw@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
-
David Tseng authored
Change-Id: I39b0b7e4ec10430f2162390db923cd58b909f0e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042034Reviewed-by:
Katie Dektar <katie@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#739235}
-
Alexandre Courbot authored
A request timing out indicates a serious issue ; this should be logged at level 1, not 3. BUG=None TEST=VDtest builds and runs. Change-Id: I5742a0f8e8de93852e30ae049b54148f16ac7d95 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2041710 Auto-Submit: Alexandre Courbot <acourbot@chromium.org> Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Reviewed-by:
Hirokazu Honda <hiroh@chromium.org> Cr-Commit-Position: refs/heads/master@{#739234}
-
Hiroki Sato authored
Currently, if Android app starts before ChromeVox or S2S starts, ARC AXTrees are not recognized from these ATs. This is because: - AXChildTreeID is not attached to the parent surface at that moment. - ARC AXTrees are not triggered to update on automationInternal.enableFrame. This CL fixes these issues. Android side change is http://ag/10167029. Bug: b:129681889 Test: manual. Open any Android app, then enable ChromeVox. Confirm that Android app gets focus. Change-Id: I38cb372f706817f5e305c4a10a1ac84176275397 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2018624Reviewed-by:
Sean Topping <seantopping@chromium.org> Reviewed-by:
Randy Rossi <rmrossi@chromium.org> Reviewed-by:
Jorge Lucangeli Obes <jorgelo@chromium.org> Reviewed-by:
Hidehiko Abe <hidehiko@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
-
Xianzhu Wang authored
This reverts commit be47c44f. Reason for revert: The reland was an accident. Forgot to uncheck automatically send revert change to CQ. Original change's description: > Reland "Ensure cc::Layer's NeedsPushProperties when blink directly sets scroll offset" > > This reverts commit 5db928ee. > > The original issue was caused by additional scroll tree scroll offset > entries. DirectSetScrollOffset should not create new entries, but just > change existing entries, otherwise after a document lifecycle update, > the scroll tree may contain entries whose element ids don't correspond > any scroll node. > > Original change's description: > > Revert "Ensure cc::Layer's NeedsPushProperties when blink directly sets scroll offset" > > > > This reverts commit 617a9c19. > > > > Reason for revert: Broke some browser tests. The tests flakily crash when pushing scroll properties at impl-side. > > > > Original change's description: > > > Ensure cc::Layer's NeedsPushProperties when blink directly sets scroll offset > > > > > > When blink directly sets scroll offset in the scroll tree, we need to > > > set the associated cc::Layer's NeedsPushProperties to ensure update of > > > cc::LayerImpl's scroll related properties > > > (see cc::LayerImpl::UpdateScrollable()). > > > > > > This reverts the change about cc::LayerTreeHost::LayerByElementId() in > > > https://chromium-review.googlesource.com/c/chromium/src/+/2020554 > > > because we still need it in layer list mode. > > > > > > Bug: 1048113 > > > Change-Id: I686beebcf54e96cc517700c0f987aaceec880c58 > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2036834 > > > Reviewed-by: Philip Rogers <pdr@chromium.org> > > > Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> > > > Cr-Commit-Position: refs/heads/master@{#739043} > > > > TBR=wangxianzhu@chromium.org,pdr@chromium.org > > > > Change-Id: Ib52291c52f8bac3f4a3c9fe0248f55d0162c556c > > No-Presubmit: true > > No-Tree-Checks: true > > No-Try: true > > Bug: 1048113 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042763 > > Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org> > > Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#739183} > > TBR=wangxianzhu@chromium.org,pdr@chromium.org > > Change-Id: Ie72b629bbdb20b0ed4c33746c33828f4d596a884 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 1048113 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2043056 > Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org> > Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> > Cr-Commit-Position: refs/heads/master@{#739220} TBR=wangxianzhu@chromium.org,pdr@chromium.org Change-Id: I23bf77e95117f2599086bef6ec4113f0fd442cd9 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1048113 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2043090Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#739232}
-
Huanzhong Huang authored
Remove NewClearBrowsingDataUI experiment, as it has become the default. Bug: 947456 Change-Id: I41f1b2eb5824d18f77bea3447b7b79bfb1469eff Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2039410Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Huanzhong Huang <huanzhong@chromium.org> Cr-Commit-Position: refs/heads/master@{#739231}
-
Wei-Yin Chen (陳威尹) authored
Show a chip on the tab card if that tab has search result page in its navigation stack. The chip contains the search term. When clicked, the tab would visit the search result page again. This is behind a Finch parameter "enable_search_term_chip" under TabGridLayoutAndroid. The CL is split into two, and this is part two, which can be verified to be behind the flag isSearchTermChipEnabled() by the Formal Equivalence Checker in http://crrev.com/c/1934235/4. See also: part one (http://crrev.com/c/2038458). Bug: 1048255 Change-Id: I03d61d97a99964168ebcd758b772231185d9223f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2036513Reviewed-by:
Tommy Li <tommycli@chromium.org> Reviewed-by:
Mei Liang <meiliang@chromium.org> Reviewed-by:
Yue Zhang <yuezhanggg@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org> Cr-Commit-Position: refs/heads/master@{#739230}
-
Yuichiro Hanada authored
This CL changes RewriteTouchEvent and RewriteScrollEvent to use the new API. These two methods seems to be the easiest to rewrite. Bug: 1049489 Test: unit_tests Change-Id: Ic133c9c829883728b4a65d04e0470c36f891c203 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2041131 Commit-Queue: Yuichiro Hanada <yhanada@chromium.org> Reviewed-by:
Kevin Schoedel <kpschoedel@chromium.org> Reviewed-by:
-
Benjamin Beaudry authored
An accessible node that represents generated content (e.g., the ::before selector) is completely ignored in AXNodePosition::MaxTextOffset and AXNodePosition::GetText. Both methods return the ax::mojom::StringAttribute::kValue's length and value, respectively. Since this attribute's value doesn't include the value of the generated content (because this generated content is either before or after the main content of the text field), we return a wrong maximum text offset and text. To fix this, we should only use ax::mojom::StringAttribute::kValue's length and value when the node has no child. When a node has a child, we need to compute the maximum text offset by summing the maximum text offset values of all of its children. The same applies for AXNodePosition::GetText. Bug: 928948 Change-Id: I624779a3e372d19ac2036de09fe9dbf1c4892867 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2037965 Commit-Queue: Benjamin Beaudry <benjamin.beaudry@microsoft.com> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Reviewed-by:
Kurt Catti-Schmidt <kschmi@microsoft.com> Cr-Commit-Position: refs/heads/master@{#739228}
-
David Van Cleve authored
This reverts commit 7f4af748. Reason for revert: Perf bot identified a memory regression; reverting while I investigate locally. Original change's description: > Add a testing config for ReducedReferrerGranularity > > We're going to 50% beta on this feature (changing the > default referrer policy to strict-origin-when-cross-origin) > so it's time to add a field trial testing config. > > Also updates the following tests, which were failing with > the testing config enabled: > - ContextMenuBrowserTest.OpenInNewTabReferrer > - DownloadReferrerPolicyTest.DownloadCrossDomainReferrerPolicy > > > Bug: 1014207 > > Change-Id: Ic50657d9c2f642af8538fea47fe2263797d45018 > Cq-Do-Not-Cancel-Tryjobs: true > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2012087 > Commit-Queue: David Van Cleve <davidvc@chromium.org> > Reviewed-by: Tom Sepez <tsepez@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: Jesse Doherty <jwd@chromium.org> > Reviewed-by: Avi Drissman <avi@chromium.org> > Cr-Commit-Position: refs/heads/master@{#737362} Bug: 1014207 Bug: 1048550 Change-Id: I4eeb5ddb45ca6685d632d2f51268455b87346779 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2036795Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/4db6affff29a..22f2b82b60a3 Created with: gclient setdep -r src-internal@22f2b82b60a3 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 Bug: chromium:1024586,chromium:1024588,chromium:1041701 Tbr: jbudorick@google.com Change-Id: I7a5c65bd4a907f118ce4aa344c45b5b8566ccad8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042668Reviewed-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@{#739226}
-
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: I9ffef5dc43cec87e71e8cce46fe572a290aae66e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2043050Reviewed-by:
-
Anand K. Mistry authored
Bug: 648275 Change-Id: I3d933396788139cefebb5ea337c5a478498e483a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1512982 Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Auto-Submit: Anand Mistry <amistry@chromium.org> Reviewed-by:
-
Su Hong Koo authored
Move DisplayManager::GetSecondaryDisplay() to DisplayManagerTestApi as it is now only used in unit tests. Update all affected unit tests to use DisplayManagerTestApi. Bug: 867966 Change-Id: I26fe6ef2d7f4ef8406c90b5f8ebd0c150f7bb09c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2032671 Commit-Queue: Su Hong Koo <sukoo@google.com> Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Reviewed-by:
-
Gyuyoung Kim authored
This CL migrates the legacy InitiateCaptureDump IPC message to the new Mojo defined in WebTestClient interface. Bug: 1039247 Change-Id: Idd38ecf67d10593f3bfc6f3c48d9832514cec595 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2032634 Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Mike West <mkwst@chromium.org> Cr-Commit-Position: refs/heads/master@{#739222}
-
Isabella Scalzi authored
In preparation for adding a delete button to the Quick View toolbar add property |canDelete| to |files-quick-view|, which indicates whether the file shown can be deleted or not. In a future CL, |shouldShowDeleteButton_| will then toggle the display of the delete button in Quick View. Test: Tests coming in future CL. Bug: 803259 Change-Id: I880fd7f2065c11435a16ec923abca12277b0c824 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040502 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
-
