[Webkit-unassigned] [Bug 51693] [chromium] editing/input/ime-composition-clearpreedit.html fails in chromium.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=51693





--- Comment #13 from James Su <suzhe at chromium.org>  2010-12-29 12:28:46 PST ---
(In reply to comment #12)
> Re-opening since I rolled out this patch with http://trac.webkit.org/changeset/74749
> 
> It causing assert failures on fast/forms/input-maxlength-ime-completed.html (http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Fforms%2Finput-maxlength-ime-completed.html&master=ChromiumWebkit):
> 
> ASSERTION FAILED: m_offset + m_count <= m_node->length()
> (third_party/WebKit/WebCore/editing/DeleteFromTextNodeCommand.cpp:42 WebCore::DeleteFromTextNodeCommand::DeleteFromTextNodeCommand(WTF::PassRefPtr<WebCore::Text>, unsigned int, unsigned int))
> [28117:28117:1434569528609:ERROR:process_util_posix.cc(106)] Received signal 11
> 
> Even before then, it needed an expectations update for another layout test that I did with http://trac.webkit.org/changeset/74748 (I'll be rolling that out too).

This failure is caused by incorrect implementation of test_shell's TextInputController::insertText() rather than this patch.

And in order to fix WebKit/LayoutTests/fast/events/ime-composition-events-001.html in chromium, we need to modify both WebWidget API in WebKit/chromium and the corresponding part in chromium. I'll post a separated patch for that part as well as the test_shell fix.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.