WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
23324
Move platform/chromium/EditorChromium.cpp into editing/chromium/
https://bugs.webkit.org/show_bug.cgi?id=23324
Summary
Move platform/chromium/EditorChromium.cpp into editing/chromium/
Darin Fisher (:fishd, Google)
Reported
2009-01-14 10:43:55 PST
Move platform/chromium/EditorChromium.cpp into editing/chromium/ When we originally created this file, other ports also put the equivalent file under the platform/ directory, but since this is part of the Editor class implementation, it clearly belongs under editing/ instead. Also, it looks like other ports have made the same change.
Attachments
v1 patch
(5.11 KB, patch)
2009-01-14 10:44 PST
,
Darin Fisher (:fishd, Google)
darin
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Darin Fisher (:fishd, Google)
Comment 1
2009-01-14 10:44:50 PST
Created
attachment 26718
[details]
v1 patch
Darin Adler
Comment 2
2009-01-14 10:49:37 PST
Comment on
attachment 26718
[details]
v1 patch r=me
> + * platform/text/chromium: Added. > + * platform/text/chromium/TextBreakIteratorInternalICUChromium.cpp: Added. > + (WebCore::currentTextBreakLocaleID):
This seems to be the ChangeLog from another change.
Darin Fisher (:fishd, Google)
Comment 3
2009-01-14 11:00:14 PST
whoops... yes. i will submit with the correct ChangeLog entry.
Darin Fisher (:fishd, Google)
Comment 4
2009-01-14 11:02:54 PST
http://trac.webkit.org/changeset/39896
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug