GitLab

Replace old API with new API. This CL is a refactor
ans has no intended behavior change.

Bug: 842194
Change-Id: I46e242b51c99ef720ee6700ab6bf0fdc8c2a4720
Reviewed-on: https://chromium-review.googlesource.com/1174582Reviewed-by: Erik Chen <erikchen@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Sang Woo Ko <sangwoo108@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583318}

Bug: 855127
Change-Id: I2942e122373c210bd0bceaa59791e2619186f79b
Reviewed-on: https://chromium-review.googlesource.com/1175935Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583317}

Flakiness dashboard shows that it has been flaky for a while:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=components_unittests&tests=SslCastSocketTest.TestMessageEndToEndWithRealSSL

TBR=wjmaclean@chromium.org
NOTRY=true

Bug: 874491
Change-Id: I5c0fe9656f3c9e74d21660db6430899e4824c6d5
Reviewed-on: https://chromium-review.googlesource.com/1176170Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583316}

Using wpt-import in Chromium a3a0b7f4.
With Chromium commits locally applied on WPT:
7dad0f17 "Added new Sec-Metadata Web Platform Tests."


Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/22763

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=raphael.kubo.da.costa@intel.com

No-Export: true
Change-Id: I2587d254f484b32739d6192d72369cbe79b3fc13
Reviewed-on: https://chromium-review.googlesource.com/1175858
Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583315}

This reverts commit 4efeaf16.

Reason for revert: <INSERT REASONING HERE>
Causes a lot of unit test failure:
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-chromeos-dbg/7319

Original change's description:
> [ozone/Wayland] Implement Wayland dmabuf approach.
> 
> This patch makes it possible to run Wayland backend with a separate gpu process.
> That is, WaylandConnection is not available on the GPU process side
> and so buffers cannot be created that way.
> 
> To overcome this, a Wayland dmabuf based approach is used:
>  - On the gpu process side, gl::Surfaceless is used, which is
>    implemented by GbmSurfaceless. It differs from the one provided by
>    the Ozone/Gbm backend (the difference is that there are no frame
>    buffers, because DRM render node is used instead.) Basically,
>    on the GPU side, a gbm_bo buffer is created and its file descriptor is
>    shared with the WaylandConnection, which then imports a wl_buffer and attaches
>    it to a requested surface.
> 
> TL:DR;
>  - There are two communicating parties - WaylandConnection, which establishes
>    a connection with Wayland, and a WaylandConnectionProxy, which serves
>    the GPU process with a forwarding mojo connection to the WaylandConnection.
>    In single process mode, WaylandConnectionProxy just forwards calls to
>    WaylandConnection, and GLSurfaceEGL is used. In a multi process mode,
>    the l::Surfaceless and dmabuf based approach is used instead.
> 
>  - The communication is established in the following way:
>    when WaylandConnectionConnector receives a OnGpuServiceLaunched,
>    it binds WaylandConnection and WaylandConnectionProxy interfaces, and
>    calls to WaylandConnectionProxy to set a mojo ptr to the WaylandConnection.
>    The WaylandConnectionProxy receives the pointer, stores it and also stores
>    a task runner to make sure mojo is called on the right sequence.
> 
> How buffers are created:
>  - Basically, the GPU process doesn't know about Wayland. What it does
>    is that it creates a GbmBuffer utilizing a DRM render node (it can't use
>    a drm card device, because authentication requires master access), takes
>    a gbm_bo file descriptor and shares it via a mojo call with the
>    WaylandConnection, which then uses the zwp_linux_dmabuf interface to
>    import a wl_buffer, which can be attached to a needed WaylandWindow
>    and commit-ed to show contents. That is, the gbm framework is used to share buffers
>    between the browser and the GPU process, where the browser process can read from them and
>    the GPU process is able to read and write using gl commands.
> 
> How buffers are committed:
>  - GbmSurfaceless uses a normal procedure and submits frames on
>    SwapBuffersAsync call: a frame is created and scheduled as an overlay plane,
>    once scheduling is done, a widget is taken from that pending plane and
>    WaylandConnection::ScheduleBufferSwap is called. In order to specify, which
>    wl_buffer must be used, gbm bo handle is used as a buffer id.
>    Once the WaylandConnection receives a buffer swap call, it finds a buffer by a
>    buffer id and attaches that buffer to a WaylandWindow, which is also found
>    by a widget passed to it. PresentationCallback is not used and will be
>    enabled in the future.
> 
> The missing feature is the software rasterization. That is, the gbm buffers cannot be
> mmap'ed using unix mmap method. Thus, it's impossible to get a mapped memory range and let the cpu
> write to it. I think this problem can be resolved in the downstream projects in different ways
> but these solutions are not generic enough to be upstreamed.
> 
> Another missing feature is presentation feedback. This will be implemented in follow up
> CLs.
> 
> COMPILE= compiles only with third_party minigbm now. For example, add use_intel_minigbm=true
> if intel gpu is ozone_platform_wayland=true. Using system libgbm will be available soon.
> TEST=run and compile ozone/wayland.
> Bug: 820047
> 
> 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: Id351edd9883a765d3e549543a50175299108c6ab
> Reviewed-on: https://chromium-review.googlesource.com/1057468
> Reviewed-by: Scott Violet <sky@chromium.org>
> Reviewed-by: Antoine Labour <piman@chromium.org>
> Reviewed-by: Ilya Sherman <isherman@chromium.org>
> Reviewed-by: Tom Sepez <tsepez@chromium.org>
> Reviewed-by: Robert Kroeger <rjkroege@chromium.org>
> Commit-Queue: Maksim Sisov <msisov@igalia.com>
> Cr-Commit-Position: refs/heads/master@{#583265}

TBR=rjkroege@chromium.org,reveman@chromium.org,sky@chromium.org,spang@chromium.org,isherman@chromium.org,tsepez@chromium.org,piman@chromium.org,msisov@igalia.com

Change-Id: I69eadb7666d275dabeceff306e3e5cde62ddf795
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 820047
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/1175468Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583314}

