Summary: | [Mac] WebKit doesn't enable or disable autocorrections when system preferences are changed | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||||||||||||
Component: | HTML Editing | Assignee: | Ryosuke Niwa <rniwa> | ||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||
Severity: | Normal | CC: | abecsi, cmarcelo, commit-queue, ddavidso, enrica, gyuyoung.kim, jiapu.mail, menard, rakuco, sam | ||||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||||
Hardware: | Unspecified | ||||||||||||||||
OS: | Unspecified | ||||||||||||||||
Attachments: |
|
Description
Ryosuke Niwa
2013-04-30 21:22:25 PDT
Created attachment 200201 [details]
Work in progress
Created attachment 200203 [details]
Work in progress 2
Created attachment 200205 [details]
Work in progress 3
Created attachment 200206 [details]
Fixes the bug
Created attachment 200256 [details]
Updated per Sam's comments
Comment on attachment 200256 [details] Updated per Sam's comments View in context: https://bugs.webkit.org/attachment.cgi?id=200256&action=review > Source/WebKit2/ChangeLog:13 > + (WebKit): > + (TextChecker): Should remove these, or fix the script so they don’t get added in the first place. > Source/WebKit2/ChangeLog:15 > + (WebContext): Ditto. > Source/WebKit2/ChangeLog:18 > + (WebKit): Ditto. > Source/WebKit2/UIProcess/mac/WebContextMac.mm:555 > + [[NSNotificationCenter defaultCenter] removeObserver:(id)m_automaticTextReplacementNotificationObserver.get()]; Can’t these be RetainPtr<NSObject> instead of RetainPtr<CFTypeRef>, to avoid all these (id) casts? Created attachment 200291 [details]
Updated per Sam's in-person review
Comment on attachment 200291 [details]
Updated per Sam's in-person review
Oops, I didn't realize Darin had already reviewed. Thanks for the review!
Committed r149476: <http://trac.webkit.org/changeset/149476> |