GitLab

Adds back call mistakenly removed in
https://chromium-review.googlesource.com/c/chromium/src/+/2496287

R=dmazzoni@chromium.org

Change-Id: I3fab6b94a8c9c80527b0b2d451ef1d6911be1f10
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532928Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826388}

The contents moved to callback_helpers.h.

Also renames a few new bind_helpers.h includes that have been added
in the meantime.

R=dcheng@chromium.org
TBR=

Change-Id: Ia511a4939d60bd2ef5b474185e3716017543f8af
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523545
Commit-Queue: danakj <danakj@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826387}

This CL adds metrics for counting the number of times a user adjusts a
capture region. Visit go/capture-mode-metrics to see other planned
metrics.

Test: manual + added
Bug: 1140182
Change-Id: Ib058be572eff6c539e8673cd9a25d340e053b521
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523492Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Reviewed-by: Weilun Shi <sweilun@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Sammie Quon <sammiequon@chromium.org>
Commit-Queue: Jeremy Chinsen <chinsenj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826386}

Sort the scanners in the scanner dropdown alphabetically and set the
first scanner as the default scanner.

Bug: 1059779
Change-Id: I5a40ea0fadeb5c6166450a537c81dfe8fecdd240
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528941Reviewed-by: Zentaro Kavanagh <zentaro@chromium.org>
Commit-Queue: Jesse Schettler <jschettler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826385}

Change-Id: Ibc08b6a720494c7234104a8089b6ad9c580e774f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528298Reviewed-by: Sunny Sachanandani <sunnyps@chromium.org>
Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Reviewed-by: Robert Kroeger <rjkroege@chromium.org>
Reviewed-by: weiliangc <weiliangc@chromium.org>
Commit-Queue: Maggie Chen <magchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826384}

Change-Id: I63536d743f3396f422fb9959eada4e0fec733a42
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528633
Auto-Submit: Ted Meyer <tmathmeyer@chromium.org>
Reviewed-by: Frank Liberato <liberato@chromium.org>
Reviewed-by: Caitlin Fischer <caitlinfischer@google.com>
Commit-Queue: Caitlin Fischer <caitlinfischer@google.com>
Cr-Commit-Position: refs/heads/master@{#826383}

Some unit tests should be removed or re-worked when non-Ozone X11 is
deprecated.  Here the TODOs are changed so they refer to the relevant
issue.

Bug: 1109112
Change-Id: I221c8447b62bb4aa4a7e38acb53cca84ff6427d8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526300Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Auto-Submit: Alexander Dunaev <adunaev@igalia.com>
Cr-Commit-Position: refs/heads/master@{#826382}

Looking at crash reports where the LocalRootImpl is null it is clear
that the HandleMouseUp can detach the frame and the GetPage accessor
will then be null.

Change-Id: If856e914135b833bfacbce84be5c15b242acd09f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533021
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826381}

Disable IncognitoBackgroundSyncBrowserTest.OffTheRecordProfile. This is
failing on Android bots.

TBR=nator@chromium.org

Bug: 1147754
Change-Id: Ib192173097ab3afe4371431d3a9b2c5d6440a57d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532856Reviewed-by: Xing Liu <xingliu@chromium.org>
Commit-Queue: Xing Liu <xingliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826380}

This adds an integration test, testing changing element conditions.

This CL also fixes several bugs that were discovered during testing:
- Fixed issue where ElementConditions were not correctly added and
cleared.
- Constructing a new instance of AssistantHeaderCoordinator no longer
has a side effect in AssistantHeaderModel.
- Fixed a possible crash when showing a new trigger script while another
was in the process of being hidden (crash comes down to timing). The fix
is to always create a new BottomSheetContent, instead of re-populating
the old one.

Bug: b/171776026
Change-Id: I7eac6673100ab21d459ec7f1ed200b2be9a44ff5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2524532
Commit-Queue: Clemens Arbesser <arbesser@google.com>
Reviewed-by: Marian Fechete <marianfe@google.com>
Cr-Commit-Position: refs/heads/master@{#826379}

ScopedObserver is being deprecated in favor of two new classes:
- base::ScopedObservation for observers that only ever observe
  a single source.
- base::ScopedMultiSourceObservation for observers that do or may
  observe more than a single source.

This CL was uploaded by git cl split.

R=vasilii@chromium.org

Bug: 1145565
Change-Id: I12660e862e5b73ed06f39d1ca4eb40a8067a4760
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533022
Auto-Submit: Sigurður Ásgeirsson <siggi@chromium.org>
Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826378}

This CL is a followup to https://crrev.com/c/2482443.
This adds simple unittest that verifies that device
scroll list in bluetooth tray view is updated correctly.

