GitLab

This reverts commit 081bb19a.

Reason for revert: crrev.com/c/1117264 got reverted, and this CL is based on that.

Original change's description:
> Roll traffic_annotation_auditor binary for Windows.
> 
> The new binary for traffic_annotation_auditor after the change in
> crrev.com/c/1117264 is rolled.
> 
> Bug: 856884
> Change-Id: I76523df5e58ace767ac19c619c9bb32da5a25674
> TBR: georgesak@chromium.org
> Reviewed-on: https://chromium-review.googlesource.com/1117686
> Reviewed-by: Ramin Halavati <rhalavati@chromium.org>
> Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#571048}

TBR=georgesak@chromium.org,rhalavati@chromium.org

Change-Id: I2706aa14d74d1c858d6e53c2703baf55c041f340
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 856884
Reviewed-on: https://chromium-review.googlesource.com/1119805Reviewed-by: Ramin Halavati <rhalavati@chromium.org>
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571412}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/ae1f03388f7a..406de133efaf


git log ae1f03388f7a..406de133efaf --date=short --no-merges --format='%ad %ae %s'
2018-06-29 yyanagisawa@chromium.org Add gclient_utils to InputAPI export.


Created with:
  gclient setdep -r src/third_party/depot_tools@406de133efaf

The AutoRoll server is located here: https://depot-tools-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=agable@chromium.org

Change-Id: I4ebdc7e46587f7903d11d5a904aabd144869f54b
Reviewed-on: https://chromium-review.googlesource.com/1119706Reviewed-by: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#571411}

Fixes Error: webkitResolveLocalFileSystemURL not found: [filesystem:drive/team_drives]

Bug: 813477
Cq-Include-Trybots: luci.chromium.try:closure_compilation
Change-Id: I6979d279f6986dba4ff5cb0ee6ac63b8f4594bfd
Reviewed-on: https://chromium-review.googlesource.com/1119733
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Reviewed-by: Luciano Pacheco <lucmult@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571410}

The AutoRoll server is located here: https://fuchsia-sdk-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=luci.chromium.try:fuchsia_arm64_cast_audio;luci.chromium.try:fuchsia_x64_cast_audio
TBR=cr-fuchsia+bot@chromium.org

Change-Id: Ia4a739ea250766e1d1747d2b39d0d2f529e41fba
Reviewed-on: https://chromium-review.googlesource.com/1119765Reviewed-by: Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#571409}

The GCMAccountTracker unittest currently plays fast-and-loose with the
notion of an "account ID", which can be either the Gaia ID or the
email address for a given account depending on the platform. This is a
blocker to porting GCMAccountTracker to use IdentityManager, as the
IdentityManager test infrastructure deliberately makes a much more
explicit and enforced separation between email address, Gaia ID, and
account ID. This CL paves the way for that porting by changing the
GCMAccountTracker unittest to be explicit about its usage of the
email address, Gaia ID, and account ID of a given account.

Interestingly, this change exposed an actual production bug
(https://crbug.com/856170).

Bug: 809923
Change-Id: I46e2f632bacd89ba55132255a95bcb089d4fb3c8
Reviewed-on: https://chromium-review.googlesource.com/1113548
Commit-Queue: Colin Blundell <blundell@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571408}

TBR=dcheng@chromium.org,jamescook@chromium.org,msw@chromium.org,xiaohuic@chromium.org

Original change's description:
> assistant: filter out incognito windows in assistant screenshot.
>
> Per privacy requirement, incognito windows should not be included
> in screenshot sent to assistant server.
>
> Bug: b/78193324
> Test:
>   ash_unittests --gtest_filter=AssistantControllerTest.Screenshot
>   browser_tests --gtest_filter=BrowserNonClientFrameViewAshTest.IncognitoMarkedAsAssistantBlocked*
>
> Change-Id: Idf55135959721d69414a1ec6fae396f9763ebe49
> Reviewed-on: https://chromium-review.googlesource.com/1114293
> Commit-Queue: Muyuan Li <muyuanli@chromium.org>
> Reviewed-by: James Cook <jamescook@chromium.org>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
> Reviewed-by: Michael Wasserman <msw@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#571385}