Bug: 847587
Change-Id: Iab2a11d1f11b6b6b703d77e7c94a01b564fd07d1
Reviewed-on: https://chromium-review.googlesource.com/1175411Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Min Chen <minch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583313}

This CL adds a try/catch block to prevent SecurityExceptions from
crashing media opened via the media intent handler. In certain cases,
we don't have permission to query for the file display name. In those
cases, instead of crashing we will instead just fall back to having no
display name.

Bug: 874149
Change-Id: I18ed34655e093cf04c9cb745eeeb6182c8fe28b6
Reviewed-on: https://chromium-review.googlesource.com/1175212Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Tommy Steimel <steimel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583312}

This change moves the logic from DeviceMotionDispatcher into
DeviceMotionEventPump and removes DeviceMotionDispatcher.
DeviceMotionController now uses DeviceMotionEventPump directly,and a
separate instance of DeviceMotionEventPump is created forevery document.
Webkit layout tests were updated to reflect this, and webkit unit tests
have been updated to use a MockDeviceMotionController.

Bug: 861902
Change-Id: Ic9a33b26c6c5771a3a7312724f1056ef311ea6a3
Reviewed-on: https://chromium-review.googlesource.com/1167994Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Tim Volodine <timvolodine@chromium.org>
Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583311}

This reverts commit 32cd8fcf.

Reason for revert: Reverting the revert because this change was proven to not have caused the issue.

