Bug 87390 - REGRESSION(r118352): Caused 6 editing/spelling tests crash/fail on chromium in debug. (Requested by vsevik on #webkit).
Summary: REGRESSION(r118352): Caused 6 editing/spelling tests crash/fail on chromium i...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: WebKit Review Bot
URL:
Keywords:
Depends on:
Blocks: 74393
  Show dependency treegraph
 
Reported: 2012-05-24 07:51 PDT by WebKit Review Bot
Modified: 2022-03-01 03:06 PST (History)
3 users (show)

See Also:


Attachments
ROLLOUT of r118352 (60.06 KB, patch)
2012-05-24 07:52 PDT, WebKit Review Bot
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description WebKit Review Bot 2012-05-24 07:51:50 PDT
http://trac.webkit.org/changeset/118352 broke the build:
Caused 6 editing/spelling tests crash/fail on chromium in debug. (Requested by vsevik on #webkit).

This is an automatic bug report generated by the sheriff-bot. If this bug
report was created because of a flaky test, please file a bug for the flaky
test (if we don't already have one on file) and dup this bug against that bug
so that we can track how often these flaky tests case pain.

"Only you can prevent forest fires." -- Smokey the Bear
Comment 1 WebKit Review Bot 2012-05-24 07:52:31 PDT
Created attachment 143832 [details]
ROLLOUT of r118352

Any committer can land this patch automatically by marking it commit-queue+.  The commit-queue will build and test the patch before landing to ensure that the rollout will be successful.  This process takes approximately 15 minutes.

If you would like to land the rollout faster, you can use the following command:

  webkit-patch land-attachment ATTACHMENT_ID

where ATTACHMENT_ID is the ID of this attachment.
Comment 2 Vsevolod Vlasov 2012-05-24 07:55:39 PDT
Flakiness dashboard:

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=editing%2Fspelling%2Fspellcheck-paste-disabled.html%2Cediting%2Fspelling%2Fspellcheck-paste.html%2Cediting%2Fspelling%2Fgrammar-paste.html%2Cediting%2Fspelling%2Fspellcheck-queue.html%2Cediting%2Fspelling%2Fspellcheck-sequencenum.html%2Cediting%2Fspelling%2Fgrammar-markers.html

Failures:
editing/spelling/grammar-markers.html - image expectations differ.

Crashes:
editing/spelling/grammar-paste.html
editing/spelling/spellcheck-paste-disabled.html
editing/spelling/spellcheck-paste.html
editing/spelling/spellcheck-queue.html
editing/spelling/spellcheck-sequencenum.html
platform/chromium/editing/spelling/move-cursor-to-misspelled-word.html

crash log for DumpRenderTree (pid 29729):
STDOUT: <empty>
STDERR: ASSERTION FAILED: badGrammarPhraseLocation == -1
STDERR: third_party/WebKit/Source/WebCore/editing/TextCheckingHelper.cpp(495) : WTF::String WebCore::TextCheckingHelper::findFirstBadGrammar(WebCore::GrammarDetail&, int&, bool)
STDERR: 1   0x14e3d60
STDERR: 2   0x14e48b0
STDERR: 3   0x14aeabc
STDERR: 4   0x14aec38
STDERR: 5   0x14b0696
STDERR: 6   0x14b5524
STDERR: 7   0x14c2da3
STDERR: 8   0x14c1e1f
STDERR: 9   0x1651aec
STDERR: 10  0x1652c3d
STDERR: 11  0x1d13671
STDERR: 12  0xb45f74
STDERR: 13  0xb40ee8
STDERR: 14  0xb40eb9
STDERR: 15  0x558f320618e
STDERR: [29729:29729:5333145594220:ERROR:process_util_posix.cc(143)] Received signal 11
STDERR: 	base::debug::StackTrace::StackTrace() [0x875d86]
STDERR: 	base::(anonymous namespace)::StackDumpSignalHandler() [0x835e49]
STDERR: 	0x7f6ffdd1caf0
STDERR: 	WebCore::TextCheckingHelper::findFirstBadGrammar() [0x14e3d6a]
STDERR: 	WebCore::TextCheckingHelper::markAllBadGrammar() [0x14e48b0]
STDERR: 	WebCore::Editor::markMisspellingsOrBadGrammar() [0x14aeabc]
STDERR: 	WebCore::Editor::markBadGrammar() [0x14aec38]
STDERR: 	WebCore::Editor::markMisspellingsAndBadGrammar() [0x14b0696]
STDERR: 	WebCore::Editor::respondToChangedSelection() [0x14b5524]
STDERR: 	WebCore::FrameSelection::setSelection() [0x14c2da3]
STDERR: 	WebCore::FrameSelection::moveTo() [0x14c1e1f]
STDERR: 	WebCore::DOMSelection::setBaseAndExtent() [0x1651aec]
STDERR: 	WebCore::DOMSelection::selectAllChildren() [0x1652c3d]
STDERR: 	WebCore::DOMSelectionV8Internal::selectAllChildrenCallback() [0x1d13671]
STDERR: 	v8::internal::HandleApiCallHelper<>() [0xb45f74]
STDERR: 	v8::internal::Builtin_Impl_HandleApiCall() [0xb40ee8]
STDERR: 	v8::internal::Builtin_HandleApiCall() [0xb40eb9]
STDERR: 	0x558f320618e
Comment 3 Vsevolod Vlasov 2012-05-24 07:59:00 PDT
Committed r118375: <http://trac.webkit.org/changeset/118375>
Comment 4 Hironori Bono 2012-05-24 19:50:26 PDT
Greetings,

Sorry for theis assertion error. It seems EditorClientImpl::checkGrammarOfString() now needs to set badGrammarLocation to -1 when it is not implemented. I will update my change to fix it.

Regards,

Hironori Bono