GitLab

https://skia.googlesource.com/skia.git/+log/cf9086c..51623cc


git log cf9086c..51623cc --date=short --no-merges --format='%ad %ae %s'
2018-05-30 swiftshader-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/swiftshader 9dfb70f..e92a88e (2 commits)
2018-05-30 egdaniel@google.com Move include into head to fix build breakage.
2018-05-30 rmistry@google.com Revert "Check for ANGLE's implementation of texture_rectangle"
2018-05-30 swiftshader-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/swiftshader 45fcfe8..9dfb70f (1 commits)
2018-05-30 reed@google.com 3d
2018-05-30 brianosman@google.com Clean up GPU configs in DM
2018-05-30 weiliangc@chromium.org Check for ANGLE's implementation of texture_rectangle
2018-05-30 egdaniel@google.com Use GrVkMemoryAllocator for vulkan memory allocations in ganesh.
2018-05-30 scroggo@google.com Add a dummy SkColorTable.cpp in src/codec
2018-05-30 angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/angle2 0ed2aa2..dc0daf8 (7 commits)
2018-05-30 borenet@google.com [recipes] Add parameters to checkout module
2018-05-28 bungeman@google.com Make sk_sp operator bool explicit.
2018-05-30 swiftshader-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/swiftshader 493fc54..45fcfe8 (1 commits)
2018-05-29 bsalomon@google.com Make GrPrimitiveProcessor::Attribute a class, hide data members.
2018-05-30 fmalita@chromium.org Add 'public' headers support to find_headers.py
2018-05-30 brianosman@google.com Make skcms mandatory
2018-05-30 mtklein@chromium.org warmup, remove clamping in append_gamut_transform()
2018-05-30 swiftshader-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/swiftshader 4b74373..493fc54 (2 commits)
2018-05-30 mtklein@chromium.org whoops

Created with:
  gclient setdep -r src/third_party/skia@51623cc

The AutoRoll server is located here: https://autoroll.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=master.tryserver.blink:linux_trusty_blink_rel;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

TBR=rmistry@chromium.org

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

NameStyleConverter.to_function_name() works well for
make_media_features.py. Also, rename the filter from
'upper_first_letter' to 'to_function_name'.

Makes upper_first_letter() private.

Bug: 843927
Change-Id: Id3cedbbc66f01c839d2ef2a4294073aae5896222
Reviewed-on: https://chromium-review.googlesource.com/1080109Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563102}

Ran it at 20% experiment for a while, so it should be fine to do this
capacity wise.

NOTRY=true

Bug: 756689
Change-Id: Ia96c0e383f0c5d631759733faf2ab8213f7796f6
Reviewed-on: https://chromium-review.googlesource.com/1080269
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563101}

This CL enables using libfuzzer on linux coverage tot builder, however,
due to crbug.com/790747, fuzzer targets won't compile on Mac yet.

Change-Id: I9bde883c382f476811883f3b5b9c05424b23c1e1
Reviewed-on: https://chromium-review.googlesource.com/1078070Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563100}

Now that we've branched to M69 and the KeyboardLock feature is enabled
by default, we can remove the blink API feature flag.

Note: I want to keep the Chrome feature flags around a bit longer, but
will be removing them at some point in the future.

BUG=680809

Change-Id: I6dcb76a0a8e7874fd20627f9126ff48a9a0625a6
Reviewed-on: https://chromium-review.googlesource.com/1073674Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Commit-Queue: Joe Downing <joedow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563099}

This reverts commit 3cb37960.

Reason for revert: there are a high volume of changes in this subdirectory right now, causing overloading of the GPU bots.

Bug: 848084, 843290
No-Try: True
Tbr: fsamuel@chromium.org
Tbr: danakj@chromium.org
Tbr: samans@chromium.org
Tbr: sadrul@chromium.org

Original change's description:
> Run WebGL 2.0 conformance tests against Viz changes.
> 
> This will increase the likelihood that intermittent failures caused by
> changes in this directory will be caught on the commit queue.
> 
> Bug: 843290
> 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: Ie4d230c0290706c956d88f483b82b98f3a62f4bf
> Reviewed-on: https://chromium-review.googlesource.com/1069598
> Reviewed-by: Fady Samuel <fsamuel@chromium.org>
> Commit-Queue: Kenneth Russell <kbr@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#560799}

TBR=kbr@chromium.org,fsamuel@chromium.org,samans@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 843290
Change-Id: Id5dddfac8e7cdb271628de739e8179459b368c49
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
Reviewed-on: https://chromium-review.googlesource.com/1080075Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563098}