Original change's description:
> Revert "cros: Enable touchable app context menus by default."
> 
> This reverts commit b58ccf6d.
> 
> Reason for revert: Possibly introduced a use-after-free bug.
> 
> This is a speculative fix for a use-after-free bug in two ash_unittests:
> 
> 1. OverflowButtonInkDropTest.MouseContextMenu
> 2. OverflowButtonActiveInkDropTest.MouseContextMenu
> 
> [ RUN      ] OverflowButtonInkDropTest.MouseContextMenu
> =================================================================
> ==28149==ERROR: AddressSanitizer: heap-use-after-free on address 0x613000031f68 at pc 0x000008c82106 bp 0x7ffea3cd7190 sp 0x7ffea3cd7188
> READ of size 8 at 0x613000031f68 thread T0
>     #0 0x8c82105 in begin buildtools/third_party/libc++/trunk/include/vector:1479:30
>     #1 0x8c82105 in begin<std::__1::vector<aura::WindowObserver *, std::__1::allocator<aura::WindowObserver *> > > buildtools/third_party/libc++/trunk/include/iterator:1670
>     #2 0x8c82105 in ContainsValue<std::__1::vector<aura::WindowObserver *, std::__1::allocator<aura::WindowObserver *> >, const aura::WindowObserver *, 0> base/stl_util.h:168
>     #3 0x8c82105 in HasObserver base/observer_list.h:267
>     #4 0x8c82105 in aura::Window::HasObserver(aura::WindowObserver const*) const ui/aura/window.cc:543
>     #5 0x8c9d363 in aura::WindowObserver::OnUnobservingWindow(aura::Window*) ui/aura/window_observer.cc:25:15
>     #6 0x8c799d7 in aura::Window::RemoveObserver(aura::WindowObserver*) ui/aura/window.cc:538:13
>     #7 0xc2f40e in RemoveAll ui/base/window_tracker_template.h:49:15
>     #8 0xc2f40e in ui::WindowTrackerTemplate<aura::Window, aura::WindowObserver>::~WindowTrackerTemplate() ui/base/window_tracker_template.h:33
>     #9 0x9225f3b in ~CompoundEventFilter ui/wm/core/compound_event_filter.cc:49:1
>     #10 0x9225f3b in wm::CompoundEventFilter::~CompoundEventFilter() ui/wm/core/compound_event_filter.cc:45
>     #11 0x6b0ae30 in operator() buildtools/third_party/libc++/trunk/include/memory:2321:5
>     #12 0x6b0ae30 in reset buildtools/third_party/libc++/trunk/include/memory:2634
>     #13 0x6b0ae30 in ~unique_ptr buildtools/third_party/libc++/trunk/include/memory:2588
>     #14 0x6b0ae30 in ash::Shell::~Shell() ash/shell.cc:969
>     #15 0x6b0c81d in ash::Shell::~Shell() ash/shell.cc:737:17
>     #16 0xc26b344 in ash::AshTestHelper::TearDown() ash/test/ash_test_helper.cc:294:3
>     #17 0xc26444c in ash::AshTestBase::TearDown() ash/test/ash_test_base.cc:192:21
>     #18 0x4724c64 in testing::TestInfo::Run() third_party/googletest/src/googletest/src/gtest.cc:2682:11
>     #19 0x4726036 in testing::TestCase::Run() third_party/googletest/src/googletest/src/gtest.cc:2800:28
>     #20 0x474b566 in testing::internal::UnitTestImpl::RunAllTests() third_party/googletest/src/googletest/src/gtest.cc:5124:43
>     #21 0x474a7b5 in testing::UnitTest::Run() third_party/googletest/src/googletest/src/gtest.cc
>     #22 0x72b431a in RUN_ALL_TESTS third_party/googletest/src/googletest/include/gtest/gtest.h:2331:46
>     #23 0x72b431a in base::TestSuite::Run() base/test/test_suite.cc:277
>     #24 0x72b9892 in Run base/callback.h:99:12
>     #25 0x72b9892 in base::(anonymous namespace)::LaunchUnitTestsInternal(base::OnceCallback<int ()>, unsigned long, int, bool, base::OnceCallback<void ()>) base/test/launcher/unit_test_launcher.cc:224
>     #26 0x72b9330 in base::LaunchUnitTests(int, char**, base::OnceCallback<int ()>) base/test/launcher/unit_test_launcher.cc:575:10
>     #27 0x178df94 in main ash/test/ash_unittests.cc:37:10
>     #28 0x7f78e2b62f44 in __libc_start_main /build/eglibc-ripdx6/eglibc-2.19/csu/libc-start.c:287
> 
> 0x613000031f68 is located 296 bytes inside of 328-byte region [0x613000031e40,0x613000031f88)
> freed by thread T0 here:
>     #0 0x6b9062 in operator delete(void*) /b/swarming/w/ir/kitchen-workdir/src/third_party/llvm/compiler-rt/lib/asan/asan_new_delete.cc:167:3
>     #1 0x6a547a0 in ash::RootWindowController::CloseChildWindows() ash/root_window_controller.cc:498:7
>     #2 0x6b0c5cf in ash::Shell::CloseAllRootWindowChildWindows() ash/shell.cc:1387:19
>     #3 0x6b06637 in ash::Shell::~Shell() ash/shell.cc:837:3
>     #4 0x6b0c81d in ash::Shell::~Shell() ash/shell.cc:737:17
>     #5 0xc26b344 in ash::AshTestHelper::TearDown() ash/test/ash_test_helper.cc:294:3
>     #6 0xc26444c in ash::AshTestBase::TearDown() ash/test/ash_test_base.cc:192:21
>     #7 0x4724c64 in testing::TestInfo::Run() third_party/googletest/src/googletest/src/gtest.cc:2682:11
>     #8 0x4726036 in testing::TestCase::Run() third_party/googletest/src/googletest/src/gtest.cc:2800:28
>     #9 0x474b566 in testing::internal::UnitTestImpl::RunAllTests() third_party/googletest/src/googletest/src/gtest.cc:5124:43
>     #10 0x474a7b5 in testing::UnitTest::Run() third_party/googletest/src/googletest/src/gtest.cc
>     #11 0x72b431a in RUN_ALL_TESTS third_party/googletest/src/googletest/include/gtest/gtest.h:2331:46
>     #12 0x72b431a in base::TestSuite::Run() base/test/test_suite.cc:277
>     #13 0x72b9892 in Run base/callback.h:99:12
>     #14 0x72b9892 in base::(anonymous namespace)::LaunchUnitTestsInternal(base::OnceCallback<int ()>, unsigned long, int, bool, base::OnceCallback<void ()>) base/test/launcher/unit_test_launcher.cc:224
>     #15 0x72b9330 in base::LaunchUnitTests(int, char**, base::OnceCallback<int ()>) base/test/launcher/unit_test_launcher.cc:575:10
>     #16 0x178df94 in main ash/test/ash_unittests.cc:37:10
>     #17 0x7f78e2b62f44 in __libc_start_main /build/eglibc-ripdx6/eglibc-2.19/csu/libc-start.c:287
> 
> previously allocated by thread T0 here:
>     #0 0x6b8422 in operator new(unsigned long) /b/swarming/w/ir/kitchen-workdir/src/third_party/llvm/compiler-rt/lib/asan/asan_new_delete.cc:106:3
>     #1 0x91cb856 in views::NativeWidgetAura::NativeWidgetAura(views::internal::NativeWidgetDelegate*, bool, aura::Env*) ui/views/widget/native_widget_aura.cc:109:15
>     #2 0x91d615d in views::internal::NativeWidgetPrivate::CreateNativeWidget(views::Widget::InitParams const&, views::internal::NativeWidgetDelegate*) ui/views/widget/native_widget_aura.cc:1113:14
>     #3 0x917dabd in CreateNativeWidget ui/views/widget/widget.cc:73:10
>     #4 0x917dabd in views::Widget::Init(views::Widget::InitParams const&) ui/views/widget/widget.cc:334
>     #5 0x6af249a in ash::ShelfWidget::ShelfWidget(aura::Window*, ash::Shelf*) ash/shelf/shelf_widget.cc:216:3
>     #6 0x6a96eaa in ash::Shelf::CreateShelfWidget(aura::Window*) ash/shelf/shelf.cc:73:27
>     #7 0x6a5847a in ash::RootWindowController::InitLayoutManagers() ash/root_window_controller.cc:699:11
>     #8 0x6a52166 in ash::RootWindowController::Init(ash::RootWindowController::RootWindowType) ash/root_window_controller.cc:663:3
>     #9 0x68ef135 in ash::WindowTreeHostManager::InitHosts() ash/display/window_tree_host_manager.cc:238:3
>     #10 0x6afadce in ash::Shell::Init(ui::ContextFactory*, ui::ContextFactoryPrivate*, std::__1::unique_ptr<base::Value, std::__1::default_delete<base::Value> >, std::__1::unique_ptr<ui::ws2::GpuInterfaceProvider, std::__1::default_delete<ui::ws2::GpuInterfaceProvider> >) ash/shell.cc:1250:30
>     #11 0x6af7163 in ash::Shell::CreateInstance(ash::ShellInitParams) ash/shell.cc:276:14
>     #12 0xc26ac31 in ash::AshTestHelper::CreateShell() ash/test/ash_test_helper.cc:390:3
>     #13 0xc26a2a0 in ash::AshTestHelper::SetUp(bool, bool) ash/test/ash_test_helper.cc:245:3
>     #14 0xc263f8d in ash::AshTestBase::SetUp() ash/test/ash_test_base.cc:159:21
>     #15 0x1322b16 in ash::ShelfViewTest::SetUp() ash/shelf/shelf_view_unittest.cc:262:18
>     #16 0x1326366 in SetUp ash/shelf/shelf_view_unittest.cc:2369:20
>     #17 0x1326366 in ash::OverflowButtonInkDropTest::SetUp() ash/shelf/shelf_view_unittest.cc:2778
>     #18 0x4722b42 in testing::Test::Run() third_party/googletest/src/googletest/src/gtest.cc
>     #19 0x4724c64 in testing::TestInfo::Run() third_party/googletest/src/googletest/src/gtest.cc:2682:11
>     #20 0x4726036 in testing::TestCase::Run() third_party/googletest/src/googletest/src/gtest.cc:2800:28
>     #21 0x474b566 in testing::internal::UnitTestImpl::RunAllTests() third_party/googletest/src/googletest/src/gtest.cc:5124:43
>     #22 0x474a7b5 in testing::UnitTest::Run() third_party/googletest/src/googletest/src/gtest.cc
>     #23 0x72b431a in RUN_ALL_TESTS third_party/googletest/src/googletest/include/gtest/gtest.h:2331:46
>     #24 0x72b431a in base::TestSuite::Run() base/test/test_suite.cc:277
>     #25 0x72b9892 in Run base/callback.h:99:12
>     #26 0x72b9892 in base::(anonymous namespace)::LaunchUnitTestsInternal(base::OnceCallback<int ()>, unsigned long, int, bool, base::OnceCallback<void ()>) base/test/launcher/unit_test_launcher.cc:224
>     #27 0x72b9330 in base::LaunchUnitTests(int, char**, base::OnceCallback<int ()>) base/test/launcher/unit_test_launcher.cc:575:10
>     #28 0x178df94 in main ash/test/ash_unittests.cc:37:10
>     #29 0x7f78e2b62f44 in __libc_start_main /build/eglibc-ripdx6/eglibc-2.19/csu/libc-start.c:287
> 
> SUMMARY: AddressSanitizer: heap-use-after-free buildtools/third_party/libc++/trunk/include/vector:1479:30 in begin
> Shadow bytes around the buggy address:
>   0x0c267fffe390: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>   0x0c267fffe3a0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>   0x0c267fffe3b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 fa
>   0x0c267fffe3c0: fa fa fa fa fa fa fa fa fd fd fd fd fd fd fd fd
>   0x0c267fffe3d0: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
> =>0x0c267fffe3e0: fd fd fd fd fd fd fd fd fd fd fd fd fd[fd]fd fd
>   0x0c267fffe3f0: fd fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
>   0x0c267fffe400: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
>   0x0c267fffe410: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
>   0x0c267fffe420: fd fd fd fd fd fd fd fd fd fd fd fd fa fa fa fa
>   0x0c267fffe430: fa fa fa fa fa fa fa fa fd fd fd fd fd fd fd fd
> Shadow byte legend (one shadow byte represents 8 application bytes):
>   Addressable:           00
>   Partially addressable: 01 02 03 04 05 06 07 
>   Heap left redzone:       fa
>   Freed heap region:       fd
>   Stack left redzone:      f1
>   Stack mid redzone:       f2
>   Stack right redzone:     f3
>   Stack after return:      f5
>   Stack use after scope:   f8
>   Global redzone:          f9
>   Global init order:       f6
>   Poisoned by user:        f7
>   Container overflow:      fc
>   Array cookie:            ac
>   Intra object redzone:    bb
>   ASan internal:           fe
>   Left alloca redzone:     ca
>   Right alloca redzone:    cb
>   Shadow gap:              cc
> ==28149==ABORTING
> 
> Original change's description:
> > cros: Enable touchable app context menus by default.
> > 
> > This will enable:
> >  - New context menu UI for:
> >   - Application icons
> >   - Shelf context menus
> >   - Desktop context menus
> > 
> > Note:
> >  - Removed tests whose functionality were already being tested.
> >    - MouseContextMenu already tests whether the ink drop is being
> >      shown, so there is no need for
> >      ShelfButtonShowsInkDropHighlightOnMenuShow.
> > 
> > Bug: 871843
> > Change-Id: I4c92ae52f75b25bc9edd1dd67778c4785d8f4058
> > Reviewed-on: https://chromium-review.googlesource.com/1170985
> > Reviewed-by: Scott Violet <sky@chromium.org>
> > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
> > Commit-Queue: Alex Newcomer <newcomer@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#582978}
> 
> TBR=xiyuan@chromium.org,sky@chromium.org,newcomer@chromium.org
> 
> Change-Id: Ib85676861251f41c84f48f956d36c01edb353266
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 871843
> Reviewed-on: https://chromium-review.googlesource.com/1174641
> Reviewed-by: Chris Mumford <cmumford@chromium.org>
> Commit-Queue: Chris Mumford <cmumford@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#583088}

