Disabling failing/crashing androidwebview tests

Disabling failing/crashing androidwebview tests

BUG=261409, 261421
NOTRY=true

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

[Ramya, 2013-07-18 00:16:06]

joth@ PTAL for lgtm

[boliu, 2013-07-18 00:19:55]

https://codereview.chromium.org/19715002/diff/1/android_webview/javatests/src...
File
android_webview/javatests/src/org/chromium/android_webview/test/AwSettingsTest.java
(right):

https://codereview.chromium.org/19715002/diff/1/android_webview/javatests/src...
android_webview/javatests/src/org/chromium/android_webview/test/AwSettingsTest.java:1606:
public void testDatabaseEnabled() throws Throwable {
Hmm, this is a different crash

[joth, 2013-07-18 00:20:19]

+mnaganov


In the linked log, I only see 3 tests failing, but you disabled 4?


C  145s Main  FAIL (3 tests):
[org.chromium.android_webview.test.AwSettingsTest#testFileAccessFromFilesIframeWithTwoViews,
org.chromium.android_webview.test.AwSettingsTest#testUniversalAccessFromFilesWithTwoViews,
org.chromium.android_webview.test.LoadDataWithBaseUrlTest#testCrossOrigin]



Do we know which blink rev (154328 ... 154374) caused this?



On 17 July 2013 17:16, <cramya@chromium.org> wrote:

> joth@ PTAL for lgtm
>
>
https://codereview.chromium.**org/19715002/<https://codereview.chromium.org/1...
>

[boliu, 2013-07-18 00:21:30]

https://codereview.chromium.org/19715002/diff/1/android_webview/javatests/src...
File
android_webview/javatests/src/org/chromium/android_webview/test/AwSettingsTest.java
(right):

https://codereview.chromium.org/19715002/diff/1/android_webview/javatests/src...
android_webview/javatests/src/org/chromium/android_webview/test/AwSettingsTest.java:1606:
public void testDatabaseEnabled() throws Throwable {
On 2013/07/18 00:19:55, boliu wrote:
> Hmm, this is a different crash

04018:  07-17 22:54:15.937  9917  9951 W WebKit  : ASSERTION FAILED: ec
04018:  07-17 22:54:15.937  9917  9951 W WebKit  :
../../third_party/WebKit/Source/bindings/v8/ExceptionState.cpp(47) : virtual
void WebCore::ExceptionState::throwDOMException(const ExceptionCode&, char
const*)

New one. File a different bug..?

[Ramya, 2013-07-18 00:29:24]

I filed crbug.com/261421 for the crashing test.
http://build.chromium.org/p/chromium.webkit/builders/Android%20Tests%20%28dbg...

[boliu, 2013-07-18 00:30:20]

lgtm

[Ramya, 2013-07-18 00:31:08]

joth@ PTAL for OWNERS lgtm

[joth, 2013-07-18 00:50:13]

lgtm

[boliu, 2013-07-18 06:00:29]

NOTRY+cq for you

[commit-bot: I haz the power, 2013-07-18 06:01:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/cramya@chromium.org/19715002/4

[commit-bot: I haz the power, 2013-07-18 09:08:09]

Change committed as 212301