Bug: b/79425648
Test: locally build and run
Change-Id: I63cb74c6eac1786f9d3c77b8d9dfafa2f70bff87
Reviewed-on: https://chromium-review.googlesource.com/1077654
Commit-Queue: Xiaohui Chen <xiaohuic@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563097}

These elemens are no longer used by chrome://md-user-manager.

Bug: None
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I3e68c45cf193585a81f833b90a73633a9e9efc66
Reviewed-on: https://chromium-review.googlesource.com/1080033Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563096}

Exo clients should not see these events and instead handle
key repeat on the client side.

Bug: 847500
Test: exo_unittests
Change-Id: I4d4c755dbe0a2fffe24f3ce85fd70a3608cfed7c
Reviewed-on: https://chromium-review.googlesource.com/1079768Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Commit-Queue: David Reveman <reveman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563095}

Remove wrong baseline LayoutTests/platform/win7/paint/invalidation/svg/zoom-foreignObject-expected.png

TBR=chrishtr@chromium.org

Bug: 842668
Change-Id: I3b729ab5b40c4c1fcefe03b259f9d1adc97810df
Reviewed-on: https://chromium-review.googlesource.com/1080157Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563094}

This ensures that the underlying vsh implementation connects to the correct
VM via Concierge service. Another CL impelments the Concierge side of this
change.

Bug: 842561
Change-Id: I9d0f84cffd111e86760369c3f7d060580aef44b3
Reviewed-on: https://chromium-review.googlesource.com/1074803
Commit-Queue: Nicholas Verne <nverne@chromium.org>
Reviewed-by: Timothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563093}

The code for tracking locked font handles on the service currently
assumes that only one DoRasterCHROMIUM command is interleaved between a
Begin/EndRaster sequence, which is an incorrect assumption. As a result
since we clear the vector of locked handles in each DoRasterCHROMIUM,
this leaks ref-counts if multiple DoRasterCHROMIUM commands are present
between a single Begin/EndRaster sequence.

Fix this by appending ref-counts on the service instead of over-writing
the current list.

R=piman@chromium.org

Bug: 829622
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;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I15025153fe887e712e0879cbeb32944c3e258e92
Reviewed-on: https://chromium-review.googlesource.com/1077819
Commit-Queue: Khushal <khushalsagar@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563092}

Update references to ExtensionService in c/b/media_galleries to use
extensions::ExtensionService.

There should be no behavior change as a result of this CL.

Bug: 117261
Change-Id: I4a097c316ead2833466274cca7c2c36e8ef0c8c0
Reviewed-on: https://chromium-review.googlesource.com/1080043Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563091}

After https://chromium-review.googlesource.com/c/chromium/src/+/1068514,
no code will try to find the specific ServiceWorkerDispatcherHost for a
given renderer process, so this CL removes the mapping management of
ServiceWorkerDispatcherHosts inside ServiceWorkerContextCore, in the
meantime simplifies ServiceWorkerDispatcherHost a little.

BUG=845341

Change-Id: Ie7747846570f64a7a4eda960878bbc0c97042952
Reviewed-on: https://chromium-review.googlesource.com/1075849Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Han Leon <leon.han@intel.com>
Cr-Commit-Position: refs/heads/master@{#563090}

If it overloads the bot, it's not a real issue. Want to see if it can
handle full CQ load

Bug: 782863
Change-Id: I776efe034ce66bb12b0ea846c706d4e8e08b7893
Reviewed-on: https://chromium-review.googlesource.com/1080270Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563089}

Bug: b/80199648

Test: tested on AT device with different cast application and activity cases.

Change-Id: Ia5ed6d6ebd6c4bf17663f60a913763ea72edd7eb
Reviewed-on: https://chromium-review.googlesource.com/1072407
Commit-Queue: Zhiheng(Vincent) Li <vincentli@google.com>
Reviewed-by: Luke Halliwell <halliwell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563088}

Bug: 790759
Change-Id: I4655e5763198a6eab8b407f36383f756d8484ba6
Reviewed-on: https://chromium-review.googlesource.com/1072878Reviewed-by: Primiano Tucci <primiano@chromium.org>
Commit-Queue: Brian Anderson <brianderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563087}

Bug: 846544
Change-Id: Ia0d38c10b7b6d42a50ca97af45a3b54ac9f0843b
Reviewed-on: https://chromium-review.googlesource.com/1079893Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563086}

This method lets the relying party determine if the browser is on a
device that supports platform authenticators (such as fingerprint).

We are plumbing the call through  mojom so that we can have separate
Android and desktop implementations.

