GitLab

Now that [SecureContext] works with interfaces, resolve some TODOs and
align the implementation of NavigationPreloadManager with the spec.

Change-Id: I6a8d156182112ee972eb4720e7fa478e784e73bc
Reviewed-on: https://chromium-review.googlesource.com/802620Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Joshua Bell <jsbell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520982}

A connection is now always obtained through one of the navigation:*
interface providers, so it doesn't have to be in this one anymore.

Bug: 595685
Change-Id: I280dfce5d876329baff8f469790ded8b320ae7ca
Reviewed-on: https://chromium-review.googlesource.com/796412Reviewed-by: Noel Gordon <noel@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Commit-Queue: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520981}

Dbus, which we use to communicate with KWallet, requires to be called
on a dedicated thread.

Bug: 782851
Change-Id: I7c55c0c6cec03d3cf688b6f1ad60326e4d5f417f
Reviewed-on: https://chromium-review.googlesource.com/803481
Commit-Queue: Christos Froussios <cfroussios@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Vaclav Brozek <vabr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520980}

Bug: 785394
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ia69c6d950726b4e6a7d5559865102e06ac2db372
Reviewed-on: https://chromium-review.googlesource.com/802623Reviewed-by: edchin <edchin@chromium.org>
Reviewed-by: Mark Cogan <marq@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Commit-Queue: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520979}

https://chromium.googlesource.com/catapult.git/+log/259a1ec4b05d..9252115c4630

$ git log 259a1ec4b..9252115c4 --date=short --no-merges --format='%ad %ae %s'
2017-12-01 simonhatch Dashboard - Limit to 100 tasks per call in /add_histograms
2017-12-01 perezju [Telemetry] Fix error resolving user profile directory

Created with:
  roll-dep src/third_party/catapult


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

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

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


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: I1b3b520b4b740ab258fb40d4a31a73f369b3e4f2
Reviewed-on: https://chromium-review.googlesource.com/803636
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520978}

Login screen UI

BUG=725124

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I374b267d4da09de8c868a32bd866444a4af64f43
Reviewed-on: https://chromium-review.googlesource.com/751667
Commit-Queue: Denis Kuznetsov <antrim@chromium.org>
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520977}

https://chromium.googlesource.com/external/khronosgroup/webgl.git/+log/e4919fa..05591bb

BUG=782317

TEST=bots

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ib9d1d373b8942214cf655d6aa132dddd233229a0
Reviewed-on: https://chromium-review.googlesource.com/802696Reviewed-by: Yunchao He <yunchao.he@intel.com>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Commit-Queue: Zhenyao Mo <zmo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520976}

If base::File was constructed with incorrect arguments such that
there was no disposition then a caller to Window's ::GetLastError()
could retrieve ERROR_SUCCESS, and also base::File::error_details()
would return Error::FILE_OK. This fixes this, and is also consistent
with the POSIX version of base::File.

Bug: None
Change-Id: Ief0e1de98f132cff28c12ab076f9b18deeaa97b3
Reviewed-on: https://chromium-review.googlesource.com/802190
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520975}

From ~@170709363 at google3/.

Bug: 766444
Change-Id: I65d94156f3f946abddbab195fc2228b5fdc032aa
Reviewed-on: https://chromium-review.googlesource.com/719636Reviewed-by: Dimitri Glazkov <dglazkov@chromium.org>
Reviewed-by: Keigo Oka <oka@chromium.org>
Commit-Queue: Mitch Foley <mitchfoley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520974}

The performance gain on an Intel Broadwell CPU is about 8-fold, and on
Intel Skylake and newer CPUs the SSE throughput should be further
doubled.

Bug: 778262
Change-Id: If4682a4d95a6729626049a38a4b1d5b1ebf830bd
Reviewed-on: https://chromium-review.googlesource.com/738243
Commit-Queue: Eero Häkkinen <eero.hakkinen@intel.com>
Reviewed-by: Raymond Toy <rtoy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520973}

Sending video frames with a picture ID allows the WebRTC receiver's
jitter buffer to improve the processing of frame references, instead of
relying on gaps in the packet sequence numbers. This should avoid false
detections of packet loss in the case of missing or corrupt ULPFEC
packets (when the frame could still be decoded).

This is only possible for VPX-encoded frames. For H264, jitter buffer
processing will fall back on using sequence-number gaps.

Bug: 773549
Change-Id: I354473a60249d85a3319ec88f07124420e078b44
Reviewed-on: https://chromium-review.googlesource.com/802455Reviewed-by: Jamie Walch <jamiewalch@chromium.org>
Commit-Queue: Lambros Lambrou <lambroslambrou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520972}

