Cleanup.
Created attachment 207706 [details] proposed patch
Comment on attachment 207706 [details] proposed patch Attachment 207706 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/1293280 New failing tests: editing/selection/leak-document-with-selection-inside.html
Created attachment 207712 [details] Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-10 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.4
Comment on attachment 207706 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=207706&action=review > Source/WebCore/editing/Editor.cpp:1870 > Vector<String> Editor::guessesForUngrammaticalSelection() > { Can we remove this function?
(In reply to comment #4) > (From update of attachment 207706 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=207706&action=review > > > Source/WebCore/editing/Editor.cpp:1870 > > Vector<String> Editor::guessesForUngrammaticalSelection() > > { > > Can we remove this function? It is still called in http://trac.webkit.org/browser/trunk/Source/WebCore/editing/Editor.cpp#L1919. Should we remove this block as well? Thanks for review!
(In reply to comment #5) > > It is still called in http://trac.webkit.org/browser/trunk/Source/WebCore/editing/Editor.cpp#L1919. Should we remove this block as well? > Thanks for review! It seems like we should if that code is never used.
(In reply to comment #6) > (In reply to comment #5) > > > > It is still called in http://trac.webkit.org/browser/trunk/Source/WebCore/editing/Editor.cpp#L1919. Should we remove this block as well? > > Thanks for review! > > It seems like we should if that code is never used. Ok, I'll get rid of it while submitting.
Committed r153515: <http://trac.webkit.org/changeset/153515>