Bug 128878 - Remove unused arguments from moveTo(Range*)
Summary: Remove unused arguments from moveTo(Range*)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords:
Depends on:
Blocks: 127832
  Show dependency tree / graph
 
Reported: 2014-02-15 18:43 PST by Ryosuke Niwa
Modified: 2014-02-15 19:02 PST (History)
4 users (show)

See Also:

Attachments
Cleanup (3.54 KB, patch)
2014-02-15 18:44 PST, Ryosuke Niwa 		koivisto: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2014-02-15 18:43:46 PST 
The second and the third argument of moveTo(Range*, ...) don't seem to be used anywhere.
Just get rid of them.
Comment 1 Ryosuke Niwa 2014-02-15 18:44:38 PST 
Created attachment 224304 [details]
Cleanup
Comment 2 Antti Koivisto 2014-02-15 18:58:03 PST 
Comment on attachment 224304 [details]
Cleanup

View in context: https://bugs.webkit.org/attachment.cgi?id=224304&action=review

> Source/WebCore/editing/FrameSelection.cpp:155
> +void FrameSelection::moveTo(const Range *r)

you could fix the coding style for Range since you are here
Comment 3 Ryosuke Niwa 2014-02-15 19:02:06 PST 
Committed r164186: <http://trac.webkit.org/changeset/164186>