Change-Id: Ieecafffe5f1e14971b3c07dc799323a29885de16
Reviewed-on: https://chromium-review.googlesource.com/1119766Reviewed-by: Muyuan Li <muyuanli@chromium.org>
Commit-Queue: Muyuan Li <muyuanli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571407}

Currently, All Sites shows all origins in their raw format, which looks crowded
and can be confusing to users. Use the host instead in most situations,
displaying non-HTTPS schemes off to the side to easily differentiate from other
hosts that may be the same. Note non-default ports will also be shown as part of
the host.

See a screenshot -
https://bugs.chromium.org/p/chromium/issues/detail?id=835712#c12

Bug: 835712
Cq-Include-Trybots: luci.chromium.try:closure_compilation
Change-Id: Id34ef01e3fe41cd428e5a97bbc4fc207dbe07d05
Reviewed-on: https://chromium-review.googlesource.com/1107579
Commit-Queue: Patti <patricialor@chromium.org>
Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Reviewed-by: Matt Giuca <mgiuca@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571406}

This CL may cause a small binary size increase, roughly proportional
to how long it's been since our last AFDO profile roll. For larger
increases (around or exceeding 100KB), please file a bug against
gbiv@chromium.org. Additional context: https://crbug.com/805539

The AutoRoll server is located here: https://afdo-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=gbiv@chromium.org

Change-Id: Ie62c720fcdce018d12a54d5b69ddf0f34d8c76a3
Reviewed-on: https://chromium-review.googlesource.com/1119767Reviewed-by: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#571405}

security_context.h is used in 3,100+ compilation units, and this CL
reduces its pre-processed size from 3.10MB to 1.74MB.

Bug: 242216
Change-Id: I0fe6dd82eea775853cfe23377f2803bd0f90b069
Reviewed-on: https://chromium-review.googlesource.com/1119613Reviewed-by: Takayoshi Kochi <kochi@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571404}

This change does the plumbing for ImageDecoder::HighBitDepthDecodingOption,
which is required to differentiate the expected behavior from PNGImageDecoder.
This is a partial CL toward supporting 16 bit PNG decode to half float backed
bitmap. This CL does not change the behavior and does not contain any tests.
Testing is covered in the main CL:
chromium-review.googlesource.com/c/chromium/src/+/1079788.

Bug: 839034
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Ie6f34b866b1fbe09bd425bdbecd7fb9726cf9621
Reviewed-on: https://chromium-review.googlesource.com/1113506
Commit-Queue: Noel Gordon <noel@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Noel Gordon <noel@chromium.org>
Reviewed-by: Urvang Joshi <urvang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571403}

There was a race condition when the policy fetch request that's initiated in CloudPolicyTest::SetUpOnMainThread is cancelled becauase the test bodies start their own request (through RefreshPolicies). The first request seems to get the python test server in a bad state and it stops responding to requests afterwards.

The fix is to skip issuing the first request by temporarily unregistering CloudPolicyRefreshScheduler.

TBR=pmarko

Bug: 858883
Change-Id: I8c158ab3dffc2f38a7712a6806f9b8d28efbcfdb
Reviewed-on: https://chromium-review.googlesource.com/1119628
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571402}

This CL follows signed exchange spec change:
https://github.com/WICG/webpackage/commit/3903da6e

Bug: 803774
Change-Id: If93b419488c166b1d8ec8a35b7a752a439d1cd7a
Reviewed-on: https://chromium-review.googlesource.com/1114407Reviewed-by: Kunihiko Sakamoto <ksakamoto@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571401}

Changing the paper-button bg-color to white was too big of a change
and would require reviewing all usages of paper-button to ensure that
style bugs similar to the one found in cr-toast do not occur.

