Disabled GetSizeForNewRenderView test as this test fails when forced in compositing mode.

Disabled GetSizeForNewRenderView test as this test fails when forced in compositing mode.
This test fails for dynamic size setting in NavigatePending. Following is comment in the code as reference.
// If WebContentsView is resized after RenderWidgetHostView is created but
// before pending navigation entry is committed, both RenderWidgetHostView and
// WebContentsView use the new size of WebContentsView.

BUG=281726
R=gab,kuan
TBR=jochen

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221146

[gab, 2013-09-03 23:29:27]

lgtm if test owners agrees.

https://codereview.chromium.org/23461020/diff/1/content/browser/web_contents/...
File content/browser/web_contents/web_contents_impl_browsertest.cc (right):

https://codereview.chromium.org/23461020/diff/1/content/browser/web_contents/...
content/browser/web_contents/web_contents_impl_browsertest.cc:231: //
force-compositing-mode is resolved (#281726).
nit: s/#281726/http://crbug.com/281726

[kuan, 2013-09-03 23:42:19]

https://codereview.chromium.org/23461020/diff/1/content/browser/web_contents/...
File content/browser/web_contents/web_contents_impl_browsertest.cc (right):

https://codereview.chromium.org/23461020/diff/1/content/browser/web_contents/...
content/browser/web_contents/web_contents_impl_browsertest.cc:34: #if
defined(TOOLKIT_GTK) || defined(OS_MACOSX)
one thing u can try is to modify these conditional flags as follows:
#if (defined(OS_WIN) && !defined(USE_AURA)) || \
    defined(TOOLKIT_GTK) || defined(OS_MACOSX)

if it doesn't work, then disabling the test is fine.

[Shrikant Kelkar, 2013-09-04 00:16:52]

https://codereview.chromium.org/23461020/diff/1/content/browser/web_contents/...
File content/browser/web_contents/web_contents_impl_browsertest.cc (right):

https://codereview.chromium.org/23461020/diff/1/content/browser/web_contents/...
content/browser/web_contents/web_contents_impl_browsertest.cc:34: #if
defined(TOOLKIT_GTK) || defined(OS_MACOSX)
On 2013/09/03 23:42:20, kuan wrote:
> one thing u can try is to modify these conditional flags as follows:
> #if (defined(OS_WIN) && !defined(USE_AURA)) || \
>     defined(TOOLKIT_GTK) || defined(OS_MACOSX)
> 
> if it doesn't work, then disabling the test is fine.

When tried this on my machine test fails with and without force-compositing-mode
flag. Without this approach test fails only in force-compositing-mode.

[kuan, 2013-09-04 00:23:49]

On 2013/09/04 00:16:52, Shrikant Kelkar wrote:
>
https://codereview.chromium.org/23461020/diff/1/content/browser/web_contents/...
> File content/browser/web_contents/web_contents_impl_browsertest.cc (right):
> 
>
https://codereview.chromium.org/23461020/diff/1/content/browser/web_contents/...
> content/browser/web_contents/web_contents_impl_browsertest.cc:34: #if
> defined(TOOLKIT_GTK) || defined(OS_MACOSX)
> On 2013/09/03 23:42:20, kuan wrote:
> > one thing u can try is to modify these conditional flags as follows:
> > #if (defined(OS_WIN) && !defined(USE_AURA)) || \
> >     defined(TOOLKIT_GTK) || defined(OS_MACOSX)
> > 
> > if it doesn't work, then disabling the test is fine.
> 
> When tried this on my machine test fails with and without
force-compositing-mode
> flag. Without this approach test fails only in force-compositing-mode.

ok, disable the test for win then.  thanks for trying! lgtm.

[commit-bot: I haz the power, 2013-09-04 00:30:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shrikant@chromium.org/23461020/3001

[commit-bot: I haz the power, 2013-09-04 04:38:57]

Change committed as 221146