- 02 Nov, 2020 40 commits
-
-
Rohit Agarwal authored
This CL removes the deprecated Profile#getOffTheRecordProfile method which invoked Profile#getPrimaryOTRProfile internally. The remaining callers of getOffTheRecordProfile are updated to directly call getPrimaryOTRProfile instead. The code requiring primary OTR profile would rely on invoking the method Profile#getPrimaryOTRProfile explicitly instead. Bug: 1060940 Change-Id: I72301c22b48e51262678673cbd4066a835c76089 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2513650 Commit-Queue: Rohit Agarwal <roagarwal@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#823190}
-
Matt Jones authored
This patch migrates all of the layout/compositor-specific animation classes and tests to the new "layouts" module. Bug: 1070281 Change-Id: If9fb504933f1e0adcc38a2e4b34800f7193a572d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2503830 Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by:
Mei Liang <meiliang@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#823189}
-
Tanmoy Mollik authored
This cl creates a version of the SigninPromo Test when MOBILE_IDENTITY_CONSISTENCY is enabled. Bug: 1142467 Change-Id: I5c7d6503c5e6c2867a43713a0ba2f957f541513e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2505811Reviewed-by:
-
Leo Lai authored
Now we are using AttestationClient directly instead of passing through CryptohomeClient. BUG=b:158955123 TEST=build ok. Change-Id: I02646e7e9b7428a5f9ad41a07e80ef00ca5ef2e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2505394 Commit-Queue: Leo Lai <cylai@google.com> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#823187}
-
Andre Le authored
Update battery icon in phone status to indicate phone's battery saver mode. This includes: - Moving BatteryImageSource in ash/system/power to a public class so that Phone Hub can extends. - Creating PhoneHubBatteryImageSource and showing battery saver mode. Bug: 1106937,1126208 Change-Id: I93bbf244148c1dc51fe5d5ee18293da2fff9c0ea Fixed: 1143374 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2507384Reviewed-by:
Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Tim Song <tengs@chromium.org> Commit-Queue: Andre Le <leandre@chromium.org> Cr-Commit-Position: refs/heads/master@{#823186}
-
Dominique Fauteux-Chapleau authored
This is to better represent what the function actually does. This is only a rename and doesn't change any existing behaviour. Bug: 1098437 Change-Id: I06888a8e6994dcaf327c48e14681b953a8cd5dfe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510194 Commit-Queue: Dominique Fauteux-Chapleau <domfc@chromium.org> Reviewed-by:
Daniel Rubery <drubery@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Cr-Commit-Position: refs/heads/master@{#823185}
-
Nicolas Ouellet-Payeur authored
This is needed to enable DestroyProfileOnBrowserClose by default in automated tests (crrev.com/c/2503629). ConfigurationRefresher did not clean up its SyncObserver properly when it received the OnSyncShutdown() event, causing a failed DCHECK() in browser tests. Bug: 88586 Change-Id: Ib37846afd26470563d58639e9fddff4928ffe907 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510797Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Commit-Queue: Nicolas Ouellet-Payeur <nicolaso@chromium.org> Cr-Commit-Position: refs/heads/master@{#823184}
-
Dave Tapuska authored
The response from CreatePopupWidget is no longer used. So this method can be async. The browser will still allocate routing IDs for popup widgets though because there are a few global tables that it uses to track widgets based on the routing ID. BUG=1097816 Change-Id: I4a2dea41e7c796423d7072a72753b4e168f8261d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510790Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#823183}
-
Mirko Bonadei authored
Without this, the test fails to run because the filter file doesn't get isolated. Bug: 1143697 Change-Id: I810319036b68e5cd073652c1abc800f9f6eee1fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514150Reviewed-by:
Olga Sharonova <olka@chromium.org> Commit-Queue: Mirko Bonadei <mbonadei@chromium.org> Cr-Commit-Position: refs/heads/master@{#823182}
-
Francois Doray authored
My team is working on power consumption at least for the next year. This CL extends the expiry of histograms that we'll use to track our efforts. Fixed: 1138308 Change-Id: Ie4c59081f3cac36b99b952ec5c88bdabbe16aa87 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510780 Auto-Submit: François Doray <fdoray@chromium.org> Commit-Queue: Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Cr-Commit-Position: refs/heads/master@{#823181}
-
Leo Lai authored
FakeCryptohomeClient doesn't have any exsiting test items after re remove TpmAttestationSignSimpleChallenge. BUG=b:158955123 TEST=build ok. Change-Id: Ib533e261d1733f4449e2b3a783757279a35eefdb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2504011 Commit-Queue: Leo Lai <cylai@google.com> Reviewed-by:
-
chromium-autoroll authored
https://dawn.googlesource.com/tint.git/+log/5bee67fced90..74023424daa4 2020-11-02 dneto@google.com Remove stale TODO 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 sarahmashay@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: sarahmashay@google.com Change-Id: If9b9664e8b9de096024f38efc3df20bf37d7bbde Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514234Reviewed-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@{#823179}
-
Nicolas Ouellet-Payeur authored
This is needed to enable DestroyProfileOnBrowserClose by default in automated tests (crrev.com/c/2503629). This caused issues during shutdown in browser tests, because TestSyncService behaved differently than ProfileSyncService. Bug: 88586 Change-Id: Ie3c6f202d251411eb316edd77180855ec0e9bc09 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2511072Reviewed-by:
-
Dominique Fauteux-Chapleau authored
This CL refactors tests in deep_scanning_browsertest.cc to use Connector policies directly instead of the indirect helpers that map to legacy policies. It also removes now unused DeepScanningClient{Request|Response} references from those tests. Bug: 1109242 Change-Id: I39f7d0cb1231d8f494ac7c58460eafee91691a54 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510239Reviewed-by: -
Jérôme Lebel authored
When the sign-in is successful in add_account_signin_coordinator.mm, an identity is needed. Change-Id: Ie1c930a7f07f4b1c14ae1a48c638df7c4644aa0f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514398 Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
Nohemi Fernandez <fernandex@chromium.org> Cr-Commit-Position: refs/heads/master@{#823176}
-
Jan Krcal authored
This is a reland of d42167eb Compared to the original CL, this CL minimizes the changes in the test that was failing on Win7. Original change's description: > [Dice sync delegate] Split sync confirmation and sync disabled > > This CL splits the ShowSyncConfirmation() function of the dice turn on > sync helper delegate into one that shows the normal sync confirmation > and one that shows a error message that sync is disabled by the admin. > > Since both the dialogs are implemented by the same webUI page, there > is not much behavioral change. The only change is for the delegate > for the signed-in profile creation flow which opens a new browser > window and shows the dialog in this window instead of showing it in > the creation flow window. > > In follow-up CLs, the variant of the UI will be passed as a param > to the WebUI page replacing the logic to decide which variant to show. > > Bug: 1126913 > Change-Id: Ibac9a339c4fb071f960f15864482c818247f5719 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2487100 > Commit-Queue: Jan Krcal <jkrcal@chromium.org> > Reviewed-by: David Roger <droger@chromium.org> > Auto-Submit: Jan Krcal <jkrcal@chromium.org> > Cr-Commit-Position: refs/heads/master@{#820222} Bug: 1126913 Change-Id: I741140df31b8ca91fe5eb1a05a643fc0d2401f58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2495069 Auto-Submit: Jan Krcal <jkrcal@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: Jan Krcal <jkrcal@chromium.org> Cr-Commit-Position: refs/heads/master@{#823175}
-
Glenn Hartmann authored
BUG=1140066 Change-Id: I68726e9d31bf935e1bba04a8b07f8e86057b3e32 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510529 Auto-Submit: Glenn Hartmann <hartmanng@chromium.org> Reviewed-by:
Ella Ge <eirage@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Jesse Doherty <jwd@chromium.org> Cr-Commit-Position: refs/heads/master@{#823174}
-
chromium-autoroll authored
This CL may cause a small binary size increase, roughly proportional to how long it's been since our last AFDO profile roll. For larger increases (around or exceeding 100KB), please file a bug against gbiv@chromium.org. Additional context: https://crbug.com/805539 Please note that, despite rolling to chrome/android, this profile is used for both Linux and Android. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/afdo-chromium-autoroll Please CC gbiv@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/+doc/master/autoroll/README.md Tbr: gbiv@chromium.org Change-Id: Ic782c3df395360cc3c6c01540edcabb68a8bcbdd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514173Reviewed-by:
-
Peter Kasting authored
Bug: 772945 Change-Id: I0d5e8a37d2c24c165737359db6f7c4af85ea0774 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2507638 Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by:
Jon Mann <jonmann@chromium.org> Auto-Submit: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#823172}
-
Javier Ernesto Flores Robles authored
- Wires the widget commands to the respective actions - Logs start action for widget - Adds TODOs in the places where specific action metrics will go Bug: 1138719 Change-Id: I2a10d029b0f0b8b1b100cc37b7a93274c939a33c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2511850 Commit-Queue: Javier Flores <javierrobles@chromium.org> Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#823171}
-
chromium-autoroll authored
Roll Chrome Win64 PGO profile from chrome-win64-master-1604307096-e67db33a1b3e023c027e084a5a169d3ae943db07.profdata to chrome-win64-master-1604318145-d42776d1d9d08b6fa8d7b373c065b46395fc90ba.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-win64-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:win64-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: I07d11bb48f96087a5fc7dfede876022d3583352c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514196Reviewed-by:
-
Justin Cohen authored
Bug: 1144212 Change-Id: Ifd5e3be45c8e2d9c9fb6195681a00657e28bc0a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510238Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium ab23b358. With Chromium commits locally applied on WPT: 074062ef "css-pseudo: :first-line should not affect inline-blocks" Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: bajones@chromium.org: external/wpt/gamepad foolip@chromium.org, lpz@chromium.org, robertma@chromium.org: external/wpt/tools foolip@chromium.org, robertma@chromium.org, smcgruer@chromium.org: external/wpt/infrastructure hongchan@chromium.org, rtoy@chromium.org: external/wpt/webaudio NOAUTOREVERT=true TBR=robertma@google.com No-Export: true Cq-Include-Trybots: luci.chromium.try:linux-wpt-identity-fyi-rel,linux-wpt-payments-fyi-rel Change-Id: I50231f90bb7bdd274320f8a2d1cb955934f36d8e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2512597Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#823168}
-
chromium-internal-autoroll authored
Release_Notes: http://go/media_app-x20/relnotes/Main/media_app_202011020600_RC00.html https://chrome-infra-packages.appspot.com/p/chromeos_internal/apps/media_app/app/+/NDZtDGoSjqh7JHOfYKjuAocve7OHond9WDS2YlbZUgEC If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/media-app-chromium-autoroll Please CC media-app@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 Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: b/165720635,b/170271605 Tbr: media-app@grotations.appspotmail.com Change-Id: I0f2505cbb3c0d8867dcd18ec46293b6bed061b83 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514231Reviewed-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@{#823167}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/6ba009e8ab84..5a41e47b6f39 2020-11-02 jobay@google.com Use Puppeteer's built-in `pierce/` handler 2020-11-02 alexrudenko@chromium.org Enable TypeScript for BackgroundServiceView.js 2020-11-02 yangguo@chromium.org [ts] Typecheck ExecutionContextSelector.js 2020-11-02 yangguo@chromium.org [ts] Typecheck CSSShadowModel.js 2020-11-02 jobay@google.com Roll Puppeteer from 5.3.1 to 5.4.1 2020-11-02 mathias@chromium.org Fix service worker capitalization 2020-11-02 yangguo@chromium.org [ts] Typecheck IsolatedFileSystemManager.js 2020-11-02 yangguo@chromium.org [ts] Typecheck BezierEditor.js 2020-11-02 changhaohan@chromium.org TypeScriptify elements/StylesSidebarPane.js part 2 2020-11-02 sigurds@chromium.org Refactor setColumnsVisiblity to use a set of string 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:1093296 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I2d8f3249f371366d710a546c12ad1afef13d6a81 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514230Reviewed-by:
-
Mark Yacoub authored
1. Change is_disabled variables in CRTC Controller and Hardware Display Controller to positive connotation(is_enabled). 2. HDC doesn't have its own enable state now; instead, it checks the states of all CRTC controllers it manages. BUG=b/172069469 TEST=ozone_unittests, modesetting and disabling an external monitor Change-Id: I0f613616ef4dd219298432904c4acabf5f5335ab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510690 Commit-Queue: Mark Yacoub <markyacoub@google.com> Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Cr-Commit-Position: refs/heads/master@{#823165}
-
Rob Clark authored
Different tracepoint, but on the kernel side, it intentially re-uses the intel_gpu_freq_change format. TEST=Locally, confirm chrome://arc-overview-tracing shows GPU freq on snapdragon devices BUG=b:145579089 Change-Id: I70aae6397736d14a0f59db9b41614aeefbc0bf48 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2451649Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Achuith Bhandarkar <achuith@chromium.org> Auto-Submit: Rob Clark <robdclark@chromium.org> Commit-Queue: Achuith Bhandarkar <achuith@chromium.org> Cr-Commit-Position: refs/heads/master@{#823164}
-
Matthias Körber authored
Change-Id: I4a0afe4bfc41a2ab18514953a2be4fe61e11c2ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2508589 Commit-Queue: Matthias Körber <koerber@google.com> Reviewed-by:
Christoph Schwering <schwering@google.com> Cr-Commit-Position: refs/heads/master@{#823163}
-
chromium-autoroll authored
Roll ChromeOS Orderfiles from 88-4296.0-1603709458-benchmark-88.0.4305.0-r1 to 88-4296.0-1603709458-benchmark-88.0.4307.0-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: I15c9e25df7b1ba23145d15d4419bc07b44e44538 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514167Reviewed-by:
-
Anders Hartvoll Ruud authored
Until now we've only supported a single compound selector as an argument to :not(). This CL adds support for complex selectors, based on the recent work on :is() and :where(). Parsing ------- Note that unlike :is() and :where(), :not() does not use the forgiving selector list behavior, but instead fails parsing the normal way. CSSSelectorParser now has both ConsumeNestedSelectorList and ConsumeForgivingNestedSelectorList for the two behaviors. Our ShadowDOMv0-disabling logic should apply to complex :not() as well, but for backwards compatibility, we need to allow it whenever :not() _doesn't_ have a complex argument. The TreatAsNonComplexArgumentToNot() was added to detect this, which replaces the previous function CSSParserSelector::IsSimple. (When v0 is removed, we can remove this mess as well). Invalidation ------------ There was a bug in ExtractInvalidationSetFeaturesFromSelectorList: we're supposed to ignore any extracted features inside a :not(). For example, for ".a :not(.b)", we should not add ".b" to the invalidation set for ".a". In ExtractInvalidationSetFeaturesFrom- SelectorList there was code that attempted to block the inner features from "crossing" the :not()-boundary, but it incorrectly used the *outer* pseudo-type. In other words, when considering ":not(.b)", the pseudo-type would be kUnknown, since we are not (yet) inside the :not(). Instead what we need to do is check the pseudo-type of of the currently held simple selector. Now that we "block" inner features correctly at the :not() boundary, the check in ExtractInvalidationSetFeaturesFromCompound is not needed anymore. This in turns means we can remove several PseudoType params, which is nice. Matching -------- Now traverses a CSSSelectorList and uses MatchSelector. Beyond that it just works. Bug: 1139865 Change-Id: Iea5513445e54303ba0685cffc3b52dabc115b7f1 I2S: https://groups.google.com/a/chromium.org/g/blink-dev/c/0alTkXvDCL8/m/-ClOGvOJBAAJ Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2507549Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Anders Hartvoll Ruud <andruud@chromium.org> Cr-Commit-Position: refs/heads/master@{#823161}
-
Ben Mason authored
TBR=govind@chromium.org Change-Id: Iba66ea8f88598ce5de73405d7f7ccbc90ad8dc79 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514457Reviewed-by:
Ben Mason <benmason@chromium.org> Commit-Queue: Ben Mason <benmason@chromium.org> Cr-Commit-Position: refs/heads/master@{#823160}
-
Ben Mason authored
This reverts commit e859a79a. Reason for revert: IDS_NEARBY_CONTACT_VISIBILITY_OWN_SOME.png.sha1 is not a sha file. Original change's description: > [Nearby] Strings updates on contact visibility page. > > Removes the "Contacts" title and updates strings in the description for > each visibility mode to match the latest specs. > > Screenshots: > https://screenshot.googleplex.com/An6Ta8EDzfS3KZU.png > https://screenshot.googleplex.com/C5WwqifMeCnLhkH.png > https://screenshot.googleplex.com/C48HQVfgouJrhHF.png > > Strings spec: > https://docs.google.com/document/d/1s-KjC1mM4ZPYeYAO7oI8lXltS_1FaYYfMMnNDdbfRag > > Bug: b:168841167 > Change-Id: I5facb117242e22b838266c2fa5e2629b221e6794 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2511852 > Reviewed-by: James Vecore <vecore@google.com> > Commit-Queue: Michael Hansen <hansenmichael@google.com> > Cr-Commit-Position: refs/heads/master@{#822914} TBR=vecore@google.com,hansenmichael@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: b:168841167 Change-Id: Ie19aa888c3914ac3e65fb330000199023e2a3b54 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514420Reviewed-by:
-
Yuki Shiino authored
Includes appropriate headers in case of not only typed arrays but also array buffer views, instead of using forward declarations. Change-Id: Ib214732bc8bbceab476cca78a9d6327919c1d4a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2512501Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#823158}
-
Leo Lai authored
We are deprecating attestation methods by CryptohomeClient. BUG=b:158955123 TEST=unit_tests. Change-Id: Ied7f9642ed3eb8d0c2fa1607468b31f8e139b7d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2505393 Commit-Queue: Leo Lai <cylai@google.com> Reviewed-by:
-
Matthias Körber authored
Change-Id: Id85a0ae43cdfab09a4bcc7bf31024b7f19e74c46 Bug: 1100231 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2507654 Commit-Queue: Matthias Körber <koerber@google.com> Reviewed-by:
-
Leo Lai authored
Now we are using AttestationClient directly instead of passing through CryptohomeClient. BUG=b:158955123 TEST=build ok. Change-Id: Ib4b7db0e640b5ff68de2ba79e80c749d09f45619 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2504010 Commit-Queue: Leo Lai <cylai@google.com> Reviewed-by:
-
Maksim Moskvitin authored
OAuth2ApiCallFlow doesn't provide a lot of value and requires putting traffic annotations in "OAuth 2.0" group, while for trusted vault requests "Chrome Sync" group. This CL refactors VaultServiceApiCallFlow into VaultServiceRequest, that uses SimpleURLLoader to serve the http requests. Bug: 1113598 Change-Id: I0bbd7c9efa10e9c90fc4eff14b6989dad528136d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2506577 Commit-Queue: Maksim Moskvitin <mmoskvitin@google.com> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Reviewed-by:
-
Jan Wilken Dörrie authored
This change increases the username hash prefix length used in the leak check logic from 24 bits to 27 bits. Taking test4 / test4 as an example, this change has the following impact on the response size: Before: PasswordManager.LeakDetection.SingleLeakResponsePrefixes: 1734.0 PasswordManager.LeakDetection.SingleLeakResponseSize: 27779.0 After: PasswordManager.LeakDetection.SingleLeakResponsePrefixes: 1347.0 PasswordManager.LeakDetection.SingleLeakResponseSize: 21587.0 Fixed: 1144689 Change-Id: Ia7ae40cdd2bdd4fd22c4c3fd5e7b59d8bd481e5d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2514146 Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#823153}
-
Greg Thompson authored
Differential updaters deliver a setup_patch.packed.7z that is applied to a previous version's setup.exe to generate the new setup.exe. The patch is no longer needed after it is applied, so delete it forthwith to reduce overall disk usage and avoid leaking it. Also, take care not to call DeleteFile with an empty string during cleanup. This could have happened in various error cases. BUG=516207 R=gab@chromium.org Change-Id: I0f446ea33662b4a448e5f7842787b3ff0523ad2e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2513587 Auto-Submit: Greg Thompson <grt@chromium.org> Commit-Queue: Gabriel Charette <gab@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#823152}
-
Francois Doray authored
This reverts commit 66c328cb. Reason for revert: We no longer plan to opt-out pages using WebRTC from normal throttling, since it causes power usage regression and is not needed by videoconferencing apps (which can use HTMLVideoElement.requestVideoFrameCallback to do video processing). Original change's description: > [blink scheduler] Disable all throttling on pages using WebRTC. > > A page that uses WebRTC may use a timer to poll video frames and do > video processing on them. Very often, these pages are already opted-out > from throttling because they play audio. However, to allow them to > work reliably when silent (e.g. when other participants stop talking > for a few seconds), this CL disables all throttling for pages using > WebRTC. > > In the long-term, there should be an event-based mechanism to do > video processing on frames when they become available. When this > mechanism is available, this policy will be revisited. > > This policy is controlled by a base::Feature so we can assess its > impact on performance. > > Bug: 1101731,b/159893852 > Change-Id: I6d7e98c544d65223fbbad41148ec201c83a63f7c > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2280161 > Reviewed-by: Florent Castelli <orphis@chromium.org> > Reviewed-by: Alexander Timin <altimin@chromium.org> > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Commit-Queue: François Doray <fdoray@chromium.org> > Cr-Commit-Position: refs/heads/master@{#787738} TBR=fdoray@chromium.org,haraken@chromium.org,altimin@chromium.org,orphis@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1101731 Bug: b/159893852 Change-Id: I7cf4f03bd5106779f8db83f64a46f38fa748aea5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510673Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
Florent Castelli <orphis@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#823151}
-