Bug: 828216
Change-Id: Iaf0b7e84606b816598b1c54e9380a5acd9a4e52b
Reviewed-on: https://chromium-review.googlesource.com/1053110
Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org>
Reviewed-by: Maria Khomenko <mariakhomenko@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563085}

Show the optin screen even when there is no server consent
flags needed to set. This will still give the the user a chance
to enable/disable the feature locally.

Bug: None
Test: locally build and test
Change-Id: If862adf4bd811e8c609233ec8c5da070ba29424a
Reviewed-on: https://chromium-review.googlesource.com/1079377Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Xiaohui Chen <xiaohuic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563084}

Show "Available" or "Connecting..." as the status text for sinks in respective
states. Default to the status text if supplied by the sink.

Bug: 844099
Change-Id: I5ba8b82960413dd7927182d34e2112ed391a12ba
Reviewed-on: https://chromium-review.googlesource.com/1071089Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563083}

https://chromium.googlesource.com/chromiumos/platform/system_api.git/+log/55139d2ab617..a9ac639b38af

$ git log 55139d2ab..a9ac639b3 --date=short --no-merges --format='%ad %ae %s'
2018-05-24 qiyuh Add new service constants in BlueZ
2018-05-30 risan Dbus constants for DiskQuota

Created with:
  roll-dep src/third_party/cros_system_api

Change-Id: Ia1da2c46a264fccb4893b78c380244c47c44da4d
Reviewed-on: https://chromium-review.googlesource.com/1079668Reviewed-by: Ben Chan <benchan@chromium.org>
Commit-Queue: Qiyu Hu <qiyuh@google.com>
Cr-Commit-Position: refs/heads/master@{#563082}

This fixes the bug in which the selected sink's inkdrop color fades in
instead of snapping to cause a flickering effect.

Bug: 842792
Change-Id: Ic4819552ee6a45f20c654a5ee4d9211b34f1f313
Reviewed-on: https://chromium-review.googlesource.com/1079695Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563081}

When BrowserNonClientFrameView is destroyed and recreated, for example
when the user installs or uninstalls a theme, the BrowserView is reused.
But the Tabstrip was only being observed on BrowserView creation.

This is probably a very old bug, but it's hard to observe the effects
without single tab mode. In particular, I found that the profile
switcher chip can get out of sync with the size of the tabstrip and
overlap the new tab button, which is also fixed.

Bug: 841643
Change-Id: Ia3c20a70f8082e1a8b90d34da35011313c344035
Reviewed-on: https://chromium-review.googlesource.com/1079958Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563080}

Change-Id: I5108679e13789cce24feda792123b5161cbcd995
Reviewed-on: https://chromium-review.googlesource.com/1079227
Commit-Queue: Brad Lassey <lassey@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563079}

This allows compilers targeting the MS ABI to select the correct
inheritance model for the member function pointer type RunnerMethodType
using the complete type of TaskHandle::Runner. It will allow us to
enable the new Clang flag -fcomplete-member-pointers globally.

Bug: 847724
Change-Id: Ib716e3c0774b7d6b46087e46057ca249e9aa72e7
Reviewed-on: https://chromium-review.googlesource.com/1079852Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563078}

In a follow up, we will make CollectGarbage the bottleneck for the GC
atomic phase.

Bug: chromium:757440
Change-Id: Icf27cfde4a61375eda52207e967c84c4ed9c7c31
Reviewed-on: https://chromium-review.googlesource.com/1078214
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563077}

The implementations are |NOTREACHED()| since users cannnot sign out of
Chrome in Chrome OS.

Bug: 820046
Change-Id: Ie559d52388418d5a8a182afdb8a08f92b726c21b
Reviewed-on: https://chromium-review.googlesource.com/1076291Reviewed-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Kush Sinha <sinhak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563076}

A test includes the cc file
components/search_engines/search_engines_test_util.cc instead of
its header file. This patch changes that to include the header
file instead.

This inclusion has caused no to me known problems and was just found
as a result of looking through the source for this pattern after
a similar instance did cause problems.

Change-Id: Iffdb68a8055d69a8a3df450c7982ec1e674dcf6e
Reviewed-on: https://chromium-review.googlesource.com/1005175
Commit-Queue: Avi Drissman <avi@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563075}

Update references to ExtensionService in c/b/renderer_context_menu to
use extensions::ExtensionService.

There should be no behavior change as a result of this CL.

Bug: 117261
Change-Id: I7a137ee97d0d884d36cf20db1c8484bae6e5c62a
Reviewed-on: https://chromium-review.googlesource.com/1079716
Commit-Queue: Avi Drissman <avi@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563074}

