Publish DEPS for Chromium 53.0.2763.0

• DEPS[diff]

Limit foreign fetch to only intercept requests made from secure contexts.

Implements step 5 of "Handle Foreign Fetch" in [1]

[1]: https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#on-foreign-fetch-request-algorithm

BUG=540509

Review-Url: https://codereview.chromium.org/2032653003
Cr-Commit-Position: refs/heads/master@{#398752}

• content/browser/loader/resource_dispatcher_host_impl.cc[diff]
• content/browser/service_worker/foreign_fetch_request_handler.cc[diff]
• content/browser/service_worker/foreign_fetch_request_handler.h[diff]
• content/child/request_extra_data.cc[diff]
• content/child/request_extra_data.h[diff]
• content/child/resource_dispatcher.cc[diff]
• content/common/resource_messages.h[diff]
• content/common/resource_request.h[diff]
• content/renderer/render_frame_impl.cc[diff]
• third_party/WebKit/LayoutTests/http/tests/serviceworker/foreign-fetch-basics.html[diff]
• third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/foreign-fetch-helper-iframe.html[Added - diff]
• third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/foreign-fetch-helpers.js[diff]

Fix jetstream unreferenced tab variable.

BUG=618513
NOTRY=true

Review-Url: https://codereview.chromium.org/2051673003
Cr-Commit-Position: refs/heads/master@{#398751}

• tools/perf/benchmarks/jetstream.py[diff]

mash: Move shelf menu code to //ash/common/shelf

It no longer has non-common dependencies.

I'll TBR since it's a purely mechanical file move.

BUG=615155
TEST=compiles
[email protected]

Review-Url: https://codereview.chromium.org/2046563012
Cr-Commit-Position: refs/heads/master@{#398750}

• ash/ash.gyp[diff]
• ash/common/shelf/shelf_alignment_menu.cc[Renamed from ash/shelf/shelf_alignment_menu.cc - diff]
• ash/common/shelf/shelf_alignment_menu.h[Renamed from ash/shelf/shelf_alignment_menu.h - diff]
• ash/common/shelf/shelf_menu_model.h[Renamed from ash/shelf/shelf_menu_model.h - diff]
• ash/shelf/shelf_view.cc[diff]
• ash/shell/context_menu.h[diff]
• ash/sysui/context_menu_mus.h[diff]
• ash/sysui/shelf_delegate_mus.cc[diff]
• chrome/browser/ui/ash/launcher/launcher_application_menu_item_model.h[diff]
• chrome/browser/ui/ash/launcher/launcher_context_menu.h[diff]

Omnibox: Don't Barf When Suggestion is Mixed-Case Equivalent to Verbatim

Make anything that's case-equivalent to the user's input / query-what-you-typed
be allowed to the default match.

BUG=617312

Review-Url: https://codereview.chromium.org/2045873002
Cr-Commit-Position: refs/heads/master@{#398749}

• chrome/browser/autocomplete/search_provider_unittest.cc[diff]
• components/omnibox/browser/base_search_provider.cc[diff]

Optimize tab switcher border PNG files

Total size of new tab switcher border assets reduced by 72%; 9256 to
2580 bytes.

BUG=606612

Review-Url: https://codereview.chromium.org/2051463003
Cr-Commit-Position: refs/heads/master@{#398748}

• chrome/android/java/res/drawable-hdpi/tabswitcher_border_frame.9.png[diff]
• chrome/android/java/res/drawable-mdpi/tabswitcher_border_frame.9.png[diff]
• chrome/android/java/res/drawable-xhdpi/tabswitcher_border_frame.9.png[diff]
• chrome/android/java/res/drawable-xxhdpi/tabswitcher_border_frame.9.png[diff]
• chrome/android/java/res/drawable-xxxhdpi/tabswitcher_border_frame.9.png[diff]

Send Navigation notifications to clients.

This teaches clients about Navigation entries & should eventually allow them to add a client lib which replicates the in-service NavigationController.

Adds back/forward dropdown menus to the browser app.

[email protected]

Review-Url: https://codereview.chromium.org/2055553002
Cr-Commit-Position: refs/heads/master@{#398747}

• mash/browser/browser.cc[diff]
• mash/webtest/webtest.cc[diff]
• services/navigation/navigation_unittest.cc[diff]
• services/navigation/public/interfaces/view.mojom[diff]
• services/navigation/view_impl.cc[diff]
• services/navigation/view_impl.h[diff]

Revert of Remove enable/disable wheel gestures setting. (patchset #3 id:40001 of https://codereview.chromium.org/2047093002/ )

Reason for revert:
Caused RenderWidgetHostViewMacTest.IgnoreEmptyUnhandledWheelEventWithWheelGestures failure on build:

https://build.chromium.org/p/chromium.mac/builders/Mac10.9%20Tests%20(dbg)/builds/25325

See
https://findit-for-me.appspot.com/build-failure?url=https://build.chromium.org/p/chromium.mac/builders/Mac10.9%20Tests%20(dbg)/builds/25325

Test output:
"""
[ RUN      ] RenderWidgetHostViewMacTest.IgnoreEmptyUnhandledWheelEventWithWheelGestures
../../content/browser/renderer_host/render_widget_host_view_mac_unittest.mm:966: Failure
Value of: view_delegate.get().unhandledWheelEventReceived
  Actual: ''
Expected: __objc_yes
Which is: '?' (1)
[3372:1287:0608/164121:1128775946716:FATAL:surface_manager.cc(43)] Check failed: namespace_client_map_.size() == 0u (1 vs. 0)
"""

Original issue's description:
> Remove enable/disable wheel gestures setting.
>
> Since wheel gesture based scrolling has shipped and the code has been
> removed from blink we can remove the runtime setting.
>
> BUG=598798
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/3f5cea527647ebc8c505b1932d680854e162d82f
> Cr-Commit-Position: refs/heads/master@{#398716}

[email protected],[email protected],[email protected],[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=598798

Review-Url: https://codereview.chromium.org/2050033002
Cr-Commit-Position: refs/heads/master@{#398746}

• cc/debug/debug_rect_history.cc[diff]
• cc/proto/layer_tree_settings.proto[diff]
• cc/trees/layer_tree_host_impl.cc[diff]
• cc/trees/layer_tree_host_impl_unittest.cc[diff]
• cc/trees/layer_tree_settings.cc[diff]
• cc/trees/layer_tree_settings.h[diff]
• components/test_runner/event_sender.cc[diff]
• components/test_runner/event_sender.h[diff]
• components/test_runner/web_test_proxy.cc[diff]
• components/test_runner/web_test_proxy.h[diff]
• content/browser/renderer_host/input/input_router_impl.cc[diff]
• content/browser/renderer_host/input/input_router_impl_unittest.cc[diff]
• content/browser/renderer_host/input/mouse_wheel_event_queue.cc[diff]
• content/browser/renderer_host/input/mouse_wheel_event_queue.h[diff]
• content/browser/renderer_host/input/mouse_wheel_event_queue_unittest.cc[diff]
• content/browser/renderer_host/input/non_blocking_event_browsertest.cc[diff]
• content/browser/renderer_host/overscroll_controller.cc[diff]
• content/browser/renderer_host/overscroll_controller.h[diff]
• content/browser/renderer_host/render_process_host_impl.cc[diff]
• content/browser/renderer_host/render_widget_host_view_aura_unittest.cc[diff]
• content/browser/renderer_host/render_widget_host_view_mac.h[diff]
• content/browser/renderer_host/render_widget_host_view_mac.mm[diff]
• content/browser/renderer_host/render_widget_host_view_mac_unittest.mm[diff]
• content/common/input/input_event_utils.cc[Added - diff]
• content/common/input/input_event_utils.h[Added - diff]
• content/content_common.gypi[diff]
• content/public/common/content_switches.cc[diff]
• content/public/common/content_switches.h[diff]
• content/renderer/gpu/render_widget_compositor.cc[diff]
• content/renderer/input/input_handler_manager.cc[diff]
• content/renderer/input/input_handler_manager.h[diff]
• content/renderer/input/input_handler_wrapper.cc[diff]
• content/renderer/input/input_handler_wrapper.h[diff]
• content/renderer/render_view_impl.cc[diff]
• content/shell/renderer/layout_test/layout_test_content_renderer_client.cc[diff]
• ui/events/blink/input_handler_proxy.cc[diff]
• ui/events/blink/input_handler_proxy.h[diff]
• ui/events/blink/input_handler_proxy_unittest.cc[diff]

Don't include wayland_object.h from wayland_display.cc

It was already included in wayland_display.h

BUG=none

Review-Url: https://codereview.chromium.org/2040373004
Cr-Commit-Position: refs/heads/master@{#398745}

• ui/ozone/platform/wayland/wayland_display.cc[diff]

[Offline Pages] Filtering expired pages and fix consistency check.

Filtering expired pages when querying offline pages. Using expiring
instead of removing page entries in consistency check. Also will remove
files in archive folder to which have no page entry pointing.

BUG=616622

Review-Url: https://codereview.chromium.org/2041983006
Cr-Commit-Position: refs/heads/master@{#398744}

• chrome/browser/android/offline_pages/offline_page_bridge.cc[diff]
• chrome/browser/android/offline_pages/offline_page_utils_unittest.cc[diff]
• components/offline_pages/offline_page_model.h[diff]
• components/offline_pages/offline_page_model_impl.cc[diff]
• components/offline_pages/offline_page_model_impl.h[diff]
• components/offline_pages/offline_page_model_impl_unittest.cc[diff]
• components/offline_pages/offline_page_test_store.cc[diff]
• components/offline_pages/offline_page_test_store.h[diff]
• components/offline_pages/stub_offline_page_model.cc[diff]
• components/offline_pages/stub_offline_page_model.h[diff]
• tools/metrics/histograms/histograms.xml[diff]

Record precache periodic task interval

Precache periodic task is invoked by GCMNetworkManager when the
conditions are right for precaching. The interval between successive
task invocations should be logged.

BUG=618386

Review-Url: https://codereview.chromium.org/2047573007
Cr-Commit-Position: refs/heads/master@{#398743}

• chrome/android/java/src/org/chromium/chrome/browser/precache/PrecacheController.java[diff]
• tools/metrics/histograms/histograms.xml[diff]

Clean up PdfMetafileSkia

- Move printing::PdfMetafileSkia::SaveToFD to the single place it is
  called.
- Simplify Page struct.
- Remove unnecessary includes.
- Factor out MakePdfDocument() in advance of supporting more document
  types provided by Skia.

Review-Url: https://codereview.chromium.org/2024813005
Cr-Commit-Position: refs/heads/master@{#398742}

• components/printing/renderer/print_web_view_helper_linux.cc[diff]
• printing/pdf_metafile_skia.cc[diff]
• printing/pdf_metafile_skia.h[diff]

Roll ANGLE 07529ff..768996c

https://chromium.googlesource.com/angle/angle.git/+log/07529ff..768996c

BUG=617410

TEST=bots
[email protected]
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.linux:linux_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2042333005
Cr-Commit-Position: refs/heads/master@{#398741}

• DEPS[diff]

ash: Remove broken OS update system tray "nagger" popup

See bug for details. We used to show a "nag" popup if you had an update
pending and your shelf had been hidden for > 24 hours. The feature has been
broken for a while and after discussing with abodenha@ we've decided to
remove it.

This is a manual revert of http://crrev.com/134812

BUG=122641
TEST=none

Review-Url: https://codereview.chromium.org/2051643002
Cr-Commit-Position: refs/heads/master@{#398740}

• ash/system/tray_update.cc[diff]
• ash/system/tray_update.h[diff]

[Mac/GN] dSYM output always goes in root_out_dir, so specify it in linker outputs.

This was causing overbuild for targets that override the output directory,
since the linker_driver.py is configured to place dSYMs in root_out_dir, but
the linker output was listed in the target_out_dir.

BUG=330301,431177
[email protected]

Review-Url: https://codereview.chromium.org/2049003004
Cr-Commit-Position: refs/heads/master@{#398739}

• build/toolchain/mac/BUILD.gn[diff]

Add unit tests for the Background Scheduler.

Adding the unit tests required a lot of related changes to be able to do dependency injection, and adding some stub classes.  The SchedulerBridgeInterface allows us to stub out the BackgroundSchedulerBridge, and we added a shadow class for GcmNetworkManager.

This work is described at go/chrome-background-offlining.

BUG=612325

Review-Url: https://codereview.chromium.org/2030773002
Cr-Commit-Position: refs/heads/master@{#398738}

• chrome/android/java/src/org/chromium/chrome/browser/ChromeBackgroundService.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundOfflinerTask.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundScheduler.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerBridge.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerProcessorImpl.java[Added - diff]
• chrome/android/java/src/org/chromium/chrome/browser/offlinepages/TaskExtrasPacker.java[Added - diff]
• chrome/android/java/src/org/chromium/chrome/browser/offlinepages/interfaces/BackgroundSchedulerProcessor.java[Added - diff]
• chrome/android/java_sources.gni[diff]
• chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/BackgroundOfflinerTaskTest.java[Added - diff]
• chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerTest.java[Added - diff]
• chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/ShadowGcmNetworkManager.java[Added - diff]
• chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/StubBackgroundSchedulerProcessor.java[Added - diff]
• chrome/browser/android/offline_pages/background_scheduler_bridge.cc[diff]
• components/offline_pages/background/request_coordinator.cc[diff]
• components/offline_pages/background/request_coordinator.h[diff]
• components/offline_pages/background/request_coordinator_unittest.cc[diff]
• components/offline_pages/background/request_picker.h[diff]

Add GetAccessibilityDelegate() to WmShell

Wires up in overview, updates overview to use WmShell in a couple of
other places and updates DEPS for overview.

BUG=581868
TEST=none
[email protected]

Review-Url: https://codereview.chromium.org/2043883007
Cr-Commit-Position: refs/heads/master@{#398737}

• ash/aura/wm_shell_aura.cc[diff]
• ash/aura/wm_shell_aura.h[diff]
• ash/common/wm_shell.h[diff]
• ash/mus/bridge/wm_shell_mus.cc[diff]
• ash/mus/bridge/wm_shell_mus.h[diff]
• ash/wm/overview/DEPS[diff]
• ash/wm/overview/window_selector.cc[diff]
• ash/wm/overview/window_selector_controller.cc[diff]

BridgedContentView: Modify IsTextRTL to use GetFirstStrongCharacterDirection.

Currently, IsTextRTL method in BridgedContentView uses
base::i18n::GetStringDirection to get the string text direction. This is
inconsistent with RenderText::GetTextDirection. This CL modifies IsTextRTL to
use base::i18n::GetFirstStrongCharacterDirection to ensure consistency with
RenderText::GetTextDirection.

This also necessitates modifying SendHomeEvent and SendEvent in
textfield_unittest.cc, to prevent the tests TextfieldTest.HitOutsideTextAreaTest
and TextfieldTest.HitOutsideTextAreaInRTLTest from regressing on MacViews. This
is because currently, SendHomeEvent and SendEndEvent correspond to logical
direction on other platforms but visual direction on Mac.

The reason that the tests TextfieldTest.HitOutsideTextAreaTest and
TextfieldTest.HitOutsideTextAreaInRTLTest passed up-till now is that the
incorrect IsTextRTL and Send{Home/End}Event implementations cancelled each other
out. IsTextRTL depends on GetStringDirection currently, and hence it always
returned false for text of mixed directionality. A SendHomeEvent would do a
Cmd+Left which would generate a moveToLeftEndOfLine. Since IsTextRTL returned
false, a moveToLeftEndOfLine mapped to a moveToBeginningOfLine in
BridgedContentView. Hence a Send{Home/End}Event mapped to
moveTo{Beginning/End}OfLine, the two incorrect implementations cancelling each
other out.

BUG=618184

Review-Url: https://codereview.chromium.org/2050583002
Cr-Commit-Position: refs/heads/master@{#398736}

• ui/views/cocoa/bridged_content_view.mm[diff]
• ui/views/controls/textfield/textfield_unittest.cc[diff]

mash: Migrate shelf menus to wm common types

This is a dependency of ShelfView.

* Inject WmShelf into Shelf and ShelfView to start breaking up dependencies on
Shelf.
* Change ShellDelegate::CreateContextMenu() to use WmShelf.
* Allow WmShelf to manipulate auto-hide state.
* Move shelf menu UMA stats to common.

I'll move the files into //ash/common in a separate CL so that the changes to
shelf_alignment_menu.cc don't look like a rewrite.

BUG=615155
TEST=existing ash_unittests and chrome unit_tests

Review-Url: https://codereview.chromium.org/2046843005
Cr-Commit-Position: refs/heads/master@{#398735}

• ash/aura/wm_shelf_aura.cc[diff]
• ash/aura/wm_shelf_aura.h[diff]
• ash/common/shelf/wm_shelf.h[diff]
• ash/common/wm/wm_user_metrics_action.h[diff]
• ash/metrics/user_metrics_recorder.cc[diff]
• ash/metrics/user_metrics_recorder.h[diff]
• ash/mus/bridge/wm_shelf_mus.cc[diff]
• ash/mus/bridge/wm_shelf_mus.h[diff]
• ash/root_window_controller.cc[diff]
• ash/shelf/shelf.cc[diff]
• ash/shelf/shelf.h[diff]
• ash/shelf/shelf_alignment_menu.cc[diff]
• ash/shelf/shelf_alignment_menu.h[diff]
• ash/shelf/shelf_view.cc[diff]
• ash/shelf/shelf_view.h[diff]
• ash/shelf/shelf_widget.cc[diff]
• ash/shell/context_menu.cc[diff]
• ash/shell/context_menu.h[diff]
• ash/shell/shell_delegate_impl.cc[diff]
• ash/shell/shell_delegate_impl.h[diff]
• ash/shell_delegate.h[diff]
• ash/sysui/context_menu_mus.cc[diff]
• ash/sysui/context_menu_mus.h[diff]
• ash/sysui/shell_delegate_mus.cc[diff]
• ash/sysui/shell_delegate_mus.h[diff]
• ash/test/shelf_test_api.h[diff]
• ash/test/test_shell_delegate.cc[diff]
• ash/test/test_shell_delegate.h[diff]
• chrome/browser/ui/ash/chrome_shell_delegate.cc[diff]
• chrome/browser/ui/ash/chrome_shell_delegate.h[diff]
• chrome/browser/ui/ash/launcher/arc_launcher_context_menu.cc[diff]
• chrome/browser/ui/ash/launcher/arc_launcher_context_menu.h[diff]
• chrome/browser/ui/ash/launcher/chrome_launcher_controller_browsertest.cc[diff]
• chrome/browser/ui/ash/launcher/desktop_shell_launcher_context_menu.cc[diff]
• chrome/browser/ui/ash/launcher/desktop_shell_launcher_context_menu.h[diff]
• chrome/browser/ui/ash/launcher/extension_launcher_context_menu.cc[diff]
• chrome/browser/ui/ash/launcher/extension_launcher_context_menu.h[diff]
• chrome/browser/ui/ash/launcher/launcher_context_menu.cc[diff]
• chrome/browser/ui/ash/launcher/launcher_context_menu.h[diff]
• chrome/browser/ui/ash/launcher/launcher_context_menu_unittest.cc[diff]

Fix wake_on_wifi_manager_unittest compile issue

StrictMock is templated on a class here, but it looks like
we tried to pass in the constructor instead.

[email protected]
BUG=

Review-Url: https://codereview.chromium.org/2049113003
Cr-Commit-Position: refs/heads/master@{#398734}

• chrome/browser/chromeos/net/wake_on_wifi_manager_unittest.cc[diff]

target_cpu -> current_cpu in media/gpu/BUILD.gn, media/cdm/ppapi/cdm_paths.gni

BUG=616819

Review-Url: https://codereview.chromium.org/2038563004
Cr-Commit-Position: refs/heads/master@{#398733}

• media/cdm/ppapi/cdm_paths.gni[diff]
• media/gpu/BUILD.gn[diff]

Log errors in FFmpegDemuxer::OnReadFrameDone

Review-Url: https://codereview.chromium.org/2047033002
Cr-Commit-Position: refs/heads/master@{#398732}

• media/filters/ffmpeg_demuxer.cc[diff]

Add support for plane offsets to NativePixmap/GbmBuffer.

GBM buffers can have different file descriptors and different offsets
for different planes.

This change adds offsets and also changes NativePixmap interface and
NativePixmapHandle accordingly.

TEST=crrev.com/2040803002 will use and test this patch.
NOTRY=true

Review-Url: https://codereview.chromium.org/2034353002
Cr-Commit-Position: refs/heads/master@{#398731}

• chrome/gpu/arc_gpu_video_decode_accelerator.cc[diff]
• components/exo/display.cc[diff]
• media/gpu/video_decode_accelerator_unittest.cc[diff]
• ui/gfx/ipc/gfx_param_traits_macros.h[diff]
• ui/gfx/native_pixmap_handle_ozone.h[diff]
• ui/gl/gl_image_ozone_native_pixmap.cc[diff]
• ui/ozone/platform/cast/surface_factory_cast.cc[diff]
• ui/ozone/platform/drm/client_native_pixmap_factory_gbm.cc[diff]
• ui/ozone/platform/drm/gpu/drm_thread.cc[diff]
• ui/ozone/platform/drm/gpu/drm_thread.h[diff]
• ui/ozone/platform/drm/gpu/drm_thread_proxy.cc[diff]
• ui/ozone/platform/drm/gpu/drm_thread_proxy.h[diff]
• ui/ozone/platform/drm/gpu/gbm_buffer.cc[diff]
• ui/ozone/platform/drm/gpu/gbm_buffer.h[diff]
• ui/ozone/platform/drm/gpu/gbm_surface_factory.cc[diff]
• ui/ozone/platform/headless/headless_surface_factory.cc[diff]
• ui/ozone/public/native_pixmap.h[diff]

Make chrome sync response compression as default

BUG=509728

Review-Url: https://codereview.chromium.org/2046493007
Cr-Commit-Position: refs/heads/master@{#398730}

• sync/BUILD.gn[diff]
• sync/internal_api/http_bridge.cc[diff]
• sync/internal_api/http_bridge_unittest.cc[diff]

Fix DrMemory test for NativeDesktopMediaListTest.UpdateThumbnail

BUG=612590

Review-Url: https://codereview.chromium.org/2045983004
Cr-Commit-Position: refs/heads/master@{#398729}

• chrome/browser/media/native_desktop_media_list_unittest.cc[diff]
• tools/valgrind/gtest_exclude/unit_tests.gtest-drmemory_win32.txt[diff]

[css-grid] Fix definite/indefinite size detection for abspos elements

LayoutBox::percentageLogicalHeightIsResolvable() was not working fine
to check if a positioned grid container has definite/indefinite height.
The positioned grid container only has definite height when it has
a fixed height or the offset properties (top and bottom) are non-auto.

The patch fixes LayoutBox::computePercentageLogicalHeight()
to take this into account for grid items too.

BUG=538513
TEST=fast/css-grid-layout/flex-item-grid-container-percentage-rows.html

Review-Url: https://codereview.chromium.org/2047093004
Cr-Commit-Position: refs/heads/master@{#398728}

• third_party/WebKit/LayoutTests/fast/css-grid-layout/positioned-grid-container-percentage-tracks.html[Added - diff]
• third_party/WebKit/Source/core/layout/LayoutBox.cpp[diff]

Update V8 to version 5.3.184.

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/76fc8514..1fc749d7

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

[email protected],[email protected],[email protected],[email protected]

Review-Url: https://codereview.chromium.org/2049963002
Cr-Commit-Position: refs/heads/master@{#398727}

• DEPS[diff]

Add rdevlin.cronin as extensions/browser/DEPS owner

BUG=None

Review-Url: https://codereview.chromium.org/2039123002
Cr-Commit-Position: refs/heads/master@{#398726}

• extensions/browser/OWNERS[diff]

ChromeOS: Update OOBE shelf on gaia screen reload.

This CL fixes the problem when shelf state is not updated after
OOBE gaia screen reload.

BUG=588099
TEST=none

Review-Url: https://codereview.chromium.org/2049633002
Cr-Commit-Position: refs/heads/master@{#398725}

• chrome/browser/chromeos/login/screens/core_oobe_actor.h[diff]
• chrome/browser/ui/webui/chromeos/login/core_oobe_handler.h[diff]
• chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc[diff]

Render the material design shelf backgrounds without assets

For the material design shelf (currently off by default
behind --ash-md), render the following backgrounds without
the use of PNG assets:

* Overflow button
* Keyboard button
* Notifications button
* System tray
* Overview mode button
* Dock window manager
* Shelf
* Shelf overflow bubble

This CL also removes dead code found in the background-painting
logic across various Ash UI surfaces. Furthermore,
the black bar is not shown at the bottom of the
material design shelf (this bar formerly contained
the horizontal white activity indicator bars).

BUG=612583, 617293, 617279
TEST=manual

Review-Url: https://codereview.chromium.org/2035313002
Cr-Commit-Position: refs/heads/master@{#398724}

• ash/common/material_design/material_design_controller.cc[diff]
• ash/common/material_design/material_design_controller.h[diff]
• ash/common/shelf/shelf_constants.cc[diff]
• ash/common/shelf/shelf_constants.h[diff]
• ash/common/wm/dock/docked_window_layout_manager.cc[diff]
• ash/shelf/overflow_bubble_view.cc[diff]
• ash/shelf/overflow_button.cc[diff]
• ash/shelf/overflow_button.h[diff]
• ash/shelf/shelf_widget.cc[diff]
• ash/system/tray/tray_background_view.cc[diff]
• ash/system/tray/tray_background_view.h[diff]
• chrome/test/base/view_event_test_platform_part_chromeos.cc[diff]

Enabling WPT serve informational bot on chromium_fyi waterfall

BUG=581386

Committed: https://chromium.googlesource.com/chromium/tools/build/+/943f1294c4cf8ab9ee92dde42e312f21bfd83d3e
Review-Url: https://codereview.chromium.org/2032903005
Cr-Commit-Position: refs/heads/master@{#398723}

• tools/mb/mb_config.pyl[diff]

PlzNavigate: fix WebContentsImplTest.DirectNavigationToViewSourceWebUI

This CL fixes WebContentsImplTest.DirectNavigationToViewSourceWebUI. The test
in its current form only works because it is missing a call to
DidStartProvsionalLoad, which means that no handle is created for the
navigation. On commit, the pending NavigationEntry is reset, and a new one is
created with no virtual url and a rewritten url used for the title. In
PlzNavigate, we have a handle, so the pending entry with the viewsource virtual
url is kept, and the virtual url is displayed as a title. The CL fixes that by
adding the missing DidStartProvsionalLoad and updating the title expectations.

BUG=439423

Review-Url: https://codereview.chromium.org/2046843004
Cr-Commit-Position: refs/heads/master@{#398722}

• content/browser/web_contents/web_contents_impl_unittest.cc[diff]

[Mac] Dismiss an open page info dialog when the location icon is clicked.

On views platforms, clicking the location (lock) icon opens the page
info dialog. Clicking the icon again dismisses the dialog. Currently,
the same actions on Mac "refreshes" the dialog in place, with a strange
animation caused by the original dialog disappearing and a new one
taking its place immediately.

This CL makes the Cocoa and toolkit-views page info dialogs on Mac
behave consistently with views platforms, so that clicking the location
icon when the dialog is open dismisses the dialog.

BUG=59289
TEST=Open Chrome on Mac, and navigate to a page. Click on the lock
icon to open the page info dialog. Click on the icon again and the
dialog should dismiss, rather than refresh in place. The same
behaviour should apply to the toolkit-views and MacViews-WebUI
browsers.

Review-Url: https://codereview.chromium.org/2041723002
Cr-Commit-Position: refs/heads/master@{#398721}

• chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm[diff]
• chrome/browser/ui/views/browser_dialogs_views_mac.cc[diff]

tab capture: Fix capturing a background tab under aura.

When a tab is being streamed, it needs to remain attached to the
compositor (i.e. it needs to be attached to a root window), so that it
receives the will-draw messages (more specifically, DelegatedFrameHost
receives the cc::SurfaceFactoryClient::WillDrawSurface() calls) from
the compositor. There already was a workaround as WindowAdoptionAgent
for aura for streaming offscreen tabs. Adapt that to work for both
offscreen and detached tabs.

BUG=616975

Review-Url: https://codereview.chromium.org/2048073002
Cr-Commit-Position: refs/heads/master@{#398720}

• chrome/browser/extensions/api/tab_capture/offscreen_tab.cc[diff]
• chrome/browser/extensions/api/tab_capture/tab_capture_registry.cc[diff]

Disable icf on chromecast android x86 builds.

The current icf settins result in segfaults in clang/gold, so disable
them.

BUG= 576197, internal b/28941634

Change-Id: I8d8a8fa2d3b6537a302c8a884897d27632dea6f6
Review-Url: https://codereview.chromium.org/2043003003
Cr-Commit-Position: refs/heads/master@{#398719}

• build/config/compiler/BUILD.gn[diff]

Add mb_config and test descriptions for Release Android GPU bots.

The Debug bots' descriptions will be removed once the waterfalls are
restarted.

BUG=617848
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
NOTRY=true

Review-Url: https://codereview.chromium.org/2054533003
Cr-Commit-Position: refs/heads/master@{#398718}

• content/test/gpu/generate_buildbot_json.py[diff]
• testing/buildbot/chromium.gpu.fyi.json[diff]
• tools/mb/mb_config.pyl[diff]

Add support for MAC address randomization

This is a feature in shill/wpa_supplicant to hide MAC addresses
while probing before we are actually connected, making it a little
more difficult to track users' devices by hardware MAC address.
The feature should be disabled for enterprise-enrolled devices
but enabled otherwise.

BUG=579598
[email protected]

Review-Url: https://codereview.chromium.org/2040603002
Cr-Commit-Position: refs/heads/master@{#398717}

• chrome/browser/chromeos/policy/device_network_configuration_updater.cc[diff]
• chrome/browser/chromeos/policy/network_configuration_updater_unittest.cc[diff]
• chromeos/network/mock_network_device_handler.h[diff]
• chromeos/network/network_device_handler.h[diff]
• chromeos/network/network_device_handler_impl.cc[diff]
• chromeos/network/network_device_handler_impl.h[diff]

Remove enable/disable wheel gestures setting.

Since wheel gesture based scrolling has shipped and the code has been
removed from blink we can remove the runtime setting.

BUG=598798
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2047093002
Cr-Commit-Position: refs/heads/master@{#398716}

• cc/debug/debug_rect_history.cc[diff]
• cc/proto/layer_tree_settings.proto[diff]
• cc/trees/layer_tree_host_impl.cc[diff]
• cc/trees/layer_tree_host_impl_unittest.cc[diff]
• cc/trees/layer_tree_settings.cc[diff]
• cc/trees/layer_tree_settings.h[diff]
• components/test_runner/event_sender.cc[diff]
• components/test_runner/event_sender.h[diff]
• components/test_runner/web_test_proxy.cc[diff]
• components/test_runner/web_test_proxy.h[diff]
• content/browser/renderer_host/input/input_router_impl.cc[diff]
• content/browser/renderer_host/input/input_router_impl_unittest.cc[diff]
• content/browser/renderer_host/input/mouse_wheel_event_queue.cc[diff]
• content/browser/renderer_host/input/mouse_wheel_event_queue.h[diff]
• content/browser/renderer_host/input/mouse_wheel_event_queue_unittest.cc[diff]
• content/browser/renderer_host/input/non_blocking_event_browsertest.cc[diff]
• content/browser/renderer_host/overscroll_controller.cc[diff]
• content/browser/renderer_host/overscroll_controller.h[diff]
• content/browser/renderer_host/render_process_host_impl.cc[diff]
• content/browser/renderer_host/render_widget_host_view_aura_unittest.cc[diff]
• content/browser/renderer_host/render_widget_host_view_mac.h[diff]
• content/browser/renderer_host/render_widget_host_view_mac.mm[diff]
• content/browser/renderer_host/render_widget_host_view_mac_unittest.mm[diff]
• content/common/input/input_event_utils.cc[Deleted - diff]
• content/common/input/input_event_utils.h[Deleted - diff]
• content/content_common.gypi[diff]
• content/public/common/content_switches.cc[diff]
• content/public/common/content_switches.h[diff]
• content/renderer/gpu/render_widget_compositor.cc[diff]
• content/renderer/input/input_handler_manager.cc[diff]
• content/renderer/input/input_handler_manager.h[diff]
• content/renderer/input/input_handler_wrapper.cc[diff]
• content/renderer/input/input_handler_wrapper.h[diff]
• content/renderer/render_view_impl.cc[diff]
• content/shell/renderer/layout_test/layout_test_content_renderer_client.cc[diff]
• ui/events/blink/input_handler_proxy.cc[diff]
• ui/events/blink/input_handler_proxy.h[diff]
• ui/events/blink/input_handler_proxy_unittest.cc[diff]

kiosk: Enable required_platform_version key on stable

BUG=618377

Review-Url: https://codereview.chromium.org/2048433004
Cr-Commit-Position: refs/heads/master@{#398715}

• extensions/common/api/_manifest_features.json[diff]

[Chromecast] Implement audio filter and default passthrough filter

BUG=internal b/28928723
TEST=none

Review-Url: https://codereview.chromium.org/2052653002
Cr-Commit-Position: refs/heads/master@{#398714}

• chromecast/media/cma/backend/alsa/BUILD.gn[diff]
• chromecast/media/cma/backend/alsa/audio_filter_factory.h[Added - diff]
• chromecast/media/cma/backend/alsa/audio_filter_factory_default.cc[Added - diff]
• chromecast/media/cma/backend/alsa/audio_filter_interface.h[Added - diff]
• chromecast/media/cma/backend/alsa/stream_mixer_alsa.cc[diff]
• chromecast/media/cma/backend/alsa/stream_mixer_alsa.h[diff]

Roll buildtools 8dd3c8e39a..099f1da55b

  In order to roll GN a8e5d2e327..b0b4c0099b (r396969:r398658) and pick up
  the following changes:

  63a623c Fix generation of Xcode project.
  8161ecd Improve the "gn path" command.
  e9efe32 Fix documentation for "pool" object in gn binary.
  40451c5 Add support for user defined "pool" to GN.
  0e85587 Fix doc on "Multiple outputs" for runtime_deps.
  5797920 [Mac/iOS/GN] Document the new rules for create_bundle and runtime_deps.
  4feb71fc [GN] Adding missing source type for extension .appex.
  3dbb1f6 Migrate WaitableEvent to enum-based constructor in tools/

[email protected]
CQ_EXTRA_TRYBOTS=tryserver.chromium.mac:mac_chromium_gn_dbg;tryserver.chromium.win:win8_chromium_gn_dbg

Review-Url: https://codereview.chromium.org/2049703003
Cr-Commit-Position: refs/heads/master@{#398713}

• DEPS[diff]

Use monotonic time to record the touch input latency in Blink.

Since PlatformEvent uses monotonical time stamp, we should use monotonicallyIncreasingTime
instead of currentTime to record the touch event latency in Blink.

BUG=617984

Review-Url: https://codereview.chromium.org/2044923004
Cr-Commit-Position: refs/heads/master@{#398712}

• third_party/WebKit/Source/core/input/TouchEventManager.cpp[diff]

Fix arc_auth_service_unittest compile issue

The literal "false" was getting implicitly converted into NULL,
which caused a type conversion issue further down the line after
some template and macro expansion of the gtest utilities.

[email protected]
BUG=

Review-Url: https://codereview.chromium.org/2049453004
Cr-Commit-Position: refs/heads/master@{#398711}

• chrome/browser/chromeos/arc/arc_auth_service_unittest.cc[diff]

Prevent playback looping after pause.

Check is_playing_ vatiable in WMPA::OnPlaybackComplete
before restarting the player.

BUG=616772

Review-Url: https://codereview.chromium.org/2053473002
Cr-Commit-Position: refs/heads/master@{#398710}

• content/renderer/media/android/webmediaplayer_android.cc[diff]

Add support for inexact resize on X11

Support switching between existing modes when XRandR is available but
exact resize is not.

BUG=596125

Review-Url: https://codereview.chromium.org/2004613002
Cr-Commit-Position: refs/heads/master@{#398709}

• remoting/host/desktop_resizer_x11.cc[diff]

Site Settings Desktop: Implement media picker for camera/mic categories.

BUG=614277, 543635
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2039833004
Cr-Commit-Position: refs/heads/master@{#398708}

• chrome/browser/resources/settings/privacy_page/privacy_page.html[diff]
• chrome/browser/resources/settings/settings_resources.grd[diff]
• chrome/browser/resources/settings/site_settings/media_picker.html[Added - diff]
• chrome/browser/resources/settings/site_settings/media_picker.js[Added - diff]
• chrome/browser/resources/settings/site_settings/site_settings_category.html[diff]
• chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js[diff]
• chrome/browser/ui/webui/settings/md_settings_ui.cc[diff]
• chrome/browser/ui/webui/settings/settings_media_devices_selection_handler.cc[Added - diff]
• chrome/browser/ui/webui/settings/settings_media_devices_selection_handler.h[Added - diff]
• chrome/chrome_browser_ui.gypi[diff]

Update CertVerifier::Verify to use RequestParams instead

CertVerifier::Verify() has some inconsistencies with
respect to how it handles additional trust anchors, which
complicates the layering and the caching.

Rather than just add an additional parameter to Verify(),
convert everything to use RequestParams, which also defines how
the caching/hashing is done.

BUG=612655
TBR=brettw

Review-Url: https://codereview.chromium.org/1994353002
Cr-Commit-Position: refs/heads/master@{#398707}

• blimp/net/exact_match_cert_verifier.cc[diff]
• blimp/net/exact_match_cert_verifier.h[diff]
• chrome/browser/chromeos/policy/policy_cert_verifier.cc[diff]
• chrome/browser/chromeos/policy/policy_cert_verifier.h[diff]
• chrome/browser/chromeos/policy/policy_cert_verifier_browsertest.cc[diff]
• chrome/browser/extensions/api/gcd_private/privet_v3_context_getter.cc[diff]
• chrome/browser/extensions/api/platform_keys/verify_trust_api.cc[diff]
• extensions/browser/api/cast_channel/cast_socket.cc[diff]
• google_apis/gcm/tools/mcs_probe.cc[diff]
• ios/web/net/cert_verifier_block_adapter.cc[diff]
• net/cert/cert_verifier.cc[diff]
• net/cert/cert_verifier.h[diff]
• net/cert/cert_verifier_unittest.cc[diff]
• net/cert/mock_cert_verifier.cc[diff]
• net/cert/mock_cert_verifier.h[diff]
• net/cert/multi_threaded_cert_verifier.cc[diff]
• net/cert/multi_threaded_cert_verifier.h[diff]
• net/cert/multi_threaded_cert_verifier_unittest.cc[diff]
• net/cert_net/nss_ocsp_unittest.cc[diff]
• net/quic/crypto/proof_verifier_chromium.cc[diff]
• net/quic/crypto/proof_verifier_chromium_test.cc[diff]
• net/socket/ssl_client_socket_impl.cc[diff]
• net/tools/quic/quic_client_bin.cc[diff]
• net/tools/quic/quic_simple_client_bin.cc[diff]
• remoting/protocol/ssl_hmac_channel_authenticator.cc[diff]

Upstream: Create WebAPK when user selects "Add to Home screen"

This CL makes "Add to Home Screen" create a WebAPK when a user runs Chrome with
the --enable-webapk command line flag and:
- Selects "Add to Home screen" from the app menu for a web app compatible app
OR
- Selects "Add to Home screen" from "Add to Home screen" infobar

BUG=609122

Review-Url: https://codereview.chromium.org/2031213004
Cr-Commit-Position: refs/heads/master@{#398706}

• chrome/android/java/src/org/chromium/chrome/browser/ChromeApplication.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkBuilder.java[Added - diff]
• chrome/android/java_sources.gni[diff]

views/mus: Run some more tests from views_unittests in mus.

BUG=577274

Review-Url: https://codereview.chromium.org/2010083002
Cr-Commit-Position: refs/heads/master@{#398705}

• .gn[diff]
• ui/views/BUILD.gn[diff]
• ui/views/accessible_pane_view_unittest.cc[diff]
• ui/views/controls/button/custom_button_unittest.cc[diff]
• ui/views/controls/button/label_button_unittest.cc[diff]
• ui/views/controls/button/menu_button_unittest.cc[diff]
• ui/views/controls/menu/menu_controller_unittest.cc[diff]
• ui/views/controls/slider_unittest.cc[diff]
• ui/views/controls/textfield/textfield_unittest.cc[diff]
• ui/views/event_monitor_unittest.cc[diff]
• ui/views/mus/BUILD.gn[diff]
• ui/views/mus/native_widget_mus.cc[diff]
• ui/views/mus/unittests_manifest.json[diff]
• ui/views/view_unittest.cc[diff]
• ui/views/views.gyp[diff]

Change //build/config/compiler:optimize_max to use -O3.

Certain components (e.g., v8) really want to be compiled with -O3,
but the current ":optimize_max" setting just used -O2. Since "max"
should theoretically mean "max", let's try making it be -O3 across
the board and see what happens.

[email protected]
BUG=616031

Review-Url: https://codereview.chromium.org/2048163002
Cr-Commit-Position: refs/heads/master@{#398704}

• build/config/compiler/BUILD.gn[diff]

Update WebGL2 conformance test expectations on Linux AMD/Intel.

BUG=
TEST=
[email protected]
NOTRY=true
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2051623002
Cr-Commit-Position: refs/heads/master@{#398703}

• content/test/gpu/gpu_tests/webgl2_conformance_expectations.py[diff]

Add cert_verify_tool utility to verify certs from command line.

BUG=167789

Review-Url: https://codereview.chromium.org/2016503002
Cr-Commit-Position: refs/heads/master@{#398702}

• net/BUILD.gn[diff]
• net/net.gyp[diff]
• net/tools/cert_verify_tool/cert_verify_tool.cc[Added - diff]
• net/tools/cert_verify_tool/cert_verify_tool_util.cc[Added - diff]
• net/tools/cert_verify_tool/cert_verify_tool_util.h[Added - diff]
• net/tools/cert_verify_tool/verify_using_cert_verify_proc.cc[Added - diff]
• net/tools/cert_verify_tool/verify_using_cert_verify_proc.h[Added - diff]

Mojo C++ bindings: Add an alias Foo::DataView for FooDataView.

This is useful when you need to write a partially specialized StructTraits to
work with both mojom::Foo and mojom::blink::Foo.

BUG=577686

Review-Url: https://codereview.chromium.org/2044233005
Cr-Commit-Position: refs/heads/master@{#398701}

• mojo/public/cpp/bindings/tests/struct_with_traits_impl_traits.cc[diff]
• mojo/public/cpp/bindings/tests/struct_with_traits_impl_traits.h[diff]
• mojo/public/tools/bindings/generators/cpp_templates/wrapper_class_declaration.tmpl[diff]

MD User Manager: Change button background/text color in disabled state

BUG=618152
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2051473002
Cr-Commit-Position: refs/heads/master@{#398700}

• chrome/browser/resources/md_user_manager/shared_styles.html[diff]

Add token field to ClientSafeBrowsingReportReqeust.

In order to calculate CTRs of bad downloads, safe browsing needs information to
link ClientSafeBrowsingReportRequest (for downloads) and ClientDownloadResponse.
To achieve this, we add a token field to identify its corresponding
ClientDownloadResponse in the ClientSafeBrowsingReportRequest, and propagate it
with the same string in ClientDownloadResponse.token.

Also add a new download report type (DANGEROUS_DOWNLOAD_BY_API) to separately
track CTR of dangerous downloads go through download API.

BUG=615511

Review-Url: https://codereview.chromium.org/2029903002
Cr-Commit-Position: refs/heads/master@{#398699}

• chrome/browser/download/download_commands.cc[diff]
• chrome/browser/download/download_danger_prompt.cc[diff]
• chrome/browser/download/download_danger_prompt.h[diff]
• chrome/browser/download/download_danger_prompt_browsertest.cc[diff]
• chrome/browser/safe_browsing/download_protection_service.cc[diff]
• chrome/browser/safe_browsing/download_protection_service.h[diff]
• chrome/browser/safe_browsing/download_protection_service_unittest.cc[diff]
• chrome/browser/ui/cocoa/download/download_danger_prompt_impl.cc[diff]
• chrome/browser/ui/views/download/download_danger_prompt_views.cc[diff]
• chrome/common/safe_browsing/csd.proto[diff]

Don't get full hashes when there are no matching prefixes

R=vakh
BUG=611316

Review-Url: https://codereview.chromium.org/2040313003
Cr-Commit-Position: refs/heads/master@{#398698}

• chrome/browser/safe_browsing/local_database_manager.cc[diff]

Roll src/third_party/skia/ 2a437e645..28215d44b (2 commits).

https://chromium.googlesource.com/skia.git/+log/2a437e64593a..28215d44b3f9

$ git log 2a437e645..28215d44b --date=short --no-merges --format='%ad %ae %s'
2016-06-08 brianosman Support running Viewer via android_gdb_app
2016-06-08 brianosman Fix mipmap generation on Nexus 5x & 6p devices.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
[email protected]

Review-Url: https://codereview.chromium.org/2049253002
Cr-Commit-Position: refs/heads/master@{#398697}

• DEPS[diff]

Change Android GPU tests to use Chromium instead of content_shell.

As an added benefit, this should allow some of the multi-tab GPU tests
to be enabled on Android.

BUG=616392
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2046833003
Cr-Commit-Position: refs/heads/master@{#398696}

• content/test/gpu/generate_buildbot_json.py[diff]
• testing/buildbot/chromium.gpu.fyi.json[diff]
• tools/perf/chrome_telemetry_build/BUILD.gn[diff]

Fix restore windows chrome icon has no running status bar on shelf

The edge case in crbug/616179 happens when browser window is not active, which means browser_list->begin_last_active() to end_last_active() cannot track it.

BUG=616179
TEST=manually test sees bug fixed

Review-Url: https://codereview.chromium.org/2051503002
Cr-Commit-Position: refs/heads/master@{#398695}

• chrome/browser/ui/ash/launcher/browser_shortcut_launcher_item_controller.cc[diff]
• chrome/browser/ui/ash/launcher/chrome_launcher_controller_unittest.cc[diff]

Unittests for ui::Event struct traits.

* Adds EchoEvent() to ui.mojom.TraitsTestService and uses it to verify
that we get the same event when we serialize and then deserialize an event.

* Fixes deserialization of tilt values.

* Changes constructor of ui::PointerEvent.

BUG=578206

Review-Url: https://codereview.chromium.org/2048513004
Cr-Commit-Position: refs/heads/master@{#398694}

• components/mus/ws/event_dispatcher.cc[diff]
• content/renderer/mus/compositor_mus_connection_unittest.cc[diff]
• ui/events/event.cc[diff]
• ui/events/event.h[diff]
• ui/events/event_unittest.cc[diff]
• ui/events/mojo/BUILD.gn[diff]
• ui/events/mojo/event_struct_traits.cc[diff]
• ui/events/mojo/struct_traits_unittest.cc[diff]
• ui/events/mojo/traits_test_service.mojom[diff]

Adapt WebKit's WaitableEvent to base's new enum-based WaitableEvent

BUG=612843
NO_DEPENDENCY_CHECKS=true

Review-Url: https://codereview.chromium.org/2052593002
Cr-Commit-Position: refs/heads/master@{#398693}

• third_party/WebKit/Source/platform/WaitableEvent.cpp[diff]

Implement cc::FilterOperation StructTraits

FilterOperation is used by RenderPassDrawQuad whcih is used by CompostiorFrame.
This CL allows FilterOperation to be serialized over mojo.

BUG=611802
[email protected]
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2042413003
Cr-Commit-Position: refs/heads/master@{#398692}

• cc/ipc/BUILD.gn[diff]
• cc/ipc/filter_operation.mojom[Added - diff]
• cc/ipc/filter_operation.typemap[Added - diff]
• cc/ipc/filter_operation_struct_traits.h[Added - diff]
• cc/ipc/struct_traits_unittest.cc[diff]
• cc/ipc/traits_test_service.mojom[diff]
• cc/ipc/typemaps.gni[diff]
• cc/output/filter_operation.cc[diff]
• cc/output/filter_operation.h[diff]
• skia/public/interfaces/image_filter_struct_traits.h[diff]
• skia/public/interfaces/test/struct_traits_unittest.cc[diff]

Add BlitFramebuffer command buffer validation.

This makes deqp/functional/gles3/negativebufferapi.html pass.

BUG=429053
TEST=webgl2_conformance
[email protected],[email protected]
NOTRY=true
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2050553002
Cr-Commit-Position: refs/heads/master@{#398691}

• content/test/gpu/gpu_tests/webgl2_conformance_expectations.py[diff]
• gpu/command_buffer/service/framebuffer_manager.cc[diff]
• gpu/command_buffer/service/framebuffer_manager.h[diff]
• gpu/command_buffer/service/gles2_cmd_decoder.cc[diff]

Add the Cryptohome.TpmResults histogram

After this CL: https://chromium-review.googlesource.com/#/c/349425/ has
landed, we need to add the TpmResults and its enum values.

BUG=430125

Review-Url: https://codereview.chromium.org/2043463002
Cr-Commit-Position: refs/heads/master@{#398690}

• tools/metrics/histograms/histograms.xml[diff]

Manual refactor to enum-based WaitableEvent for remaining mac/ios files

BUG=612843
NO_DEPENDENCY_CHECKS=true
[email protected]
[email protected]
(Francois please fact check me, I'm TBRing dana for otherwise mechanical change)

Review-Url: https://codereview.chromium.org/2056443002
Cr-Commit-Position: refs/heads/master@{#398689}

• chrome/browser/media_galleries/mac/mtp_device_delegate_impl_mac_unittest.mm[diff]
• components/cronet/ios/cronet_environment.cc[diff]
• components/cronet/ios/test/cronet_bidirectional_stream_test.mm[diff]
• components/cronet/ios/test/quic_test_server.cc[diff]
• content/browser/mach_broker_mac_unittest.cc[diff]
• ios/chrome/browser/browser_state/chrome_browser_state_impl.cc[diff]
• media/audio/mac/audio_auhal_mac_unittest.cc[diff]
• ui/accelerated_widget_mac/window_resize_helper_mac.cc[diff]

Move ash/material_design to ash/common/material_design

Move ash/material_design files to a new
directory in ash/common.

BUG=618306
TEST=none

Review-Url: https://codereview.chromium.org/2047273003
Cr-Commit-Position: refs/heads/master@{#398688}

• ash/ash.gyp[diff]
• ash/common/material_design/DEPS[Added - diff]
• ash/common/material_design/material_design_controller.cc[Renamed from ash/material_design/material_design_controller.cc - diff]
• ash/common/material_design/material_design_controller.h[Renamed from ash/material_design/material_design_controller.h - diff]
• ash/common/material_design/material_design_controller_unittest.cc[Renamed from ash/material_design/material_design_controller_unittest.cc - diff]
• ash/material_design/DEPS[Deleted - diff]
• ash/shelf/shelf_button.cc[diff]
• ash/sysui/sysui_application.cc[diff]
• ash/test/ash_test_helper.cc[diff]
• ash/test/material_design_controller_test_api.h[diff]
• ash/wm/overview/window_selector_item.cc[diff]
• chrome/browser/chrome_browser_main.cc[diff]
• chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.cc[diff]

[Media Router] Assign each route a current cast mode if possible

This change gives each media route in the WebUI a current cast mode
value if its media source corresponds to one of the currently available
cast modes. This is to prevent showing the 'cast' button that allows
replacing the route when the new route would effectively be a copy of
the original route.

BUG=614144
[email protected]
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2040883002
Cr-Commit-Position: refs/heads/master@{#398687}

• chrome/browser/resources/media_router/elements/media_router_container/media_router_container.html[diff]
• chrome/browser/resources/media_router/elements/media_router_container/media_router_container.js[diff]
• chrome/browser/resources/media_router/elements/route_details/route_details.html[diff]
• chrome/browser/resources/media_router/elements/route_details/route_details.js[diff]
• chrome/browser/resources/media_router/media_router_data.js[diff]
• chrome/browser/ui/webui/media_router/media_router_ui.cc[diff]
• chrome/browser/ui/webui/media_router/media_router_ui.h[diff]
• chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc[diff]
• chrome/browser/ui/webui/media_router/media_router_webui_message_handler.cc[diff]
• chrome/browser/ui/webui/media_router/media_router_webui_message_handler.h[diff]
• chrome/browser/ui/webui/media_router/media_router_webui_message_handler_unittest.cc[diff]
• chrome/test/data/webui/media_router/media_router_container_route_tests.js[diff]
• chrome/test/data/webui/media_router/media_router_container_test_base.js[diff]
• chrome/test/data/webui/media_router/route_details_tests.js[diff]

MruWindowTrackerTest for mash

BUG=617789,618058
TEST=test only change
[email protected]

Review-Url: https://codereview.chromium.org/2046633006
Cr-Commit-Position: refs/heads/master@{#398686}

• ash/mus/BUILD.gn[diff]
• ash/mus/wm/mru_window_tracker_unittest.cc[diff]

Move 'frame-src' CSP checks into FrameFetchContext.

Currently, we're doing 'frame-src' checks inside 'FrameLoader', which
ends up being too early in the navigation cycle to handle the results
of 'upgrade-insecure-requests'. Happily, we've refactored enough of the
loading code in the last ~3 years that we can pretty easily move this
check into 'FrameFetchContext::canRequest' alongside the rest of CSP.

BUG=615910

Review-Url: https://codereview.chromium.org/2022083002
Cr-Commit-Position: refs/heads/master@{#398685}

• content/browser/site_per_process_browsertest.cc[diff]
• third_party/WebKit/LayoutTests/http/tests/security/contentSecurityPolicy/frame-src-child-frame-navigates-to-blocked-origin-expected.txt[diff]
• third_party/WebKit/LayoutTests/http/tests/security/upgrade-insecure-requests/iframe-upgrade-csp.https.html[Added - diff]
• third_party/WebKit/Source/core/loader/DocumentLoader.cpp[diff]
• third_party/WebKit/Source/core/loader/DocumentLoader.h[diff]
• third_party/WebKit/Source/core/loader/FrameFetchContext.cpp[diff]
• third_party/WebKit/Source/core/loader/FrameFetchContext.h[diff]
• third_party/WebKit/Source/core/loader/FrameLoader.cpp[diff]
• third_party/WebKit/Source/core/loader/FrameLoader.h[diff]
• third_party/WebKit/Source/web/tests/WebFrameTest.cpp[diff]
• third_party/WebKit/Source/web/tests/data/iframe_redirect_blocked.html[Added - diff]

Auto-rebaseline for r398645

https://chromium.googlesource.com/chromium/src/+/142a5307d

BUG=618402
[email protected]

Review URL: https://codereview.chromium.org/2052683002 .

Cr-Commit-Position: refs/heads/master@{#398684}

• third_party/WebKit/LayoutTests/TestExpectations[diff]
• third_party/WebKit/LayoutTests/platform/linux-precise/fast/images/color-profile-reflection-expected.png[Deleted - diff]
• third_party/WebKit/LayoutTests/platform/linux/fast/images/color-profile-reflection-expected.png[diff]

[Media Router] Close dialog after resolving presentation request

This change causes the Media Router dialog to close after it resolves a
route that is from a presentation request. This prevents the dialog from
staying open (by the user keeping their mouse over it) and reusing the
same presentation request for another route request (which is an error).

BUG=617321
[email protected]

Review-Url: https://codereview.chromium.org/2038423006
Cr-Commit-Position: refs/heads/master@{#398683}

• chrome/browser/ui/webui/media_router/media_router_ui.cc[diff]
• chrome/browser/ui/webui/media_router/media_router_ui.h[diff]

Auto-rebaseline for r398650

https://chromium.googlesource.com/chromium/src/+/cbe578b0c

BUG=618402
[email protected]

Review URL: https://codereview.chromium.org/2056543002 .

Cr-Commit-Position: refs/heads/master@{#398682}

• third_party/WebKit/LayoutTests/TestExpectations[diff]
• third_party/WebKit/LayoutTests/inspector/layers/layer-compositing-reasons-expected.txt[Renamed from third_party/WebKit/LayoutTests/platform/linux/inspector/layers/layer-compositing-reasons-expected.txt - diff]
• third_party/WebKit/LayoutTests/platform/mac/inspector/layers/layer-compositing-reasons-expected.txt[Deleted - diff]
• third_party/WebKit/LayoutTests/platform/win/inspector/layers/layer-compositing-reasons-expected.txt[Deleted - diff]
• third_party/WebKit/LayoutTests/platform/win/media/video-layer-crash-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/media/video-layer-crash-expected.txt[diff]
• third_party/WebKit/LayoutTests/platform/win7/inspector/layers/layer-compositing-reasons-expected.txt[Deleted - diff]
• third_party/WebKit/LayoutTests/platform/win7/media/video-layer-crash-expected.png[Deleted - diff]
• third_party/WebKit/LayoutTests/platform/win7/media/video-layer-crash-expected.txt[Deleted - diff]

Fix 2D canvas entiring invalid state when painted during hibernation

painting a canvas while the canvas is in a hibernation state was causing
the canvas to allocate a non-accelerated surface backing, which would
later cause GPU texture preparation to fail after exiting from hibernation.
This fix prevents "Canvas2DLayerBridge::newImageSnapshot()" from triggering
the creation of a surface whil the canvas is in a hibernating state.

BUG=613513

Review-Url: https://codereview.chromium.org/2051513002
Cr-Commit-Position: refs/heads/master@{#398681}

• third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridge.cpp[diff]
• third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridgeTest.cpp[diff]

[css-grid] Fix definite/indefinite size detection on block axis

LayoutBox::hasDefiniteLogicalHeight() was wrong as
it was not taking into account several cases
where the height is definite/indefinite.
E.g. it was not checking the special situations for flex items,
that despite of having height "auto" can definite sometimes.

As the method was buggy, this patch is directly removing it
and using LayoutBox::percentageLogicalHeightIsResolvable() instead.

Add new test to check that the flexbox case works as expected.

BUG=617876
TEST=fast/css-grid-layout/flex-item-grid-container-percentage-rows.html

Review-Url: https://codereview.chromium.org/2037383002
Cr-Commit-Position: refs/heads/master@{#398680}

• third_party/WebKit/LayoutTests/fast/css-grid-layout/flex-item-grid-container-percentage-rows-expected.html[Added - diff]
• third_party/WebKit/LayoutTests/fast/css-grid-layout/flex-item-grid-container-percentage-rows.html[Added - diff]
• third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp[diff]
• third_party/WebKit/Source/core/layout/LayoutBox.cpp[diff]
• third_party/WebKit/Source/core/layout/LayoutBox.h[diff]
• third_party/WebKit/Source/core/layout/LayoutGrid.cpp[diff]

Implement pointer lock API for out-of-process iframes.

This patch moves the MouseLockDispatcher from RenderView to RenderWidget and
implements the necessary plumbing to allow pointer locking from out-of-process
iframes.

TEST=Manual test please:
Navigate to http://scheib.github.io/HTMLMisc/PointerLockAndFullscreen-iframes.html
and verify that toggling pointer lock from all iframes work
as well as exiting* pointer lock. Verify that the mouse
movement continues to be tracked in the frame that locked
the pointer.
* Exiting can be done by ESC key, ALT-TAB (on Mac Command
Tab), Windows key to bring up start menu etc.

BUG=601926,419087
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/1980133002
Cr-Commit-Position: refs/heads/master@{#398679}

• content/browser/frame_host/cross_process_frame_connector.cc[diff]
• content/browser/frame_host/cross_process_frame_connector.h[diff]
• content/browser/frame_host/render_widget_host_view_child_frame.cc[diff]
• content/browser/frame_host/render_widget_host_view_child_frame.h[diff]
• content/browser/renderer_host/render_widget_host_impl.cc[diff]
• content/browser/renderer_host/render_widget_host_impl.h[diff]
• content/browser/web_contents/web_contents_impl.cc[diff]
• content/browser/web_contents/web_contents_impl.h[diff]
• content/content_renderer.gypi[diff]
• content/renderer/mouse_lock_dispatcher_browsertest.cc[diff]
• content/renderer/render_view_impl.cc[diff]
• content/renderer/render_view_impl.h[diff]
• content/renderer/render_view_mouse_lock_dispatcher.h[Deleted - diff]
• content/renderer/render_widget.cc[diff]
• content/renderer/render_widget.h[diff]
• content/renderer/render_widget_mouse_lock_dispatcher.cc[Renamed from content/renderer/render_view_mouse_lock_dispatcher.cc - diff]
• content/renderer/render_widget_mouse_lock_dispatcher.h[Added - diff]
• third_party/WebKit/Source/core/dom/Document.cpp[diff]
• third_party/WebKit/Source/core/page/ChromeClient.h[diff]
• third_party/WebKit/Source/core/page/PointerLockController.cpp[diff]
• third_party/WebKit/Source/web/ChromeClientImpl.cpp[diff]
• third_party/WebKit/Source/web/ChromeClientImpl.h[diff]
• third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp[diff]
• third_party/WebKit/Source/web/WebFrameWidgetImpl.h[diff]
• third_party/WebKit/Source/web/WebViewImpl.cpp[diff]
• third_party/WebKit/Source/web/WebViewImpl.h[diff]
• third_party/WebKit/public/web/WebViewClient.h[diff]

Create a KeyboardEventManager class

Extract Keyboard related logic and events from
EventHandler and put it in a dedicate class.

BUG=616836

Review-Url: https://codereview.chromium.org/2036643003
Cr-Commit-Position: refs/heads/master@{#398678}

• third_party/WebKit/Source/core/core.gypi[diff]
• third_party/WebKit/Source/core/input/EventHandler.cpp[diff]
• third_party/WebKit/Source/core/input/EventHandler.h[diff]
• third_party/WebKit/Source/core/input/KeyboardEventManager.cpp[Added - diff]
• third_party/WebKit/Source/core/input/KeyboardEventManager.h[Added - diff]
• third_party/WebKit/Source/platform/PlatformKeyboardEvent.cpp[diff]
• third_party/WebKit/Source/platform/PlatformKeyboardEvent.h[diff]
• third_party/WebKit/Source/web/WebAXObject.cpp[diff]

cc: Separate raster and decode prepaint regions.

This patch separates raster and decode distances to be
independent of each other. Specifically, any tile in prepaint
that is further than 1000 screen space pixels from the
viewport will not be rasterized. Instead, we will pull out
the images on those tiles and predecode those instead.

R=enne, ericrk
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2007163002
Cr-Commit-Position: refs/heads/master@{#398677}

• cc/test/fake_tile_manager.cc[diff]
• cc/tiles/tile_manager.cc[diff]
• cc/tiles/tile_manager.h[diff]
• cc/trees/layer_tree_host_impl.cc[diff]
• cc/trees/layer_tree_settings.h[diff]

Bump smaller tests to only require 4 devices.

NOTRY=True
[email protected]
BUG=

Review-Url: https://codereview.chromium.org/2055533002
Cr-Commit-Position: refs/heads/master@{#398676}

• testing/buildbot/chromium.linux.json[diff]

DevTools: update V8Inspector to work with the new v8_inspector API.

BUG=580337
NOTRY=true

Review-Url: https://codereview.chromium.org/2044343002
Cr-Commit-Position: refs/heads/master@{#398675}

• third_party/WebKit/Source/core/inspector/InspectorSession.cpp[diff]
• third_party/WebKit/Source/core/inspector/InspectorSession.h[diff]
• third_party/WebKit/Source/platform/blink_platform.gypi[diff]
• third_party/WebKit/Source/platform/inspector_protocol/Array.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/BackendCallback.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/CodeGenerator.py[diff]
• third_party/WebKit/Source/platform/inspector_protocol/Collections.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/CollectionsSTL.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/DispatcherBase.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/ErrorSupport.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/FrontendChannel.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/Maybe.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/Object.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/Parser.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/Platform.h[Added - diff]
• third_party/WebKit/Source/platform/inspector_protocol/PlatformSTL.h[Added - diff]
• third_party/WebKit/Source/platform/inspector_protocol/PlatformWTF.h[Added - diff]
• third_party/WebKit/Source/platform/inspector_protocol/String16STL.cpp[diff]
• third_party/WebKit/Source/platform/inspector_protocol/String16STL.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/TypeBuilder_h.template[diff]
• third_party/WebKit/Source/platform/inspector_protocol/ValueConversions.h[diff]
• third_party/WebKit/Source/platform/inspector_protocol/Values.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/Atomics.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/InjectedScript.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/InjectedScriptNative.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/JavaScriptCallFrame.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/JavaScriptCallFrame.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/RemoteObjectId.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/RemoteObjectId.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8Console.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8DebuggerAgentImpl.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8DebuggerImpl.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8DebuggerImpl.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8FunctionCall.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8HeapProfilerAgentImpl.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8InspectorSessionImpl.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8InspectorSessionImpl.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8Regex.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8StackTraceImpl.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/V8StackTraceImpl.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/public/V8ContentSearchUtil.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/public/V8Debugger.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/public/V8DebuggerClient.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/public/V8Inspector.cpp[diff]
• third_party/WebKit/Source/platform/v8_inspector/public/V8Inspector.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/public/V8InspectorSession.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/public/V8InspectorSessionClient.h[diff]
• third_party/WebKit/Source/platform/v8_inspector/public/V8StackTrace.h[diff]

Do not build the IPC fuzzer on chromium.lkgr debug bots.

[email protected]
BUG=

Review-Url: https://codereview.chromium.org/2047323003
Cr-Commit-Position: refs/heads/master@{#398674}

• tools/mb/mb_config.pyl[diff]

DevTools: introduce browser domain for basic target discovery.

TBR=reillyg trivial change to internal function signature.
NOTRY=true

Review-Url: https://codereview.chromium.org/2047833002
Cr-Commit-Position: refs/heads/master@{#398673}

• chrome/browser/devtools/devtools_ui_bindings.cc[diff]
• chrome/browser/extensions/api/debugger/debugger_api.cc[diff]
• chrome/browser/prerender/prerender_browsertest.cc[diff]
• components/devtools_http_handler/devtools_http_handler.cc[diff]
• content/browser/devtools/browser_devtools_agent_host.cc[diff]
• content/browser/devtools/browser_devtools_agent_host.h[diff]
• content/browser/devtools/devtools_agent_host_impl.cc[diff]
• content/browser/devtools/devtools_agent_host_impl.h[diff]
• content/browser/devtools/devtools_manager_unittest.cc[diff]
• content/browser/devtools/protocol/browser_handler.cc[Added - diff]
• content/browser/devtools/protocol/browser_handler.h[Added - diff]
• content/browser/devtools/protocol/devtools_protocol_browsertest.cc[diff]
• content/browser/devtools/protocol/service_worker_handler.cc[diff]
• content/browser/devtools/render_frame_devtools_agent_host.cc[diff]
• content/browser/devtools/render_frame_devtools_agent_host.h[diff]
• content/browser/devtools/shared_worker_devtools_manager_unittest.cc[diff]
• content/browser/devtools/site_per_process_devtools_browsertest.cc[diff]
• content/content_browser.gypi[diff]
• content/public/browser/devtools_agent_host.h[diff]
• content/shell/browser/shell_devtools_frontend.cc[diff]
• headless/lib/browser/headless_devtools_client_impl.cc[diff]
• third_party/WebKit/Source/core/inspector/browser_protocol.json[diff]

Add web platform pointerevents tests to Layouttest

Adding two of the tests and using eventsender and
gpu benchmarking to give inputs automatically to
them.

BUG=612924

Review-Url: https://codereview.chromium.org/1994683004
Cr-Commit-Position: refs/heads/master@{#398672}

• content/shell/app/shell_main_delegate.cc[diff]
• third_party/WebKit/LayoutTests/TestExpectations[diff]
• third_party/WebKit/LayoutTests/imported/wpt/html/semantics/embedded-content/the-img-element/update-the-source-set-expected.txt[diff]
• third_party/WebKit/LayoutTests/imported/wpt/resources/testharnessreport.js[diff]
• third_party/WebKit/LayoutTests/imported/wpt_automation/pointerevents/pointerevent_button_attribute_mouse-manual-input.js[Added - diff]
• third_party/WebKit/LayoutTests/imported/wpt_automation/pointerevents/pointerevent_change-touch-action-onpointerdown_touch-manual-input.js[Added - diff]
• third_party/WebKit/LayoutTests/resources/testharnessreport.js[diff]
• third_party/WebKit/LayoutTests/virtual/stable/webexposed/global-interface-listing-expected.txt[diff]
• third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt[diff]

Update V8 to version 5.3.183.

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/af2ed87b..76fc8514

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

[email protected],[email protected],[email protected],[email protected]

Review-Url: https://codereview.chromium.org/2049193002
Cr-Commit-Position: refs/heads/master@{#398671}

• DEPS[diff]

Upstream: Only allow Chrome to use the WebAPK service

This CL checks the uid of the process calling into the WebAPK service against a
whitelist prior to allowing the WebAPK service to be used.

BUG=524670
TBR=hanxi (TBR for testing/ dependency in libs/runtime_library/junit)
R=rsesek

Review-Url: https://codereview.chromium.org/2028413002
Cr-Commit-Position: refs/heads/master@{#398670}

• chrome/android/webapk/libs/runtime_library/BUILD.gn[diff]
• chrome/android/webapk/libs/runtime_library/junit/DEPS[Added - diff]
• chrome/android/webapk/libs/runtime_library/junit/src/org/chromium/webapk/lib/runtime_library/WebApkServiceImplTest.java[Added - diff]
• chrome/android/webapk/libs/runtime_library/src/org/chromium/webapk/lib/runtime_library/WebApkServiceImpl.java[diff]
• chrome/android/webapk/shell_apk/src/org/chromium/webapk/shell_apk/WebApkServiceFactory.java[diff]

Fixes for turning on offliner request.

BUG=601173

Review-Url: https://codereview.chromium.org/2048923002
Cr-Commit-Position: refs/heads/master@{#398669}

• chrome/browser/android/offline_pages/prerender_adapter.cc[diff]
• chrome/browser/android/offline_pages/prerendering_offliner.cc[diff]
• chrome/browser/android/offline_pages/prerendering_offliner.h[diff]

Make the focus, caret, and cursor highlights fade out.

To minimize the distraction from having these highlights on the screen,
make all of them behave similarly to the cursor highlight and just fade
in quickly when changing, and then fade out slowly.

Because this conflicts with how the focus ring behaves when spoken
feedback is on, add a separate mode for that focus ring behavior.

BUG=314889

Review-Url: https://codereview.chromium.org/2046563009
Cr-Commit-Position: refs/heads/master@{#398668}

• chrome/app/chromeos_strings.grdp[diff]
• chrome/browser/accessibility/accessibility_extension_api.cc[diff]
• chrome/browser/chromeos/accessibility/accessibility_highlight_manager.cc[diff]
• chrome/browser/chromeos/accessibility/accessibility_manager.cc[diff]
• chrome/browser/chromeos/ui/accessibility_cursor_ring_layer.cc[diff]
• chrome/browser/chromeos/ui/accessibility_cursor_ring_layer.h[diff]
• chrome/browser/chromeos/ui/accessibility_focus_ring_controller.cc[diff]
• chrome/browser/chromeos/ui/accessibility_focus_ring_controller.h[diff]
• chrome/browser/chromeos/ui/focus_ring_layer.cc[diff]
• chrome/browser/chromeos/ui/focus_ring_layer.h[diff]

Revert of Only trigger session restore logic for urls if it is not first run. (patchset #2 id:20001 of https://codereview.chromium.org/2040193003/ )

Reason for revert:
broke tests in official builds

Original issue's description:
> Only trigger session restore logic for urls if it is not first run.
>
> The current logic trumped over start up urls specified in master_prefs
> because session restore simply ignored them. Instead of adding them to
> session restore we gate session restore logic on the absence of the
> first run flag. There should be no session to restore in this case anyway.
> Even if the user manually used the --force-first-run it makes sense to
> disregard the session restore code as this sounds more in line with the
> flag name.
>
> BUG=419580
> TEST=StartupBrowserCreatorFirstRunTest.FirstRunTabsWithRestoreSession
>
> Committed: https://crrev.com/da14eddd225351ab39f35acea8daf1b59d3fe4e5
> Cr-Commit-Position: refs/heads/master@{#398512}

[email protected],[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=419580

Review-Url: https://codereview.chromium.org/2049263002
Cr-Commit-Position: refs/heads/master@{#398667}

• chrome/browser/chrome_browser_main.cc[diff]
• chrome/browser/ui/startup/startup_browser_creator_browsertest.cc[diff]
• chrome/browser/ui/startup/startup_browser_creator_impl.cc[diff]

target_cpu -> current_cpu in build/config/sanitizers/BUILD.gn

BUG=616819

Review-Url: https://codereview.chromium.org/2041433002
Cr-Commit-Position: refs/heads/master@{#398666}

• build/config/sanitizers/BUILD.gn[diff]

Moving prefs observer code form ChromeLauncherController into a separate class.

BUG=NONE

Review-Url: https://codereview.chromium.org/2042843004
Cr-Commit-Position: refs/heads/master@{#398665}

• chrome/browser/ui/ash/chrome_launcher_prefs.cc[diff]
• chrome/browser/ui/ash/chrome_launcher_prefs.h[diff]
• chrome/browser/ui/ash/launcher/chrome_launcher_controller.cc[diff]
• chrome/browser/ui/ash/launcher/chrome_launcher_controller.h[diff]

clang-tidy WaitableEvent refactor (ChromeOS side)

BUG=612843
[email protected]
(purely mechanical and automated C++ change doesn't warrant another round of components OWNER IMO)

Review-Url: https://codereview.chromium.org/2056493002
Cr-Commit-Position: refs/heads/master@{#398664}

• chrome/browser/chromeos/extensions/default_app_order.cc[diff]
• chrome/browser/chromeos/login/signin/oauth2_browsertest.cc[diff]
• chromeos/dbus/blocking_method_caller.cc[diff]
• chromeos/system/statistics_provider.cc[diff]
• components/exo/wayland/server_unittest.cc[diff]

gpu: Disallow null client ids in Gen* functions

Gen* functions never return 0 as a valid id. Do not accept 0 as a valid id from an untrusted client.

BUG=None
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2043983002
Cr-Commit-Position: refs/heads/master@{#398663}

• gpu/command_buffer/build_gles2_cmd_buffer.py[diff]
• gpu/command_buffer/service/gles2_cmd_decoder.cc[diff]
• gpu/command_buffer/service/gles2_cmd_decoder_autogen.h[diff]
• gpu/command_buffer/service/gles2_cmd_decoder_unittest.cc[diff]
• gpu/command_buffer/service/gles2_cmd_decoder_unittest_1_autogen.h[diff]
• gpu/command_buffer/service/gles2_cmd_decoder_unittest_attribs.cc[diff]

Roll src/third_party/skia/ 7942f22c6..2a437e645 (4 commits).

https://chromium.googlesource.com/skia.git/+log/7942f22c607c..2a437e64593a

$ git log 7942f22c6..2a437e645 --date=short --no-merges --format='%ad %ae %s'
2016-06-08 liyuqian Provide filter when there are many options
2016-06-08 csmartdalton Replace targetHasUnifiedMultisampling in GrPB constructor
2016-06-08 bsalomon When setting up a copySurface dst texture make the orientation match the src when glBlitFramebuffer requires it
2016-06-08 msarett Modify QCMS test to match use by Chrome

BUG=618122

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
[email protected]

Review-Url: https://codereview.chromium.org/2044293003
Cr-Commit-Position: refs/heads/master@{#398662}

• DEPS[diff]

Update GN docs about target_cpu to encourage use of current_cpu

BUG=616819

Review-Url: https://codereview.chromium.org/2038733003
Cr-Commit-Position: refs/heads/master@{#398661}

• tools/gn/docs/cross_compiles.md[diff]
• tools/gn/variables.cc[diff]

Revert of service worker: Don't control a subframe of an insecure context (patchset #21 id:440001 of https://codereview.chromium.org/2009453002/ )

Reason for revert:
ServiceWorkerProviderHost::SetControllerVersionAttribute CHECK is firing in release builds: https://crbug.com/618365

Original issue's description:
> service worker: Don't control a subframe of an insecure context
>
> We must check isSecureContext when creating the network provider to
> adhere to https://w3c.github.io/webappsec/specs/powerfulfeatures/#settings-privileged.
>
> We already did this for getRegistration(), register(), unregister() but must
> also do this when deciding whether to control an in-scope document.
>
> BUG=607543
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/59a2e54eeb0e61971a0c27c44c54dd0c30b5d06d
> Cr-Commit-Position: refs/heads/master@{#398229}

[email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected]
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=607543

Review-Url: https://codereview.chromium.org/2055433002
Cr-Commit-Position: refs/heads/master@{#398660}

• chrome/browser/chrome_content_browser_client.cc[diff]
• chrome/browser/chrome_content_browser_client.h[diff]
• chrome/browser/extensions/service_worker_apitest.cc[diff]
• content/browser/service_worker/service_worker_browsertest.cc[diff]
• content/browser/service_worker/service_worker_context_core.cc[diff]
• content/browser/service_worker/service_worker_context_request_handler_unittest.cc[diff]
• content/browser/service_worker/service_worker_context_unittest.cc[diff]
• content/browser/service_worker/service_worker_controllee_request_handler.cc[diff]
• content/browser/service_worker/service_worker_controllee_request_handler_unittest.cc[diff]
• content/browser/service_worker/service_worker_dispatcher_host.cc[diff]
• content/browser/service_worker/service_worker_dispatcher_host.h[diff]
• content/browser/service_worker/service_worker_dispatcher_host_unittest.cc[diff]
• content/browser/service_worker/service_worker_handle_unittest.cc[diff]
• content/browser/service_worker/service_worker_job_unittest.cc[diff]
• content/browser/service_worker/service_worker_provider_host.cc[diff]
• content/browser/service_worker/service_worker_provider_host.h[diff]
• content/browser/service_worker/service_worker_provider_host_unittest.cc[diff]
• content/browser/service_worker/service_worker_request_handler_unittest.cc[diff]
• content/browser/service_worker/service_worker_storage_unittest.cc[diff]
• content/browser/service_worker/service_worker_url_request_job_unittest.cc[diff]
• content/browser/service_worker/service_worker_version_unittest.cc[diff]
• content/browser/service_worker/service_worker_write_to_cache_job_unittest.cc[diff]
• content/child/service_worker/service_worker_network_provider.cc[diff]
• content/child/service_worker/service_worker_network_provider.h[diff]
• content/common/service_worker/service_worker_messages.h[diff]
• content/public/browser/content_browser_client.h[diff]
• content/renderer/render_frame_impl.cc[diff]
• content/renderer/service_worker/service_worker_context_client.cc[diff]
• content/renderer/shared_worker/embedded_shared_worker_stub.cc[diff]
• third_party/WebKit/LayoutTests/http/tests/serviceworker/insecure-parent-frame.html[Deleted - diff]
• third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/insecure-inscope.html[Deleted - diff]
• third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/insecure-parent.html[Deleted - diff]
• third_party/WebKit/Source/core/dom/Document.cpp[diff]
• third_party/WebKit/Source/core/dom/Document.h[diff]
• third_party/WebKit/Source/core/frame/Frame.cpp[diff]
• third_party/WebKit/Source/core/frame/Frame.h[diff]
• third_party/WebKit/Source/web/WebFrame.cpp[diff]
• third_party/WebKit/Source/web/tests/WebFrameTest.cpp[diff]
• third_party/WebKit/public/web/WebFrame.h[diff]

Add DisableIf annotation to EME test

EME test does not pass on AOSP Android, therefore, add DisableIf annotation to
it (from BaseInstrumentationTestRunner)

Reorganized BaseInstrumentationTestRunner so it doesn't include anything
unrelated to WebViewLayout Testing.

BUG=607350

Review-Url: https://codereview.chromium.org/1952573002
Cr-Commit-Position: refs/heads/master@{#398659}

• android_webview/tools/system_webview_shell/layout_tests/src/org/chromium/webview_shell/test/WebViewLayoutTest.java[diff]
• android_webview/tools/system_webview_shell/layout_tests/src/org/chromium/webview_shell/test/WebViewLayoutTestRunner.java[diff]
• base/test/android/javatests/src/org/chromium/base/test/util/DisableIf.java[diff]
• base/test/android/javatests/src/org/chromium/base/test/util/DisableIfSkipCheck.java[diff]

Add role lists to ChromeVox.

- add lists for forms, headings, landmarks, links, and tables.
- fix some focusing issues with menus

BUG=617670

CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2042083002
Cr-Commit-Position: refs/heads/master@{#398658}

• chrome/browser/resources/chromeos/chromevox/chromevox/background/keymaps/next_keymap.json[diff]
• chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js[diff]
• chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js[diff]
• chrome/browser/resources/chromeos/chromevox/cvox2/background/panel_menu.js[diff]
• chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings.grd[diff]

Manual refactor to enum-based WaitableEvent for remaining _win files

BUG=612843
[email protected]
[email protected]
(Francois please fact check me, I'm TBRing dana for otherwise mechanical change)

Review-Url: https://codereview.chromium.org/2047163003
Cr-Commit-Position: refs/heads/master@{#398657}

• base/sync_socket_win.cc[diff]
• chrome/browser/chrome_select_file_dialog_factory_win.cc[diff]
• chrome/browser/password_manager/password_store_win_unittest.cc[diff]
• components/policy/core/common/policy_loader_win.cc[diff]
• net/proxy/dhcp_proxy_script_adapter_fetcher_win_unittest.cc[diff]
• net/proxy/dhcp_proxy_script_fetcher_win_unittest.cc[diff]

StructTraits optimizations

Pass more things by const reference and read directly into output.

BUG=611802
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2047963004
Cr-Commit-Position: refs/heads/master@{#398656}

• cc/ipc/returned_resource_struct_traits.h[diff]
• cc/ipc/selection_struct_traits.h[diff]
• cc/ipc/shared_quad_state_struct_traits.h[diff]
• cc/ipc/transferable_resource_struct_traits.h[diff]
• gpu/ipc/common/mailbox_holder_struct_traits.h[diff]

Remove type converters for ui::Event.

We are using struct traits for serializing/deserializing ui::Event and don't
need type converters anymore.

BUG=578206

Review-Url: https://codereview.chromium.org/2048633004
Cr-Commit-Position: refs/heads/master@{#398655}

• ash/mus/BUILD.gn[diff]
• ash/mus/frame/non_client_frame_view_mash.cc[diff]
• ash/mus/window_manager_application.cc[diff]
• ash/sysui/BUILD.gn[diff]
• components/mus/public/cpp/BUILD.gn[diff]
• components/mus/public/cpp/lib/window_tree_client.cc[diff]
• components/mus/public/cpp/tests/BUILD.gn[diff]
• components/mus/public/cpp/tests/window_tree_client_private.cc[diff]
• components/mus/public/cpp/tests/window_tree_client_unittest.cc[diff]
• components/mus/ws/BUILD.gn[diff]
• components/mus/ws/event_dispatcher.cc[diff]
• components/mus/ws/event_matcher.cc[diff]
• components/mus/ws/platform_display.cc[diff]
• components/mus/ws/window_server.cc[diff]
• components/mus/ws/window_tree.cc[diff]
• content/renderer/mus/BUILD.gn[diff]
• content/renderer/mus/compositor_mus_connection.cc[diff]
• ui/events/mojo/BUILD.gn[diff]
• ui/events/mojo/input_events_type_converters.cc[Deleted - diff]
• ui/events/mojo/input_events_type_converters.h[Deleted - diff]
• ui/events/mojo/mojo_extended_key_event_data.cc[Deleted - diff]
• ui/events/mojo/mojo_extended_key_event_data.h[Deleted - diff]
• ui/events/mojo/mojo_input_events_export.h[Deleted - diff]
• ui/views/mus/BUILD.gn[diff]

Promote DCHECK to CHECK to help track down crashes.

BUG=595980
TEST=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2045283003
Cr-Commit-Position: refs/heads/master@{#398654}

• cc/layers/video_layer.cc[diff]

Remove IPC::ParamTraits for ui::Event.

We decided to use struct traits for ui::Event and don't need the param traits
anymore.

This CL reverts the changes we don't need anymore in the following CLs:
* https://codereview.chromium.org/1695783002/
* https://codereview.chromium.org/1800633002/

BUG=584687

Review-Url: https://codereview.chromium.org/2049053003
Cr-Commit-Position: refs/heads/master@{#398653}

• components/mus/common/BUILD.gn[diff]
• components/mus/common/event_param_traits.cc[Deleted - diff]
• components/mus/common/event_param_traits.h[Deleted - diff]
• components/mus/common/event_param_traits_macros.h[Deleted - diff]
• components/mus/common/event_param_traits_unittest.cc[Deleted - diff]
• ui/events/event.cc[diff]
• ui/events/event.h[diff]
• ui/events/gesture_event_details.h[diff]
• ui/events/keycodes/dom/dom_key.h[diff]

Migrate WaitableEvent to enum-based constructor for callsites using non-inlined bools.

As expected: clang-tidy couldn't auto-refactor those non-inlined bools

BUG=612843
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
NO_DEPENDENCY_CHECKS=true
TBR=Have owners approval, skipping NO_DEPENDENCY_CHECKS bug for presubmit (http://crbug.com/616912)

Review-Url: https://codereview.chromium.org/2052573002
Cr-Commit-Position: refs/heads/master@{#398652}

• cc/layers/texture_layer_unittest.cc[diff]
• components/mus/gpu/gpu_service_mus.cc[diff]

Always disable session invalidations when not on the recent tabs page

This CL removes checking for the AndroidSessionNotifications field trial

Review-Url: https://codereview.chromium.org/2022303002
Cr-Commit-Position: refs/heads/master@{#398651}

• chrome/android/java/src/org/chromium/chrome/browser/invalidation/ChromeBrowserSyncAdapter.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/invalidation/InvalidationController.java[diff]
• tools/metrics/histograms/histograms.xml[diff]

Rebase two layout tests on Win10

Failure here:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10/builds/11104

BUG=618402
NOTRY=true
[email protected]

Review-Url: https://codereview.chromium.org/2048783003
Cr-Commit-Position: refs/heads/master@{#398650}

• third_party/WebKit/LayoutTests/TestExpectations[diff]

Use correct URL in Authenticator webview loadabort event handler

Fix to use the URL whose load was aborted, instead of the current
webview URL (which could be updated multiple times and therefore does
not necessarily correspond to this loadabort event).

This is just a cleanup fix, because the passed URL was anyway not used
by the listeners in the current code.

BUG=
TEST=manual: add logging to 'loadAbort' Authenticator event
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2052503002
Cr-Commit-Position: refs/heads/master@{#398649}

• chrome/browser/resources/gaia_auth_host/authenticator.js[diff]

Move leak_detector_controller to its own directory

This new directory will hold future code that relates to the leak
detector as well.

BUG=chromium:615223
TEST=unit tests pass

Review-Url: https://codereview.chromium.org/2047163002
Cr-Commit-Position: refs/heads/master@{#398648}

• chrome/browser/BUILD.gn[diff]
• chrome/browser/metrics/chromeos_metrics_provider.h[diff]
• chrome/browser/metrics/leak_detector/OWNERS[Added - diff]
• chrome/browser/metrics/leak_detector/leak_detector_controller.cc[Renamed from chrome/browser/metrics/leak_detector_controller.cc - diff]
• chrome/browser/metrics/leak_detector/leak_detector_controller.h[Renamed from chrome/browser/metrics/leak_detector_controller.h - diff]
• chrome/browser/metrics/leak_detector/leak_detector_controller_unittest.cc[Renamed from chrome/browser/metrics/leak_detector_controller_unittest.cc - diff]
• chrome/chrome_browser.gypi[diff]
• chrome/chrome_tests_unit.gypi[diff]

Mark fast/js/regress/to-int32-boolean.html as timeout

This test seems to be timed out in multiple bots

BUG=618399
NOTRY=true
[email protected], [email protected]

Review-Url: https://codereview.chromium.org/2054493002
Cr-Commit-Position: refs/heads/master@{#398647}

• third_party/WebKit/LayoutTests/TestExpectations[diff]

Revert of Mojo: Eliminate duplicate C API symbols (patchset #1 id:1 of https://codereview.chromium.org/2048053003/ )

Reason for revert:
trying to revert again, and then clobber. seemed to have fixed the error locally

Original issue's description:
> Reland of Mojo: Eliminate duplicate C API symbols (patchset #1 id:1 of https://codereview.chromium.org/2047813003/ )
>
> Reason for revert:
> Reverting this didn't fix the failure after all
>
> Original issue's description:
> > Revert of Mojo: Eliminate duplicate C API symbols (patchset #3 id:80001 of https://codereview.chromium.org/2044023004/ )
> >
> > Reason for revert:
> > tentative revert due to linker errors. this doesn't make sense. nothing makes sense. RIP sanity
> >
> > Original issue's description:
> > > Mojo: Eliminate duplicate C API symbols
> > >
> > > This gets rid of the duplicate definitions of public
> > > Mojo C API symbols, instead making mojo/public/c/system
> > > the singular source of these definitions. This allows
> > > targets to be linked against mojo/public libraries
> > > without requiring any additional EDK dependencies at
> > > build time.
> > >
> > > To facilitate this the EDK uses the same set of API
> > > thunks as the public library, and the public library
> > > exposes a special embedder API (distinct from the DSO
> > > MojoSetSystemThunks call used by the native app loader)
> > > for setting these thunks. The MojoSetSystemThunks API
> > > is moved to a module which is only linked directly into
> > > app DSOs.
> > >
> > > Because mojo/public/c/system and mojo/edk/system are
> > > now both components, and because there are no longer
> > > redundant exports between the two, it's now safe for
> > > targets to depend on either one or both of them as
> > > needed without encountering duplicate definitions.
> > >
> > > Also some opportunistic cleanup of Mojo build rules.
> > >
> > > BUG=612500
> > >
> > > Committed: https://crrev.com/464e306a82ead839f02f1930a684400228b72495
> > > Cr-Commit-Position: refs/heads/master@{#398605}
> >
> > [email protected]
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=612500
> >
> > Committed: https://crrev.com/e8e242161954d86ce8e064f8a6515bc818d5b600
> > Cr-Commit-Position: refs/heads/master@{#398614}
>
> [email protected]
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=612500
>
> Committed: https://crrev.com/26ec28cabbdcc430483d5f4996b3afaafb1ba1bf
> Cr-Commit-Position: refs/heads/master@{#398622}

[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=612500

Review-Url: https://codereview.chromium.org/2054513002
Cr-Commit-Position: refs/heads/master@{#398646}

• mojo/edk/embedder/BUILD.gn[diff]
• mojo/edk/embedder/embedder.cc[diff]
• mojo/edk/embedder/entrypoints.cc[diff]
• mojo/edk/embedder/entrypoints.h[Deleted - diff]
• mojo/edk/system/BUILD.gn[diff]
• mojo/mojo.gyp[diff]
• mojo/mojo_base.gyp[diff]
• mojo/mojo_edk.gyp[diff]
• mojo/mojo_edk_nacl.gyp[diff]
• mojo/mojo_edk_tests.gyp[diff]
• mojo/mojo_public.gyp[diff]
• mojo/mojo_variables.gypi[Added - diff]
• mojo/public/c/system/BUILD.gn[diff]
• mojo/public/c/system/set_thunks_for_app.cc[Deleted - diff]
• mojo/public/c/system/system_export.h[diff]
• mojo/public/mojo_application.gni[diff]
• mojo/public/platform/native/BUILD.gn[Added - diff]
• mojo/public/platform/native/system_thunks.cc[Renamed from mojo/public/c/system/thunks.cc - diff]
• mojo/public/platform/native/system_thunks.h[Renamed from mojo/public/c/system/thunks.h - diff]
• mojo/public/platform/native/thunk_export.h[Added - diff]
• services/shell/runner/host/BUILD.gn[diff]
• services/shell/runner/host/native_application_support.cc[diff]
• ui/gfx/gfx.gyp[diff]

Rebaseline fast/images/color-profile-reflection.html

BUG=618402
NOTRY=true
[email protected]

Review-Url: https://codereview.chromium.org/2051553002
Cr-Commit-Position: refs/heads/master@{#398645}

• third_party/WebKit/LayoutTests/TestExpectations[diff]

ios: Default GYP_GENERATORS to ninja, and error out if GYP_GENERATORS is xcode

m51 has shipped to stable, so nothing / nobody should still use
Xcode for building chrome/ios.

BUG=336575

Review-Url: https://codereview.chromium.org/2050793002
Cr-Commit-Position: refs/heads/master@{#398644}

• build/gyp_chromium.py[diff]
• build/gyp_environment.py[diff]

Disable page_cycler(_v2).typical_25 on Nexus 5X WebView

[email protected]
BUG=616781
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq

Review-Url: https://codereview.chromium.org/2042323002
Cr-Commit-Position: refs/heads/master@{#398643}

• tools/perf/benchmarks/page_cycler.py[diff]
• tools/perf/benchmarks/page_cycler_v2.py[diff]

Add UMA for rtcp-mux policy.

Add a function to WebRTCPeerConnectionHandler to log the selected Rtcp-mux policy.
Rtcp-mux policy is recorded after parsing the RTCConfiguration when creating the RTCPeerConnection object.

Review-Url: https://codereview.chromium.org/2013573005
Cr-Commit-Position: refs/heads/master@{#398642}

• components/test_runner/mock_webrtc_peer_connection_handler.cc[diff]
• components/test_runner/mock_webrtc_peer_connection_handler.h[diff]
• content/renderer/media/rtc_peer_connection_handler.cc[diff]
• content/renderer/media/rtc_peer_connection_handler.h[diff]
• third_party/WebKit/Source/modules/mediastream/RTCPeerConnection.cpp[diff]
• third_party/WebKit/public/platform/WebRTCPeerConnectionHandler.h[diff]
• tools/metrics/histograms/histograms.xml[diff]

Mark printing/thead-repeats-at-top-of-each-page.html as failure

printing/thead-repeats-at-top-of-each-page.html
virtual/threaded/printing/thead-repeats-at-top-of-each-page.html

fails on Win10 even after rebaseline

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10/builds/11102

BUG=618082
NOTRY=true
[email protected]

Review-Url: https://codereview.chromium.org/2045013006
Cr-Commit-Position: refs/heads/master@{#398641}

• third_party/WebKit/LayoutTests/TestExpectations[diff]

IntersectionObserver: throw exceptions as spec mandates.

BUG=617396
[email protected],[email protected]

Review-Url: https://codereview.chromium.org/2046973003
Cr-Commit-Position: refs/heads/master@{#398640}

• third_party/WebKit/LayoutTests/intersection-observer/observer-exceptions-expected.txt[diff]
• third_party/WebKit/LayoutTests/intersection-observer/observer-exceptions.html[diff]
• third_party/WebKit/Source/core/dom/IntersectionObserver.cpp[diff]
• third_party/WebKit/Source/core/dom/IntersectionObserver.h[diff]
• third_party/WebKit/Source/core/dom/IntersectionObserver.idl[diff]
• third_party/WebKit/Source/web/tests/IntersectionObserverTest.cpp[diff]

Add 64-bit bisect bot configs.

Renaming some bots in https://codereview.chromium.org/1972023002/ .

BUG=611484

Review-Url: https://codereview.chromium.org/2046823002
Cr-Commit-Position: refs/heads/master@{#398639}

• tools/mb/mb_config.pyl[diff]
• tools/perf/core/trybot_command.py[diff]

Update WebGL2 conformanc test expectations for Linux NVidia with ANGLE

BUG=618408
TEST=linux gpu bot
[email protected]
NOTRY=true
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2055493002
Cr-Commit-Position: refs/heads/master@{#398638}

• content/test/gpu/gpu_tests/webgl2_conformance_expectations.py[diff]

Reland of cc : Make LayerImpl destruction independent of tree hierarchy (patchset #1 id:1 of https://codereview.chromium.org/2044253004/ )

Reason for revert:
around and around we go :D

Original issue's description:
> Revert of cc : Make LayerImpl destruction independent of tree hierarchy (patchset #2 id:20001 of https://codereview.chromium.org/2043963005/ )
>
> Reason for revert:
> Didn't fix anything.
>
> Original issue's description:
> > cc : Make LayerImpl destruction independent of tree hierarchy
> >
> > Instead of deleting all layers when the root is deleted, we delete
> > all layers when the layer tree impl is shutdown.
> >
> > BUG=617791
> > CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> >
> > Committed: https://crrev.com/e536d2292fa548ee6231ff23ac65203c261e4a14
> > Cr-Commit-Position: refs/heads/master@{#398606}
>
> [email protected],[email protected],[email protected]
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=617791
>
> Committed: https://crrev.com/ef1625ae9feafee9ddaa2a58b9476175d7956b8b
> Cr-Commit-Position: refs/heads/master@{#398633}

[email protected],[email protected],[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=617791

Review-Url: https://codereview.chromium.org/2049883002
Cr-Commit-Position: refs/heads/master@{#398637}

• cc/layers/layer_impl.cc[diff]
• cc/layers/layer_impl.h[diff]
• cc/layers/picture_layer_impl_perftest.cc[diff]
• cc/layers/picture_layer_impl_unittest.cc[diff]
• cc/trees/damage_tracker_unittest.cc[diff]
• cc/trees/layer_tree_host_common_unittest.cc[diff]
• cc/trees/layer_tree_host_impl.cc[diff]
• cc/trees/layer_tree_host_impl_unittest.cc[diff]
• cc/trees/layer_tree_impl.cc[diff]
• cc/trees/layer_tree_impl.h[diff]
• cc/trees/tree_synchronizer.cc[diff]
• cc/trees/tree_synchronizer_unittest.cc[diff]

Remove SkDevice, obsolete constructor

We already have all the API we need to rewrite skia_utils_mac without
referring to the to-be-deprecated SkBaseDevice or its
BitmapPlatformDevice subclasses. Also get rid of a constructor that
was marked "TODO: delete this" and all of the expensive code behind
it which was *only* activated by unit tests.

[email protected]
BUG=543755

Review-Url: https://codereview.chromium.org/2001653002
Cr-Commit-Position: refs/heads/master@{#398636}

• skia/ext/skia_utils_mac.h[diff]
• skia/ext/skia_utils_mac.mm[diff]
• skia/ext/skia_utils_mac_unittest.mm[diff]

We did not set the pointer type for WebMouseEvent, which is created from NSEvent in
WebMouseEventBuilder::Build. Now we set the pointer type based on the NSEvent's
type to see if it is a tablet event or subtype of a mouse event to see its input device.

Reference:
https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/EventOverview/EventObjectsTypes/EventObjectsTypes.html#//apple_ref/doc/uid/10000060i-CH4-SW4

BUG=615122

Review-Url: https://codereview.chromium.org/2022843002
Cr-Commit-Position: refs/heads/master@{#398635}

• content/browser/renderer_host/input/web_input_event_builders_mac.mm[diff]

Add ArrayTraits<CArray<T>> utility template

This CL adds a utility ArrayTraits class to read and write from and into a c array from StructTraits.

This CL also deletes duplicate code across various traits.

BUG=611802
[email protected], [email protected]
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2046563005
Cr-Commit-Position: refs/heads/master@{#398634}

• gpu/ipc/common/mailbox_struct_traits.cc[diff]
• gpu/ipc/common/mailbox_struct_traits.h[diff]
• mojo/mojo_public.gyp[diff]
• mojo/public/cpp/bindings/BUILD.gn[diff]
• mojo/public/cpp/bindings/array_traits_carray.h[Added - diff]
• mojo/public/cpp/bindings/lib/serialization.h[diff]
• skia/public/interfaces/bitmap_skbitmap_struct_traits.cc[diff]
• skia/public/interfaces/bitmap_skbitmap_struct_traits.h[diff]
• ui/events/mojo/latency_info_struct_traits.cc[diff]
• ui/events/mojo/latency_info_struct_traits.h[diff]

Revert of cc : Make LayerImpl destruction independent of tree hierarchy (patchset #2 id:20001 of https://codereview.chromium.org/2043963005/ )

Reason for revert:
Didn't fix anything.

Original issue's description:
> cc : Make LayerImpl destruction independent of tree hierarchy
>
> Instead of deleting all layers when the root is deleted, we delete
> all layers when the layer tree impl is shutdown.
>
> BUG=617791
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/e536d2292fa548ee6231ff23ac65203c261e4a14
> Cr-Commit-Position: refs/heads/master@{#398606}

[email protected],[email protected],[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=617791

Review-Url: https://codereview.chromium.org/2044253004
Cr-Commit-Position: refs/heads/master@{#398633}

• cc/layers/layer_impl.cc[diff]
• cc/layers/layer_impl.h[diff]
• cc/layers/picture_layer_impl_perftest.cc[diff]
• cc/layers/picture_layer_impl_unittest.cc[diff]
• cc/trees/damage_tracker_unittest.cc[diff]
• cc/trees/layer_tree_host_common_unittest.cc[diff]
• cc/trees/layer_tree_host_impl.cc[diff]
• cc/trees/layer_tree_host_impl_unittest.cc[diff]
• cc/trees/layer_tree_impl.cc[diff]
• cc/trees/layer_tree_impl.h[diff]
• cc/trees/tree_synchronizer.cc[diff]
• cc/trees/tree_synchronizer_unittest.cc[diff]

Reland of cc : Make LayerImpl destruction independent of tree hierarchy (patchset #1 id:1 of https://codereview.chromium.org/2054483002/ )

Reason for revert:
Didn't fix anything

Original issue's description:
> Revert of cc : Make LayerImpl destruction independent of tree hierarchy (patchset #2 id:20001 of https://codereview.chromium.org/2043963005/ )
>
> Reason for revert:
> Tentative revert for strange linker failures:
>
> https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac%20Builder%20%28dbg%29/builds/156821
>
> I already tried reverting the other CL in the blame list (mine) with no success. Doesn't make sense that this CL would cause it either, but just ruling it out. I'll revert the revert if the builder stays red.
>
> Original issue's description:
> > cc : Make LayerImpl destruction independent of tree hierarchy
> >
> > Instead of deleting all layers when the root is deleted, we delete
> > all layers when the layer tree impl is shutdown.
> >
> > BUG=617791
> > CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> >
> > Committed: https://crrev.com/e536d2292fa548ee6231ff23ac65203c261e4a14
> > Cr-Commit-Position: refs/heads/master@{#398606}
>
> [email protected],[email protected],[email protected]
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=617791
>
> Committed: https://crrev.com/f3a9e5397cce794930f73f6c6a43c0621ecf06a6
> Cr-Commit-Position: refs/heads/master@{#398623}

[email protected],[email protected],[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=617791

Review-Url: https://codereview.chromium.org/2048863003
Cr-Commit-Position: refs/heads/master@{#398632}

• cc/layers/layer_impl.cc[diff]
• cc/layers/layer_impl.h[diff]
• cc/layers/picture_layer_impl_perftest.cc[diff]
• cc/layers/picture_layer_impl_unittest.cc[diff]
• cc/trees/damage_tracker_unittest.cc[diff]
• cc/trees/layer_tree_host_common_unittest.cc[diff]
• cc/trees/layer_tree_host_impl.cc[diff]
• cc/trees/layer_tree_host_impl_unittest.cc[diff]
• cc/trees/layer_tree_impl.cc[diff]
• cc/trees/layer_tree_impl.h[diff]
• cc/trees/tree_synchronizer.cc[diff]
• cc/trees/tree_synchronizer_unittest.cc[diff]

blacklist some devices with buggy MediaCodec support

Some old Samsung devices have buggy support for MediaCodec, better fallback to MediaPlayer for them.

BUG=615872
[email protected],[email protected]
TEST=Play several youtube videos sequentially in the same tab on Samsung Galaxy S3 phone

Review-Url: https://codereview.chromium.org/2028433002
Cr-Commit-Position: refs/heads/master@{#398631}

• media/base/android/media_codec_util.cc[diff]

[tools/perf] Delete unused variable

BUG=475714

Review-Url: https://codereview.chromium.org/1914233003
Cr-Commit-Position: refs/heads/master@{#398630}

• tools/perf/benchmarks/blink_perf.py[diff]
• tools/perf/benchmarks/dummy_benchmark.py[diff]
• tools/perf/benchmarks/jetstream.py[diff]
• tools/perf/benchmarks/media.py[diff]
• tools/perf/benchmarks/oortonline.py[diff]
• tools/perf/benchmarks/service_worker.py[diff]
• tools/perf/benchmarks/spaceport.py[diff]
• tools/perf/measurements/draw_properties.py[diff]
• tools/perf/measurements/image_decoding.py[diff]
• tools/perf/measurements/media.py[diff]
• tools/perf/measurements/memory.py[diff]
• tools/perf/measurements/oilpan_gc_times.py[diff]
• tools/perf/measurements/page_cycler.py[diff]
• tools/perf/measurements/power.py[diff]
• tools/perf/measurements/rasterize_and_record_micro.py[diff]
• tools/perf/measurements/startup.py[diff]
• tools/perf/measurements/tab_switching.py[diff]
• tools/perf/measurements/task_execution_time.py[diff]
• tools/perf/measurements/thread_times.py[diff]
• tools/perf/measurements/v8_detached_context_age_in_gc.py[diff]
• tools/perf/measurements/v8_gc_times.py[diff]
• tools/perf/measurements/webrtc.py[diff]

Enable GpuRasterization.ConcavePaths on Nexus devices

Enable GpuRasterization.ConcavePaths on Nexus devices.

The test inspects browser screenshot pixels.

In order to take a screenshot, device pixel ratio is used
to scale the screenshot rect.

In order to correlate logical pixels to physical pixels with device
pixel ratio, viewport scale needs to be set to 1.

BUG=617805
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2045993004
Cr-Commit-Position: refs/heads/master@{#398629}

• content/test/data/gpu/concave_paths.html[diff]
• content/test/gpu/gpu_tests/gpu_rasterization_expectations.py[diff]

Fix bug in EOF handling during payload reads and writes.

Fix bug in EOF handling during payload reads and writes.
Add new regression tests to exercise that functionality.
Factor out socket read/write calls into DoRead()/DoWrite(), which manage the mapping from the zero EOF return value to net::ERR_CONNECTION_CLOSED.

[email protected]
BUG=617729

Review-Url: https://codereview.chromium.org/2040263002
Cr-Commit-Position: refs/heads/master@{#398628}

• blimp/net/stream_packet_reader.cc[diff]
• blimp/net/stream_packet_reader.h[diff]
• blimp/net/stream_packet_reader_unittest.cc[diff]
• blimp/net/stream_packet_writer.cc[diff]
• blimp/net/stream_packet_writer.h[diff]
• blimp/net/stream_packet_writer_unittest.cc[diff]

Check for registry and extension existence before web_accessible_resources.

BUG=618300

Review-Url: https://codereview.chromium.org/2048693004
Cr-Commit-Position: refs/heads/master@{#398627}

• extensions/browser/extension_navigation_throttle.cc[diff]

Roll src/third_party/skia/ 9a6cf800b..7942f22c6 (3 commits).

https://chromium.googlesource.com/skia.git/+log/9a6cf800bc39..7942f22c607c

$ git log 9a6cf800b..7942f22c6 --date=short --no-merges --format='%ad %ae %s'
2016-06-08 csmartdalton Fix instances where hwaa was wrong for mixed samples
2016-06-08 martina.kollarova Document dm error about missing skps
2016-06-08 herb Handle single pixel case on tiled.

BUG=None

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
[email protected]

Review-Url: https://codereview.chromium.org/2050723002
Cr-Commit-Position: refs/heads/master@{#398626}

• DEPS[diff]

Shift rdsmith@ to watching all of content/browser/loader.

BUG=None

Review-Url: https://codereview.chromium.org/2047423002
Cr-Commit-Position: refs/heads/master@{#398625}

• WATCHLISTS[diff]

Fix SAML single-password scrapping authenticating with an empty password

Fix a JS syntax error that led to authentication with an empty password.
Also, remove setting enterpriseInfo in <offline-gaia> as it doesn't have
this property anymore.

BUG=618279,617962
TEST=manually with login/logout, lock/unlock.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2049103002
Cr-Commit-Position: refs/heads/master@{#398624}

• chrome/browser/chromeos/login/saml/saml_browsertest.cc[diff]
• chrome/browser/resources/gaia_auth_host/authenticator.js[diff]
• ui/login/display_manager.js[diff]

Revert of cc : Make LayerImpl destruction independent of tree hierarchy (patchset #2 id:20001 of https://codereview.chromium.org/2043963005/ )

Reason for revert:
Tentative revert for strange linker failures:

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac%20Builder%20%28dbg%29/builds/156821

I already tried reverting the other CL in the blame list (mine) with no success. Doesn't make sense that this CL would cause it either, but just ruling it out. I'll revert the revert if the builder stays red.

Original issue's description:
> cc : Make LayerImpl destruction independent of tree hierarchy
>
> Instead of deleting all layers when the root is deleted, we delete
> all layers when the layer tree impl is shutdown.
>
> BUG=617791
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/e536d2292fa548ee6231ff23ac65203c261e4a14
> Cr-Commit-Position: refs/heads/master@{#398606}

[email protected],[email protected],[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=617791

Review-Url: https://codereview.chromium.org/2054483002
Cr-Commit-Position: refs/heads/master@{#398623}

• cc/layers/layer_impl.cc[diff]
• cc/layers/layer_impl.h[diff]
• cc/layers/picture_layer_impl_perftest.cc[diff]
• cc/layers/picture_layer_impl_unittest.cc[diff]
• cc/trees/damage_tracker_unittest.cc[diff]
• cc/trees/layer_tree_host_common_unittest.cc[diff]
• cc/trees/layer_tree_host_impl.cc[diff]
• cc/trees/layer_tree_host_impl_unittest.cc[diff]
• cc/trees/layer_tree_impl.cc[diff]
• cc/trees/layer_tree_impl.h[diff]
• cc/trees/tree_synchronizer.cc[diff]
• cc/trees/tree_synchronizer_unittest.cc[diff]

Reland of Mojo: Eliminate duplicate C API symbols (patchset #1 id:1 of https://codereview.chromium.org/2047813003/ )

Reason for revert:
Reverting this didn't fix the failure after all

Original issue's description:
> Revert of Mojo: Eliminate duplicate C API symbols (patchset #3 id:80001 of https://codereview.chromium.org/2044023004/ )
>
> Reason for revert:
> tentative revert due to linker errors. this doesn't make sense. nothing makes sense. RIP sanity
>
> Original issue's description:
> > Mojo: Eliminate duplicate C API symbols
> >
> > This gets rid of the duplicate definitions of public
> > Mojo C API symbols, instead making mojo/public/c/system
> > the singular source of these definitions. This allows
> > targets to be linked against mojo/public libraries
> > without requiring any additional EDK dependencies at
> > build time.
> >
> > To facilitate this the EDK uses the same set of API
> > thunks as the public library, and the public library
> > exposes a special embedder API (distinct from the DSO
> > MojoSetSystemThunks call used by the native app loader)
> > for setting these thunks. The MojoSetSystemThunks API
> > is moved to a module which is only linked directly into
> > app DSOs.
> >
> > Because mojo/public/c/system and mojo/edk/system are
> > now both components, and because there are no longer
> > redundant exports between the two, it's now safe for
> > targets to depend on either one or both of them as
> > needed without encountering duplicate definitions.
> >
> > Also some opportunistic cleanup of Mojo build rules.
> >
> > BUG=612500
> >
> > Committed: https://crrev.com/464e306a82ead839f02f1930a684400228b72495
> > Cr-Commit-Position: refs/heads/master@{#398605}
>
> [email protected]
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=612500
>
> Committed: https://crrev.com/e8e242161954d86ce8e064f8a6515bc818d5b600
> Cr-Commit-Position: refs/heads/master@{#398614}

[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=612500

Review-Url: https://codereview.chromium.org/2048053003
Cr-Commit-Position: refs/heads/master@{#398622}

• mojo/edk/embedder/BUILD.gn[diff]
• mojo/edk/embedder/embedder.cc[diff]
• mojo/edk/embedder/entrypoints.cc[diff]
• mojo/edk/embedder/entrypoints.h[Added - diff]
• mojo/edk/system/BUILD.gn[diff]
• mojo/mojo.gyp[diff]
• mojo/mojo_base.gyp[diff]
• mojo/mojo_edk.gyp[diff]
• mojo/mojo_edk_nacl.gyp[diff]
• mojo/mojo_edk_tests.gyp[diff]
• mojo/mojo_public.gyp[diff]
• mojo/mojo_variables.gypi[Deleted - diff]
• mojo/public/c/system/BUILD.gn[diff]
• mojo/public/c/system/set_thunks_for_app.cc[Added - diff]
• mojo/public/c/system/system_export.h[diff]
• mojo/public/c/system/thunks.cc[Renamed from mojo/public/platform/native/system_thunks.cc - diff]
• mojo/public/c/system/thunks.h[Renamed from mojo/public/platform/native/system_thunks.h - diff]
• mojo/public/mojo_application.gni[diff]
• mojo/public/platform/native/BUILD.gn[Deleted - diff]
• mojo/public/platform/native/thunk_export.h[Deleted - diff]
• services/shell/runner/host/BUILD.gn[diff]
• services/shell/runner/host/native_application_support.cc[diff]
• ui/gfx/gfx.gyp[diff]

[perf] Re-enable system_health.memory_* benchmarks

Hopefully, the perf dashboard infrastructure will now be able to handle
the amount of data generated by the benchmark.

Nevertheless, we throw away all Numeric statistics (standard deviation,
sum, count, max, min and percentiles) except for the average using the
trick from https://codereview.chromium.org/2005233002/. This should
reduce the amount of data pushed to the dashboard approximately 6 times.

BUG=589726,610962,617117,catapult:#2340
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq

Review-Url: https://codereview.chromium.org/2012743002
Cr-Commit-Position: refs/heads/master@{#398621}

• tools/perf/benchmarks/system_health.py[diff]
• tools/perf/page_sets/system_health/loading_stories.py[diff]

[perf] Prepare benchmarks for memory metric normalization

This is patch is necessary so that benchmarks using the memory metric
wouldn't stop reporting values once
https://codereview.chromium.org/2018503002/ lands.

BUG=617117
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq

Review-Url: https://codereview.chromium.org/2044853002
Cr-Commit-Position: refs/heads/master@{#398620}

• tools/perf/benchmarks/memory_infra.py[diff]
• tools/perf/benchmarks/v8.py[diff]

Renaming CreateAdvertisementData() by AdvertisementData().

CreateAdvertisementData() doesn't return an object that should be released
anymore.

BUG=NONE

Review-Url: https://codereview.chromium.org/2041643002
Cr-Commit-Position: refs/heads/master@{#398619}

• device/bluetooth/bluetooth_adapter_mac_unittest.mm[diff]

Remove the default wheel event handler from blink.

Wheel gesture scrolls are used solely now. Remove the
defaultWheelEventHandler code as it is unused.

Also remove the WebSetting for wheel gesture support.

BUG=598798

Review-Url: https://codereview.chromium.org/2049493002
Cr-Commit-Position: refs/heads/master@{#398618}

• content/renderer/render_view_impl.cc[diff]
• third_party/WebKit/Source/core/dom/Node.cpp[diff]
• third_party/WebKit/Source/core/frame/Settings.in[diff]
• third_party/WebKit/Source/core/input/EventHandler.cpp[diff]
• third_party/WebKit/Source/core/input/EventHandler.h[diff]
• third_party/WebKit/Source/web/WebSettingsImpl.cpp[diff]
• third_party/WebKit/Source/web/WebSettingsImpl.h[diff]
• third_party/WebKit/Source/web/WebViewImpl.cpp[diff]
• third_party/WebKit/Source/web/tests/WebFrameTest.cpp[diff]
• third_party/WebKit/public/web/WebSettings.h[diff]

Update V8 to version 5.3.182.

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/ff6dd4f4..af2ed87b

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

[email protected],[email protected],[email protected],[email protected]

Review-Url: https://codereview.chromium.org/2045353003
Cr-Commit-Position: refs/heads/master@{#398617}

• DEPS[diff]

exo: Avoid cross-fade animations for window without a configure callback.

When transitioning in/out of maximized or fullscreen mode we need to
make sure we have a configure callback before we allow the default
cross-fade animations. The configure callback provides a mechanism for
the client to inform us that a frame has taken the state change into
account and without this cross-fade animations are unreliable.

BUG=549781
TEST=exo_unittests

Review-Url: https://codereview.chromium.org/2049623002
Cr-Commit-Position: refs/heads/master@{#398616}

• components/exo/shell_surface.cc[diff]
• components/exo/shell_surface.h[diff]

Add UMA histograms IOSurface creation and binding to a GL texture

There are reports that these calls sometimes hang for tens of
milliseconds. These reports appear to be coming from particular OS and
GPU combinations, using particular features.

These histograms will allow us to test these hypotheses.

BUG=580616

Review-Url: https://codereview.chromium.org/2044683003
Cr-Commit-Position: refs/heads/master@{#398615}

• content/browser/compositor/software_output_device_mac.mm[diff]
• tools/metrics/histograms/histograms.xml[diff]
• ui/gfx/mac/io_surface.cc[diff]
• ui/gl/gl_image_io_surface.mm[diff]

Revert of Mojo: Eliminate duplicate C API symbols (patchset #3 id:80001 of https://codereview.chromium.org/2044023004/ )

Reason for revert:
tentative revert due to linker errors. this doesn't make sense. nothing makes sense. RIP sanity

Original issue's description:
> Mojo: Eliminate duplicate C API symbols
>
> This gets rid of the duplicate definitions of public
> Mojo C API symbols, instead making mojo/public/c/system
> the singular source of these definitions. This allows
> targets to be linked against mojo/public libraries
> without requiring any additional EDK dependencies at
> build time.
>
> To facilitate this the EDK uses the same set of API
> thunks as the public library, and the public library
> exposes a special embedder API (distinct from the DSO
> MojoSetSystemThunks call used by the native app loader)
> for setting these thunks. The MojoSetSystemThunks API
> is moved to a module which is only linked directly into
> app DSOs.
>
> Because mojo/public/c/system and mojo/edk/system are
> now both components, and because there are no longer
> redundant exports between the two, it's now safe for
> targets to depend on either one or both of them as
> needed without encountering duplicate definitions.
>
> Also some opportunistic cleanup of Mojo build rules.
>
> BUG=612500
>
> Committed: https://crrev.com/464e306a82ead839f02f1930a684400228b72495
> Cr-Commit-Position: refs/heads/master@{#398605}

[email protected]
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=612500

Review-Url: https://codereview.chromium.org/2047813003
Cr-Commit-Position: refs/heads/master@{#398614}

• mojo/edk/embedder/BUILD.gn[diff]
• mojo/edk/embedder/embedder.cc[diff]
• mojo/edk/embedder/entrypoints.cc[diff]
• mojo/edk/embedder/entrypoints.h[Deleted - diff]
• mojo/edk/system/BUILD.gn[diff]
• mojo/mojo.gyp[diff]
• mojo/mojo_base.gyp[diff]
• mojo/mojo_edk.gyp[diff]
• mojo/mojo_edk_nacl.gyp[diff]
• mojo/mojo_edk_tests.gyp[diff]
• mojo/mojo_public.gyp[diff]
• mojo/mojo_variables.gypi[Added - diff]
• mojo/public/c/system/BUILD.gn[diff]
• mojo/public/c/system/set_thunks_for_app.cc[Deleted - diff]
• mojo/public/c/system/system_export.h[diff]
• mojo/public/mojo_application.gni[diff]
• mojo/public/platform/native/BUILD.gn[Added - diff]
• mojo/public/platform/native/system_thunks.cc[Renamed from mojo/public/c/system/thunks.cc - diff]
• mojo/public/platform/native/system_thunks.h[Renamed from mojo/public/c/system/thunks.h - diff]
• mojo/public/platform/native/thunk_export.h[Added - diff]
• services/shell/runner/host/BUILD.gn[diff]
• services/shell/runner/host/native_application_support.cc[diff]
• ui/gfx/gfx.gyp[diff]

Remove unused NQE histograms

All these histograms have been replaced by other histograms.

BUG=615929,608587,608585

Review-Url: https://codereview.chromium.org/2036453003
Cr-Commit-Position: refs/heads/master@{#398613}

• net/nqe/network_quality_estimator.cc[diff]
• net/nqe/network_quality_estimator.h[diff]
• net/nqe/network_quality_estimator_unittest.cc[diff]
• tools/metrics/histograms/histograms.xml[diff]

[Cronet] Make StartNetLog and StopNetLog synchronous in Cronet

CronetEngine.StartNetLogToFile and StopNetLog post asynchronous tasks to the
network thread. This results truncated logs if executing those tasks is slow on
the network thread. Since NetLog is a global and thread-safe, this CL creates
and tears down WriteToFileNetLogObserver on the calling Java thread. The access
to WriteToFileNetLogObserver is protected by a lock to ensure thread safety.

BUG=617632

Review-Url: https://codereview.chromium.org/2035313008
Cr-Commit-Position: refs/heads/master@{#398612}

• components/cronet/android/cronet_url_request_context_adapter.cc[diff]
• components/cronet/android/cronet_url_request_context_adapter.h[diff]

sandwich: Fix an assertion failure cause by bad initiator inference

Devtools' inference of request initiatior is not that reliable,
loading to some javascript originated resources marked as
initiated from the parser.

This CL removes an assertion in sandwich_prefetch.py to not
unecessarly fail on some URLs where it happens more often.

BUG=582080

Review-Url: https://codereview.chromium.org/2043413002
Cr-Commit-Position: refs/heads/master@{#398611}

• tools/android/loading/sandwich_prefetch.py[diff]

views/mus: Chaneg how Widget activation works in mus.

Install an aura::client::FocusClient implementation on NativeWidgetMus's aura
window-tree which focuses the mus::Window when the aura::Window is focused.

This change allows running some tests in views/focus/focus_manager_unittest.cc
under mus.

BUG=577274

Review-Url: https://codereview.chromium.org/2049723002
Cr-Commit-Position: refs/heads/master@{#398610}

• ui/views/focus/focus_manager_unittest.cc[diff]
• ui/views/mus/BUILD.gn[diff]
• ui/views/mus/native_widget_mus.cc[diff]
• ui/wm/core/focus_controller.h[diff]

Add option to android test_runner to specify what devices to run on.

BUG=612360

Review-Url: https://codereview.chromium.org/2045583004
Cr-Commit-Position: refs/heads/master@{#398609}

• build/android/pylib/local/device/local_device_environment.py[diff]
• build/android/test_runner.py[diff]

[chromedriver] Drop support for M49 and add 52.

BUG=

Review-Url: https://codereview.chromium.org/2041813005
Cr-Commit-Position: refs/heads/master@{#398608}

• chrome/test/chromedriver/chrome/version.cc[diff]
• chrome/test/chromedriver/test/run_all_tests.py[diff]

[MIPS] Don't use Gold for mipsel Android

Accidentally enabled Gold for mipsel Android in
https://codereview.chromium.org/2027173002/
Fixing it here.

BUG=webrtc:5977

Review-Url: https://codereview.chromium.org/2048063002
Cr-Commit-Position: refs/heads/master@{#398607}

• build/common.gypi[diff]

cc : Make LayerImpl destruction independent of tree hierarchy

Instead of deleting all layers when the root is deleted, we delete
all layers when the layer tree impl is shutdown.

BUG=617791
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2043963005
Cr-Commit-Position: refs/heads/master@{#398606}

• cc/layers/layer_impl.cc[diff]
• cc/layers/layer_impl.h[diff]
• cc/layers/picture_layer_impl_perftest.cc[diff]
• cc/layers/picture_layer_impl_unittest.cc[diff]
• cc/trees/damage_tracker_unittest.cc[diff]
• cc/trees/layer_tree_host_common_unittest.cc[diff]
• cc/trees/layer_tree_host_impl.cc[diff]
• cc/trees/layer_tree_host_impl_unittest.cc[diff]
• cc/trees/layer_tree_impl.cc[diff]
• cc/trees/layer_tree_impl.h[diff]
• cc/trees/tree_synchronizer.cc[diff]
• cc/trees/tree_synchronizer_unittest.cc[diff]

Mojo: Eliminate duplicate C API symbols

This gets rid of the duplicate definitions of public
Mojo C API symbols, instead making mojo/public/c/system
the singular source of these definitions. This allows
targets to be linked against mojo/public libraries
without requiring any additional EDK dependencies at
build time.

To facilitate this the EDK uses the same set of API
thunks as the public library, and the public library
exposes a special embedder API (distinct from the DSO
MojoSetSystemThunks call used by the native app loader)
for setting these thunks. The MojoSetSystemThunks API
is moved to a module which is only linked directly into
app DSOs.

Because mojo/public/c/system and mojo/edk/system are
now both components, and because there are no longer
redundant exports between the two, it's now safe for
targets to depend on either one or both of them as
needed without encountering duplicate definitions.

Also some opportunistic cleanup of Mojo build rules.

BUG=612500

Review-Url: https://codereview.chromium.org/2044023004
Cr-Commit-Position: refs/heads/master@{#398605}

• mojo/edk/embedder/BUILD.gn[diff]
• mojo/edk/embedder/embedder.cc[diff]
• mojo/edk/embedder/entrypoints.cc[diff]
• mojo/edk/embedder/entrypoints.h[Added - diff]
• mojo/edk/system/BUILD.gn[diff]
• mojo/mojo.gyp[diff]
• mojo/mojo_base.gyp[diff]
• mojo/mojo_edk.gyp[diff]
• mojo/mojo_edk_nacl.gyp[diff]
• mojo/mojo_edk_tests.gyp[diff]
• mojo/mojo_public.gyp[diff]
• mojo/mojo_variables.gypi[Deleted - diff]
• mojo/public/c/system/BUILD.gn[diff]
• mojo/public/c/system/set_thunks_for_app.cc[Added - diff]
• mojo/public/c/system/system_export.h[diff]
• mojo/public/c/system/thunks.cc[Renamed from mojo/public/platform/native/system_thunks.cc - diff]
• mojo/public/c/system/thunks.h[Renamed from mojo/public/platform/native/system_thunks.h - diff]
• mojo/public/mojo_application.gni[diff]
• mojo/public/platform/native/BUILD.gn[Deleted - diff]
• mojo/public/platform/native/thunk_export.h[Deleted - diff]
• services/shell/runner/host/BUILD.gn[diff]
• services/shell/runner/host/native_application_support.cc[diff]
• ui/gfx/gfx.gyp[diff]

Auto-rebaseline for r398556

https://chromium.googlesource.com/chromium/src/+/2feb7b70f

BUG=436475
[email protected]

Review URL: https://codereview.chromium.org/2045393002 .

Cr-Commit-Position: refs/heads/master@{#398604}

• third_party/WebKit/LayoutTests/TestExpectations[diff]
• third_party/WebKit/LayoutTests/platform/android/compositing/reflections/load-video-in-reflection-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/android/compositing/reflections/nested-reflection-anchor-point-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/android/compositing/reflections/nested-reflection-opacity-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/android/compositing/reflections/nested-reflection-transformed-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/android/compositing/reflections/nested-reflection-transformed2-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/android/compositing/reflections/reflection-in-composited-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/android/compositing/reflections/reflection-on-composited-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/android/compositing/reflections/reflection-opacity-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/android/fast/reflections/opacity-reflection-transform-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/android/fast/reflections/reflection-overflow-hidden-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/android/inspector/layers/layer-compositing-reasons-expected.txt[Renamed from third_party/WebKit/LayoutTests/inspector/layers/layer-compositing-reasons-expected.txt - diff]
• third_party/WebKit/LayoutTests/platform/linux-precise/fast/images/color-profile-reflection-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/animation-inside-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/load-video-in-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/masked-reflection-on-composited-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/nested-reflection-anchor-point-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/nested-reflection-animated-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/nested-reflection-mask-change-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/nested-reflection-opacity-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/nested-reflection-transformed-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/nested-reflection-transformed2-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/reflection-in-composited-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/reflection-on-composited-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/reflection-opacity-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/css3/filters/composited-reflected-expected.png[Deleted - diff]
• third_party/WebKit/LayoutTests/platform/linux/fast/css/transformed-mask-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/fast/images/color-profile-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/fast/reflections/opacity-reflection-transform-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/fast/reflections/reflection-overflow-hidden-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/inspector/layers/layer-compositing-reasons-expected.txt[Added - diff]
• third_party/WebKit/LayoutTests/platform/linux/media/video-layer-crash-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/linux/virtual/gpu-rasterization/fast/images/color-profile-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac-retina/inspector/layers/layer-compositing-reasons-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/mac-retina/inspector/layers/layer-compositing-reasons-expected.txt[Added - diff]
• third_party/WebKit/LayoutTests/platform/mac-retina/media/video-layer-crash-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/mac-retina/media/video-layer-crash-expected.txt[Added - diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/animation-inside-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/load-video-in-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/masked-reflection-on-composited-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/nested-reflection-anchor-point-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/nested-reflection-animated-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/nested-reflection-mask-change-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/nested-reflection-transformed-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/nested-reflection-transformed2-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/reflection-in-composited-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/reflection-on-composited-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/reflection-opacity-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/css3/blending/effect-background-blend-mode-stacking-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/css3/filters/composited-reflected-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/fast/css/transformed-mask-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/fast/images/color-profile-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/fast/reflections/opacity-reflection-transform-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/fast/reflections/reflection-masks-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/fast/reflections/reflection-masks-opacity-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/fast/reflections/reflection-overflow-hidden-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/fast/reflections/reflection-with-zoom-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/inspector/layers/layer-compositing-reasons-expected.txt[Added - diff]
• third_party/WebKit/LayoutTests/platform/mac/media/video-layer-crash-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/mac/virtual/gpu-rasterization/fast/images/color-profile-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/animation-inside-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/load-video-in-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/masked-reflection-on-composited-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/nested-reflection-anchor-point-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/nested-reflection-animated-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/nested-reflection-mask-change-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/nested-reflection-transformed-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/nested-reflection-transformed2-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/reflection-in-composited-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/reflection-on-composited-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/compositing/reflections/reflection-opacity-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/css3/blending/effect-background-blend-mode-stacking-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/css3/filters/composited-reflected-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/fast/css/transformed-mask-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/fast/images/color-profile-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/fast/reflections/opacity-reflection-transform-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/fast/reflections/reflection-masks-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/fast/reflections/reflection-masks-opacity-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/fast/reflections/reflection-overflow-hidden-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/fast/reflections/reflection-with-zoom-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/inspector/layers/layer-compositing-reasons-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/win/inspector/layers/layer-compositing-reasons-expected.txt[Added - diff]
• third_party/WebKit/LayoutTests/platform/win/media/video-layer-crash-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win/media/video-layer-crash-expected.txt[diff]
• third_party/WebKit/LayoutTests/platform/win/virtual/gpu-rasterization/fast/images/color-profile-reflection-expected.png[diff]
• third_party/WebKit/LayoutTests/platform/win7/inspector/layers/layer-compositing-reasons-expected.txt[Added - diff]
• third_party/WebKit/LayoutTests/platform/win7/media/video-layer-crash-expected.png[Added - diff]
• third_party/WebKit/LayoutTests/platform/win7/media/video-layer-crash-expected.txt[Added - diff]

Moves ShellObserver to ash/common

BUG=581868
TEST=none
[email protected], [email protected], [email protected]
[email protected], [email protected], [email protected]

Review-Url: https://codereview.chromium.org/2047373002
Cr-Commit-Position: refs/heads/master@{#398603}

• ash/accelerators/accelerator_interactive_uitest_chromeos.cc[diff]
• ash/app_list/app_list_presenter_delegate.h[diff]
• ash/aura/wm_root_window_controller_aura.h[diff]
• ash/aura/wm_shell_aura.h[diff]
• ash/common/shell_observer.h[Renamed from ash/shell_observer.h - diff]
• ash/desktop_background/desktop_background_controller.h[diff]
• ash/display/display_change_observer_chromeos.h[diff]
• ash/display/projecting_observer_chromeos.h[diff]
• ash/display/screen_orientation_controller_chromeos.h[diff]
• ash/frame/custom_frame_view_ash.cc[diff]
• ash/high_contrast/high_contrast_controller.h[diff]
• ash/root_window_controller.h[diff]
• ash/shelf/shelf_layout_manager.h[diff]
• ash/shelf/shelf_locking_manager.h[diff]
• ash/system/cast/tray_cast.h[diff]
• ash/system/chromeos/brightness/tray_brightness.cc[diff]
• ash/system/chromeos/rotation/tray_rotation_lock.h[diff]
• ash/system/chromeos/screen_security/screen_capture_tray_item.h[diff]
• ash/system/chromeos/session/logout_confirmation_controller.h[diff]
• ash/system/overview/overview_button_tray.h[diff]
• ash/system/toast/toast_manager.h[diff]
• ash/system/tray_accessibility.h[diff]
• ash/system/web_notification/ash_popup_alignment_delegate.h[diff]
• ash/utility/screenshot_controller.h[diff]
• ash/virtual_keyboard_controller.h[diff]
• ash/wm/lock_state_controller.h[diff]
• ash/wm/maximize_mode/maximize_mode_controller.h[diff]
• ash/wm/maximize_mode/maximize_mode_window_manager.h[diff]
• ash/wm/overlay_event_filter.h[diff]
• ash/wm/panels/attached_panel_window_targeter.h[diff]
• ash/wm/screen_dimmer.h[diff]
• ash/wm/video_detector.h[diff]
• ash/wm/workspace/workspace_layout_manager_unittest.cc[diff]
• chrome/browser/chromeos/accessibility/accessibility_manager.h[diff]
• chrome/browser/chromeos/policy/display_rotation_default_handler.h[diff]
• chrome/browser/chromeos/preferences.h[diff]
• chrome/browser/notifications/login_state_notification_blocker_chromeos.h[diff]
• chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.h[diff]
• chrome/browser/ui/webui/chromeos/touch_view_controller_delegate.h[diff]
• components/arc/window_manager/arc_window_manager_bridge.h[diff]
• components/exo/wayland/server.cc[diff]

In DevicePermissionsDialogView disable Connect button if no device selected

When no device selected in the dialog, the "Connect" button needs to be
disabled.

BUG=613596

Review-Url: https://codereview.chromium.org/2045093002
Cr-Commit-Position: refs/heads/master@{#398602}

• chrome/browser/ui/views/extensions/device_permissions_dialog_view.cc[diff]
• chrome/browser/ui/views/extensions/device_permissions_dialog_view.h[diff]

Reland #2: Move side-loaded test data /sdcard -> /sdcard/gtestdata

Reason for reland:
- content_browsertests and component_browsertests updated to
clear the correct private data directory between test runs.

Uses delete_device_stale=True when pushing. This will prevent tests
mistakenly passing when they depend on stale data files from a previous test.

[email protected],[email protected],[email protected],[email protected]
BUG=607169,617213,616155

Review-Url: https://codereview.chromium.org/2043803003
Cr-Commit-Position: refs/heads/master@{#398601}

• android_webview/tools/system_webview_shell/layout_tests/src/org/chromium/webview_shell/test/WebViewLayoutTest.java[diff]
• android_webview/tools/system_webview_shell/test/data/run_tests.sh[diff]
• base/base_paths_android.cc[diff]
• base/test/android/javatests/src/org/chromium/base/test/util/UrlUtils.java[diff]
• base/test/test_support_android.cc[diff]
• base/test/test_support_android.h[diff]
• build/android/pylib/local/device/local_device_gtest_run.py[diff]
• build/android/pylib/local/device/local_device_instrumentation_test_run.py[diff]
• build/android/pylib/remote/device/remote_device_test_run.py[diff]
• build/apk_test.gypi[diff]
• chrome/common/chrome_paths.cc[diff]
• components/BUILD.gn[diff]
• components/cronet/android/BUILD.gn[diff]
• components/cronet/android/test/mock_cert_verifier.cc[diff]
• components/cronet/android/test/native_test_server.cc[diff]
• components/cronet/android/test/quic_test_server.cc[diff]
• components/cronet/android/test/src/org/chromium/net/MockCertVerifier.java[diff]
• components/cronet/android/test/src/org/chromium/net/NativeTestServer.java[diff]
• components/cronet/android/test/src/org/chromium/net/QuicTestServer.java[diff]
• components/test/android/browsertests_apk/src/org/chromium/components_browsertests_apk/ComponentsBrowserTestsActivity.java[diff]
• content/shell/android/browsertests_apk/src/org/chromium/content_browsertests_apk/ContentBrowserTestsActivity.java[diff]
• content/test/BUILD.gn[diff]
• net/test/android/javatests/src/org/chromium/net/test/EmbeddedTestServer.java[diff]
• net/test/android/javatests/src/org/chromium/net/test/EmbeddedTestServerImpl.java[diff]
• net/test/android/javatests/src/org/chromium/net/test/util/CertTestUtil.java[diff]
• net/test/embedded_test_server/android/embedded_test_server_android.cc[diff]
• testing/android/native_test.gyp[diff]
• testing/android/native_test/BUILD.gn[diff]
• testing/android/native_test/java/src/org/chromium/native_test/NativeTest.java[diff]
• testing/android/native_test/native_test_launcher.cc[diff]

IntersectionObserver: use border bounding box for target rect.

BUG=617392
[email protected]

Review-Url: https://codereview.chromium.org/2046113002
Cr-Commit-Position: refs/heads/master@{#398600}

• third_party/WebKit/LayoutTests/intersection-observer/client-rect-expected.txt[Added - diff]
• third_party/WebKit/LayoutTests/intersection-observer/client-rect.html[Added - diff]
• third_party/WebKit/Source/core/dom/IntersectionObservation.cpp[diff]

Moves MruWindowTracker to WmShell

We could also go with a ShellCommon and keep WmShell pure virtual, but
as this code is temporary I'm ok with making WmShell not pure virtual.

BUG=617789
TEST=covered by tests
[email protected]

Review-Url: https://codereview.chromium.org/2041423002
Cr-Commit-Position: refs/heads/master@{#398599}

• ash/ash.gyp[diff]
• ash/aura/wm_shell_aura.cc[diff]
• ash/aura/wm_shell_aura.h[diff]
• ash/common/wm/window_positioner.cc[diff]
• ash/common/wm_shell.h[diff]
• ash/common/wm_shell_common.cc[Added - diff]
• ash/common/wm_shell_common.h[Added - diff]
• ash/mus/bridge/wm_shell_mus.cc[diff]
• ash/mus/bridge/wm_shell_mus.h[diff]
• ash/shell.cc[diff]
• ash/shell.h[diff]
• ash/wm/overview/window_selector.cc[diff]
• ash/wm/overview/window_selector_controller.cc[diff]