- 07 Feb, 2020 40 commits
-
-
shrekshao authored
Bug: 1000354, skia:9861 Change-Id: I4763b847526e04fe21e875110417a6bfb03013d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040611Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Shrek Shao <shrekshao@google.com> Cr-Commit-Position: refs/heads/master@{#739274}
-
Eriko Kurimoto authored
This CL adds Timeout in shared-worker-import-referrer.html expectation. This test is currently flaky because of Timeout, and the similar issue is happening as well in dedicated-worker-import-referrer.html. I will continue the investigation about this issue, but we add Timeout expectation for now. Bug: 1049689 Change-Id: I891b80b51186d623c304ecb59a835f725f407a0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2041374Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Commit-Queue: Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#739273}
-
Hiroshi Ichikawa authored
To clarify the behavior. Change-Id: Ie9a1ff59b934615c353fb6c11c06b2678fca5c97 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2041836 Commit-Queue: Hiroshi Ichikawa <ichikawa@chromium.org> Reviewed-by:
John Wu <jzw@chromium.org> Cr-Commit-Position: refs/heads/master@{#739272}
-
Yutaka Hirano authored
These functions were "shared between network service and the old loading path in content/" but now the network service path is the only path, so we don't need to have them separately. Merge them into network::URLLoader. Bug: 1049483 Change-Id: I26026b2f2bc5b763c834cee004d2f7bc04c7d402 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040800 Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/7457e5e4ba27..6737082f1029 git log 7457e5e4ba27..6737082f1029 --date=short --first-parent --format='%ad %ae %s' 2020-02-07 devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com Update DevTools DEPS. Created with: gclient setdep -r src/third_party/devtools-frontend/src@6737082f1029 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/+/master/autoroll/README.md Bug: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I9bfb41e53269796a229df704a73f9a144d2abab4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042912Reviewed-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@{#739270}
-
John Chen authored
Updated baseline for the following two tests, due to https://crrev.com/738119: * fast/css/css2-system-color.html * fast/css/test-setting-canvas-color.html Updated TestExpectations for a test that sometimes timeout: * http/tests/misc/slow-loading-image-in-pattern.html Change-Id: I695fab8468a23aa37e84ebfa72984a4ef2aba8ab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042196Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#739269}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/22f2b82b60a3..fb6c493bb000 Created with: gclient setdep -r src-internal@fb6c493bb000 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: None Tbr: jbudorick@google.com Change-Id: I4dd5895754510d68c7873f4550fe0fca52423929 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2043022Reviewed-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@{#739268}
-
Isabella Scalzi authored
No-try: true No-presubmit: true Test: No change in behavior, no new tests. Bug: 803259 Change-Id: Ifcb8ad66dc70aa94e7dd1d8e3bd0085924fd094a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2041718 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#739267}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=govind@chromium.org Change-Id: I57754be5aa941dbd995f996f22a1c9517546bab1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2043034Reviewed-by:
Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#739266}
-
Ryan Sturm authored
It is possible for the default search provider to be empty, and we should avoid crashing when it is. Bug: 1049861 Change-Id: I21ad12dfa7f6be5a582101a2081ae2725b3c3da8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042055 Auto-Submit: Ryan Sturm <ryansturm@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Ryan Sturm <ryansturm@chromium.org> Cr-Commit-Position: refs/heads/master@{#739265}
-
Kyle Horimoto authored
Before this CL, navigating to a settings subpage on browser and OS settings emitted to the same histogram. This caused issues, since some separate subpages have the same name in each settings surface. For example, the "/printing" subpage means a completely different thing in OS vs. browser settings. Bug: 1049417, 1026353 Change-Id: Ifbe15866c61377a8da0413694b8578228401a6fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040951 Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Auto-Submit: Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#739264}
-
Jon Mann authored
Bug: 1047815 Change-Id: I15889b977329b3d4ed1e042b76f2076360c288aa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040744 Commit-Queue: Jon Mann <jonmann@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#739263}
-
Jason Lin authored
In addition, CrostiniInstallerUI::ClickInstallForTesting() is updated so that it works no matter whether this flag is on or not, so if in an unlikely event that we need to re-disable this flag, we do not need to modify it again. Bug: 1016195 Change-Id: I254057ffdff281c2ca01035953adfc737214b646 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040635Reviewed-by:
Julian Watson <juwa@google.com> Commit-Queue: Jason Lin <lxj@google.com> Cr-Commit-Position: refs/heads/master@{#739262}
-
Austin Eng authored
Bug: 1049864 Change-Id: I3041a776006a53f4268a38a1786ab5b5c88bb4f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2043005Reviewed-by:
Sunny Sachanandani <sunnyps@chromium.org> Reviewed-by:
-
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:
-
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:
-
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:
-
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:
-
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:
-
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:
-
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:
-
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}
-
