Add a content_browsertest for LoadDataWithBaseURL + Back.

Add a content_browsertest for LoadDataWithBaseURL + Back.

The current navigation logic has a bug that loads the history URL but
lies and claims the data URL is committed.  This bug is fixed in the
new navigation logic with subframe FrameNavigationEntries.

This CL adds a test showing these two expectations, mimicing
org.chromium.android_webview.test.LoadDataWithBaseUrlTest's

It's easier to switch to the new navigation logic than fix the bug
in the old logic.

BUG=612196
TEST=Test passes by default and in --site-per-process.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/fa6742e4236b8efa17c8eb60f23f566f64b1c500
Cr-Commit-Position: refs/heads/master@{#401912}

[Charlie Reis, 2016-06-24 06:48:55]

Description was changed from

==========
Add a content_browsertest for LoadDataWithBaseURL + Back.

The current navigation logic has a bug that loads the history URL but
lies and claims the data URL is committed.  This bug is fixed in the
new navigation logic with subframe FrameNavigationEntries.

This CL adds a test showing these two expectations, mimicing
org.chromium.android_webview.test.LoadDataWithBaseUrlTest's

It's easier to switch to the new navigation logic than fix the bug
in the old logic.

BUG=612196
TEST=Test passes by default and in --site-per-process.
==========

to

==========
Add a content_browsertest for LoadDataWithBaseURL + Back.

The current navigation logic has a bug that loads the history URL but
lies and claims the data URL is committed.  This bug is fixed in the
new navigation logic with subframe FrameNavigationEntries.

This CL adds a test showing these two expectations, mimicing
org.chromium.android_webview.test.LoadDataWithBaseUrlTest's

It's easier to switch to the new navigation logic than fix the bug
in the old logic.

BUG=612196
TEST=Test passes by default and in --site-per-process.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[Charlie Reis, 2016-06-24 06:51:13]

Patchset #1 (id:1) has been deleted

[Charlie Reis, 2016-06-24 17:02:11]

creis@chromium.org changed reviewers:
+ boliu@chromium.org

[Charlie Reis, 2016-06-24 17:02:12]

Bo, can you take a look?  This ports
LoadDataWithBaseUrlTest#testHistoryUrlNavigation to a content_browsertest and
updates the expectation based on my findings in https://crbug.com/612196#c10.

https://codereview.chromium.org/2099623002/diff/40001/content/browser/frame_h...
File content/browser/frame_host/navigation_controller_impl_browsertest.cc
(right):

https://codereview.chromium.org/2099623002/diff/40001/content/browser/frame_h...
content/browser/frame_host/navigation_controller_impl_browsertest.cc:181: if
(SiteIsolationPolicy::UseSubframeNavigationEntries())
Here's the bit where I switch the test's expectation between the old and new
navigation paths.  We can remove surprising the "form" expectation after we've
switched over.

[boliu, 2016-06-24 17:34:38]

lgtm

[Charlie Reis, 2016-06-24 17:43:36]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2016-06-24 17:44:17]

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-06-24 18:36:49]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-06-24 18:38:43]

Description was changed from

==========
Add a content_browsertest for LoadDataWithBaseURL + Back.

The current navigation logic has a bug that loads the history URL but
lies and claims the data URL is committed.  This bug is fixed in the
new navigation logic with subframe FrameNavigationEntries.

This CL adds a test showing these two expectations, mimicing
org.chromium.android_webview.test.LoadDataWithBaseUrlTest's

It's easier to switch to the new navigation logic than fix the bug
in the old logic.

BUG=612196
TEST=Test passes by default and in --site-per-process.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Add a content_browsertest for LoadDataWithBaseURL + Back.

The current navigation logic has a bug that loads the history URL but
lies and claims the data URL is committed.  This bug is fixed in the
new navigation logic with subframe FrameNavigationEntries.

This CL adds a test showing these two expectations, mimicing
org.chromium.android_webview.test.LoadDataWithBaseUrlTest's

It's easier to switch to the new navigation logic than fix the bug
in the old logic.

BUG=612196
TEST=Test passes by default and in --site-per-process.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/fa6742e4236b8efa17c8eb60f23f566f64b1c500
Cr-Commit-Position: refs/heads/master@{#401912}
==========

[commit-bot: I haz the power, 2016-06-24 18:38:44]

Patchset 2 (id:??) landed as
https://crrev.com/fa6742e4236b8efa17c8eb60f23f566f64b1c500
Cr-Commit-Position: refs/heads/master@{#401912}