WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 84024
[chromium] Layout Test editing/pasteboard/emacs-ctrl-k-with-move.html is failing (CRASH)
https://bugs.webkit.org/show_bug.cgi?id=84024
Summary
[chromium] Layout Test editing/pasteboard/emacs-ctrl-k-with-move.html is fail...
Takashi Toyoshima
Reported
2012-04-16 05:23:53 PDT
The following layout test is failing on Chromium Mac10.5, 10.5 (dbg), 10.6, 10.6 (dbg), and 10.7. editing/pasteboard/emacs-ctrl-k-with-move.html
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=editing%2Fpasteboard%2Femacs-ctrl-k-with-move.html
I'm not sure who is familiar to this crash. But I notice one of stack trace contains WebCore::executeYank and this test is for editing. I guess Niwa-san may be familiar to this.
Attachments
Add attachment
proposed patch, testcase, etc.
Takashi Toyoshima
Comment 1
2012-04-16 07:00:53 PDT
update expectations.
http://trac.webkit.org/changeset/114248
Kent Tamura
Comment 2
2012-04-16 17:41:58 PDT
(In reply to
comment #1
)
> update expectations. >
http://trac.webkit.org/changeset/114248
You must not skip this test on Mac. It tests a feature for Mac. You need to resolve the failure.
Kent Tamura
Comment 3
2012-04-16 17:44:43 PDT
Clearly
http://trac.webkit.org/changeset/114220/
is the trigger. Don't hesitate to roll it out.
Jia Pu
Comment 4
2012-04-16 17:51:28 PDT
(In reply to
comment #3
)
> Clearly
http://trac.webkit.org/changeset/114220/
is the trigger. Don't hesitate to roll it out.
If this failure is caused a crash in Editor::insertTextWithoutSendingTextEvent, which seems to be the case to em, the patch in
bug 84010
fixes it. Can someone review it?
Takashi Toyoshima
Comment 5
2012-04-17 00:44:55 PDT
Oops, I miss reverting. Now it's difficult to revert related change 114220, 114231, and 114266. Jia Pu, Could you fix this crush or manually revert them?
Jia Pu
Comment 6
2012-04-17 10:26:19 PDT
(In reply to
comment #5
)
> Oops, I miss reverting. > Now it's difficult to revert related change 114220, 114231, and 114266. > > Jia Pu, > Could you fix this crush or manually revert them?
It seems
http://trac.webkit.org/changeset/114311
has already fixed this. You can revert the test back to original now.
Takashi Toyoshima
Comment 7
2012-04-17 21:12:50 PDT
Thanks Jia! I revert my previous expectation update.
http://trac.webkit.org/changeset/114479
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug