Bug 109548 - [chromium-android] WebFrameTest.ReplaceMisspelledRange crashes
Summary: [chromium-android] WebFrameTest.ReplaceMisspelledRange crashes
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Android Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-11 23:40 PST by jochen
Modified: 2013-04-08 11:10 PDT (History)
5 users (show)

See Also:

Attachments
Patch (1.48 KB, patch)
2013-02-11 23:48 PST, jochen 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description jochen 2013-02-11 23:40:16 PST 
Since the WebKit roll 142426:142523, WebFrameTest.ReplaceMisspelledRange crashes on Android Tests and Android Tests (dbg)

Note: Google Test filter = WebFrameTest.ReplaceMisspelledRange:WebPageNewSerializeTest.SVGImageDontCrash
[==========] Running 2 tests from 2 test cases.
[----------] Global test environment set-up.
[----------] 1 test from WebFrameTest
[ RUN      ] WebFrameTest.ReplaceMisspelledRange
[WARNING:proxy_service.cc(887)] PAC support disabled because there is no system implementation
[ CRASHED      ]
Comment 1 jochen 2013-02-11 23:48:32 PST 
Created attachment 187784 [details]
Patch
Comment 2 jochen 2013-02-12 00:04:55 PST 
Comment on attachment 187784 [details]
Patch

Clearing flags on attachment: 187784

Committed r142590: <http://trac.webkit.org/changeset/142590>
Comment 3 jochen 2013-02-12 00:05:00 PST 
All reviewed patches have been landed.  Closing bug.
Comment 4 jochen 2013-02-12 00:05:56 PST 
reopening to track the underlying issue
Comment 5 Rouslan Solomakhin 2013-02-12 11:00:37 PST 
Is there a stack trace?
Comment 7 Rouslan Solomakhin 2013-02-22 14:04:18 PST 
Why is this reopened? Did disabling the test on Android not solve the problem?
Comment 8 jochen 2013-02-23 01:48:39 PST 
No, it didn't solve the problem.

Fixing the crash will solve the problem
Comment 9 Rouslan Solomakhin 2013-02-25 11:24:43 PST 
Is this the chromium-android platform?
Comment 10 jochen 2013-02-25 11:29:20 PST 
Yes, it's chromium android
Comment 11 John Knottenbelt 2013-03-18 10:16:49 PDT 
It could be that this test is failing because, on Android, the data: URLs are going through the network stack instead of being mocked. See WebURLLoaderImpl::Context::CanHandleDataURL in src/webkit/glue/weburlloader_impl.cc.

A fix might be to use registerMockedHttpURLLoad
Comment 12 Peter Beverloo 2013-04-08 11:10:05 PDT 
Resolving as WontFix given that Chromium moved to Blink.