TBR=xiyuan@chromium.org,sky@chromium.org,cmumford@chromium.org,newcomer@chromium.org

Change-Id: I4edeef479d40e1e591ac906aa89ede363668b6a3
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 871843
Reviewed-on: https://chromium-review.googlesource.com/1174777Reviewed-by: Alex Newcomer <newcomer@chromium.org>
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583310}

Change-Id: I34b87a55631eda7a76c5291c95444848c051b6cf
Reviewed-on: https://chromium-review.googlesource.com/1175861Reviewed-by: Tommy Nyquist <nyquist@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583309}

TBR=perezju@chromium.org

Bug: 871708
Change-Id: I49d63079721102aa58396f5efc21ca428e9647c2
Reviewed-on: https://chromium-review.googlesource.com/1176065Reviewed-by: Becky Zhou <huayinz@chromium.org>
Commit-Queue: Becky Zhou <huayinz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583308}

MASH_DEPRECATED is never set anymore.

BUG=692193
TEST=none

Change-Id: I49a99e9e3c6bbc7f3c77238e450e18c4ffd6b992
Reviewed-on: https://chromium-review.googlesource.com/1176102Reviewed-by: Dan Erat <derat@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583307}

Bug: 755840
Change-Id: I5eba22ebcc2e8a11c1d0df7d249b0dd5c0aa5066
Reviewed-on: https://chromium-review.googlesource.com/1175847
Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org>
Commit-Queue: Chris Hamilton <chrisha@chromium.org>
Reviewed-by: Chris Hamilton <chrisha@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583306}

