[Webkit-unassigned] [Bug 39433] editingBehavior settings needs to be set back to a reasonable default between tests

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu May 20 11:18:14 PDT 2010


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


Martin Robinson <mrobinson at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #56608|review?                     |
               Flag|                            |




--- Comment #2 from Martin Robinson <mrobinson at webkit.org>  2010-05-20 11:18:14 PST ---
(From update of attachment 56608)
via IRC:
(10:57:24 AM) ojan: mrobinson: i'd rather see a shared method in Settings
(10:57:53 AM) ojan: mrobinson: that way, if a port changes which editing behavior they use ona  platform, they only change it in one place
(11:01:09 AM) ojan: mrobinson: so, i'm thinking...add Settings::resetEditingBehavior and move the logic from the Settings constructor into that method. then call the method instead of setEditingBehavior.
(11:05:48 AM) ojan: so, i'm just staying to move those 7 lines of code into Settings::resetEditingBehavior and instead call that method from the constructor

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



More information about the webkit-unassigned mailing list