Bug 144949

Summary: REGRESSION(r183770): Crash inside WebEditorClient::shouldApplyStyle when applying underline
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: HTML EditingAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: darin, enrica
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Fixes the crash darin: review+

Description Ryosuke Niwa 2015-05-13 01:37:13 PDT
>  1 com.apple.WebKitLegacy         0x7fff8b329185 WebEditorClient::shouldApplyStyle(WebCore::StyleProperties*, WebCore::Range*) + 0x15
   2 com.apple.WebCore              0x7fff87e6864c WebCore::Editor::applyStyleToSelection(WTF::Ref<WebCore::EditingStyle>&&, WebCore::EditAction) + 0x9c
   3 com.apple.WebCore              0x7fff87e77c4c WebCore::applyCommandToFrame(WebCore::Frame&, WebCore::EditorCommandSource, WebCore::EditAction, WTF::Ref<WebCore::EditingStyle>&&) + 0x5c
   4 com.apple.WebCore              0x7fff87e77698 WebCore::executeUnderline(WebCore::Frame&, WebCore::Event*, WebCore::EditorCommandSource, WTF::String const&) + 0x68
   5 com.apple.WebCore              0x7fff87ba7d86 WebCore::Editor::Command::execute(WTF::String const&, WebCore::Event*) const + 0xb6
   6 com.apple.WebKitLegacy         0x7fff8b2cd819 -[WebHTMLView executeCoreCommandBySelector:] + 0x69
Comment 1 Ryosuke Niwa 2015-05-13 01:37:27 PDT
<rdar://problem/20895753>
Comment 2 Ryosuke Niwa 2015-05-13 01:52:58 PDT
Created attachment 253026 [details]
Fixes the crash
Comment 3 Ryosuke Niwa 2015-05-13 14:13:25 PDT
Comment on attachment 253026 [details]
Fixes the crash

View in context: https://bugs.webkit.org/attachment.cgi?id=253026&action=review

Thanks for the review!

> LayoutTests/ChangeLog:12
> +        * editing/style/underline-by-user.html: Added.

Oops, I forgot to add -expected.txt in svn.
Will do that before landing it.
Comment 4 Ryosuke Niwa 2015-05-13 15:00:07 PDT
Committed r184308: <http://trac.webkit.org/changeset/184308>