Failure here:
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-chromeos-rel/11980

TBR=wjmaclean@chromium.org
NOTRY=true

Bug: 669227
Change-Id: I4836948e52e9b1b2a1cbb5d6dde6b9707c27fdbc
Reviewed-on: https://chromium-review.googlesource.com/1176123Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583305}

This reverts commit 56d3597e and
relands the original CL.

Reason for revert: The failure (revert reason) doesn't seem to be
caused by this CL.

Original change's description:
> media: MediaBrowserTest not watching for plugin crash
>
> Now the CDM is not running in a plugin, there's no need for
> MediaBrowserTest to watch for plugin crash.

Bug: 772160
Change-Id: I387abcd37e85b703e2d0d1bf67be1f82dd68bf68
Reviewed-on: https://chromium-review.googlesource.com/1174852Reviewed-by: John Rummell <jrummell@chromium.org>
Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583304}

Previously, these could only be set on construction, but the Pepper
Mojo API lets consumers change these at runtime.

Bug: 848078
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: Ia75aeeda26e3ea09ba3b2647ab42b97161a6bae9
Reviewed-on: https://chromium-review.googlesource.com/1173341Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Helen Li <xunjieli@chromium.org>
Reviewed-by: Xiangjun Zhang <xjz@chromium.org>
Commit-Queue: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583303}

