- 23 Jan, 2020 40 commits
-
-
Ben Smith authored
It has been enabled by default on desktop for a while now. The feature is still required, since SharedArrayBuffer is still not enabled by default on Android. Bug: 821270 Change-Id: Ie58c4bb4118648391f533f310a8f093f9fa4e19b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015564Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Adam Klein <adamk@chromium.org> Commit-Queue: Ben Smith <binji@chromium.org> Cr-Commit-Position: refs/heads/master@{#734576}
-
Dan Harrington authored
- Restructure to match chrome/browser/java/res - Replace strings with references to existing .grd strings - I kept Piet resources separate, since piet may eventually be moved into a different library. - Move feed/library sources into chrome_java Strings changed: snackbar_fetch_no_new_suggestions -> ntp_suggestions_fetch_no_new_suggestions snackbar_fetch_failed -> ntp_suggestions_fetch_failed snackbar_default_action -> undo It looks like chromium's strings are translated to more languages, so this might provide better localization. Some of these strings are translated differently between feed and zine. Bug: 1024945 Change-Id: If9862a4965756e28ba812bf315556d6b8db09e62 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1956275Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Peter Wen <wnwen@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Carlos Knippschild <carlosk@chromium.org> Commit-Queue: Dan H <harringtond@chromium.org> Cr-Commit-Position: refs/heads/master@{#734575}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=sczs@chromium.org Bug: 1042208 Change-Id: I402bf098fad7e7266b8e312ca013a8e375639bd9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016652 Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Commit-Queue: Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#734574}
-
Joshua Berenhaus authored
This is the first step to enabling the ukm service to reset the ukm client id when a cloned install is detected. By moving the responsibility of the pref, both services can call in to understand if we are in a cloned state. Bug: 981407 Change-Id: I0a27bc0f14f5458cf685ff5abf7ea792e73246d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1992009 Commit-Queue: Joshua Berenhaus <joshber@microsoft.com> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Auto-Submit: Joshua Berenhaus <joshber@microsoft.com> Cr-Commit-Position: refs/heads/master@{#734573}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=jeffreycohen@chromium.org Bug: 1042208 Change-Id: Ib396345f37691f2b87d6596c5c96ce50ee913442 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015091 Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Jeffrey Cohen <jeffreycohen@chromium.org> Commit-Queue: Jeffrey Cohen <jeffreycohen@chromium.org> Cr-Commit-Position: refs/heads/master@{#734572}
-
Ryan Daum authored
Now that internal is always passing this value it's safe to remove the default nullptr to avoid any possible future regressions for webview window corner placement. Bug: none Test: manual Change-Id: I465c888bc07eb6d339969ca0eefd8efdcc770df5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016284Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Commit-Queue: Ryan Daum <rdaum@chromium.org> Cr-Commit-Position: refs/heads/master@{#734571}
-
Toni Barzic authored
This is a reland of 97dfeb90 Original change was not the culprit for TabletModeControllerTest.VerticalHingeTest crashes on MSan bots (for which it was reverted). Original change's description: > Prepare shelf layout manager for shutdown earlier > > Starts shelf layout manager shutdown earlier at the beginning of shell > shutdown sequence (during OnShellDestroying call issued to shell > observers when the shell tear down starts). > > The main goal is to prevent shelf state updates that might happen during > shutdown. For example, tablet mode controller destruction, and overview > controller destruction might cause window stacking changes, that in > turn can cause shelf state updates - both of which happen before > Shell::CloseAllRootWindowChildWindows(), which is from where > ShelfLayoutManager::PrepareForShutdown is currently called (with some > indirection). > > NOTE: Calls to start shelf shutdown issued from > RootWindowController::CloseChildWindows() are kept around to handle root > window controller shutdowns that happen outside shell shutdown. > > BUG=1033343 > > Change-Id: I86211392b45bdafd5f095bd60c774fe88e8accce > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015358 > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> > Commit-Queue: Toni Baržić <tbarzic@chromium.org> > Cr-Commit-Position: refs/heads/master@{#734253} Bug: 1033343 Change-Id: I1d5535e7d2a0a35d5ec59e7755588d3745caae86 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016289Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Toni Baržić <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#734570}
-
chromium-autoroll authored
Roll ChromeOS orderfile from 81-3987.39-1578912426-benchmark-81.0.4019.0-r1 to 81-4025.0-1579517934-benchmark-81.0.4019.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/+/master/autoroll/README.md Tbr: c-compiler-chrome@google.com Change-Id: Ic95aee3f1d0bb1ac3d27c483a06809aba331156d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017648Reviewed-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@{#734569}
-
Armando Miraglia authored
This histogram are related to OutputController and are used to monitor how the lifetime of audio output controllers. For debugging and analysis of the audio pipeline we need such histograms indefinitely. Fixed: 1037594 Change-Id: I01ecbafa8fc6c549764114d0109bbdc0278b92e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2012342Reviewed-by:
Armando Miraglia <armax@chromium.org> Reviewed-by:
Olga Sharonova <olka@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Armando Miraglia <armax@chromium.org> Cr-Commit-Position: refs/heads/master@{#734568}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=sczs@chromium.org Bug: 1042208 Change-Id: I5511f8af83fe19a3df0a21071792650766b6d49c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017224 Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Commit-Queue: Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#734567}
-
yilkal authored
This CL makes sure that navigation has committed in SupervisedUserNavigationObserver::DidFinishNavigation before trying to access the RenderFrameHost that hosted the navigation. Bug: 1042620 Change-Id: I8258023537c769338c82e54927d38107759b1db3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2008317Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Aga Wronska <agawronska@chromium.org> Commit-Queue: Yilkal Abe <yilkal@chromium.org> Cr-Commit-Position: refs/heads/master@{#734566}
-
Anupam Snigdha authored
Previously we used to report the client rectangle instead of actual control bounds of the focused editable element. With this change, we are fetching this data from the focused editable element's bounding client rect and sending this info to TSF. Bug: 1043424 Change-Id: I20e18e3ec4e861311636ced9dc97d0a82f49838d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2008774 Commit-Queue: Anupam Snigdha <snianu@microsoft.com> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Yohei Yukawa <yukawa@chromium.org> Cr-Commit-Position: refs/heads/master@{#734565}
-
Fernando Serboncini authored
Bug: 1044993 Change-Id: I14c8c09b04aba9c2e81f76792f6f9f9a8e7fbd35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017042 Commit-Queue: Fernando Serboncini <fserb@chromium.org> Reviewed-by:
Juanmi Huertas <juanmihd@chromium.org> Reviewed-by:
Fernando Serboncini <fserb@chromium.org> Auto-Submit: Fernando Serboncini <fserb@chromium.org> Cr-Commit-Position: refs/heads/master@{#734564}
-
Trent Begin authored
Add TimezoneInfo to mojom file to keep it in sync with Chrome OS. Bug: 1030361 Test: browser_tests --gtest_filter=DeviceStatus* && chromeos_unittests Change-Id: Ide7cf1bdd7dcb1b17369125048bb4742298a14bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2013218 Commit-Queue: Trent Begin <tbegin@chromium.org> Reviewed-by:
Jesse Schettler <jschettler@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Jorge Lucangeli Obes <jorgelo@chromium.org> Cr-Commit-Position: refs/heads/master@{#734563}
-
Jan Wilken Dörrie authored
This change adds a JNI bridge for the BiometricAuthenticator and implements BiometricAuthenticator::CanAuthenticate() in a backwards compatible way. Bug: 1031483 Change-Id: I273aa655879c4abfb0afedd841744a53f914c71a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2012024 Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#734562}
-
Elly Fong-Jones authored
Bug: 1011446 Change-Id: I1af5ea914f2e4ffef7069f53f984c8d69cb99eac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015607Reviewed-by:
Daniel Rubery <drubery@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#734561}
-
Avery Musbach authored
Bug: None Change-Id: I51a8ef2deb6f9c569e03dd6d8680e896c7a24ab8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016340Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Commit-Queue: Avery Musbach <amusbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#734560}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/cdd73e095cd4..48655cfdbfd9 git log cdd73e095cd4..48655cfdbfd9 --date=short --first-parent --format='%ad %ae %s' 2020-01-23 minyue@webrtc.org Send absolute capture time through audio coding module. Created with: gclient setdep -r src/third_party/webrtc@48655cfdbfd9 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@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 Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: Ic3049f1ff76f02183d8114802cccce6f529d7539 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016332Reviewed-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@{#734559}
-
Ahmed Mehfooz authored
Bug: 960116 Change-Id: I2e6868f2ec1873273614cfe1ae902714dd882df8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015572 Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#734558}
-
Sebastien Lalancette authored
This reverts commit 8fd76b23. Reason for revert: Failing on bots. Bug: 1045071 Original change's description: > Add browser tests for smart card SAML login > > Provide test coverage for the scenario where a new user authenticates > via SAML using a smart card (or, generally speaking, using a security > token via an extension using the chrome.certificateProvider API). > > This also provides some test coverage for the <security-token-pin> > Polymer element used on the Login Screen. > > Bug: 1033936 > Change-Id: I5ded32e0570eb7227c77b954c33d12c1a8a62914 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1968989 > Reviewed-by: Alexander Alekseev <alemate@chromium.org> > Reviewed-by: Denis Kuznetsov [CET] <antrim@chromium.org> > Reviewed-by: Alexander Hendrich <hendrich@chromium.org> > Commit-Queue: Maksim Ivanov <emaxx@chromium.org> > Cr-Commit-Position: refs/heads/master@{#734510} TBR=antrim@chromium.org,alemate@chromium.org,emaxx@chromium.org,hendrich@chromium.org,fabiansommer@google.com Change-Id: Ibfff9d60cd3ed0d2470435e67b3fd563c98c888a No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1033936 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016971Reviewed-by:
Sebastien Lalancette <seblalancette@chromium.org> Commit-Queue: Sebastien Lalancette <seblalancette@chromium.org> Cr-Commit-Position: refs/heads/master@{#734557}
-
Robert Sesek authored
Bug: 1042912 Change-Id: I64b0e5c0e9f6c36e9fdb7dd7d766e6a78a5f5734 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015364 Commit-Queue: Greg Kerr <kerrnel@chromium.org> Reviewed-by:
Greg Kerr <kerrnel@chromium.org> Cr-Commit-Position: refs/heads/master@{#734556}
-
manukh authored
Drive suggestions display icons corresponding to their type. We currently display specific icons for docs, sheets, slides, forms, pdfs, images, and videos. Other types (e.g. folders) display the generic drive icon. Apparently, some users search for folders often. To distinguish folder results, this CL adds the folder icon for folder results. These icons are the same as those displayed on drive.com. This CL also adjusts the image and slide icon colors to match those used on drive.com. Bug: 1032663 Change-Id: I555baa5d3dc4027cc399d8fb6e1507c5204586f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016906Reviewed-by:
manuk hovanesian <manukh@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: manuk hovanesian <manukh@chromium.org> Cr-Commit-Position: refs/heads/master@{#734555}
-
Henrique Nakashima authored
Register it in ChromePreferenceKeys and use SharedPreferencesManager consistently instead of SharedPreferences directly. These keys are dynamic, so this CL uses a grandfathered KeyPrefix to generate them. Bug: 1022108 Change-Id: I3ce28a329619d144130d60c9fad275518129b232 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2005374Reviewed-by:
Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Commit-Queue: Henrique Nakashima <hnakashima@chromium.org> Cr-Commit-Position: refs/heads/master@{#734554}
-
Ian Kilpatrick authored
This fixes the above testcase, marked as failing in: https://chromium-review.googlesource.com/c/chromium/src/+/2015101 Essentially a flex-item, if it has any %-block-size children, needs to mark itself as depending on %-block-size, as its definiteness can change. See note in ng_physical_container_fragment.cc as this is a *little* too broad, as we should only check if the definiteness changes, not the percentage block resolution size. Bug: 845235 Change-Id: I78120f5dd65186ef9ad597f8a5717923cfa49144 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016634 Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
David Grogan <dgrogan@chromium.org> Cr-Commit-Position: refs/heads/master@{#734553}
-
Dave Tapuska authored
Define synchronous APIs on mojo to handle the appropriate dialog messaging. Move this solely into blink, the API is still exposed on the ChromeClient to still support PopupOpeningObservers. Introduce LocalFrameHostInterceptor on the content/public/test side to help support stubbing out the LocalFrameHostInterface. Move handling of Javascript dialogs in test_runner to the browser side. Some code already existed, the logging is just moved from the renderer to browser side. BUG=1039695 Change-Id: I2596c4a9ad35027e3ffde1c29b1525114c2cdfbd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1994604 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#734552}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=vitaliii@chromium.org Bug: 1042208 Change-Id: I3ad84430894a8b0bd355680007ee07cebf5a46e8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2014976 Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
vitaliii <vitaliii@chromium.org> Commit-Queue: vitaliii <vitaliii@chromium.org> Cr-Commit-Position: refs/heads/master@{#734551}
-
Mike Wasserman authored
Plumb the internal display id to populate the Screen.internal attribute. Follows the primary display id and Screen.primary attribute pattern. This requires the ScreenEnumeration experimental flag to be enabled: chrome://flags#enable-experimental-web-platform-features OR $ chrome --enable-blink-features=ScreenEnumeration Test with this demo: https://github.com/michaelwasserman/window-placement-demo Bug: 994889 Test: getScreens()[i].internal returns an accurate value. Change-Id: I71ed5eb8b164eff6f2f5d93a7a400e69df219490 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015555 Commit-Queue: Michael Wasserman <msw@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#734550}
-
James Cook authored
Today, all Chrome OS users must consent to sync during login. For the SplitSettingsSync project we want to allow them to use their device without consenting to browser sync. However, Chrome OS always sets the IdentityManager primary account during login, which implies sync consent. We would like to switch to using the "unconsented" primary account, which exists whether or not the user has consented to sync. However, there are a large number of places in the Chrome OS code that use GetPrimaryAccount() to get the "main" user's account ID, email or tokens. Add a command line switch that skips setting the primary account during login and instead directly sets the "unconsented" primary account. This allows me to manually test to identify areas that need to change. Convert some login and ARC ToS code to use the unconsented primary account. This is a no-op if the switch isn't set, because without the switch the primary account and unconsented primary account are the same. Bug: 1042400 Test: existing automated tests, also sign-in and ARC++ work on device without the switch set Change-Id: Idfeece6ac79b97a102111fb165062614a81f5a9a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2001018 Commit-Queue: James Cook <jamescook@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#734549}
-
Sorin Jianu authored
The certificate tool needs an un-tagged, Authenticode-signed binary for unit testing purposes. This binary comes from the official Google Update build, where it is used for a similar purpose. BUG=1031734 R=waffles CC=agl Change-Id: I2fb5299bd7434ebdfa147f38a69959e7860427df Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015804 Commit-Queue: Adam Langley <agl@chromium.org> Reviewed-by:
Adam Langley <agl@chromium.org> Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Cr-Commit-Position: refs/heads/master@{#734548}
-
Abhijeet Kandalkar authored
This CL removes old legacy IPC to adopt new mojo types. It also removes unwanted implementations form RenderFrameImpl class. The goal of this CL is to tell renderer to perform the given action on the media player location at the given point in the view coordinate space. Bug: 1040105 Change-Id: I4b52e1352dc69f433bf8c965459bbd8b060c6f45 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2002488Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Abhijeet Kandalkar <abhijeet@igalia.com> Cr-Commit-Position: refs/heads/master@{#734547}
-
Nate Fischer authored
This separates developer mode state from the mechanism for plumbing developer mode information. These were previously tied together (we examined the enabled state of the ContentProvider which provided flags over IPC), but we observed an issue on Android Q related to disabling and re-enabling a ContentProvider in the same process. Instead, DeveloperModeContentProvider is always enabled and exported, and we examine the state of DeveloperModeState, which is a dummy component we never launch. While this creates slightly more complex code, this significantly simplifies the runtime behavior, because we no longer need to worry about functional code falling into a non-recoverable disabled state. We use an <activity-alias> as the dummy component because it requires the least amount of boilerplate, and there's little harm if the user somehow successfully launches this component. This also includes temporary workaround code: because the previous versions explicitly disabled the ContentProvider, and PackageManager persists this across upgrades, we need to explicitly reset the component state. To make potential future migrations easier, this updates the disableDeveloperMode() to set the component to DEFAULT rather than DISABLED. Bug: 1041943 Test: Manual - try enabling and disabling developer mode on Android Q Test: Repro the bug with buggy APK, upgrade to this APK, UI works Test: Disable dev mode with buggy APK, upgrade to this APK, UI works Change-Id: I410f36fa590026ddc53837f076e6b5912fa075f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2005973Reviewed-by:
Richard Coles <torne@chromium.org> Commit-Queue: Nate Fischer <ntfschr@chromium.org> Cr-Commit-Position: refs/heads/master@{#734546}
-
Takumi Fujimoto authored
Set the controller type to kMirroring so that local mirroring sessions don't show up in Global Media Controls. Bug: 1043302 Change-Id: Ib667dae4afc3b078b6aaa7b76618024cf0de1810 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2007318 Commit-Queue: Takumi Fujimoto <takumif@chromium.org> Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Cr-Commit-Position: refs/heads/master@{#734545}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=olivierrobin@chromium.org Bug: 1042208 Change-Id: I9953fbcbd08b61ac98b77be0d838649534a671a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2014917 Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Olivier Robin <olivierrobin@chromium.org> Cr-Commit-Position: refs/heads/master@{#734544}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=javierrobles@chromium.org Bug: 1042208 Change-Id: I951945d01287beb4c2f57357da575ec093fa89e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015170 Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Javier Ernesto Flores Robles <javierrobles@chromium.org> Commit-Queue: Javier Ernesto Flores Robles <javierrobles@chromium.org> Cr-Commit-Position: refs/heads/master@{#734543}
-
gogerald authored
Bug: 1045018 Change-Id: I7be550116de433e73fc8c61001110edca9f8e594 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017045Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Commit-Queue: Ganggui Tang <gogerald@chromium.org> Cr-Commit-Position: refs/heads/master@{#734542}
-
Jit Yao Yap authored
crrev.com/c/1877088 updated the default behavior of web dialog views to close when the Escape accelerator is pressed. This CL updates the login screen extension UI dialog to not close when receiving the accelerator. Bug: 957573 Change-Id: I72d283f38dd95af6d3909d5c2871b46880117d2b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2014961 Commit-Queue: Jit Yao Yap <jityao@google.com> Reviewed-by:
Alexander Hendrich <hendrich@chromium.org> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#734541}
-
Charlie Harrison authored
These are useful to have around for a while longer. Bug: 1034588 Change-Id: Iaddeed9b331e10d2b9409c1ec0273fa5146cbb37 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015739 Auto-Submit: Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Nate Chapin <japhet@chromium.org> Commit-Queue: Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#734540}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 707e7c14. With Chromium commits locally applied on WPT: 0cbea1f0 "SharedWorker: Add meta.url WPT for SharedWorker with type 'module'" 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: foolip@chromium.org, lpz@chromium.org, robertma@chromium.org: external/wpt/tools NOAUTOREVERT=true TBR=smcgruer No-Export: true Change-Id: Ie763c7bae19d091bb94dd8c9b0e56699c62f42bb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017122Reviewed-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@{#734539}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/261523f9..1b20535e Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues 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 CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I28adf7d618e11bd8edfbcc4cbeec1887e8874a6b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017102Reviewed-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@{#734538}
-
Askar Aitzhan authored
Change the dates to 2021-03-01 for all of them: 1. Because the feature "Cloud Policy over FCM" is not yet released to prod 2. For all of them to be consistent Bug: 1043709, 1043709 Change-Id: I2443e056dd22c141834e9402f7ba0cddbf34ca36 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2012023Reviewed-by:
Ivan Šandrk <isandrk@chromium.org> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Reviewed-by:
Bartosz Fabianowski <bartfab@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Commit-Queue: Askar Aitzhan <askaraitzhan@google.com> Cr-Commit-Position: refs/heads/master@{#734537}
-