- 17 Nov, 2020 40 commits
-
-
Tim Dresser authored
Add flackr@, and remove nzolghadr@. Change-Id: I105114ad7c89b9a2162a79880c6fc71d33578cb0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2541810Reviewed-by:
Robert Flack <flackr@chromium.org> Reviewed-by:
Navid Zolghadr <nzolghadr@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Nicolás Peña Moreno <npm@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#828375}
-
chromium-autoroll authored
Roll Chrome Linux PGO profile from chrome-linux-master-1605612997-0b538d267d59602889f3873bf24b3a402bd4dd04.profdata to chrome-linux-master-1605635959-3de00f504d9de3eba6702b2e7e69c522ca7d3882.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-linux-chromium Please CC pgo-profile-sheriffs@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.chrome.try:linux-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: I92b4601d5f52c6d19a3e6c2946f6be6a41f11f37 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2545228Reviewed-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@{#828374}
-
Henrique Nakashima authored
tools/android/modularization/getowners.py traverses the codebase reading OWNERS files and uses git commands to read a number of metrics at the granularity of modules. This script will be run periodically in a builder to feed information into a dashboard. Bug: 1135347 Change-Id: I8ef4050396c796cc96aff757d99ad15cb14b9483 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2451326 Commit-Queue: Henrique Nakashima <hnakashima@chromium.org> Reviewed-by:
Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#828373}
-
Shakti Sahu authored
This CL removes the loading animation for the video player. We will use chrome media's in-built loading animation for this purpose which also provides better feedback on slow networks and other video failures. Bug: 1149946 Change-Id: Ib0e7ad4ab131e10e0bda01e819bb7314ec6d57b4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2542860Reviewed-by:
Min Qin <qinmin@chromium.org> Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Cr-Commit-Position: refs/heads/master@{#828372}
-
Adam Langley authored
BUG=1002262 Change-Id: Ief209ab2904ade456afa3dcdfd11d001aeb8330e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2538158 Commit-Queue: Adam Langley <agl@chromium.org> Reviewed-by:
Martin Kreichgauer <martinkr@google.com> Cr-Commit-Position: refs/heads/master@{#828371}
-
Etienne Bergeron authored
This CL is adding unittests to ensures the layout is recomputed when the behavior or whitespace elision is updated. This fix is cleaning a TODO left in the code. R=msw@chromium.org Bug: 1085014 Change-Id: I803b6285bc03771b5026db63950bd60ebee1231c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2545222Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: Etienne Bergeron <etienneb@chromium.org> Cr-Commit-Position: refs/heads/master@{#828370}
-
Jeffrey Kardatzke authored
This was not being handled correctly before. We now require passing back in the prior decoded slice data so that dependent slice header parsing will be correct. Previously we relied on the decoder to do this, but even then we didn't handle all the fields properly. Fixes new fuzzer case and removes need for prior fuzzer fix in this area. This also fixes a range check issue where values needed to be in the 0 to 2^15 - 1 range for a few things that fixes another fuzzer case. BUG=b:153111783,chrome:1149206,chrome:1148863,chrome:1149205 TEST=H265 playback still works, unit/fuzzer tests pass Change-Id: I722dd5d2996685fd716f2218b3d9031a51fc4c79 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2542946 Commit-Queue: Jeffrey Kardatzke <jkardatzke@google.com> Reviewed-by:
Sergey Volk <servolk@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#828369}
-
edchin authored
Design doc: go/bling-phishguard PASSWORD_REUSE_DETECTION_ENABLED is only defined for !is_ios. This CL removes the use of this flag in files that are already not compiled in iOS. Bug: 1147962 Change-Id: Ibf646ea2536ceb15de4dd5e15c140bbbe2b5269a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543803Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Commit-Queue: edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#828368}
-
Mounir Lamouri authored
The feature is enabled by default and the flag is no longer required. Change-Id: Ifd7825e03edc2b166209f232bd1b5bd40a7f062a Fixed: 1112537 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2538352Reviewed-by:
dpapad <dpapad@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Commit-Queue: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#828367}
-
Luciano Pacheco authored
Fix the export of `MediaSessionPlaybackState` to export the whole enum. Bug: 1133186 Change-Id: I67897f77d34741e9bf343c157b839fd6f282243f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543648 Commit-Queue: Luciano Pacheco <lucmult@chromium.org> Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
Jeremie Boulic <jboulic@chromium.org> Cr-Commit-Position: refs/heads/master@{#828366}
-
Marijn Kruisselbrink authored
This reverts commit 84cf3e38. Reason for revert: Some of the added tests have been very flaky since landing. The worst being All/ExperimentsManagerESAE2ETest.FetchingExperiments/0, see for example on https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=All%2FExperimentsManagerESAE2ETest.FetchingExperiments%2F0&testType=gcp_unittests Example test output: ../../chrome/credential_provider/gaiacp/experiments_manager_unittests.cc(191): error: Expected equality of these values: experiment1_value Which is: "abc" ExperimentsManager::Get()->GetExperimentForUser( base::UTF16ToUTF8(OLE2W(sid)), Experiment::TEST_CLIENT_FLAG) Which is: "false" Stack trace: Backtrace: credential_provider::testing::ExperimentsManagerESAE2ETest_FetchingExperiments_Test::TestBody [0x00007FF624EC0ED6+3350] (o:\chrome\credential_provider\gaiacp\experiments_manager_unittests.cc:189) testing::internal::HandleExceptionsInMethodIfSupported<testing::Test,void> [0x00007FF6252EE67F+79] (o:\third_party\googletest\src\googletest\src\gtest.cc:2674) ../../chrome/credential_provider/gaiacp/experiments_manager_unittests.cc(194): error: Expected equality of these values: experiment2_value Which is: "def" ExperimentsManager::Get()->GetExperimentForUser( base::UTF16ToUTF8(OLE2W(sid)), Experiment::TEST_CLIENT_FLAG2) Which is: "false" Stack trace: Backtrace: credential_provider::testing::ExperimentsManagerESAE2ETest_FetchingExperiments_Test::TestBody [0x00007FF624EC1040+3712] (o:\chrome\credential_provider\gaiacp\experiments_manager_unittests.cc:192) testing::internal::HandleExceptionsInMethodIfSupported<testing::Test,void> [0x00007FF6252EE67F+79] (o:\third_party\googletest\src\googletest\src\gtest.cc:2674) Original change's description: > Add support for fetching experiments > > Bug: 1143836 > Change-Id: I0c787bf1c663b28d94e67c8015910dbcb48a4779 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2508633 > Commit-Queue: Yusuf Sengul <yusufsn@google.com> > Reviewed-by: Rakesh Soma <rakeshsoma@google.com> > Cr-Commit-Position: refs/heads/master@{#827540} TBR=yusufsn@google.com,rakeshsoma@google.com,wrlewis@google.com,ssanapala@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1143836 Change-Id: Iad3912ce90d3d7f95a18b5747a8a6efbc68b11e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544934Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Yusuf Sengul <yusufsn@google.com> Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Commit-Queue: Yusuf Sengul <yusufsn@google.com> Cr-Commit-Position: refs/heads/master@{#828365}
-
Yu Su authored
Bug: 1148031 Change-Id: I7e44ba488428a31479728d8c6d267070c1043edb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2542950Reviewed-by:
David Schinazi <dschinazi@chromium.org> Commit-Queue: Yu Su <yuyansu@google.com> Cr-Commit-Position: refs/heads/master@{#828364}
-
Tarun Bansal authored
Bug: 989569 Change-Id: I0b669beade05c7d9d4e9af3215712868e149207a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2521882 Commit-Queue: Theresa <twellington@chromium.org> Auto-Submit: Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#828363}
-
Xi Han authored
When Instant Start is enabled, the calculateButtonData() is called pre-native and early exits without calling the IdentityDiscController#ensureProfileDataCache(). As a result, the IdentityDiscController misses the event of the onProfileDataUpdated() and thus doesn't notify its observer StartSurfaceToolbarMediator to update the visibility of the identity disc button. To fix the issue, we cache the call of calculateButtonData() from the Start surface if it is called pre-native. When a new Profile is set, we check if a cached request exists, and call the calculateButtonData() if yes. Thus, it guarantees to update the button data if Profile has updated, as well as to notify Start Surface to update the visibility of the button. Bug: 1137535 Change-Id: I3aca4434ecc74850ee226affd3c3425931cf02a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533522Reviewed-by:
Pavel Yatsuk <pavely@chromium.org> Commit-Queue: Xi Han <hanxi@chromium.org> Cr-Commit-Position: refs/heads/master@{#828362}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/f69ec827f69b..70f0ab5795d6 2020-11-17 sigurds@chromium.org Fix usage of style.setProperty 2020-11-17 andoli@chromium.org Refactor ui/Action.js into ui/ActionRegistration.js and update dependencies accordingly 2020-11-17 andoli@chromium.org Remove treeElement property bound to Node intances 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,chromium:1134103 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I0b3dcfb42cbfb87334740272a54cd4b5a701403c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544031Reviewed-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@{#828361}
-
Ian Kilpatrick authored
When flex did a 2-pass layout (once to determine the intrinsic size, and a followup to shrink - and the block-size changed, we'd just perform a simplified layout pass. The fieldset layout algorithm "stretches" its content to its block-size upfront. Due to this using the simplified layout pass resulted in an incorrect layout, never correctly sizing the anonymous fieldset content. This patch disables the simplified layout pass for fieldsets. Bug: 1149714 Change-Id: I454a65363c019240dd4b30531009a13e8b596ea4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544755Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#828360}
-
Jan Krcal authored
This CL unifies the size of avatar icons in the Profiles native menu on Mac with the Chrome profile menu. Bug: 857064 Change-Id: Ie0336b272506ba10972921261eda8b055372de1f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2545162 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Commit-Queue: David Roger <droger@chromium.org> Auto-Submit: Jan Krcal <jkrcal@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#828359}
-
Liviu Tinta authored
Bug: 989958 Change-Id: I2faf1553f355290838b6467e8a0fa1c6baf97531 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544025Reviewed-by:
Caitlin Fischer <caitlinfischer@google.com> Commit-Queue: Caitlin Fischer <caitlinfischer@google.com> Cr-Commit-Position: refs/heads/master@{#828358}
-
Sebastien Lalancette authored
Realized there were no tests for this action in that context. Also added coverage for the fix for the associate bug. Bug: 1140517 Change-Id: Ia16ff23d3dd4c95d35dcefe9d760305f058e5341 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544048Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: Sebastien Lalancette <seblalancette@chromium.org> Cr-Commit-Position: refs/heads/master@{#828357}
-
edchin authored
Design doc: go/bling-phishguard PASSWORD_REUSE_DETECTION_ENABLED is only defined for !is_ios. This CL removes the use of this flag in files that are already not compiled in iOS. Bug: 1147962 Change-Id: I3ac6fca7db743442645ebe0bde4dda6a5a3fa554 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543584Reviewed-by:
Bettina Dea <bdea@chromium.org> Commit-Queue: edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#828356}
-
Edward Lesmes authored
Generate DIR_METADATA files and remove metadata from OWNERS files for directories under //chrome. Bug: 1113033 Change-Id: Ia64ed766a212bd6399273909a0e4b95d6d07bce8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544925 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#828355}
-
Becca Hughes authored
Change-Id: I64be99576fe6557dd9adc7c987c3c113254be6fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544076Reviewed-by:
Tommy Steimel <steimel@chromium.org> Commit-Queue: Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#828354}
-
Yann Dago authored
Bug: 1095223 Change-Id: Idf5089542203d6541661d89a0254b958a4960d3e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543132 Commit-Queue: Yann Dago <ydago@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Auto-Submit: Yann Dago <ydago@chromium.org> Reviewed-by:
Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#828353}
-
Alice Gong authored
Bug: 1148561 Change-Id: I77051c7af4f9ca67d954d31d2ce7a596b2b89a58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535670Reviewed-by:
Owen Min <zmin@chromium.org> Reviewed-by:
Albert J. Wong <ajwong@chromium.org> Commit-Queue: Alice Gong <alicego@google.com> Cr-Commit-Position: refs/heads/master@{#828352}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/9abde8c3f3a2..90b4c0f76b33 2020-11-17 apolito@google.com add support for main in git_cache 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 sokcevic@google.com,ajp@google.com,apolito@google.com,ehmaldonado@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: sokcevic@google.com,ajp@google.com,apolito@google.com,ehmaldonado@google.com Change-Id: I0195fa990c3524340cc3bec6cfb2ad451ab25d09 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544029Reviewed-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@{#828351}
-
Daniel Hosseinian authored
Pass in a base::GUID in lieu of a std::string GUID. Bug: 1026195 Change-Id: Ie76c20a61bc8b42aeadbf41721b7d2666bef0f27 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528632 Commit-Queue: Daniel Hosseinian <dhoss@chromium.org> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#828350}
-
Edward Lesmes authored
Generate DIR_METADATA files and remove metadata from OWNERS files for directories under //chrome. Bug: 1113033 Change-Id: I942c8de231523281ee6840aae14d367972c8702a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544754Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Ted Choc <tedchoc@chromium.org> Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Cr-Commit-Position: refs/heads/master@{#828349}
-
Daniel Murphy authored
This fixes an issue that I caused by my previous patch where file handlers are not enabled by default, which is incorrect. It also addresses a few issue that loyso@ found in my previous CL: https://chromium-review.googlesource.com/c/chromium/src/+/2505685 R=phillis@chromium.org Bug: 1149981 Change-Id: I70221ff76ef576fb6f09ab89d3db0d90ffe30a60 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544843 Commit-Queue: Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Phillis Tang <phillis@chromium.org> Auto-Submit: Daniel Murphy <dmurph@chromium.org> Cr-Commit-Position: refs/heads/master@{#828348}
-
Tarun Bansal authored
Add UMA metrics to connectivity detector. Also, add some adb logging to make it easier to understand the functionality of connectivity detector. Finally, prevent the retry delay from overlflowing. Change-Id: I3de32e01c20aeef04648685bb074012618e73831 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514065 Commit-Queue: Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Jian Li <jianli@chromium.org> Cr-Commit-Position: refs/heads/master@{#828347}
-
chromium-autoroll authored
https://dawn.googlesource.com/tint.git/+log/20a8d31849c9..8f26453c7db4 2020-11-17 dsinclair@chromium.org Fixup SPIR-V include of builder 2020-11-17 dsinclair@chromium.org Add missing override 2020-11-17 bclayton@google.com writer/spirv: Inherit TestHelperBase from ast::Builder 2020-11-17 bclayton@google.com ast::Builder: Add more builder methods 2020-11-17 bclayton@google.com ast::Builder: Make the interface more fluent 2020-11-16 dneto@google.com spirv-reader: pretend handles are void 2020-11-16 rharrison@chromium.org Stop setting mac_xcode_version in DEPS 2020-11-16 dsinclair@chromium.org Fixup include 2020-11-16 dneto@google.com spirv-reader: better message for unknown SPIR-V type 2020-11-16 dneto@google.com spirv-reader: add conversions for image Dim and Format 2020-11-16 bclayton@google.com Big cleanup now that AST nodes are raw pointers 2020-11-16 bclayton@google.com Replace use of std::unique_ptr<T> with T* for AST nodes 2020-11-16 bclayton@google.com Replace remaining std::make_unique<T> -> create<T> for ast::Nodes 2020-11-16 bclayton@google.com src/transform: Replace std::make_unique<T> -> create<T> 2020-11-16 bclayton@google.com reader/wgsl: Replace std::make_unique<T> -> create<T> 2020-11-16 dj2@everburning.com [spirv-writer] Fixup generation of temporary variable. 2020-11-16 bclayton@google.com reader/spirv: Replace std::make_unique<T> -> create<T> 2020-11-16 bclayton@google.com ast: Remove no-arg constructor for ast::IfStatement 2020-11-16 dj2@everburning.com [spirv-writer] Generate loads for const constructors. 2020-11-16 dsinclair@chromium.org Add language check 2020-11-16 dj2@everburning.com [ast] Add the result_type into the AST dump 2020-11-16 sarahmashay@google.com [validation] add error number to invalid member accessor error: v-0007 2020-11-16 bclayton@google.com ast: Remove no-arg constructor for ast::CastStatement 2020-11-14 bclayton@google.com ast: Remove no-arg constructor for ast::ElseStatement 2020-11-14 bclayton@google.com ast: Remove no-arg constructor for ast::LoopStatement 2020-11-14 bclayton@google.com ast: Add body parameter to ast::Function constructors 2020-11-14 bclayton@google.com writer/wgsl tests: Replace std::make_unique<T> -> create<T> 2020-11-14 bclayton@google.com writer/spirv tests: Replace std::make_unique<T> -> create<T> 2020-11-13 bclayton@google.com writer/msl tests: Replace std::make_unique<T> -> create<T> 2020-11-13 bclayton@google.com writer/hlsl tests: Replace std::make_unique<T> -> create<T> 2020-11-13 bclayton@google.com validator tests: Replace std::make_unique<T> -> create<T> 2020-11-13 bclayton@google.com transform tests: Replace std::make_unique<T> -> create<T> 2020-11-13 bclayton@google.com inspector_test: Replace std::make_unique<T> -> create<T> 2020-11-13 bclayton@google.com type_determiner_test: Replace std::make_unique<T> -> create<T> 2020-11-13 bclayton@google.com ast/types tests: Replace std::make_unique<T> -> create<T> 2020-11-13 bclayton@google.com ast tests: Replace std::make_unique<T> -> create<T> 2020-11-13 bclayton@google.com ast/type: Have all tests derive from new TestHelper 2020-11-13 dsinclair@chromium.org [spirv-writer] Generate load of conditionals. 2020-11-13 bclayton@google.com ast: Have all tests derive from new TestHelper 2020-11-13 bclayton@google.com ast: Have all AST types derive from ast::Node 2020-11-13 dsinclair@chromium.org Place the namer into the context object. 2020-11-12 rharrison@chromium.org [inspector] Add Multisampled Texture data extraction If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/tint-chromium-autoroll Please CC kainino@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:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel Bug: chromium:1147839 Tbr: kainino@google.com Change-Id: I35d93bb01a6b8d2bec699493e5465c70534728dc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543490Reviewed-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@{#828346}
-
Anthony Vallee-Dubois authored
Bug: None Change-Id: I5ce839a67e9b05bc90641c90b7b24944efd68c47 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2542606 Commit-Queue: Caitlin Fischer <caitlinfischer@google.com> Reviewed-by:
Caitlin Fischer <caitlinfischer@google.com> Cr-Commit-Position: refs/heads/master@{#828345}
-
Edward Lesmes authored
Generate DIR_METADATA files and remove metadata from OWNERS files for directories under //chrome/browser/ui. Bug: 1113033 Change-Id: Ie4670d92e06d1c02f251590c9a26bb5a9a65a818 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2545064 Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#828344}
-
Joe Downing authored
The original file logger CL did not include PID/TID in the log output which I had intended to do but didn't include. This CL adds that data to each log statement. Change-Id: I6767db2c8962283ff1f6890466f02eb17c6621d3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2544747 Commit-Queue: Joe Downing <joedow@chromium.org> Reviewed-by:
Jamie Walch <jamiewalch@chromium.org> Cr-Commit-Position: refs/heads/master@{#828343}
-
Kyle Horimoto authored
Implement OnPayloadReceived() by: (1) Verifying that the received payload is represented in bytes rather than a file (the phone will never send a file). (2) Converting the received bytes into a string and passing it to the NearbyMessageReceiver interface. Tweak sending payloads by generating a random payload ID for each message sent rather than starting a count from 0 for each connection. Bug: 1149576, 1106937 Change-Id: I2310c17df2cd854648c141cda0066f83089a54d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543682 Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
James Vecore <vecore@google.com> Cr-Commit-Position: refs/heads/master@{#828342}
-
Scott Violet authored
Specifically, blocking only happens while in CCT and only if a parameter is supplied along with the intent. The ability to do this will be restricted to certain apps. BUG=1149582 TEST=CustomTabActivityTest and NotificationPermissionContextTest Change-Id: Ieb6d76ff2326406872764973ff31d81d8e088449 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2538359 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Peter Conn <peconn@chromium.org> Cr-Commit-Position: refs/heads/master@{#828341}
-
Josip Sokcevic authored
Generate DIR_METADATA files and remove metadata from OWNERS files for directories under //content. R=boliu@chromium.org Bug: 1113033 Change-Id: Iab77bab240ce5482ad7ae97ae5c37555f15f0241 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543250Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Auto-Submit: Josip Sokcevic <sokcevic@google.com> Cr-Commit-Position: refs/heads/master@{#828340}
-
Jackson Tadie authored
We use the same CSS style for both now and don't need an independent CSS for the demo dialog. Bug: b:172388847 Change-Id: Id1f0510c30a09806d0d8487b5e7287bf435c0ae7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533393Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Reviewed-by:
Yves Arrouye <drcrash@chromium.org> Reviewed-by:
Aga Wronska <agawronska@chromium.org> Commit-Queue: Jackson Tadie <jacksontadie@google.com> Cr-Commit-Position: refs/heads/master@{#828339}
-
Ken Rockot authored
We don't use the Service Manager anymore and the relevant bits here (e.g. download manager) no longer have anything to do with the Service Manager. To avoid confusion as Service Manager code is removed, symbol names and documentation have been renamed here. A few symbols have not been renamed despite having potentially misleading names now, because they correspond to feature flag strings which may need to retain their existing values. Bug: 977637 Change-Id: I017c02f026e52f68e844012883b0d3856c518524 Tbr: carlosk@chromium.org Tbr: cduvall@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2541662Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#828338}
-
Marijn Kruisselbrink authored
This reverts commit 269f7161. Reason for revert: Tests are failing on MSAN https://ci.chromium.org/p/chromium/builders/ci/Linux%20MSan%20Tests/26262 Original change's description: > cc: Set up a Flag for Displaying Performance Metrics > > Set up a feature flag and corresponding state inside cc to enable > using HUD for displaying performance metrics. Currently when we enable > the show performance metrics flag we only add WebVitals metrics, which > comes from Blink. We might want to add other metrics in the future. > > R=sadrul > > Bug: 1149385 > Change-Id: I437e113a5fc4f44c6fbaa542c92d1f2da6cbb393 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2540570 > Commit-Queue: weiliangc <weiliangc@chromium.org> > Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org> > Reviewed-by: David Bokan <bokan@chromium.org> > Cr-Commit-Position: refs/heads/master@{#828221} TBR=sadrul@chromium.org,bokan@chromium.org,weiliangc@chromium.org Change-Id: Ie5b290290078f669f487f8792020fe4e822798fe No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1149385 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2545250Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Cr-Commit-Position: refs/heads/master@{#828337}
-
Tanmoy Mollik authored
This cl moves the logic for Signin Promo from SignInPreference to SyncPromoPreference. Briefly the main changes in this cl are: 1. SignInPreference no longer has reference to SigninPromoController. 2. Introduce a new state in SignInPreference to hide it when promos are shown by SyncPromoPreference. 3. setChangedCallBack() in SignInPreference has been moved to SyncPromoPreference. This callback is used by the MainSettings page to hide "Account" category section when signin promo is shown. Bug: 1110889 Change-Id: I5c0805c30f5bde04cde79d9135aef4fdaf382df1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2505144Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org> Cr-Commit-Position: refs/heads/master@{#828336}
-