https://www.w3.org/TR/webauthn/#ref-for-dom-authenticatorassertionresponse-userhandle

Bug: 847878
Change-Id: I8071207c15d3150a486133eb50cc7d00cbba5401
Reviewed-on: https://chromium-review.googlesource.com/1079200Reviewed-by: Balazs Engedy <engedy@chromium.org>
Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563073}

Moves HandleViewResources to ui/resources.
Also removes the DEPS exception rule that becomes unnecessary.

Bug: 617324
Change-Id: Ideb9b2d858ffe8a9f5ea7f345c7a9cdd3a9c38cd
Reviewed-on: https://chromium-review.googlesource.com/1071148Reviewed-by: Mohsen Izadi <mohsen@chromium.org>
Reviewed-by: Bo <boliu@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563072}

No actual logic is added, simply the plumbing to call between Java and
native code. In subsequent patches logic will be added to the
FeedSchedulerHost.

Bug: 831642, 831644
Change-Id: Icec552cbf4d482739d7211a9cc81bb669e0df413
Reviewed-on: https://chromium-review.googlesource.com/1068199
Commit-Queue: Sky Malice <skym@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563071}

https://pdfium.googlesource.com/pdfium.git/+log/0789714..a7b65b8


git log 0789714..a7b65b8 --date=short --no-merges --format='%ad %ae %s'
2018-05-30 rharrison@chromium.org Migrate coverage_report.py to use upstream Chromium scripts
2018-05-30 npm@chromium.org Roll third_party/freetype/src/ 9e345c911..d45d4b97e (27 commits)
2018-05-30 thestig@chromium.org Remove dead code in various write function.
2018-05-30 thestig@chromium.org Add pixel tests for PDFs that use generation numbers.
2018-05-30 thestig@chromium.org Exclude certain colorspace types for calculating transparency backdrop color.
2018-05-30 thomasanderson@chromium.org Remove manual references to exe_and_shlib_deps
2018-05-30 tsepez@chromium.org Make common page base class for XFA and non-XFA.

Created with:
  gclient setdep -r src/third_party/pdfium@a7b65b8

The AutoRoll server is located here: https://pdfium-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:847346,chromium:845700

TBR=dsinclair@chromium.org

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

- Only 1 Console test dumps classNames: 'console-format-classes.js'
- Classes dumped with indentation

Bug: none
Change-Id: Ieb919f05e3a69f374ed6baecbaef99f66cf20e33
Reviewed-on: https://chromium-review.googlesource.com/1077747Reviewed-by: Joel Einbinder <einbinder@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Erik Luo <luoe@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563069}

Sort gtests in the same way the src/testing/buildbot version does, to
minimize differences when switching over to that script.

Apply swarming_settings to gtests, not just isolated_scripts.

Tbr: ynovikov@chromium.org
Tbr: geofflang@chromium.org
Bug: 844815
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: Ifd791b9e459230f78b020bf46f031b3d3691e64a
Reviewed-on: https://chromium-review.googlesource.com/1079850Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563068}

Adjust the dialog based on UI review.
1) Align the enterprise logo with the head of button instead of tail.
2) Append "..." into the launching information.
3) Using close button instead of cancel button.

Bug: 840018
Change-Id: I5689eceef9ad9b9061d5e4a3d1e314456704b6a2
Reviewed-on: https://chromium-review.googlesource.com/1072207Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Roger Tawa <rogerta@chromium.org>
Commit-Queue: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563067}

For incognito profiles, the infobar is not being shown since
the DownloadManagerService only starts observing the download item updates
after we open download home in incognito. Fixed this bug in this CL.

Bug: 846139
Change-Id: I507c285901474ce47f09a1610639805afe3c904f
Reviewed-on: https://chromium-review.googlesource.com/1073078
Commit-Queue: Shakti Sahu <shaktisahu@chromium.org>
Reviewed-by: Min Qin <qinmin@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563066}

Previous attempts of adding and removing to the top layer were done
synchronously. The spec has been updated to do this asynchronously and
this change matches those changes.

Bug: 240576
Change-Id: Ic57a651596c685daa4b32d78421de5db912c106e
Reviewed-on: https://chromium-review.googlesource.com/1066600
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563065}

Based on UX feedback.

BUG=848070

Change-Id: I794b8069d74379887ad5ad04070873eb8aeaaf94
Reviewed-on: https://chromium-review.googlesource.com/1080170Reviewed-by: Tao Wu <wutao@chromium.org>
Commit-Queue: Ahmed Fakhry <afakhry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563064}