- 18 Dec, 2019 40 commits
-
-
Demetrios Papadopoulos authored
On ChromeOS, this subpage is now displayed from the OS Settings instead. Same holds for the settings-captions subpage, even though it is still in the browser settings on non-ChromeOS. Also removing chromeos/localized_link/localized_link.{html/js} from settings_resources.grd again, since they were accidentally re-added at r725414. Will move the related files that are now only used by OS Settings to c/b/r/chromeos in a follow up CL. Bug: 1026455 Change-Id: Id3eeae79c75feb5886a6091797d105743bdd7db8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970845 Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Auto-Submit: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#726052}
-
Kenneth Russell authored
Canvases with display:none don't participate in compositing, so if they're drawn by applications, they never get a chance to reset their count of frames since the last commit. Handle visibility changes similarly. Bug: 1032171 Tbr: fserb@chromium.org Change-Id: I0eb7c3965627df533eb0617575618ec2dccb5ea9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1968324 Commit-Queue: Kenneth Russell <kbr@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Reviewed-by:
Juanmi Huertas <juanmihd@chromium.org> Cr-Commit-Position: refs/heads/master@{#726051}
-
sczs authored
Adds infobarBannerActionWillPresentModal to InfobarCoordinatorImplementation so the banner is not dismissed (causing the Modal to be dismissed) after its action button has been tapped. Bug: 1032623 Change-Id: Ie0b0d81b451353c3d4815e43ada08de36ce54d7a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970836Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Commit-Queue: Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#726050}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/a8a5c96f0b89..a02676614519 Created with: gclient setdep -r src-internal@a02676614519 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: chromium:702834 Tbr: jbudorick@google.com Change-Id: I9a1c4342426606bfa4e5d4f77ff242526173cd09 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974206Reviewed-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@{#726049}
-
Bettina authored
Log UMA metric when a non-sync signed-in user first saves their password on sign in and when they sign out. Right now, we only log when the signed-in non-sync account is saved in content area or it's changed. @tienmai inline_login_handler file only @vasilii password_manger files only Bug: 1034766 Change-Id: Ia6b407570433aa4f7df6c3365bd5cdf5f3ce4cc0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970768Reviewed-by:
Tien Mai <tienmai@chromium.org> Reviewed-by:
Daniel Rubery <drubery@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Commit-Queue: Bettina Dea <bdea@chromium.org> Cr-Commit-Position: refs/heads/master@{#726048}
-
Kurt Horimoto authored
This is an auxiliary OverlayRequest config that will be used to set up infobar banner UI. This CL also adds code to produce the appropriate banner config for save passwords overlays. Bug: 1030357 Change-Id: I9b455a3ffb696138b4cff1dad7c0cd4a827c8736 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1963384 Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#726047}
-
minch authored
1. Always activate the snapped window that is underneath the back start point. 2. Put the affordance below the finger instead of above if it will exceed or overlap with the splitview divider while in portrait orientation. Bug: 1034040 Change-Id: I62c22bdb716ccd5540ebfd67f290a13a4ab989dc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1967130Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Commit-Queue: Min Chen <minch@chromium.org> Cr-Commit-Position: refs/heads/master@{#726046}
-
Tarun Bansal authored
Bug: 988641 Change-Id: I0ac5898783fd01c90a50accb9a7270a3c0c3336c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973553Reviewed-by:
Doug Arnett <dougarnett@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#726045}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/703bcab9106c..5bddc336299d git log 703bcab9106c..5bddc336299d --date=short --first-parent --format='%ad %ae %s' 2019-12-18 fmayer@google.com Merge "Remove leftover reference to idle page tracking." 2019-12-18 fmayer@google.com Merge "Document args that are only valid in new versions." 2019-12-18 fmayer@google.com Remove Idle Page Tracking instructions. 2019-12-18 khokhlov@google.com Merge changes Ic732c2e5,Ib843c04c,I6b340067 Created with: gclient setdep -r src/third_party/perfetto@5bddc336299d If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@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: perfetto-bugs@google.com Change-Id: Iec81ddfc4625fe56366cc8d3a09759aee377240d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974205Reviewed-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@{#726044}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/65e2f03bd23e..17b3c2f3f8b1 git log 65e2f03bd23e..17b3c2f3f8b1 --date=short --first-parent --format='%ad %ae %s' 2019-12-18 shaobo.yan@intel.com Implement SamplerVideoWEBGL for WEBGL_video_texture extension on desktop Created with: gclient setdep -r src/third_party/angle@17b3c2f3f8b1 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC jmadill@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win-asan;luci.chromium.try:win_optional_gpu_tests_rel Bug: chromium:776222 Tbr: jmadill@google.com Change-Id: Ie1b6d0f84f5e7041a88dab8dc835e757e9cfde12 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973946Reviewed-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@{#726043}
-
Bo Liu authored
The zygote can only use the system linker for loading library as it cannot communicate with the browser process at all. Relax the assert that was not aware of zygote. Bug: 1035101 Change-Id: If88e09224992aff5e28c3b540d232b6f38fbd16f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973215 Commit-Queue: Bo <boliu@chromium.org> Commit-Queue: Benoit L <lizeb@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Auto-Submit: Bo <boliu@chromium.org> Reviewed-by:
Benoit L <lizeb@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#726042}
-
Andreas Haas authored
This CL replaces calls to DeprecatedByteLengthAsUnsigned by calls to ByteLengthAsSizeT. Unfortunately the current implementation cannot deal with huge ArrayBuffers yet. Therefore I reject the incoming ArrayBuffers with a RangeError if its size is too big. Background: we prepare ArrayBuffers to be bigger than 4GB. Therefore we changed the size field to size_t. Now we are changing all uses of ByteLength to be able to deal with size_t, either by accepting a size_t, or by throwing an exception if the size is too big. R=reillyg@chromium.org Bug: chromium:1008840 Change-Id: Ia0af7fb8c57ec522cf8cd0963d6e0dd3fad9dd92 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973910Reviewed-by:
Rijubrata Bhaumik <rijubrata.bhaumik@intel.com> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#726041}
-
Ken Rockot authored
This reverts commit c6e3166e. Reason for revert: https://crbug.com/1036299 It looks like the refresh callback used throughout UsbServiceImpl might need to be Repeating... Original change's description: > Convert base::Bind etc in services/ (1/2) > > This covers removal of deprecated base::Bind, base::Callback, > base::Closure, base::Passed across services/data_decoder/ and > services/device/. > > Bug: 1007831 > Change-Id: I03ca32424d53e91e6720e9209cb19a564a19de96 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970231 > Reviewed-by: Matt Reynolds <mattreynolds@chromium.org> > Reviewed-by: Ovidio de Jesús Ruiz-Henríquez <odejesush@chromium.org> > Commit-Queue: Ken Rockot <rockot@google.com> > Cr-Commit-Position: refs/heads/master@{#725374} TBR=rockot@google.com,mattreynolds@chromium.org,odejesush@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1007831 Fixed: 1036299 Change-Id: Ice2f343d2f25df88a9c4740449026975f04f4612 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974311 Commit-Queue: Ken Rockot <rockot@google.com> Reviewed-by:
Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#726040}
-
Yusuke Sato authored
https://chromium.googlesource.com/chromiumos/platform2/system_api.git/+log/4bab6cb11c26..cdae7443f8cf $ git log 4bab6cb11..cdae7443f --date=short --no-merges --format='%ad %ae %s' 2019-12-06 cylai system_api: change licenses terms 2019-12-17 yusukes system_api: Add cpus field to StartTerminaVm 2019-12-12 garrick system_api: Add subnet index field for plugin VM. 2019-12-02 andrewlassalle dlcservice: Create/delete metadata folder for DLCs 2019-11-14 xzhou system_api: allow multiple exclude credentials. 2019-11-22 henryhsu system_api: Add dobule tap power policy 2019-11-19 garrick system_api: Update patchpanel interface for Crostini 2019-12-05 norvez system_api: remove norvez@ from OWNERS Created with: roll-dep src/third_party/cros_system_api BUG=1011559 TEST=try Change-Id: Idff9ee27bb2f4e01ead3b8dbe303d02f7409282f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974290 Commit-Queue: Yusuke Sato <yusukes@chromium.org> Commit-Queue: Yury Khmel <khmel@chromium.org> Auto-Submit: Yusuke Sato <yusukes@chromium.org> Reviewed-by:
Yury Khmel <khmel@chromium.org> Cr-Commit-Position: refs/heads/master@{#726039}
-
John Abd-El-Malek authored
This reverts commit 96d8be10. Reason for revert: this is a problem with that bot, not weblayer_browsertests. Original change's description: > Sheriff: Disable many weblayer tests on Windows > > ErrorPageBrowserTest.404WithEmptyBody > ErrorPageBrowserTest.NameNotResolved > NavigationBrowserTest.HttpClientError > NavigationBrowserTest.HttpConnectivityError > NavigationBrowserTest.HttpServerError > NavigationBrowserTest.NoError > NavigationBrowserTest.SSLError > SSLBrowserTest.NavigateAway > SSLBrowserTest.Proceed > SSLBrowserTest.Reload > SSLBrowserTest.TakeMeBack > SSLBrowserTest.TakeMeBackEmptyNavigationHistory > WebLayerBrowserTest.Basic > WebLayerWebUIBrowserTest.WebUI > > Bug: 1034764 > > TBR=weblayer-dev@chromium.org > > Change-Id: I3b25c2ecca3ca123e5e40d47f78c6f6f1f33440d > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970299 > Reviewed-by: Fergus Dall <sidereal@google.com> > Commit-Queue: Fergus Dall <sidereal@google.com> > Cr-Commit-Position: refs/heads/master@{#725333} TBR=sidereal@google.com,weblayer-dev@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1034764 Change-Id: I6bf5bd29cad53d8d837ead018160559f98634ef0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974392Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#726038}
-
sczs authored
Bug: 1032624 Change-Id: I0a4edcbf26806da4448e2aa102df5b988c896711 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970088Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Commit-Queue: Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#726037}
-
Tom Anderson authored
libvulkan is dlopen()'ed, so a manual dependency is necessary. It's available on all RPM-based distros, so a basic dependency is added. But it isn't available everywhere on Debian-based systems, so it's added as a recommended dependency instead. If the system doesn't have libvulkan, OpenGL will be used as a fallback. R=thestig BUG=1034606 CC=penghuang Change-Id: Ibb0a1a94cc163c93a6428b1b408f6b364f9b6bb1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973042 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Auto-Submit: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#726036}
-
sczs authored
-Increases the Compresion rate for the Title and Modal Buttons so they're not compressed. -Increases the max banner height by 30 pts. Bug: 1032995,1032621 Change-Id: I87233e835f18f83d4dbdda8349af2bd322118947 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1969521 Commit-Queue: Sergio Collazos <sczs@chromium.org> Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Cr-Commit-Position: refs/heads/master@{#726035}
-
Aran Gilman authored
This CL also dynamically generates the tickmarks on the font size slider based on the supported font sizes provided in the JS code. Bug: 1027612 Change-Id: I4179430993ef31381f84099b6f4109b8b18fecd8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1960070 Commit-Queue: Aran Gilman <gilmanmh@google.com> Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Cr-Commit-Position: refs/heads/master@{#726034}
-
Marc Treib authored
The FCMInvalidations.SubscriptionResponseCodeForTopic histogram already had suffixes for all Sync ModelTypes. However, FCM invalidations are also used for other features (Drive, Policy). Previously, such cases where not properly covered. This CL introduces an "OTHER" suffix to catch all non-Sync subscriptions. It might be possible to split this up further, but it's not easy to do at this point in the code, and anyway it's not clear if the possible set of topics is always known statically. Bug: 1023813 Change-Id: I8571fa5061a6b7bb842353a3fbbddda0397acf5c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970994Reviewed-by:
Alex Ilin <alexilin@chromium.org> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#726033}
-
Ben Kelly authored
A nullptr manager can be encountered if a cross-sequence operation is begun during the shutdown of the storage partition. Bug: 1033251 Change-Id: I98c77a714dc7eb04a31a103c95012a7a989b9580 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1964931Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Rayan Kanso <rayankans@chromium.org> Commit-Queue: Ben Kelly <wanderview@chromium.org> Cr-Commit-Position: refs/heads/master@{#726032}
-
Nick Burris authored
Per the spec, the common ancestor element of the match range becomes the target element[1] and should have the :target pseudo-class applied. [1] https://html.spec.whatwg.org/multipage/browsing-the-web.html#target-element Bug: 959163 Change-Id: Idb462a17972f4758df9617653f03b88196fd565a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1965273Reviewed-by:
David Bokan <bokan@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/master@{#726031}
-
Caleb Rouleau authored
This means that if you abridge a benchmark or remove stories from a benchmark then that information will be reflected in http://go/chrome-benchmark-schedules as soon as you check in your change. Also, add a --filter-only option so that we can keep this up to date without needing to run a slow bigquery query each time. Also, add validation to presubmit so that this stays up to date. Also, make the output deterministic by sorting it. Bug: 1030840 Change-Id: I2cda6110a029953559874707852230693d0ac7b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973232Reviewed-by:
John Chen <johnchen@chromium.org> Auto-Submit: Caleb Rouleau <crouleau@chromium.org> Cr-Commit-Position: refs/heads/master@{#726030}
-
Caleb Rouleau authored
I somehow checked in the wrong schedules file in https://chromium-review.googlesource.com/c/chromium/src/+/1956369 that didn't account for the changes in https://chromium-review.googlesource.com/c/chromium/src/+/1956369 In this change, I manually skip the bigquery work to simply get the whole thing in sync. Later I will add presubmit checks for all of this. Bug: 1030840 Change-Id: I20cc91b574396bdbb78642430d0effef86525025 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973239 Auto-Submit: Caleb Rouleau <crouleau@chromium.org> Reviewed-by:
John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#726029}
-
Shivani Sharma authored
Bug: 1018718 Change-Id: I1f43af0d604d58abe37f2cacf58283b795e1d725 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948230 Commit-Queue: Shivani Sharma <shivanisha@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#726028}
-
John Budorick authored
Bug: 922145 Change-Id: Iefd23b8fc535628e6165b6a09608b205ed705096 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973788 Commit-Queue: John Budorick <jbudorick@chromium.org> Commit-Queue: Yun Liu <yliuyliu@google.com> Reviewed-by:
Yun Liu <yliuyliu@google.com> Auto-Submit: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#726027}
-
Mitsuru Oshima authored
* Added name for wallpaper widget * Renamed EndPendingAnimation to StopAnimating * Renamed WallpaperWidgetController::property to GetWallpaperProperty(). (It'll be delegated to wallpaper view in next CL). * Added WallpaperView::proeprty(), instead of returning blur_sigma/opacity separately. Bug: None Test: None Change-Id: Ic8aa86990a45e6a99472733f4949861be22dd89e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973963Reviewed-by:
Sammie Quon <sammiequon@chromium.org> Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#726026}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/8d97c155be33..ee94693c7ab1 git log 8d97c155be33..ee94693c7ab1 --date=short --first-parent --format='%ad %ae %s' 2019-12-18 robertphillips@google.com Fix a sprinkling of GPU issues 2019-12-18 rmistry@google.com Make OUTPUT_DIRECTORY in ProdDoxyfile configurable 2019-12-18 csmartdalton@google.com Add a kWireframe flag to GrPipeline Created with: gclient setdep -r src/third_party/skia@ee94693c7ab1 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC borenet@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: borenet@google.com Change-Id: Iab150c8bd8dc87a161348be0d2a267aa352dfba4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974196Reviewed-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@{#726025}
-
Josh Nohle authored
The flag to disable v1 DeviceSync is was named "CryptAuthV1DeviceSyncDeprecate". The word "deprecate" is ambiguous. Rename to "DisableCryptAuthV1DeviceSync". Also, change helper function name from ShouldDeprecateV1DeviceSync() to ShouldUseV1DeviceSync() in order to avoid double negation in the code. Bug: 951969 Change-Id: Ibbc4286a9c2d6eac50dbfb428ecb3ca06abcacd5 Fixed: 1034041 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972237Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Martin Barbella <mbarbella@chromium.org> Commit-Queue: Josh Nohle <nohle@chromium.org> Cr-Commit-Position: refs/heads/master@{#726024}
-
kylechar authored
The feature is launched everywhere so the feature checks are no longer necessary. Bug: 936425 Change-Id: I7ad1800cc44dc9260ea2348825b7d11aa330e842 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972314 Commit-Queue: kylechar <kylechar@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Cr-Commit-Position: refs/heads/master@{#726023}
-
Andrey Kosyakov authored
After we switched to using mojo host resolver in https://chromium-review.googlesource.com/c/chromium/src/+/1199848/ we need to make sure HostResolver is destroyed on the same thread that it was created on. Ensuring this with ref-counted DeviceProvider would require a helper class, so just re-create it each time we need to resolve a host for simplicity. The overhead should be small considering to other things we do (like not caching host resolution lately etc). Bug: 1028292, 1025369, 921608 Change-Id: I6f8d2e663f0ac872964e1725e261232493bbbfe5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972781 Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#726022}
-
Kurt Horimoto authored
Updates InfobarOverlayRequestCancelHandler to handle InfoBar replacements. When the corresponding request's InfoBar has been replaced, the cancel handler creates an OverlayRequest for the replacement and inserts it in the queue after the replaced InfoBar's request. Then, the existing request is cancelled, removing it from the queue. Bug: 1030357 Change-Id: Idff53a277c4115adb6075017db5aed4c4e3afac7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1960810 Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#726021}
-
Devlin Cronin authored
Improve and possibly de-flake the browser test BrowserActionApiTest, CloseBackgroundPage. This includes: - Moving away from the content::NotificationService system to an ExtensionHostObserver; this ensures we receive events for the right ExtensionHost. - Waiting for the setBadgeText() call to finish before closing the background page. Neither of these is guaranteed to fix the problem (which I was unable to reproduce locally), but they're still desirable, so may as well give it a shot. Bug: 1035075 Change-Id: I6b24d64d7ddd95b46bb26aae411e8a7ce269e3d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972292 Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Cr-Commit-Position: refs/heads/master@{#726020}
-
Chromium WPT Sync authored
Using wpt-import in Chromium ccf01bd7. With Chromium commits locally applied on WPT: c15c1083 "Add test that datachannels close when connection does." fadf6027 "[css-pseudo] Recalculate viewport units in ::marker" 044ad8ad "[OT-PW] Make it work with OOP-R" 2ad15546 "Make interpolation-testcommon.js support composition" 9899425d "[css-layout-api] IntrinsicSizes Implementation" 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 NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: I75fd3a0d9f748e42ba11b91ed1f051c1701e49e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973995Reviewed-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@{#726019}
-
behdad authored
source_id and sequence_number is used on BeginFrameArgs and BeginFrameAck for identifying the frame. This Change creates a single struct for these info. TBR=sky@chromium.org, eseckler@chromium.org, meacer@chromium.org Bug: none Change-Id: Iabafca83472af0facd39ada5f11c6ad2017bf84d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1966606 Commit-Queue: Behdad Bakhshinategh <behdadb@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#726018}
-
David Tseng authored
- significant changes in this change when compared to upgrade to v3.11 (which was reverted) t/l/w/liblouis_wrapper.js. c/t/d/c/liblouis_wasm/test.js also exercises the en-ueb-g2.ctb table which actually caused many of the crashes with the original change. Most notably, LibLouis now seems to more often than not, require a null terminated string. This is an improvement, though lou_translateString is quite unpredictable as to whether it counts the terminator in its out param (inLen). Between the upgrade to v3.12, and the compile of the wasm module using the latest Emsdk (though fastcomp), seems to have resolved instability. Bug: 1033082 Test: browser_tests --gtest_filter=ChormeVox*.* and --gtest_filter=LibLouis*.*. Manually with a Focus 40 which has a hardware braille keyboard. Also, test with en-ueb-g2 which was causing crashes. Change-Id: I930c6dbe771dafb7df39f50eb16ba15fda418609 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970643 Commit-Queue: David Tseng <dtseng@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#726017}
-
Eugene But authored
Notable EG2 changes: - Use ensureAppLaunchedWithFeaturesEnabled to enable the feature - Do not tap back to dismiss settings (this does not fork in EG2 and tappoing back is irrrelevant for acessibility testing, which is the purpose of this test). Bug: 987646 Change-Id: Ib3794661e09cd6a40d4e46999c31626f47d70124 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972911 Auto-Submit: Eugene But <eugenebut@chromium.org> Reviewed-by:
Peter Lee <pkl@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#726016}
-
Rakib M. Hasan authored
This CL also adds the ability to remove configurations from expectations that do not have bugs. This CL also affects the behavior of the rebaseline and try-flag tools so that it will create expectations which do not have more than one mutually exclusive specifier. It also adds a new presubmit check which prevents users from creating expectations that contain more than one mutually exclusive specifier. We are doing this because the new test expectations format does not allow expectations to have more than one mutually exclusive specifier. Bug: 986447 Change-Id: Ic97c4f1bba1e57b71f4624ef88ad69b5c8122a9f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970297 Commit-Queue: Rakib Hasan <rmhasan@google.com> Reviewed-by:
Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#726015}
-
Kurt Horimoto authored
This CL adds BrowserObserver to the public Browser target and updates some BUILD.gn files to reference the public target. Bug: none Change-Id: I78a009fb6569121cd745f7f5771c43c1c538f24d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972586Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> Auto-Submit: Kurt Horimoto <kkhorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#726014}
-
Evan Stade authored
It's unused. Bug: none Change-Id: Ic080066b9502f834a5f77abdffe53a5967c8e79b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972457Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#726013}
-