See failure here:
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.12%20Tests/14935

TBR=wjmaclean@chromium.org

Bug: 669227
Change-Id: I75f036c0c4bf3cb883a094ed4f60741b53f3338b
Reviewed-on: https://chromium-review.googlesource.com/1175843Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583302}

Currently, luci-logdog-dev isn't letting anything write to project
chromium.

Bug: 868525
Change-Id: I970ccc0b6959f3d5830ca007df963a6edaf6693a
Reviewed-on: https://chromium-review.googlesource.com/1176079Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Commit-Queue: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583301}

The test fails at shutdown because of
contextual_suggestions.EnabledStateMonitor trying to access the
ProfileSyncService at shutdown. So, restore the original service at
tearDown.

Change-Id: I02e7f14caf44ed9c0d4dc5a70d0f353a4428e34f
Reviewed-on: https://chromium-review.googlesource.com/1170311
Commit-Queue: Siddhartha S <ssid@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583300}

This reverts commit 31f1f7d3.

Reason for revert: Bypasses the form's required fields when clicking
"Done"

Original change's description:
> [NTP] Fix cancel button submitting when adding custom link
> 
> "Cancel" was calling the form's submit event, which triggered the add
> link flow.
> 
> Bug: 874018
> Change-Id: Iad822d3884484f5dc0d9f3e6667c9d4c9e0a2728
> Reviewed-on: https://chromium-review.googlesource.com/1174908
> Reviewed-by: Fernando Serboncini <fserb@chromium.org>
> Commit-Queue: Kristi Park <kristipark@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#583004}

TBR=fserb@chromium.org,kristipark@chromium.org

Change-Id: Ie605d0a5d5a5043fbb76d30eb98b7565818e41f7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 874018
Reviewed-on: https://chromium-review.googlesource.com/1174775
Commit-Queue: Kristi Park <kristipark@chromium.org>
Reviewed-by: Kristi Park <kristipark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583299}

https://skia.googlesource.com/skia.git/+log/137ca523a36e..c97a339cd601


