fix unittests

base/clipboard_util.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/clipboard_util.h"
 
 #include <shellapi.h>
 #include <shlwapi.h>
 #include <wininet.h>
 
@@ skipping 448 matching lines @@
                                  std::string* html,
                                  std::string* base_url) {
   // Obtain base_url if present.
   static std::string src_url_str("SourceURL:");
   size_t line_start = cf_html.find(src_url_str);
   if (line_start != std::string::npos) {
     size_t src_end = cf_html.find("\n", line_start);
     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);
     }
   }
 
   // Find the markup between "<!--StartFragment -->" and "<!--EndFragment-->".
   std::string cf_html_lower = StringToLowerASCII(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) {
     *html = cf_html.substr(fragment_start, fragment_end - fragment_start);
     TrimWhitespace(*html, TRIM_ALL, html);
   }
 }