This is a regression test case for https://crbug.com/789975.

BUG=789975
TEST=None

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I73a3fca41b11106e8ebc01d9f003604605432c9d
Reviewed-on: https://chromium-review.googlesource.com/800892Reviewed-by: Mark Cogan <marq@chromium.org>
Commit-Queue: Rohit Rao <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520971}

The finch experiment has been running as a kill-switch for a quarter,
which is not the right way to run this. We should just enable the feature,
and if it needs to be killed only at that point push out an experiment.

Bug: 710720
Change-Id: Id9b0ae2d0d66e8082a98346921a51838e3c6a381
Reviewed-on: https://chromium-review.googlesource.com/803815Reviewed-by: Egor Pasko <pasko@chromium.org>
Commit-Queue: Matthew Cary <mattcary@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520970}

This patch rebaselines the following paint invalidation tests:
paint/invalidation/resize-iframe-text.html
paint/invalidation/scroll/overflow-scroll-body-appear.html
paint/invalidation/svg/absolute-sized-document-no-scrollbars.svg
paint/invalidation/svg/deep-nested-embedded-svg-size-changes-no-layout-triggers-1.html
paint/invalidation/svg/deep-nested-embedded-svg-size-changes-no-layout-triggers-2.html
paint/invalidation/svg/window.svg

Bug: 781419
Change-Id: I1fc2a3cf849d6aef28f5cd58de062f563a51edab
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_root_layer_scrolls
Reviewed-on: https://chromium-review.googlesource.com/797616Reviewed-by: Stefan Zager <szager@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520969}

This flag is private to content shell and should not be included in
//content/public/.

Bug: 
Change-Id: If8d3588b5d7002356f532199738f92c309b2c43b
Reviewed-on: https://chromium-review.googlesource.com/803496Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Commit-Queue: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520968}

ShouldEnableLitePages and ShouldEnableLoFi methods are identical,
and both are checked at the same time, which could be run more times
that they should.

Removed one, and change the method name to ShouldEnableServerPreviews.

Bug: 790733
Change-Id: I9c787b4a0b8919fc250d260638b5b62b067e375c
Reviewed-on: https://chromium-review.googlesource.com/801994Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Doug Arnett <dougarnett@chromium.org>
Commit-Queue: Thanh Le <thanhdle@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520967}

Bug: 766653
Change-Id: Iff945c3b637fc566380b6ab8e2ab8e1c409088ba
Reviewed-on: https://chromium-review.googlesource.com/756196Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Nate Chapin <japhet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520966}

Do this to reduce the likelihood of running out of time before
finishing with the corpus on ClusterFuzz.

TBR=mmoroz@chromium.org

Bug: 791052
Change-Id: I8c91dc0ff3e64dbde9bc6e5b17ef85e456c64911
Reviewed-on: https://chromium-review.googlesource.com/803894Reviewed-by: Jonathan Metzman <metzman@chromium.org>
Reviewed-by: Abhishek Arya <inferno@chromium.org>
Commit-Queue: Jonathan Metzman <metzman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520965}

Change-Id: I295442cc7b3f5168fc69d26bdba17450ff0cc105
Reviewed-on: https://chromium-review.googlesource.com/801987Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Commit-Queue: Pavel Feldman <pfeldman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520964}

Data bound fields need to have the backend value updated before sending
the input event.
Example website:
- nordstrom.com
- walgreens.com

Bug: 788099
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I5b8467a76d15b6ad83061730b3a627f274342c27
Reviewed-on: https://chromium-review.googlesource.com/787433
Commit-Queue: Olivier Robin <olivierrobin@chromium.org>
Reviewed-by: Peter Lee <pkl@chromium.org>
Reviewed-by: Mathieu Perreault <mathp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520963}

Changing names of variables from subclass
to make them different from variables in superclass
(which will remove Prone warnings)

