Disable WebViewTest.ClearDataCache on Windows as it's flakily failing.

Disable WebViewTest.ClearDataCache on Windows as it's flakily failing.

BUG=489088
Test=None, internal change.

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

[lazyboy, 2015-05-18 22:14:28]

lazyboy@chromium.org changed reviewers:
+ brucedawson@chromium.org

[brucedawson, 2015-05-18 22:43:14]

lgtm

[lazyboy, 2015-05-18 23:10:04]

The CQ bit was checked by lazyboy@chromium.org

[commit-bot: I haz the power, 2015-05-18 23:10:30]

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

[Nico, 2015-05-18 23:12:11]

thakis@chromium.org changed reviewers:
+ thakis@chromium.org

[Nico, 2015-05-18 23:12:11]

Since the test is new, can we revert the whole CL that added the test instead?

[brucedawson, 2015-05-18 23:14:02]

That would disable it for all platforms and it seems to be working reliably on
other platforms, I think?

[lazyboy, 2015-05-18 23:25:44]

On 2015/05/18 23:14:02, brucedawson wrote:
> That would disable it for all platforms and it seems to be working reliably on
> other platforms, I think?

IIUC, the test is failing on windows because of some EmbeddedTestServer
issues, responses are being cached where it shouldn't be cached.

[Nico, 2015-05-18 23:39:47]

On 2015/05/18 23:25:44, lazyboy wrote:
> On 2015/05/18 23:14:02, brucedawson wrote:
> > That would disable it for all platforms and it seems to be working reliably
on
> > other platforms, I think?
> 
> IIUC, the test is failing on windows because of some EmbeddedTestServer
> issues, responses are being cached where it shouldn't be cached.

In general, new code shouldn't add flakiness anywhere. Don't all other tests
work with EmbeddedTestServer? Can the server be fixed?

[commit-bot: I haz the power, 2015-05-19 00:06:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-05-19 00:07:24]

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

[lazyboy, 2015-05-19 00:34:58]

On 2015/05/18 23:39:47, Nico wrote:
> On 2015/05/18 23:25:44, lazyboy wrote:
> > On 2015/05/18 23:14:02, brucedawson wrote:
> > > That would disable it for all platforms and it seems to be working
reliably
> on
> > > other platforms, I think?
> > 
> > IIUC, the test is failing on windows because of some EmbeddedTestServer
> > issues, responses are being cached where it shouldn't be cached.
> 
> In general, new code shouldn't add flakiness anywhere. Don't all other tests
> work with EmbeddedTestServer? Can the server be fixed?

It's not an issue with EmbeddedTestServer by itself only, it is related to
mocking server's response with cache-control header (that's my guess):
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/app...