git log 137ca523a36e..c97a339cd601 --date=short --no-merges --format='%ad %ae %s'
2018-08-15 ruiqimao@google.com added optimizations to speed up skinning


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

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=kjlubick@chromium.org

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

pointerevent fractional coordinates was shipped from M64, but
offsetx/y were left behind.
This CL makes them aligned with other coordinates, expose fractions
for pointerevents and rounded version for mouseevents.

Intent to Ship: https://groups.google.com/a/chromium.org/d/msg/blink-dev/tl2sOodhtCs/oe0ZMNXPCAAJ

Bug: 867609
Change-Id: I31af4ce01fa820e3281932e0b0a7e8bd5a6c0042
Reviewed-on: https://chromium-review.googlesource.com/1151534
Commit-Queue: Ella Ge <eirage@chromium.org>
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583297}

Rather than Env::GetInstance(). This ensures the right instance is used.

BUG=847992
TEST=covered by tests

Change-Id: I9b75458c1e4d21402ef2e58833c57ff9a289b7a0
Reviewed-on: https://chromium-review.googlesource.com/1176062Reviewed-by: Michael Wasserman <msw@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583296}

Bug: 859594
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: I5026f51b6bc005215827de6d8d4a15b870b42cf4
Reviewed-on: https://chromium-review.googlesource.com/1175981Reviewed-by: Matt Menke <mmenke@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583295}

This instrumentation flag is recommended by libFuzzer authors, as it may enable
various different instrumentation options which are actual and recommended for use.
The flag is implemented in clang, so we won't have to update individual
instrumentation flags anymore.

See crbug.com/764514 and https://github.com/google/oss-fuzz/issues/832

AFL still supports only trace-pc-guard instrumentation, so we have to use separate
compilation flags for AFL and libFuzzer, the same way as we do in OSS-Fuzz.

Previous reverts:

1) https://chromium-review.googlesource.com/c/chromium/src/+/852826

The issue with sancov report generation is not resolved yet. We can either land
this and break coverage on CF for some time, until I resolve crbug.com/818467 OR
we can wait until crbug.com/818467 gets resolved first. I'd prefer landing this now
and fixing coverage on CF after that.

2) https://chromium-review.googlesource.com/c/chromium/src/+/1087431

The issue with linker trying to link in symbols which were not used and thus
were not properly instrumented has been reported to the sanitizers project
and fixed on LLVM side: https://github.com/google/sanitizers/issues/971.

3) https://chromium-review.googlesource.com/c/chromium/src/+/1110348

Dead code stripping led to libFuzzer complaining about a mismatch of the size of
coverage PC tables: https://crbug.com/856239.
Fixed upstream: https://reviews.llvm.org/rL336941.

Bug: 764514
Change-Id: Ic19a6c27ee37143f3ca126e3bc57c9182e962750
Reviewed-on: https://chromium-review.googlesource.com/1175851Reviewed-by: Abhishek Arya <inferno@chromium.org>
Reviewed-by: Jonathan Metzman <metzman@chromium.org>
Commit-Queue: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583294}

TBR=wjmaclean@chromium.org
NOTRY=true

Bug: 874500
Change-Id: I16f3b600c597b3243c5ad5723b7f845cc0e76e61
Reviewed-on: https://chromium-review.googlesource.com/1176080Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583293}

These stories depend on tabs and/or omnibox. The fact that they don't
run on WebView is intentional, not a bug. So remove them from
expectations and clarify that they shouldn't run on WebView in the
code.

Bug: 865464,865466,865471,867568
Change-Id: I9bbd6a4a87aaa423bdf8d106d3de2d64cf6c8d52
Reviewed-on: https://chromium-review.googlesource.com/1174700
Commit-Queue: Annie Sullivan <sullivan@chromium.org>
Commit-Queue: Juan Antonio Navarro Pérez <perezju@chromium.org>
Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#583292}

Change-Id: I20cd1fe7eeb2eac30d7913599a8dc1285ae9023e

******************************Resource Sizes Diff******************************
Monochrome.apk_Dex
    +1,728 entries fields
    +2,366 entries methods
      +581 entries types
    +1,193 entries strings
Monochrome.apk_Breakdown (+169,703 bytes)
    +3,241 bytes Zip Overhead
    +1,510 bytes Non-compiled Android resources size
   +12,528 bytes Native code size
      +129 bytes Package metadata size
  +148,695 bytes Java code size
    +3,600 bytes Compiled Android resources size