The default paper-button bg-color will not be set. The bg-color of
paper-button in the user-manager will be set to white. And for
cr-toast, the bg-color will be set to transparent (including the hover
state).

Bug: 845576
Cq-Include-Trybots: luci.chromium.try:closure_compilation
Change-Id: I5b7a513012c43e2da0163555c1a5d25e69cf8328
Reviewed-on: https://chromium-review.googlesource.com/1115354
Commit-Queue: Esmael El-Moslimany <aee@chromium.org>
Reviewed-by: Scott Chen <scottchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571400}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/6d97dd7fb70f..898687843450


git log 6d97dd7fb70f..898687843450 --date=short --no-merges --format='%ad %ae %s'
2018-06-29 vapier@chromium.org gerrit: update old googlecode URLs


Created with:
  gclient setdep -r src/third_party/chromite@898687843450

The AutoRoll server is located here: https://chromite-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.



BUG=chromium:None
TBR=chrome-os-gardeners@chromium.org

Change-Id: Ia948fd9c24c9d0a0c4b259ffc3f612467b7dc1ad
Reviewed-on: https://chromium-review.googlesource.com/1119705Reviewed-by: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#571399}

crrev.com/570859 removed a trace-event that was used to find the frames,
but failed to update the trace-name correctly. This fixes the script so
that frames can correctly be determined again.

BUG=855328

Change-Id: I22980d8473272a256e348b1d3f54e041be51dda1
Reviewed-on: https://chromium-review.googlesource.com/1119625
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#571398}

(1) MRUFeatures are calculated and saved when WillCloseAllTabs() is
    invoked but returned when the WebContentsDestroyed is called.
(2) MRUFeatures is calculated in WebContentsDestroyed if not in
    closing_all mode.
(3) Change tab_strip_model to respond to WillCloseAllTabs,
    CloseAllTabsCanceled and CloseAllTabsFinished.

Bug: 817174

Change-Id: Idea15b5c48666fbc0163ccba62ee37b23bb192f0
Reviewed-on: https://chromium-review.googlesource.com/1086552Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Commit-Queue: Charles . <charleszhao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571397}

Bug: 858194
Change-Id: I051084347da2f395e25b090ba54439c866b0c044
Reviewed-on: https://chromium-review.googlesource.com/1119357
Commit-Queue: Wez <wez@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571396}

This CL restores IME mode icon to UnifiedSystemTray. The icon was
removed as a part of UnifiedSystemTray & Shelf spec, but it turned out
the icon is important especially for CJK users with laptop mode.
We still hide the icon for tablet mode.

TEST=manual
BUG=854948

Change-Id: I55bdd88627f91c9e285242b725c4a55451d0c60a
Reviewed-on: https://chromium-review.googlesource.com/1116412
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571395}

This reverts commit 9f6cd533.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 571385 as the
culprit for failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzlmNmNkNTMzZGIxNGJmZjY3OWJkOGNkNjcxNGI3MzI0NTVlNDRjMTYM

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.chromiumos/linux-chromeos-dbg/6550

Sample Failed Step: compile

Original change's description:
> assistant: filter out incognito windows in assistant screenshot.
> 
> Per privacy requirement, incognito windows should not be included
> in screenshot sent to assistant server.
> 
>   ash_unittests --gtest_filter=AssistantControllerTest.Screenshot
>   browser_tests --gtest_filter=BrowserNonClientFrameViewAshTest.IncognitoMarkedAsAssistantBlocked*
> 
> Bug: b/78193324
> Test: 
> Change-Id: Idf55135959721d69414a1ec6fae396f9763ebe49
> Reviewed-on: https://chromium-review.googlesource.com/1114293
> Commit-Queue: Muyuan Li <muyuanli@chromium.org>
> Reviewed-by: James Cook <jamescook@chromium.org>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
> Reviewed-by: Michael Wasserman <msw@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#571385}

