fix unittests

base/clipboard_util.cc

Index: base/clipboard_util.cc
diff --git a/base/clipboard_util.cc b/base/clipboard_util.cc
index 5b64732adac86d632c972a4fb032f608cdc62f8f..a9e7d2afbf4b40fb651b05df1783eb12e50531a4 100644
--- a/base/clipboard_util.cc
+++ b/base/clipboard_util.cc
@@ -466,6 +466,7 @@ void ClipboardUtil::CFHtmlToHtml(const std::string& cf_html,
     size_t src_start = line_start + src_url_str.length();
     if (src_end != std::string::npos && src_start != std::string::npos) {
       *base_url = cf_html.substr(src_start, src_end - src_start);
+      TrimWhitespace(*base_url, TRIM_ALL, base_url);
     }
   }
 
@@ -474,7 +475,7 @@ void ClipboardUtil::CFHtmlToHtml(const std::string& cf_html,
   size_t markup_start = cf_html_lower.find("<html", 0);

[Evan Stade, 2008/11/19 02:39:33]

The function we had previously (which looks like it was based off the same
webkit code) has

  .find('<', 0);

here instead of

  .find("<html",0);

I don't know which one is better. I guess you could see which one is more
recent.

   size_t tag_start = cf_html.find("StartFragment", markup_start);
   size_t fragment_start = cf_html.find('>', tag_start) + 1;
-  size_t tag_end = cf_html.find("EndFragment", fragment_start);
+  size_t tag_end = cf_html.rfind("EndFragment", std::string::npos);
   size_t fragment_end = cf_html.rfind('<', tag_end);
   if (fragment_start != std::string::npos &&
       fragment_end != std::string::npos) {