Monochrome.apk_Specifics
  +523,907 bytes normalized apk size
        +3 zip entries file count
   +12,528 bytes main lib size
  +354,204 bytes main dex size

Change-Id: I20cd1fe7eeb2eac30d7913599a8dc1285ae9023e
Reviewed-on: https://chromium-review.googlesource.com/1175218Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Commit-Queue: Patrick Noland <pnoland@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583291}

The test currently asserts that, in list-based hit test results, floats
precede other descendants of the block container. However, according to
spec:

- Floats are painted earlier than other descendants [1]
- Elements should be in reverse paint order in hit test result [2]

Hence, this patch changes the test to assert that non-float descendants
come first in hit test result. It also marks it failure in legacy and
pass in LayoutNG.

[1] https://www.w3.org/TR/CSS22/zindex.html#painting-order
[2] https://drafts.csswg.org/cssom-view/#dom-document-elementsfrompoint

Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
Change-Id: I8778d68b4d47f616a653e51f745c10c6b96cfe72
Reviewed-on: https://chromium-review.googlesource.com/1173398Reviewed-by: Koji Ishii <kojii@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583290}

Flakiness dashboard shows that it has been flaky for a while

TBR=wjmaclean@chromium.org
NOTRY=true

Bug: 874497
Change-Id: I017096a3a33ba69cab80cf9f78caa59bd4672319
Reviewed-on: https://chromium-review.googlesource.com/1175939Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583289}

Preparation for hooking up Blink on Android to the font unique name
matching service in RenderProcessHost.

Bug: 870223
Change-Id: I89bd326e78dbd1bdf989265ca6f8820f8d242621
Reviewed-on: https://chromium-review.googlesource.com/1160648Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583288}

MASH_DEPRECATED is never set anymore.

BUG=640773
TEST=none

Change-Id: Iab913023f56e7f832cb987b10b5725d55ccc1ead
Reviewed-on: https://chromium-review.googlesource.com/1176072Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583287}

This fails in single-process mash mode because it uses EventMonitorAura,
which uses Env::GetInstance().

BUG=874481
TEST=test only change

Change-Id: I1f71b68db1a1b52c8e1c2ecc6c87828eea131700
Reviewed-on: https://chromium-review.googlesource.com/1176067
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583286}

Bug: 852735
Change-Id: I56752deea548f5aa01f9ae3d71f3551841737809
Reviewed-on: https://chromium-review.googlesource.com/1175489Reviewed-by: Alice Boxhall <aboxhall@chromium.org>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583285}

TBR=wjmaclean@chromium.org
NOTRY=true

Bug: 874491
Change-Id: Idda7c1bbd55468c33dc6bf2aaafceeb16eb699c5
Reviewed-on: https://chromium-review.googlesource.com/1176075Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583284}

Change-Id: I24e7febdedac576c7a8e77c5412b657383e065bd
Reviewed-on: https://chromium-review.googlesource.com/1176064Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Commit-Queue: Quan Nguyen <qnnguyen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583283}

Changes:
In homecher, folder or search result page is closed when an app is
launched or a search result is opened.

Bug: 853846
Change-Id: I92edf732b01bb9c5b0f97ff2206bd09b2b6130ca
Reviewed-on: https://chromium-review.googlesource.com/1175417
Commit-Queue: Weidong Guo <weidongg@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583282}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/52c721180793..af03ae194f78


git log 52c721180793..af03ae194f78 --date=short --no-merges --format='%ad %ae %s'
2018-08-15 recipe-roller@chromium.org Roll recipe dependencies (trivial).


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

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: I567c98c6cbdae16c6b753b9dcff1cd958d3a7d60
Reviewed-on: https://chromium-review.googlesource.com/1174805Reviewed-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@{#583281}

This CL just adds the strings and drawables for the Previews badge.
The actual triggering and interaction with the native code will come in
another CL.

Bug: 871839
Change-Id: Iad6d5543492bf78e9b8f54e525d979b66217eefb
Reviewed-on: https://chromium-review.googlesource.com/1173266Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Commit-Queue: Robert Ogden <robertogden@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583280}

Adds render tests for JavaScript confirm and prompt dialogs in the VR
browser.

Additionally, cleans the test file up a bit and adds the golden image
directory as a data dependency so that the tests will actually fail on
a mismatch instead of constantly producing new golden images.

Bug: 810847
Change-Id: I6e4ea25cf6fb84f73ed3a7d90fb02eb5adb9e783
Reviewed-on: https://chromium-review.googlesource.com/1174990Reviewed-by: Amirhossein Simjour <asimjour@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583279}