[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


Darin Adler <darin at apple.com> changed:

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




--- Comment #2 from Darin Adler <darin at apple.com>  2009-11-12 08:36:04 PST ---
(From update of attachment 43020)
> -        // We set m_userEdited to false since this change was not explicty made by the user (say, via typing on the keyboard), see <rdar://problem/5359921>.
> -        m_userEdited = false;
> +        // We set m_lastChangeWasUserEdit to false since this change was not explicty made by the user (say, via typing on the keyboard), see <rdar://problem/5359921>.
> +        m_lastChangeWasUserEdit = false;

It would be nice to fix the spelling of explicitly.

I don't think the "see rdar" comment is really all that useful, even for people
at Apple.

I think the comment is no longer needed; the new name of the data member speaks
for itself. Or if we really want one, the comment could say, "Set
m_lastChangeWasUserEdit to false since this function is only used for
programmatic changes, not for user editing."

As you mentioned, we should make other changes here. We don't want people to
directly set these with set functions.

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