WebFrameImpl::GetSelection(): Replace NBSPs with spaces....

webkit/glue/webframe_impl.cc

 /*
  * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2006 Apple Computer, Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 58 matching lines @@
 #include "base/compiler_specific.h"
 #include "build/build_config.h"
 
 #include <algorithm>
 #include <string>
 
 MSVC_PUSH_WARNING_LEVEL(0);
 #include "HTMLFormElement.h"  // need this before Document.h
 #include "Chrome.h"
 #include "ChromeClientChromium.h"
+#include "ClipboardUtilitiesChromium.h"
 #include "Console.h"
 #include "Document.h"
 #include "DocumentFragment.h"  // Only needed for ReplaceSelectionCommand.h :(
 #include "DocumentLoader.h"
 #include "DocumentMarker.h"
 #include "DOMWindow.h"
 #include "Editor.h"
 #include "EventHandler.h"
 #include "FormState.h"
 #include "Frame.h"
@@ skipping 1291 matching lines @@
 
 std::string WebFrameImpl::GetSelection(bool as_html) {
   RefPtr<Range> range = frame()->selection()->toNormalizedRange();
   if (!range.get())
     return std::string();
 
   if (as_html) {
     String markup = WebCore::createMarkup(range.get(), 0);
     return webkit_glue::StringToStdString(markup);
   } else {
-    return webkit_glue::StringToStdString(range->text());
+    String text = range->text();
+#if defined(OS_WIN)
+    WebCore::replaceNewlinesWithWindowsStyleNewlines(text);
+#endif
+    WebCore::replaceNBSPWithSpace(text);
+    return webkit_glue::StringToStdString(text);
   }
 }
 
 std::string WebFrameImpl::GetFullPageHtml() {
   return webkit_glue::StringToStdString(createFullMarkup(frame_->document()));
 }
 
 void WebFrameImpl::CreateFrameView() {
   ASSERT(frame_);  // If frame_ doesn't exist, we probably didn't init properly.
 
@@ skipping 417 matching lines @@
 
   SecurityOrigin* security_origin = frame_->document()->securityOrigin();
 
   if (!frame_->loader()->isScheduledLocationChangePending()) {
     frame_->loader()->stopAllLoaders();
     frame_->loader()->begin(frame_->loader()->url(), true, security_origin);
     frame_->loader()->write(script_result);
     frame_->loader()->end();
   }
 }