PlzNavigate: Fix missing navigation information.

PlzNavigate: Fix missing navigation information.

The navigation type "blink::WebNavigationTypeReload" is now
forwarded to the browser process.

Two more tests pass when browser-side-navigation is enabled.
http/tests/w3c/webperf/approved/navigation-timing/html/test_navigation_type_reload.html
http/tests/w3c/webperf/approved/navigation-timing/html/test_timing_reload.html

BUG=576272
Committed: https://crrev.com/69824eb6bff5487e1fc8060bbc25d6b0eb01df65
Cr-Commit-Position: refs/heads/master@{#405179}

[arthursonzogni, 2016-07-11 09:53:51]

The CQ bit was checked by arthursonzogni@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-11 09:53:58]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-11 10:41:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-11 10:41:52]

Dry run: This issue passed the CQ dry run.

[arthursonzogni, 2016-07-11 10:47:52]

Description was changed from

==========
PlzNavigate: Fix missing navigation information.

The navigation type "blink::WebNavigationTypeReload" is now
forwarded to the browser process.

Two more tests pass when browser-side-navigation is enabled.
http/tests/w3c/webperf/approved/navigation-timing/html/test_navigation_type_reload.html
http/tests/w3c/webperf/approved/navigation-timing/html/test_timing_reload.html

BUG=576272
==========

to

==========
PlzNavigate: Fix missing navigation information.

The navigation type "blink::WebNavigationTypeReload" is now
forwarded to the browser process.

Two more tests pass when browser-side-navigation is enabled.
http/tests/w3c/webperf/approved/navigation-timing/html/test_navigation_type_reload.html
http/tests/w3c/webperf/approved/navigation-timing/html/test_timing_reload.html

BUG=576272
==========

[arthursonzogni, 2016-07-11 10:47:52]

arthursonzogni@chromium.org changed reviewers:
+ clamy@chromium.org

[clamy, 2016-07-11 12:37:57]

Thanks! Could you also remove the passing tests from
third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation ?

https://codereview.chromium.org/2136963002/diff/20001/content/renderer/render...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/2136963002/diff/20001/content/renderer/render...
content/renderer/render_frame_impl.cc:5829: 
nit: no extra line here.

[arthursonzogni (do not use), 2016-07-11 12:55:27]

On 2016/07/11 12:37:57, clamy wrote:
> Thanks! Could you also remove the passing tests from
> third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation
?
> 
>
https://codereview.chromium.org/2136963002/diff/20001/content/renderer/render...
> File content/renderer/render_frame_impl.cc (right):
> 
>
https://codereview.chromium.org/2136963002/diff/20001/content/renderer/render...
> content/renderer/render_frame_impl.cc:5829: 
> nit: no extra line here.

It is done. I will remember this for next time.

[clamy, 2016-07-11 14:03:16]

Thanks! Lgtm.

[arthursonzogni (do not use), 2016-07-13 13:56:23]

The CQ bit was checked by arthursonzogni@google.com

[arthursonzogni (do not use), 2016-07-13 13:56:29]

The CQ bit was unchecked by arthursonzogni@google.com

[arthursonzogni (do not use), 2016-07-13 13:56:44]

The CQ bit was checked by arthursonzogni@google.com

[commit-bot: I haz the power, 2016-07-13 13:56:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-13 16:22:25]

Description was changed from

==========
PlzNavigate: Fix missing navigation information.

The navigation type "blink::WebNavigationTypeReload" is now
forwarded to the browser process.

Two more tests pass when browser-side-navigation is enabled.
http/tests/w3c/webperf/approved/navigation-timing/html/test_navigation_type_reload.html
http/tests/w3c/webperf/approved/navigation-timing/html/test_timing_reload.html

BUG=576272
==========

to

==========
PlzNavigate: Fix missing navigation information.

The navigation type "blink::WebNavigationTypeReload" is now
forwarded to the browser process.

Two more tests pass when browser-side-navigation is enabled.
http/tests/w3c/webperf/approved/navigation-timing/html/test_navigation_type_reload.html
http/tests/w3c/webperf/approved/navigation-timing/html/test_timing_reload.html

BUG=576272
==========

[commit-bot: I haz the power, 2016-07-13 16:22:26]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-13 16:22:39]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-13 16:23:40]

Description was changed from

==========
PlzNavigate: Fix missing navigation information.

The navigation type "blink::WebNavigationTypeReload" is now
forwarded to the browser process.

Two more tests pass when browser-side-navigation is enabled.
http/tests/w3c/webperf/approved/navigation-timing/html/test_navigation_type_reload.html
http/tests/w3c/webperf/approved/navigation-timing/html/test_timing_reload.html

BUG=576272
==========

to

==========
PlzNavigate: Fix missing navigation information.

The navigation type "blink::WebNavigationTypeReload" is now
forwarded to the browser process.

Two more tests pass when browser-side-navigation is enabled.
http/tests/w3c/webperf/approved/navigation-timing/html/test_navigation_type_reload.html
http/tests/w3c/webperf/approved/navigation-timing/html/test_timing_reload.html

BUG=576272

Committed: https://crrev.com/69824eb6bff5487e1fc8060bbc25d6b0eb01df65
Cr-Commit-Position: refs/heads/master@{#405179}
==========

[commit-bot: I haz the power, 2016-07-13 16:23:41]

Patchset 4 (id:??) landed as
https://crrev.com/69824eb6bff5487e1fc8060bbc25d6b0eb01df65
Cr-Commit-Position: refs/heads/master@{#405179}