diff options
author | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-10-16 19:19:41 +0000 |
---|---|---|
committer | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-10-16 19:19:41 +0000 |
commit | bd1aecfa71424dfa796e88a048b36e1ba5cc76bb (patch) | |
tree | d28a13f6bd2a1b5a5a3c3e0a733c3745862f0ba5 | |
parent | 4d608716c5f8fe564c6ec1e05267831802d9101c (diff) | |
download | chromium_src-bd1aecfa71424dfa796e88a048b36e1ba5cc76bb.zip chromium_src-bd1aecfa71424dfa796e88a048b36e1ba5cc76bb.tar.gz chromium_src-bd1aecfa71424dfa796e88a048b36e1ba5cc76bb.tar.bz2 |
Remove some more extraneous NOTIMPLEMENTEDs to "fix" failing layout tests.
It looks as though shouldMoveRangeAfterDelete exists for the purpose of non-browser embeddings of WebKit. WebKit on Windows also just answers true here without asking the embedder, whereas WebKit on Mac asks the embedder. I'm guessing that Mail.app may make use of this API to customize the behavior of WebKit's rich text editor. For us, just answering true seems fine.
TBR=tony
Review URL: http://codereview.chromium.org/7444
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@3473 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | webkit/glue/editor_client_impl.cc | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/webkit/glue/editor_client_impl.cc b/webkit/glue/editor_client_impl.cc index 5067466..74fce21 100644 --- a/webkit/glue/editor_client_impl.cc +++ b/webkit/glue/editor_client_impl.cc @@ -218,7 +218,6 @@ bool EditorClientImpl::shouldApplyStyle(WebCore::CSSStyleDeclaration* style, bool EditorClientImpl::shouldMoveRangeAfterDelete( WebCore::Range* /*range*/, WebCore::Range* /*rangeToBeReplaced*/) { - NOTIMPLEMENTED(); return true; } @@ -274,11 +273,9 @@ void EditorClientImpl::didEndEditing() { } void EditorClientImpl::didWriteSelectionToPasteboard() { - NOTIMPLEMENTED(); } void EditorClientImpl::didSetSelectionTypesForPasteboard() { - NOTIMPLEMENTED(); } void EditorClientImpl::registerCommandForUndo( |