Fixed: 1141995
Change-Id: I8237871ff6c06f864a804d106a0556d3552b6109
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2508234Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Azeem Arshad <azeemarshad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826377}

Bug: 1145379
Change-Id: I5284a9c080e5b3d951eff65ca8ef4426b3377cd6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532918
Commit-Queue: Sergio Collazos <sczs@chromium.org>
Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826376}

This change enables by default the feature where the
long search suggestion text would be wrapped to the
second line.

Bug: 1085184
Change-Id: Id1e8848457250e8bf35cca2a7b8bf6723374661c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2504071Reviewed-by: Caitlin Fischer <caitlinfischer@google.com>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Tomasz Wiszkowski <ender@google.com>
Cr-Commit-Position: refs/heads/master@{#826375}

The ArcSharesheetBridge will receive share requests from Android
and handle them with Chrome's sharesheet in a Chrome Custom Tab.

This change sets up the skeleton to accept connections from
Android.

BUG=b:172963488
TEST=Deployed chrome, and verified share intents is received from
     Android using Android's APIDemos apk.

Change-Id: Ib14e414f65c3bb7b675a0d1a0d846fc3a17b0769
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531394
Auto-Submit: Prameet Shah <phshah@chromium.org>
Commit-Queue: Josh Horwich <jhorwich@chromium.org>
Reviewed-by: Josh Horwich <jhorwich@chromium.org>
Reviewed-by: Sam McNally <sammc@chromium.org>
Reviewed-by: Maggie Cai <mxcai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826374}

ARC provisioning can fail due to ARC instance abruptly stopping. This
can have three possible reasons however we show the same generic error
"Something went wrong" instead of a more useful error.

With this change, we're adding the ability to show a unique error for
each of the three reasons for ARC instance stopping.

BUG=b:166322619
TEST=force the arc stop error, deploy to dut, ensure new metric is
logged in chrome://histograms

Change-Id: I5d73fcf9f845237e6d4d52e52a24a6cfe7f34e74
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2518664Reviewed-by: Yury Khmel <khmel@chromium.org>
Reviewed-by: Hidehiko Abe <hidehiko@chromium.org>
Reviewed-by: Jorge Lucangeli Obes <jorgelo@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Commit-Queue: Muhammad Hasan Khan <mhasank@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826373}

All non-primitive types need explicit non/null modifier as per style guide.

chrome/test/data/webui/chromeos/diagnostics:closure_compile

Bug: 1125150
Test: autoninja
Change-Id: Ic1a6ff8c08cb1fb8770158c3154eb0bec8879712
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2530191
Auto-Submit: Joon Ahn <joonbug@chromium.org>
Reviewed-by: Zentaro Kavanagh <zentaro@chromium.org>
Commit-Queue: Joon Ahn <joonbug@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826372}

ScopedObserver is being deprecated in favor of two new classes:
- base::ScopedObservation for observers that only ever observe
  a single source.
- base::ScopedMultiSourceObservation for observers that do or may
  observe more than a single source.

This CL was uploaded by git cl split.

R=harringtond@chromium.org

Bug: 1145565
Change-Id: I00ea07dd484293ba9cc164f97838def2a4e15615
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532657
Auto-Submit: Sigurður Ásgeirsson <siggi@chromium.org>
Reviewed-by: Dan H <harringtond@chromium.org>
Commit-Queue: Dan H <harringtond@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826371}

Change-Id: Iece225a990448b954161d3286a1f595f60e6f04e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2530262Reviewed-by: Peter Williamson <petewil@chromium.org>
Commit-Queue: Cathy Li <chili@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826370}

ScopedObserver is being deprecated in favor of two new classes:
- base::ScopedObservation for observers that only ever observe
  a single source.
- base::ScopedMultiSourceObservation for observers that do or may
  observe more than a single source.

This CL was uploaded by git cl split.

R=msramek@chromium.org

Bug: 1145565
Change-Id: I10357ac1f5c2ae101b2a63a7e0841f890cf3134a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532717
Auto-Submit: Sigurður Ásgeirsson <siggi@chromium.org>
Reviewed-by: Martin Šrámek <msramek@chromium.org>
Commit-Queue: Martin Šrámek <msramek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826369}

- Current full battery capacity was moved to a different struct

Bug: 1125150
Test: browser_tests --gtest_filter=DiagnosticsApp*
Change-Id: I64e6c3ff810a38e53c4ea1bf95e58f56469d81de
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531114
Commit-Queue: Zentaro Kavanagh <zentaro@chromium.org>
Reviewed-by: Bailey Berro <baileyberro@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826368}