Change-Id: Ieb8299a17fca3e3033e2c5a67933ed5ef642335b
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: b/78193324
Reviewed-on: https://chromium-review.googlesource.com/1119745
Cr-Commit-Position: refs/heads/master@{#571394}

This is a reland of ea1974d4

Original change's description:
> [chrome_elf, third-party block support] Pass file path in registry.
> 
> Due to rare case of install_static::GetUserDataDir() path expansion,
> this API cannot be used inside DllMain().  The full, expanded path to
> the packed blacklist file will be dropped into install_static::GetRegistryPath(),
> in the kThirdPartyRegKeyName subkey, in the kBlFilePathRegValue REG_SZ value.
> This has no other impact on existing functionality.
> 
> - This change includes comments, constants, and tests.
> 
> - LogLoadAttempt() will now include the section path for any 'blocked' logs
> as well.  Friendly reminder that log.AddEntry() will still not add duplicate
> 'blocked' logs, to protect against spammy logging.
> 
> - Removed old bit of uninstaller touching old blacklist.
> 
> Test: chrome_elf_unittests.exe, ThirdParty*
> Bug: 769590
> Change-Id: Id9ada49215aa110db9fb72a4712da9d3fb7ccf31
> Reviewed-on: https://chromium-review.googlesource.com/1109600
> Reviewed-by: Robert Shield <robertshield@chromium.org>
> Reviewed-by: Tom Sepez <tsepez@chromium.org>
> Commit-Queue: Penny MacNeil <pennymac@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#570849}

Bug: 769590
Change-Id: I80fd3a322a9d578a9d4b716f81f5a8d110e9eab4
Reviewed-on: https://chromium-review.googlesource.com/1117498
Commit-Queue: Penny MacNeil <pennymac@chromium.org>
Reviewed-by: Patrick Monette <pmonette@chromium.org>
Reviewed-by: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571393}

blink::ReadableStreamOperations::GetReader() should throw an exception
when the operation failed. Take an ExceptionState& parameter and rethrow
the exception on failure.

BodyStreamBuffer::CloseAndLockAndDisturb used the exception from
GetReader() to detect when the stream was already locked. Since it
there's no straightforward way to distinguish between the expected
exception and some other exception, instead check IsLocked() beforehand
to eliminate the possibility of an expected exception from GetReader().

Bug: 853189
Change-Id: Ib7bdf1cda9d392e4434ceff352c8f10ddf8ff951
Reviewed-on: https://chromium-review.googlesource.com/1118205Reviewed-by: Tsuyoshi Horo <horo@chromium.org>
Commit-Queue: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571392}

This CL fixes label and icon color of notifier settings view and IME
list view in UnifiedSystemTray.

TEST=manual
BUG=850396

Change-Id: I11bc88e3420b45641773ac7b1185db477a294455
Reviewed-on: https://chromium-review.googlesource.com/1117728Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571391}

This CL changes feature pod toggle behavior as follows:

Current behavior:
The feature was enabled. -> The feature is disabled.
The feature was disabled. -> The feature is enabled.

Expected behavior (This CL):
The feature was enabled. -> The feature is disabled.
The feature was disabled. -> The feature is enabled &
                             transition to the detailed view.

As currently we only have three feature pod buttons which are togglable
& having a detailed view, so it changes behavior of these buttons:
* Network
* Bluetooth
* Quiet Mode

TEST=manual
BUG=856028

Change-Id: I860bd9501bc651d74d982130d22b5006e0fba6e0
Reviewed-on: https://chromium-review.googlesource.com/1116503Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571390}

Suppressing the following flaky test on Linux NVIDIA:
  deqp/functional/gles3/texturespecification/
    random_teximage2d_cube.html

Tbr: ynovikov@chromium.org
Bug: 618447
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_optional_gpu_tests_rel
Change-Id: Ia22432c9cfd71fc4d26e21eea26576ecedfe8593
Reviewed-on: https://chromium-review.googlesource.com/1119542Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571389}

