[Webkit-unassigned] [Bug 109781] Rename some files in WebCore/editing/

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Feb 25 06:12:32 PST 2013


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





--- Comment #3 from Kent Tamura (ooo until Mar 15) <tkent at chromium.org>  2013-02-25 06:14:55 PST ---
(In reply to comment #2)
> I'm not sure if having "Helper" in file name is an improvement. What is the additional information provided by "Helper" part of the name?
> 
> If the longer name is not more helpful, I prefer a shorter one.

Yeah, "Helper" is not so meaningful. However I think adding this word is helpful in this case.

* htmlediting.*
  Renaming them to editing/Editing.* would be confusing.  Editing.* sounds the main file in editing/ directory. 
  Renaming them to editing/HTMLEditing.* won't work. AFAIK, case-only renaming makes troubles.

* markup.*
  Renaming them to editing/Markup.* won't work.  Ditto.

* visible_units.*
  ok, I'd like to rename them to VisibleUnits.*.

-- 
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