Bug: 
Change-Id: I3591cb6b64062c94e9f84f30dd505d0c31fbf3d2
Reviewed-on: https://chromium-review.googlesource.com/802435Reviewed-by: Theresa <twellington@chromium.org>
Commit-Queue: Marcin Wiącek <marcin@mwiacek.com>
Cr-Commit-Position: refs/heads/master@{#520962}

This reverts commit 52affce6.

Reason for reland: Fixes bug that causes webview to fail

Original change's description:
> Revert "Merges process and package resources script"
>
> This reverts commit ea0f96c8.
>
> Reason for revert:
> Causing resource errors in the real webview (not on cq)
> https://build.chromium.org/p/chromium.android/builders/Android%20WebView%20L%20%28dbg%29/builds/10729
>
> Original change's description:
> > Merges process and package resources script
> >
> > process_resources.py and package_resources.py are currently both required to
> > create the final apk. They currently communicate using zipfiles. This ends up
> > having 3-4 zips/unzips during the final build of the same apk in order to
> > perform modifications. This merger allows all the required processing to be
> > done before apk creation with no need for zipping. It also simplifies the
> > process_resources.py script to separate the code that runs during library
> > builds vs apk builds.
> >
> > Bug: 782316
> > Change-Id: I958c1d107d593b0932275d3c4452c2c51c499b55
> > Reviewed-on: https://chromium-review.googlesource.com/791152
> > Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
> > Reviewed-by: agrieve <agrieve@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#519850}
>
> TBR=agrieve@chromium.org,mheikal@chromium.org
>
> Change-Id: If0d30f7e57b18ff56bd17c2c918398158fb3dffa
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 782316
> Reviewed-on: https://chromium-review.googlesource.com/796717
> Reviewed-by: Bo <boliu@chromium.org>
> Commit-Queue: Bo <boliu@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#520149}

Bug: 782316
Change-Id: I4b6d7650c84706b55bfab7ae6943251342f2e0b3
Reviewed-on: https://chromium-review.googlesource.com/801595
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520961}

The http://crrev.com/449177 removed the last file guarded by these
OWNERS rules a while ago. These seem unlikely to be re-introduced.

Bug: None
Change-Id: I3da1654092f5cfdc705f03e9f8f70b6cc0360e7e
Reviewed-on: https://chromium-review.googlesource.com/803574Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Egor Pasko <pasko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520960}

The cc/scheduler/scheduler_state_machine uses a lot of enums.
enums are not real types and hence have subtle problems.
C++ introduced the concept of "enum class" types which would
be better replacements.

BUG=440759

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I12ce9ee05b668bee5c2f84bbe522edd741d2bd56
Reviewed-on: https://chromium-review.googlesource.com/709094
Commit-Queue: Alexander Timin <altimin@chromium.org>
Reviewed-by: Sami Kyöstilä <skyostil@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520959}

* Add AecDumpMessageFilter::GetOverrideAec3().
* Don't set aec3 in AecDumpMessageFilter::AddDelegate().
* Instead, get the override value before init in MSAP.
* In init, the override value is already honored, although that was always not set before this fix.

Bug: 791016
Change-Id: I0d58ab3dac13321b4a686b6f726c449930b185d5
Reviewed-on: https://chromium-review.googlesource.com/803486Reviewed-by: Max Morin <maxmorin@chromium.org>
Commit-Queue: Henrik Grunell <grunell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520958}

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I557e788d4cb5fe68c7eb5d61c32dbc6d1b017b46
Reviewed-on: https://chromium-review.googlesource.com/777712Reviewed-by: Robert Sesek <rsesek@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Reviewed-by: Stephen White <senorblanco@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520957}

R=aleventhal@chromium.org, dmazzoni@chromium.org
Tested; manually

Bug: 730965
Change-Id: I0819a7e053f665ae5150147fd90b5dcab6ca0994
Reviewed-on: https://chromium-review.googlesource.com/786544Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Nektarios Paisios <nektar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520956}

When the BottomSheet is open, clear the URL. Reset the URL when the
sheet is closed.

Also moves security icon logic from LocationBarLayout to
ToolbarDataProvider and updates tests.

BUG=789775

Change-Id: Iceb934eefa32a71cbb1c3c4ec2b372eded72a237
Reviewed-on: https://chromium-review.googlesource.com/798059
Commit-Queue: Theresa <twellington@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520955}

This single function will make it easier to support more scenario
(events, frameworks...).

Bug: 788099
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ib76aadebbb3c6999b20b5261a8b0045689bd2785
Reviewed-on: https://chromium-review.googlesource.com/786239
Commit-Queue: Olivier Robin <olivierrobin@chromium.org>
Reviewed-by: Peter Lee <pkl@chromium.org>
Reviewed-by: Mathieu Perreault <mathp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520954}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/009150b7afe1..f6b946d71d43

$ git log 009150b7a..f6b946d71 --date=short --no-merges --format='%ad %ae %s'
2017-11-29 achuith Also print key in commit message.

Created with:
  roll-dep src/third_party/depot_tools


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

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

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


TBR=phajdan.jr@chromium.org

Change-Id: I06bee3c72b420aa6925c67f1d7a18417d2657d30
Reviewed-on: https://chromium-review.googlesource.com/803594Reviewed-by: depot-tools-roller . <depot-tools-roller@chromium.org>
Commit-Queue: depot-tools-roller . <depot-tools-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520953}