The ActiveURL is now set via messages sent on the LocalFrame.
Hopefully this will help improve crash reports. This use to occur
in RenderFrameImpl::OnMessageReceived.

BUG=993189

Change-Id: I184ef2892b550fb79ced43c2c48315f68dd6c358
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2521078Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826367}

Bug: 1147858,1116601,1142017
Change-Id: I1611af0945f340f704055bedb3e1bffd5765c8f7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533015
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: Mark Mentovai <mark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826366}

ScriptExecutor is used to drive programmatic script execution by
extensions into different web frames. While it is reasonably tested
transitively through end-to-end tests for the tabs.executeScript()
method and friends, we should have more targetted testing for its
functionality as well.

Add a browser test with a simple test case of injecting script into
the main world and returning the result as a starting point.

Bug: None
Change-Id: I9a00a6b1141935d35cf1c0317c0ccda734c991e4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528912
Auto-Submit: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: David Bertoni <dbertoni@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826365}

Move the duplicate implementation of these methods into the base class.

BUG=1097816

Change-Id: Ibc97bc14dbc246cbc7bdced0a53bb43afa96cfbf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526912
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826364}

TBR=robertma@chromium.org

Change-Id: I8c48c7408c5f91077bb07c7e73897fbdd1c9313f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533016Reviewed-by: Luke Z <lpz@chromium.org>
Reviewed-by: Robert Ma <robertma@chromium.org>
Commit-Queue: Luke Z <lpz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826363}

Bug: 1130482
Change-Id: I1ea101821632f331e37e2ea0eecc04716a93b41f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532261Reviewed-by: Roman Sorokin [CET] <rsorokin@chromium.org>
Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826362}

Change-Id: I18846c0f5e331678b7f0c9bd21a142e22c30c991
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2520021
Commit-Queue: Liquan (Max) Gu <maxlg@chromium.org>
Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826361}

https://chromium.googlesource.com/catapult.git/+log/994d3835f535..ac95abdb4b12

2020-11-11 gab@chromium.org [telemetry] Add "base" as a key low-overhead tracing category

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/catapult-autoroll
Please CC oysteine@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:chromeos-kevin-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
Bug: None
Tbr: oysteine@google.com
Change-Id: I4a4de4e1131d384d8adfaf93e3123699b4d4b317
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533094Reviewed-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@{#826360}

Dark mode was not working correctly since layouts were being inflated
with the feed isolated split context and not the activity context. This
change pulls only the ClassLoader from the feed isolated split context,
and adds a util which can be used elsewhere.

Bug: 1126301
Change-Id: I9f1da598cb1df2a33c1095bc9afb8e451b518991
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531516
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826359}

This fixes an issue where textures sampling parameters are incorrect
when there is a concurrent read of the texture. For example when an
upscaled video texture is read back to draw to a canvas, the texture
will end up pixelated some frames and smooth in others.
GL_TEXTURE_MAG_FILTER is expected to be GL_LINEAR but it's changed to
GL_NEAREST for readback and not reset.

There is no way to know if a concurrent read happened in Chrome today so
we have to always reset GL texture parameters to avoid it. For
GLRenderer always resetting the sampling parameters when binding the
texture fixes the provided examples. With SkiaRenderer texture
parameters are reset right after BeginAccess() is called on the texture.

Bug: 1092080
Change-Id: I968202f3dfff8c3c300c7e98583c55aca8639f81
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2431550Reviewed-by: Sunny Sachanandani <sunnyps@chromium.org>
Reviewed-by: Khushal <khushalsagar@chromium.org>
Commit-Queue: kylechar <kylechar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826358}

I have ~2 more CLs to land before this feature is ready for release.
I suspect that this won't be before M88 branch point so I am disabling
the feature and will re-enable it once we can control it via the
registry in a follow-up CL.  This way, if the required CLs don't land
before branch point, the host is still in a releasable state w/o
requiring any merges.

Change-Id: I03ec977f563679d6b4b84fa6fbbdf849d185e36b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532924
Commit-Queue: Jamie Walch <jamiewalch@chromium.org>
Auto-Submit: Joe Downing <joedow@chromium.org>
Reviewed-by: Jamie Walch <jamiewalch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826357}

Bug: 1141674
Change-Id: I7c9e4854e7dd9d0c94500c61ccd45e0a1f1f6f27
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528164Reviewed-by: Taylor Bergquist <tbergquist@chromium.org>
Commit-Queue: Charlene Yan <cyan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826356}

