- 23 Jan, 2019 40 commits
-
-
Gabriel Charette authored
I just learned about these guidelines through https://chromium-review.googlesource.com/c/chromium/src/+/1340802/18#message-b68ecbafe6212025dd856998736431eea6a19991 and think they should be surfaced in base/OWNERS (which points to base/README.md). R=dcheng@chromium.org Change-Id: I0acf3eec7a2d4e888eac4d15e9e8fbf8ad7acf02 Reviewed-on: https://chromium-review.googlesource.com/c/1422742Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#625211}
-
Avi Drissman authored
This is a followup to http://crrev.com/366963 . That change made the '--disable-web-security' flag have no effect unless the '--user-data-dir' flag was also specified. This was done to encourage the use of the '--disable-web-security' flag only for development and not for general purpose use. However, that change (locking '--disable-web-security' to '--user-data-dir') was made only in one place in the Chromium code. There are quite a few other places in Chromium that turn off security features when '--disable-web-security' is specified, and none of them lock it to the use of the '--user-data-dir' flag. Therefore, move the lock to much earlier in Chromium startup, and if the '--user-data-dir' flag isn't specified, remove the '--disable-web-security' flag entirely from the CommandLine. That way, this correctly locks together those two flags for all of Chromium. BUG=923523,327804 Change-Id: I9000f1fbd2c236a435d8dae25870dead870ddd55 Reviewed-on: https://chromium-review.googlesource.com/c/1423342Reviewed-by:
Mark Mentovai <mark@chromium.org> Reviewed-by:
Chris Palmer <palmer@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#625210}
-
Mark Cogan authored
This CL changes the init of the main BrowserInterface in BrowserViewWrangler to be explicit, and to be done as part of creating the main Browser instance. BrowserViewWrangler's -mainInterface implementation is now a simple (synthesized) getter, and thus will return nil before -createMainBrowser is called. BrowserViewWrangler's -incognitoInterface implementation now returns nil if there's no mainInterface. This mitigates the risks around calling -mainInterface prior to the main browser being created. It does slightly change the timing of BVC creation (it's slightly earlier), but at least now it's explicit. TEST: The crash for this bug is triggered by auth service init forcing a sign-out of a managed account, which causes MainController's removeBrowsingDataForBrowserState: method to be called. The completion block in that method uses mainInterface to disable user interactions. To test this fix, I added an explicit call to SignOut() in AuthService::Initialize() and launched Chrome with a signed-in managed account. It crashes as expected before this fix, and works fine afterwards. Bug: 919932 Change-Id: Iaffb29cd7d0dfb5b54f3347755fc9e15f0aa4539 Reviewed-on: https://chromium-review.googlesource.com/c/1430239 Commit-Queue: Mark Cogan <marq@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#625209}
-
Denis Kuznetsov authored
Bug: 916991 Change-Id: I1d22e80d3f942464a5e577210e8be3ae242565bd Reviewed-on: https://chromium-review.googlesource.com/c/1424889 Commit-Queue: Denis Kuznetsov <antrim@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Cr-Commit-Position: refs/heads/master@{#625208}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/2aeeb2d0..21b1ee52 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ief63e9c080afea332a1caacfff8526e5167879bb Reviewed-on: https://chromium-review.googlesource.com/c/1430499Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#625207}
-
kylechar authored
Remove usage of deprecated base::Bind, base::Callback, base::Closure, base::CancelableCallback and base::CancelableClosure types from gpu/*. Where possible convert to the corresponding once type. Otherwise replace with the repeating type which is equivalent to the deprecated type. This CL is the second of multiple to cleanup callback types in gpu/*. Bug: 714018 Change-Id: Ifc2af8ee36feed7815f053a5f53147fc983cb005 Reviewed-on: https://chromium-review.googlesource.com/c/1426010Reviewed-by:
Jonathan Backer <backer@chromium.org> Reviewed-by:
-
Finnur Thorarinsson authored
Bug: 860467 Change-Id: I71bab32bd386f425eee3837ceb7180d6a40b26eb Reviewed-on: https://chromium-review.googlesource.com/c/1426787Reviewed-by:
Mike West <mkwst@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> Cr-Commit-Position: refs/heads/master@{#625205}
-
Peter Kotwicz authored
This CL refactors WebappSplashScreenController in preparation for not having the WebApkActivity show a splash screen if WebappInfo#isSplashProvidedByWebApk(). The code which should not run for new-style WebAPKs is moved to SameActivityWebappSplashDelegate BUG=918459 R=dominickn TBR=yfriedman (For WebappUma rename) Change-Id: Ie47b4b26590f6fa1606e0594c979591def9aaf75 Reviewed-on: https://chromium-review.googlesource.com/c/1408353 Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#625204}
-
Denis Kuznetsov authored
https://chromium.googlesource.com/chromiumos/platform2/system_api.git/+log/88d06ccdd29b..e409915310e1 $ git log 88d06ccdd..e40991531 --date=short --no-merges --format='%ad %ae %s' 2019-01-07 ejcaruso cryptohome: add UnmountEx method 2019-01-21 hmchu runtime_probe: report additional information for generic battery 2019-01-21 hmchu runtime_probe: Report "path" and "size" for storage device 2019-01-22 jkardatzke vm_tools: Cleanup IsContainerRunning dead code 2019-01-16 jkardatzke vm_tools: Remove unneeded VM ip, subnet, netmask from cicerone 2019-01-15 kdlee debugd: Sanitize print URIs before calling lpadmin 2019-01-14 ljusten system_api: Add login manager go protos 2018-11-28 briannorris shill: wifi: add RxBitrate 2019-01-07 jimmyxgong smbprovider: Add PremounOptionsProto Created with: roll-dep src/third_party/cros_system_api Bug: 916991 Change-Id: I5538b70753139a3f041107cf370b4e721d080bf0 Reviewed-on: https://chromium-review.googlesource.com/c/1429988Reviewed-by:
-
Anders Hartvoll Ruud authored
This CL adds a function to get a CSSPropertyName from a CSSProperty. Change-Id: I76f0a09367a082142dd38bc48f82200a630be527 Reviewed-on: https://chromium-review.googlesource.com/c/1429643Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Anders Hartvoll Ruud <andruud@chromium.org> Cr-Commit-Position: refs/heads/master@{#625202}
-
Nektarios Paisios authored
This patch depends on https://chromium-review.googlesource.com/c/chromium/src/+/1229234 which implemented a virtual accessibility view class. I propose that we use the grid role for the table view and mark it readonly because in Views a table view is interactive like an ARIA grid. On Windows it will be exposed as a list view control and on Mac as a table view. R=dmazzoni@chromium.org, ellyjones@chromium.org, sky@chromium.org, aleventhal@chromium.org Bug: 811277 Change-Id: I9e02563f1220f300e7fd78dc4cbbf5ee5e585acb Reviewed-on: https://chromium-review.googlesource.com/c/1363793 Commit-Queue: Nektarios Paisios <nektar@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#625201}
-
Mikel Astiz authored
This makes the datatype less special and reduces the complexity of "core" sync, although further patches are needed to complete that. Short term, the main goal is to avoid circular dependencies across keyed services without subtle tricks to get references lazily, as it was done for SessionSyncService (and more datatypes are coming). TBR=asvitkine@chromium.org,pkasting@chromium.org,olivierrobin@chromium.org Bug: 922971 Change-Id: I944f1a74fa9b843185393d0d34bb590a6e6a27f6 Reviewed-on: https://chromium-review.googlesource.com/c/1420737Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#625200}
-
Kevin Ellis authored
* Adds MutateAynchronously method to the mutator dispatcher. * Adds tests for ansynchronous mutation. * Stubs in API for notification of mutation state (pending/complete). Note async mutations are not used outside of testing with the patch pending completion of the notification plumbing in a separate CL. Bug: 791280 Change-Id: I91545306c71899945f797a641831257247849b86 Reviewed-on: https://chromium-review.googlesource.com/c/1409600Reviewed-by:
Robert Flack <flackr@chromium.org> Reviewed-by:
Majid Valipour <majidvp@chromium.org> Commit-Queue: Kevin Ellis <kevers@chromium.org> Cr-Commit-Position: refs/heads/master@{#625199}
-
Yuichiro Hanada authored
It's enabled by default now. Bug: 924383 Test: unit_tests --gtest_filter=ArcInputMethod* Change-Id: Idaa38371fede902b1779f2482442388dece4721b Reviewed-on: https://chromium-review.googlesource.com/c/1429322Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Auto-Submit: Yuichiro Hanada <yhanada@chromium.org> Cr-Commit-Position: refs/heads/master@{#625198}
-
Zakhar Voit authored
Policies considered risky (besides force-installed extensions and ONC, it was handled in the previous CLs): 1. Any policies marked with 'website-sharing' tag. 2. 'PacHttpsUrlStripping' policy when it's value is 'false'. Bug: 910218 Change-Id: Ieade2fc159801597a3ede3b1c115abc285ae716a Reviewed-on: https://chromium-review.googlesource.com/c/1412478 Commit-Queue: Zakhar Voit <voit@google.com> Reviewed-by:
Jacob Dufault <jdufault@chromium.org> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Cr-Commit-Position: refs/heads/master@{#625197}
-
Olivier Robin authored
When distilling a page with media content, the media starts to play even if the webState is in the background. This cause audio playing in case of an audio, and the video player to start in case of a video. This CL blocks loading the media pages when distilling. Bug: 919326 Change-Id: I3717618ba31019fbc173cb721436a323acfbc6aa Reviewed-on: https://chromium-review.googlesource.com/c/1409514 Commit-Queue: Eric Noyau <noyau@chromium.org> Auto-Submit: Olivier Robin <olivierrobin@chromium.org> Reviewed-by:
Eric Noyau <noyau@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#625196}
-
Xida Chen authored
This CL cleans up the CrossThreadStyleValueTest, which does proper initialization for the WebThreadSupportingGC, and shut down the thread at the end of the test, just like what we do in the PaintWorkletStylePropertyMapTest. Bug: 895579 Change-Id: I7cf93fe5a192cd221a7f9490982d1a104110a232 Reviewed-on: https://chromium-review.googlesource.com/c/1427605Reviewed-by:
Anders Hartvoll Ruud <andruud@chromium.org> Reviewed-by:
-
Peng Huang authored
Also convert RasterDecoderContextState from struct to class. TODO: * rename raster_decoder_context_state.h to decoder_context_state.h Bug: 919865 Change-Id: I77cb8d369c1e0c736434da6bbb144c863d9ec3f6 Reviewed-on: https://chromium-review.googlesource.com/c/1426060 Commit-Queue: Peng Huang <penghuang@chromium.org> Auto-Submit: Peng Huang <penghuang@chromium.org> Reviewed-by:
-
Christos Froussios authored
Currently, Chrome uses the user default (umask), which may be too broad for sensitive files. We will restrict all permissions to users other than the owner. Bug: 921308 Change-Id: I9c03b57ae1392926a6f912372a8eae8e4c6b3cfb Reviewed-on: https://chromium-review.googlesource.com/c/1426678 Commit-Queue: Christos Froussios <cfroussios@chromium.org> Reviewed-by:
Maxim Kolosovskiy <kolos@chromium.org> Cr-Commit-Position: refs/heads/master@{#625193}
-
Alan Cutter authored
This CL sets desktop-pwas-team@google.com to be the owner of the following flags: - enable-desktop-pwas - enable-desktop-pwas-custom-tab-ui - enable-desktop-pwas-link-capturing - enable-desktop-pwas-stay-in-window - enable-experimental-app-banners - pwa-improved-splash-screen - pwa-persistent-notification Change-Id: I06c9d53532e09ee8c9d0400d92457c248451fcb5 Reviewed-on: https://chromium-review.googlesource.com/c/1428139Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Auto-Submit: Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#625192}
-
Xi Han authored
The STARTED count in the MobileDownload.Background metrics is more than a double of total count of completed + cancelled + failed. This is due to a download task might be interrupted multiple times before it finishes. To fix it, we add a INTERRUPTED bucket in this metrics. We can roughly estimate the crash rate by using: (start - interrupted - completed - cancelled - failed) / start We can't distinguish the following cases from the crash rate though: 1) A download task has been paused by user before Chrome crashes. 2) Chrome is killed by rebooting the devices / users, rather than killed by OOM. Bug: 864625 Change-Id: Ic1f03bc665d5411c73ab16138cda259f0cab6ed1 Reviewed-on: https://chromium-review.googlesource.com/c/1427563Reviewed-by:
Min Qin <qinmin@chromium.org> Reviewed-by:
Henrique Nakashima <hnakashima@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Xi Han <hanxi@chromium.org> Cr-Commit-Position: refs/heads/master@{#625191}
-
Daniel Nicoara authored
Move the zcr_vsync_feedback interface into its own files. This change only moves code around, no functional changes. Bug: 896710 Test: Build and run exo_unittests Change-Id: I74885b5a6692306b05a1d86350f627ed7046045b Reviewed-on: https://chromium-review.googlesource.com/c/1427571 Commit-Queue: Daniel Nicoara <dnicoara@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Cr-Commit-Position: refs/heads/master@{#625190}
-
Florent Castelli authored
Some codecs don't support directly creating simulcast layers with non-optimal parameters. This proxy will detect this and create multiple encoders then, one for each layer as a fallback. Bug: 857048 Change-Id: Ib352621363cb3cdc0f73385f7f74d4e473263fc1 Reviewed-on: https://chromium-review.googlesource.com/c/1430084Reviewed-by:
Henrik Boström <hbos@chromium.org> Commit-Queue: Florent Castelli <orphis@chromium.org> Cr-Commit-Position: refs/heads/master@{#625189}
-
David Jean authored
Bug: 907527 Change-Id: I346f28df9938f56b948c2f4fdbbbe427ca0fa52a Reviewed-on: https://chromium-review.googlesource.com/c/1429981Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: David Jean <djean@chromium.org> Cr-Commit-Position: refs/heads/master@{#625188}
-
Anatoliy Potapchuk authored
Bug: 918101 Test: chromeos_unittest --gtest_filter=*OncParsedCertificatesTest* Change-Id: I02dd141d8a2b809a3424f453bf5c9df3ea1c1321 Reviewed-on: https://chromium-review.googlesource.com/c/1425705 Commit-Queue: Anatoliy Potapchuk <apotapchuk@chromium.org> Reviewed-by:
Pavol Marko <pmarko@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#625187}
-
Morten Stenshorne authored
TBR=eae@chromium.org Bug: 924254,915204 Change-Id: Ib551a93ce37d47c87989add7c10d22ad0499efc7 Reviewed-on: https://chromium-review.googlesource.com/c/1430005Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#625186}
-
Stepan Khapugin authored
Instead of erasing the text in the textfield before inserting new chars, just select the text that is displayed as selected with preedit label. This produces exactly the same result as if the selection was real, fixing a few small glitches with third party keyboards. Bug: 875918,873544 Change-Id: Ia1628f0c0207aa7db8318f60e64bc73ac27872e6 Reviewed-on: https://chromium-review.googlesource.com/c/1183495 Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by:
-
David Roger authored
In certain edge cases (involving adding a GCMS observer from inside a GCMS notification) , OnGaiaAccountsInCookieUpdated() could be fired after a ListAccounts call that returns true (i.e. cookies are fresh). The reconcilor assumed that OnGaiaAccountsInCookieUpdated() could only be called after ListAccounts returns false. This CL adds support for receiving unexpected notifications while the reconciliation is in progress. The reconcilor will simply ignore them, and rely on the fact that the reconciliation will touch the cookies and trigger new ListAccounts calls at the end, triggering additional reconciliation cycles if needed. Bug: 923716 Change-Id: I89f7ed720f44b75d7a3c215fad6e06fb078f12cf Reviewed-on: https://chromium-review.googlesource.com/c/1424953 Commit-Queue: David Roger <droger@chromium.org> Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#625184}
-
Gyuyoung Kim authored
As a step to make to use IdentityManager,this CL replaces all uses of ATS with IdentityManager's ones in authentication_flow_performer.mm. Bug: 922798 Change-Id: I732c92e96b1174537afa2273fddf984da665477e Reviewed-on: https://chromium-review.googlesource.com/c/1429326Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com> Cr-Commit-Position: refs/heads/master@{#625183}
-
Mirko Bonadei authored
Bug: None Change-Id: I95e63c6efba18cb2692f6ac570cd68d3fb9c7476 Reviewed-on: https://chromium-review.googlesource.com/c/1429940Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#625182}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/f6929d6c..2aeeb2d0 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I014d8686db17ee4461e1b53a27773eeaf7f9db13 Reviewed-on: https://chromium-review.googlesource.com/c/1430159Reviewed-by:
-
Stepan Khapugin authored
Exposes the OmniboxUIExperimentMaxAutocompleteMatches flag on iOS flags page. Unfortunately, there seems to be no good way to share flags with iOS at the moment, so this involves a lot of copy-paste. Also extend maximum numbers of rows in UI implementation to 12 to prevent crashes when this flag is used. Bug: 908478 Change-Id: I5347400289cd10ac2ae735e9ed6cb930c6a0d405 Reviewed-on: https://chromium-review.googlesource.com/c/1425722Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Auto-Submit: Stepan Khapugin <stkhapugin@chromium.org> Cr-Commit-Position: refs/heads/master@{#625180}
-
Mario Sanchez Prada authored
Following the cleanup to remove any dependency on includes from //content, let's move this delegate of ProfileOAuth2TokenService into //components now as necessary step in the direction of hiding the PO2TS delegates behind the IdentityManager. Bug: 922555 Change-Id: I6b54723b232efea24a7001a2f7e25733ccbbabb7 Reviewed-on: https://chromium-review.googlesource.com/c/1425500Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
-
Dominic Battre authored
The new form parser may decide that a form has no current-password element, but only new-password / confirmation elements. In this case, the parser sets the unique_renderer_id to 'undefined'. The credentials will be sent to the renderer for filling via manual fallback but the renderer won't ever fill (as there is no password element defined for filling. This CL ensures that this case does not get recorded as a failure to fill. This is important because we record two metrics about the first fill attempt: PasswordManager.FirstWaitForUsernameReason and PasswordManager.FirstRendererFillingResult For websites that have a sign-up form followed by a sign-in form, we don't want to record the failure to fill on the sign-up form. Bug: 918846 Change-Id: I79e9a73ae3573a81e121eaa483e0ecef0889184f Reviewed-on: https://chromium-review.googlesource.com/c/1422018 Commit-Queue: Dominic Battré <battre@chromium.org> Reviewed-by:
Vadym Doroshenko <dvadym@chromium.org> Cr-Commit-Position: refs/heads/master@{#625178}
-
Antonio Gomes authored
This is the second out of 2 CLs that migrate DiceTurnSyncOnHelper to IdentityManager. The first part [1] migrated it away from using SigninManager APIs directly. [1] https://chromium-review.googlesource.com/c/1387393 Additionally, the CL also takes the opportunity to remove AccountTrackerService usage, and replace the PO2TSObserver inheritance by IdentityManagerObserver. Both SigninManager and PO2TS are becoming implementation details of the new IdentityManager API. This CL migrates the direct PO2TS API calls to their respective IdentityManager APIs. BUG=890811,922789 Change-Id: I928390d4a336a6b42f56661586c90b3d2f5d4635 Reviewed-on: https://chromium-review.googlesource.com/c/1426457 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
David Roger <droger@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#625177}
-
Mario Sanchez Prada authored
Add a new static SigninManagerFactory::RegisterPrefs() method to the relevant factory for //ios/web_view, have it call SigninManagerBase::RegisterPrefs() instead and call that static method from ios_web_view::ApplicationContext, similar to what's already done in //chrome and //ios//chrome. Bug: 920191 Change-Id: If8f1c86b4f0dc069fe66aa013235d9e695543a2b Reviewed-on: https://chromium-review.googlesource.com/c/1425733 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
-
Simon Que authored
Bug: 922354 Change-Id: I0d168efeea407409ab234bdcb1ebf307a9c7b551 Reviewed-on: https://chromium-review.googlesource.com/c/1426248Reviewed-by:
Shinya Kawanaka <shinyak@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Simon Que <sque@chromium.org> Cr-Commit-Position: refs/heads/master@{#625175}
-
Zakhar Voit authored
[Managed Session] Force managed sessions mode when network certificates are set via ONC device policy The same way as force-installed extensions, forced network certificates are considered "risky" should activate "Managed Session" mode (i.e. show warning about admin capabilities on the login page). Bug: 910218 Change-Id: Ic34898ed255f51b48b948b00ae4563bcd7939347 Reviewed-on: https://chromium-review.googlesource.com/c/1391590 Commit-Queue: Zakhar Voit <voit@google.com> Reviewed-by:
-
Julie Jeongeun Kim authored
This Cl removes unused headers, account_tracker_service_factory.h and account_tracker_service.h. Bug: 922783 Change-Id: Ibfc8379b0aa4d8ed2e49bf00bcaa9651121b051f Reviewed-on: https://chromium-review.googlesource.com/c/1428507Reviewed-by:
Thomas Tangl <tangltom@chromium.org> Commit-Queue: Julie Jeongeun Kim <jkim@igalia.com> Cr-Commit-Position: refs/heads/master@{#625173}
-
Mirko Bonadei authored
Change Log: https://chromium.googlesource.com/external/github.com/abseil/abseil-cpp/+log/455dc17ba1..0b1e6d417b Full diff: https://chromium.googlesource.com/external/github.com/abseil/abseil-cpp/+/455dc17ba1..0b1e6d417b Bug: None Change-Id: I8faff7205ec2f433a4bed90b58df788d4fcc3063 Reviewed-on: https://chromium-review.googlesource.com/c/1429642Reviewed-by:
-
