Add FrameNavigationEntry class.

Add FrameNavigationEntry class.

This will be used to track session history items for subframes.
For now, each NavigationEntry owns a single one for its main frame.

BUG=236848
TEST=No behavior change.

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

[Charlie Reis, 2015-02-18 21:41:47]

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

[Charlie Reis, 2015-02-18 21:41:48]

Avi: Let's get these changes going!  Here's the first piece I've carved out from
https://codereview.chromium.org/281653003/.  Sound like a reasonable starting
point?

[Avi (use Gerrit), 2015-02-18 21:56:24]

That was simple enough. Only nits; LGTM with them fixed.

https://codereview.chromium.org/931333004/diff/1/content/browser/frame_host/f...
File content/browser/frame_host/frame_navigation_entry.h (right):

https://codereview.chromium.org/931333004/diff/1/content/browser/frame_host/f...
content/browser/frame_host/frame_navigation_entry.h:6: #define
CONTENT_BROWSER_FRAME_HOST_FRAME_NAVIGATION_ENTRY_IMPL_H_
Bad guard constant.

https://codereview.chromium.org/931333004/diff/1/content/browser/frame_host/f...
content/browser/frame_host/frame_navigation_entry.h:60: 
too many blank lines

[Charlie Reis, 2015-02-18 22:02:12]

New patchsets have been uploaded after l-g-t-m from avi@chromium.org

[Charlie Reis, 2015-02-18 22:04:39]

Thanks!  I ran git cl format as well.

https://codereview.chromium.org/931333004/diff/1/content/browser/frame_host/f...
File content/browser/frame_host/frame_navigation_entry.h (right):

https://codereview.chromium.org/931333004/diff/1/content/browser/frame_host/f...
content/browser/frame_host/frame_navigation_entry.h:6: #define
CONTENT_BROWSER_FRAME_HOST_FRAME_NAVIGATION_ENTRY_IMPL_H_
On 2015/02/18 21:56:24, Avi wrote:
> Bad guard constant.

Done.

https://codereview.chromium.org/931333004/diff/1/content/browser/frame_host/f...
content/browser/frame_host/frame_navigation_entry.h:60: 
On 2015/02/18 21:56:24, Avi wrote:
> too many blank lines

Done.

[Charlie Reis, 2015-02-18 22:05:47]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2015-02-18 22:06:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/931333004/20001

[commit-bot: I haz the power, 2015-02-18 23:30:38]

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

[commit-bot: I haz the power, 2015-02-18 23:30:39]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Charlie Reis, 2015-02-18 23:38:45]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2015-02-18 23:41:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/931333004/20001

[commit-bot: I haz the power, 2015-02-19 01:06:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-19 01:07:54]

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