The should let us deprecate waitForNavigation parameter to
Emulation.setVirtualTimePolicy and make its contract a bit
more sensible: instead of virtual time being set in advance
and waiting to kick-in after the navigation, we let the
navigation wait for the virtual time budget to be granted
before the document is actually loaded. This removes the
race between document resources loading (and, conditionally,
consuming 10ms each) and the client granting the budget.

Change-Id: Iffc794a450dbb8cb6ae1d37100386f00c7638abf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531363Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Andrey Kosyakov <caseq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826355}

we are obsoleting the unnecessary `TpmIsBeingOwned()`.

BUG=b:172775884
TEST=build ok.

Change-Id: I593ae2c86994b997d4e1ac9d98e096f73f0375f7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532156
Commit-Queue: Leo Lai <cylai@google.com>
Reviewed-by: Achuith Bhandarkar <achuith@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826354}

This reverts commit 3aed6673.

Reason for revert: suspected in build failures on ios-simulator, ios-simulator-noncq

E.g. https://ci.chromium.org/p/chromium/builders/ci/ios-simulator/59564 and https://ci.chromium.org/p/chromium/builders/ci/ios-simulator-noncq/13473

Original change's description:
> Reland "mac/ios: Use real response files instead of -filelist."
>
> This is a reland of 5e46febb
>
> Original change's description:
> > mac/ios: Use real response files instead of -filelist.
> >
> > Xcode 12 added support for real response files, so let's use them.
> > Allows making response file contents more like on other systems
> > (libs, solibs, frameworks, swiftmodules now go in the rsp file too),
> > and makes it theoretically possible to have object files with spaces
> > in their name.
> >
> > No intentional behavior change.
> >
> > Bug: none
> > Change-Id: I8c2d6ff1abd47261bf7674dc7bf4506814681ad4
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527621
> > Auto-Submit: Nico Weber <thakis@chromium.org>
> > Commit-Queue: Brett Wilson <brettw@chromium.org>
> > Reviewed-by: Brett Wilson <brettw@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#825480}
>
> Bug: none
> Change-Id: Ib355508d2d45ab3ddf9ece76b6e95106c784fde9
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2530350
> Auto-Submit: Nico Weber <thakis@chromium.org>
> Reviewed-by: Tommy Martino <tmartino@chromium.org>
> Reviewed-by: Mark Mentovai <mark@chromium.org>
> Commit-Queue: Nico Weber <thakis@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#826282}

TBR=thakis@chromium.org,tmartino@chromium.org,mark@chromium.org

Change-Id: Ie98fc38610c34711ce14d4e76582c78d4e3280bb
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: none
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533176Reviewed-by: James MacLean <wjmaclean@chromium.org>
Commit-Queue: James MacLean <wjmaclean@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826353}

This reverts commit 3c30ec0c.

Reason for revert: catapult fix has rolled in: crrev.com/c/2530627

Original change's description:
> Disable resultdb.result_sink experiment on all chromium try builders.
>
> Leading to many telemetry_unittest failures on mac-rel.
>
> TBR=gbeaty
>
> Bug: 1147657
> Change-Id: I6456649d1eff2e7f700b7ea37b2ff4da793cb374
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531295
> Reviewed-by: Ben Pastene <bpastene@chromium.org>
> Commit-Queue: Ben Pastene <bpastene@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#826052}

TBR=bpastene@chromium.org,gbeaty@chromium.org

Change-Id: Ic44cf6dc06d92a40eb64f5ddbe6d2dd68570d5d9
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1147657
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532857Reviewed-by: Ben Pastene <bpastene@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826352}

This change will allow an admin to set a policy to prevent any UI
from being shown when a user connects to a non-curtained session.
This change is targeted at improving the user experience when the
machine is not physically accessible (i.e. we know that a local
user cannot interact with the machine) and we want to remove the
disconnect window and any other UI in that case.

Change-Id: I3ba89e7aaf7538ae5cce4385fb65256cb202ef5a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527984
Commit-Queue: Joe Downing <joedow@chromium.org>
Reviewed-by: Jamie Walch <jamiewalch@chromium.org>
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826351}

BUG=1029038

Change-Id: I81ccafffacc1188e485d9950ee84141143e7a8d4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527627Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Peter Wen <wnwen@chromium.org>
Reviewed-by: Paul Jensen <pauljensen@chromium.org>
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826350}

Note: Based on a few historical bugs that address a similar issue. Could not repro
on my personal test device, will coordinate verification with reporter.

Bug: 1145497
Change-Id: I3d1f9781037fad41cb1f7af578781eed9b7c1ec3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2529971Reviewed-by: Jeffrey Cohen <jeffreycohen@chromium.org>
Commit-Queue: Travis Skare <skare@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826349}