virtual/threaded/fast/animationworklet/animation-worklet-scroll-timeline.html

is very broken with SPv2.

TBR=majidvp@chromium.org

Bug: 778875
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ic94109ad74c09dfcbc48f4bf44a3ba134fda0918
Reviewed-on: https://chromium-review.googlesource.com/802637Reviewed-by: James MacLean <wjmaclean@chromium.org>
Commit-Queue: James MacLean <wjmaclean@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520952}

The way the current code is set up, selecting a shipping address
will kick off a view update under the spinner screen while waiting
for the merchant to call update with. Not only is this wasteful
(the view will update again on updateWith, as the spinner is hidden)
but it's also a source of use-after-free crashes when updateWith
is called in the middle of the view update because the two updates
race to updating and deleting some view objects.

BUG=788514

Change-Id: I86f48907632d679e7702b3bbf9e09d893b1b8c7a
Reviewed-on: https://chromium-review.googlesource.com/801835Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Reviewed-by: Tommy Martino <tmartino@chromium.org>
Commit-Queue: anthonyvd <anthonyvd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520951}

Bug: 789532
Change-Id: Iad9ed5998dd245df7c8def83cc3123f960a8fdc7
Reviewed-on: https://chromium-review.googlesource.com/802734Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Commit-Queue: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520950}

Import wpt@691ab2189860490becdf2c4813ad62cad62c44ba

Using wpt-import in Chromium a5f4b6fa.

(Backed out WPT_BASE_MANIFEST.json changes.)

No-Export: true
Change-Id: I8db6cc540c633cd3252765c92586af5da328496b
Reviewed-on: https://chromium-review.googlesource.com/793152Reviewed-by: Robert Ma <robertma@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520949}

This reverts commit 1db67406.

Reason for revert: 

NEVER, EVER, EVER!!!! revert one of these chrome-release-bot commits without consulting with TPMs or release team. This can cause major havoc with Chrome releases.

Change-Id: Ib5f4bf96c81e2fe0fac017efa7ee9fa7245a039a
Reviewed-on: https://chromium-review.googlesource.com/803734Reviewed-by: Michael Moss <mmoss@chromium.org>
Commit-Queue: Michael Moss <mmoss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520948}

Bug: 784442
Change-Id: I00bab82f67d2b47f02394831e9ef5126e3fd7a38
Reviewed-on: https://chromium-review.googlesource.com/794133
Commit-Queue: Greg Kraynov <kraynov@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520947}

Some javascript frameworks (e.g. React) or pages have a cache version of input fields in
pages.
This cache will override the autofill value of the fields.
When value is changed, events must be sent to update these cache.
When a user type in a field, the following events are usually sent:
- keydown
- input
- change
- keyup
By testing, most framework use use one of these events, and some framework use
the data field of the event as new value.

Send these events when updating fields.

Bug: 783025, 728631, 732360
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I147a4471ecd2e389cfca57d2aa9d78f61360aa61
Reviewed-on: https://chromium-review.googlesource.com/771674
Commit-Queue: Olivier Robin <olivierrobin@chromium.org>
Reviewed-by: Moe Ahmadi <mahmadi@chromium.org>
Reviewed-by: Peter Lee <pkl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520946}

NetworkContexts can now be created with an optional
initial proxy configuration and/or a pipe to pass
updated proxy configurations over.

Will make chrome/ use this in a followup CL.

Bug: 754007
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Idc92e3cb6e52716d50e1780dd296bef694411c43
Reviewed-on: https://chromium-review.googlesource.com/755797
Commit-Queue: Matt Menke <mmenke@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520945}

I expected this to be a no-op, but broke Docs. C'est la vie.
Manually verified that Docs isn't broken after this patch.

Bug: 790955
Change-Id: Id8290a5c1f826d1f374e6531de8e8433a814b21c
Reviewed-on: https://chromium-review.googlesource.com/803478
Commit-Queue: Mike West <mkwst@chromium.org>
Reviewed-by: Andy Paicu <andypaicu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520944}

- Replace the old BeginNavigation IPC by the equivalent mojo call in
the FrameHost interface. Removed all reference to the old IPC and the
render_frame_host callback.

- Convert BeginNavigationParams to a mojo structure. Most of this patch
is simply modifying function prototypes to match the new mojo syntax.

Bug: 784904
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation
Change-Id: I6f288c1d44c6dd7ab34726689ebf241ae45f73de
Reviewed-on: https://chromium-review.googlesource.com/763529Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Commit-Queue: Arthur Hemery <ahemery@chromium.org>
Cr-Commit-Position: refs/heads/master@{#520943}