Adds a unit test for NavigationEntry. Also adds a comment to BrowsingInstanc...

Adds a unit test for NavigationEntry.  Also adds a comment to BrowsingInstance
to explain that it is unit tested in site_instance_unittest.cc.

BUG=7371
BUG=7369
TEST=NavigationEntryTest.*



Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=9349

[Charlie Reis, 2009-02-06 19:03:36]

Hi Brett--
  Could you review this unit test for NavigationEntry (and comment for the test
for BrowsingInstance)?  I've CC'd Mike since he filed the bugs.

Thanks!
Charlie

[brettw, 2009-02-06 19:10:46]

Jolly good change. Looks ducky to me.

http://codereview.chromium.org/21132/diff/1/4
File chrome/browser/navigation_entry_unittest.cc (right):

http://codereview.chromium.org/21132/diff/1/4#newcode1
Line 1: // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
s/2006\-2008/2009/

[Charlie Reis, 2009-02-06 22:41:45]

Great, thanks.

http://codereview.chromium.org/21132/diff/1/4
File chrome/browser/navigation_entry_unittest.cc (right):

http://codereview.chromium.org/21132/diff/1/4#newcode1
Line 1: // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
Fixed.

On 2009/02/06 19:10:46, brettw wrote:
> s/2006\-2008/2009/

[pink (ping after 24hrs), 2009-02-09 17:20:53]

why was this added to chrome/browser? NavigationEntry lives is
chrome/browser/tab_contents.

Can this be moved?