Currently, network feature pod button may show Ethernet, mobile data &
tether in addition to WiFi. However, clicking on the button always
toggled WiFi enabled / disabled.

TEST=manual
BUG=856030

Change-Id: If39e3360a045e0b0ba64cb89c43999dd11151549
Reviewed-on: https://chromium-review.googlesource.com/1116510Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571388}

Adds new Event.Touch.GestureTarget histogram for browser UI elements,
and stops including browser events in Ash.GestureTarget.

Removes header and runtime dependency between browser and ash for
gesture metric handling.

BUG: 826476
Change-Id: I87e8de2c4cae6f5e30939726917f5f811cb61ba8
Reviewed-on: https://chromium-review.googlesource.com/1116250Reviewed-by: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Ryan Cui <rcui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571387}

This is a reland of e2cac39f

Original change's description:
> Settings: Content settings can now be reset for a group of origins in All Sites.
>
> Add a three-dot / overflow menu to groups of origins in All Sites. This overflow
> menu allows the entire list of origins to have all their content settings reset
> in one go. This will be preceded with a confirmation dialog.
>
> Manual test - With #enable-site-settings turned on, change a content setting via
> Page Info > Site settings on both https://permission.site and
> http://permission.site. Navigate to chrome://settings/content/all and see that
> both sites are listed under 'permission.site'. Verify there is an overflow menu
> on this entry and that inside the overflow menu, there is a 'Reset permissions'
> option. Clicking this should bring up a reset permission dialog and confirming
> the dialog should remove this entry from All Sites.
>
> Bug: 835712, 717468
> Cq-Include-Trybots: luci.chromium.try:closure_compilation
> Change-Id: I777f759c26bf0fe7f8da086fa887969897cf9833
> Reviewed-on: https://chromium-review.googlesource.com/1098577
> Commit-Queue: Patti <patricialor@chromium.org>
> Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#568742}

TBR=stevenjb@chromium.org

Bug: 835712, 717468
Change-Id: I88522c41a536e86e27b831358c61842d69f42d34
Cq-Include-Trybots: luci.chromium.try:closure_compilation
Reviewed-on: https://chromium-review.googlesource.com/1116318
Commit-Queue: Patti <patricialor@chromium.org>
Reviewed-by: calamity <calamity@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571386}

Per privacy requirement, incognito windows should not be included
in screenshot sent to assistant server.

  ash_unittests --gtest_filter=AssistantControllerTest.Screenshot
  browser_tests --gtest_filter=BrowserNonClientFrameViewAshTest.IncognitoMarkedAsAssistantBlocked*

Bug: b/78193324
Test: 
Change-Id: Idf55135959721d69414a1ec6fae396f9763ebe49
Reviewed-on: https://chromium-review.googlesource.com/1114293
Commit-Queue: Muyuan Li <muyuanli@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571385}

Add async trace-events for selected commands to see how long they
spend in the queue. This can then be used to measure the impact of
the gpu-scheduler. This can be done by adding 'trace_queueing_time'
attribute to the commands. This CL adds this only to SwapBuffers.

BUG=858895

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_optional_gpu_tests_rel
Change-Id: I75146dd806f54357d0d669e92c1d461dc04d912c
Reviewed-on: https://chromium-review.googlesource.com/1114057
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: Jonathan Backer <backer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571384}

Bug: 850243
Cq-Include-Trybots: luci.chromium.try:closure_compilation
Change-Id: I7f028f7541a60f0aa15495823969fdd243609845
Reviewed-on: https://chromium-review.googlesource.com/1115589
Commit-Queue: Esmael El-Moslimany <aee@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571383}

This CL
* Fixes the behavior when date is clicked while system does not allow
  manual time change (we still allow users to change time zone & 12/24
  clock).
* Closes system tray bubble when managed button is clicked.
* Adds material design ripple to date button and managed button.

TEST=manual
BUG=none


