Move various methods from glue/webview.h to api/public/WebView.h...

Move various methods from glue/webview.h to api/public/WebView.h

I'll re-order the methods in webview_impl.cc in a follow-up CL.  I
wanted to keep this one easy to review.

SetBackForwardListSize is no longer necessary given that
BackForwardListChromium.cpp doesn't care about its capacity.

R=dglazkov
BUG=10033
TEST=none


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

[dglazkov, 2009-09-30 23:28:32]

LGTM, with comments:

http://codereview.chromium.org/251051/diff/1/17
File chrome/renderer/render_view.cc (left):

http://codereview.chromium.org/251051/diff/1/17#oldcode324
Line 324: webview()->SetBackForwardListSize(1);
Did you forget to add this back somewhere? Or is this initialized elsewhere?

http://codereview.chromium.org/251051/diff/1/14
File webkit/glue/webview_unittest.cc (right):

http://codereview.chromium.org/251051/diff/1/14#newcode23
Line 23: EXPECT_TRUE(view->isActive());
Nice testing there, trungl.

[darin (slow to review), 2009-09-30 23:46:52]

http://codereview.chromium.org/251051/diff/1/17
File chrome/renderer/render_view.cc (left):

http://codereview.chromium.org/251051/diff/1/17#oldcode324
Line 324: webview()->SetBackForwardListSize(1);
On 2009/09/30 23:28:32, Dimitri Glazkov wrote:
> Did you forget to add this back somewhere? Or is this initialized elsewhere?

It is not needed anymore.  It was important when we were using the same
implementation of BackForwardList.cpp as other WebKit ports.  Now, it doesn't
matter, so I killed it.  I will update the CL description to mention this.