Fixes for frame proxy to avoid double output in layout tests.

Fixes for frame proxy to avoid double output in layout tests.

Calling out to m_baseProxy and the base object results right now in duplicate prints, since the base object also calls into WebTestProxyBase. I'm removing the extra calls right now and will add them as I migrate methods from RenderView to RenderFrame.

BUG=245126
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=153250

[nasko, 2013-06-28 00:19:27]

Hey Jochen,
I didn't catch this earlier, but there are duplicate logs in some cases, which
caused layout tests to fail. Can you review this fix?
Thanks,
Nasko

[jochen (gone - plz use gerrit), 2013-06-28 08:48:23]

lgtm

https://codereview.chromium.org/18031013/diff/1/public/testing/WebFrameTestPr...
File public/testing/WebFrameTestProxy.h (right):

https://codereview.chromium.org/18031013/diff/1/public/testing/WebFrameTestPr...
public/testing/WebFrameTestProxy.h:46: m_baseProxy = 0;
nit: move to initializer list

https://codereview.chromium.org/18031013/diff/1/public/testing/WebFrameTestPr...
public/testing/WebFrameTestProxy.h:107: // into the base proxy.
why not implement all these in RenderFrameImpl?

[nasko, 2013-06-28 15:12:48]

https://codereview.chromium.org/18031013/diff/1/public/testing/WebFrameTestPr...
File public/testing/WebFrameTestProxy.h (right):

https://codereview.chromium.org/18031013/diff/1/public/testing/WebFrameTestPr...
public/testing/WebFrameTestProxy.h:46: m_baseProxy = 0;
On 2013/06/28 08:48:24, jochen wrote:
> nit: move to initializer list

Done.

https://codereview.chromium.org/18031013/diff/1/public/testing/WebFrameTestPr...
public/testing/WebFrameTestProxy.h:107: // into the base proxy.
On 2013/06/28 08:48:24, jochen wrote:
> why not implement all these in RenderFrameImpl?

They aren't implemented in RenderViewImpl either, so I chose not to include them
for now, to make code transition easier to follow.

[commit-bot: I haz the power, 2013-06-28 15:13:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nasko@chromium.org/18031013/6001

[commit-bot: I haz the power, 2013-06-28 16:56:26]

Change committed as 153250