Change-Id: I43df968100952940d21e87132bbb1007a7bacdca
Reviewed-on: https://chromium-review.googlesource.com/1118091Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571382}

Bug: 825010
Change-Id: I384e1fc1e24572a2fc185d2ceac1bd9fa0b8a5bc
Reviewed-on: https://chromium-review.googlesource.com/1119611
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571381}

Bug: 856896
Change-Id: I5897da89ffbf38a5ca7fbce3205aefb770d81ecc
TBR: kylixrd
Reviewed-on: https://chromium-review.googlesource.com/1119646Reviewed-by: Peter Kasting <pkasting@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571380}

I ran into a display bug when I tried to pass the host name and the
mode separately from the multidevice-container-page to the multidevice-
page in which only the first property sent updated the display.

To solve this, I am refactoring to pass an object with all the data
the page needs from the container in a single object, the new typedef
PageContentData.

This CL involves a refactor to handle the infrastructural changes. It
uses this functionality to pass the host device name from the container
to the page.

Manual testing revealed that it is no longer showing the same buggy
display behavior (i.e. the heading shows up and changes appropriately
regardless of the initial mode set by the container).

Cq-Include-Trybots: luci.chromium.try:closure_compilation
Change-Id: I3e19bcaa8ed8ed774a947ffc9f24fc71dc406038
Reviewed-on: https://chromium-review.googlesource.com/1117201Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Jordy Greenblatt <jordynass@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571379}

Bug: 680046
Change-Id: I8b3894bfbc8df999952fa82e8f2b3abb4088f434
Reviewed-on: https://chromium-review.googlesource.com/1114566Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571378}

Some CLs are affected by a nondeterministic package install bug
which is occurring in content_unittests, preventing those CLs from
being landed.

This CL unblocks those CL owners by temporarily removing content_unittests from the
CQ.

TBR=jbudorick@chromium.org

Bug: 858692
Change-Id: I8416f238f248ed1a16e406ef80e801859df10a0b
Reviewed-on: https://chromium-review.googlesource.com/1119198Reviewed-by: John Budorick <jbudorick@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Kevin Marshall <kmarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571377}

When apply masks, we only use the alpha channel. For TileMaskLayer that
appended SolidColorDrawQuads, only cheak the alpha channel instead of
the entire color.

R=danakj

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Ic66497ed3aa4e8b4b9befc80649ea09f21930467
Reviewed-on: https://chromium-review.googlesource.com/1087778Reviewed-by: danakj <danakj@chromium.org>
Commit-Queue: weiliangc <weiliangc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571376}

ServiceWorkerSubresourceLoader does not necessarily run on a
SingleThreadTaskRunner.

Bug: 857040
Change-Id: I8e1d65da88a25a49a01d0d8da951c85be54f6190
Reviewed-on: https://chromium-review.googlesource.com/1118404
Commit-Queue: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571375}

This is one step of "moving //device/geolocation into //services".

TBR=benchan@chromium.org
BUG=800659

Change-Id: I566db835e7df596374cd341048d9734bf82398f6
Reviewed-on: https://chromium-review.googlesource.com/1107637Reviewed-by: Ben Chan <benchan@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Ke He <ke.he@intel.com>
Cr-Commit-Position: refs/heads/master@{#571374}

Introduces an internal PlatformHandleInTransit type to safely
scope the lifetime of platform handles which may belong to a remote
process.

Uses this everywhere for handles stored in messages, eliminating
all remaining use of some InternalPlatformHandle properties which
aren't covered by PlatformHandle (namely |owning_process| and the
MACH_NAME "handle" type).

This is a precursor to removing InternalPlatformHandle and
ScopedInternalPlatformHandle altogether.

Bug: 753541
Change-Id: Ibc223669c935bdde72c2af3b6b3d7cd1474e2bdf
Reviewed-on: https://chromium-review.googlesource.com/1114462
Commit-Queue: Ken Rockot <rockot@chromium.org>
Reviewed-by: Jay Civelli <jcivelli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571373}