[Webkit-unassigned] [Bug 31186] Cleanup: Rename fields RenderTextControl::m_edited and RenderTextControl::m_userEdited

[bugzilla-daemon at webkit.org]

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


Daniel Bates <dbates at webkit.org> changed:

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




--- Comment #1 from Daniel Bates <dbates at webkit.org>  2009-11-11 16:55:40 PST ---
Created an attachment (id=43020)
 --> (https://bugs.webkit.org/attachment.cgi?id=43020)
Patch

No functionality was changed so no tests were included in this patch.

On Darin's suggestion <https://bugs.webkit.org/show_bug.cgi?id=20780#c24>, I
chose to rename RenderTextControl::m_edited to wasChangedSinceLastChangeEvent,
and RenderTextControl::m_userEdited to m_lastChangeWasUserEdit. I felt these
names were consistent (in wording - both use the word "change" to describe the
modification to the text control) and best described what these fields are for.
It would have been nice if we could have more concise names for these fields,
but I could not think of any.

We should probably look into re-factoring the functionality of these
methods/cleanup RenderTextControl as outlined in
<https://bugs.webkit.org/show_bug.cgi?id=20780#c14>. Then we may have a cleaner
API.

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