Implement visited link coloring.

webkit/glue/chromium_bridge_impl.cc

 // Copyright (c) 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 "config.h"
 #include "ChromiumBridge.h"
 
 #include "BitmapImage.h"
 #include "ClipboardUtilitiesChromium.h"
 #include "Cursor.h"
@@ skipping 15 matching lines @@
 
 #undef LOG
 #include "base/clipboard.h"
 #include "base/file_util.h"
 #include "base/message_loop.h"
 #include "base/stats_counters.h"
 #include "base/string_util.h"
 #include "base/time.h"
 #include "base/trace_event.h"
 #include "build/build_config.h"
+#include "googleurl/src/url_util.h"
 #include "net/base/mime_util.h"
 #if USE(V8)
 #include <v8.h>
 #endif
 #include "webkit/glue/chrome_client_impl.h"
 #include "webkit/glue/glue_util.h"
 #include "webkit/glue/plugins/plugin_instance.h"
 #include "webkit/glue/scoped_clipboard_writer_glue.h"
 #include "webkit/glue/webcursor.h"
 #include "webkit/glue/webkit_glue.h"
@@ skipping 468 matching lines @@
                                    const char* extra) {
   TRACE_EVENT_END(name, id, extra);
 }
 
 // URL ------------------------------------------------------------------------
 
 KURL ChromiumBridge::inspectorURL() {
   return webkit_glue::GURLToKURL(webkit_glue::GetInspectorURL());
 }
 
+// Visited links --------------------------------------------------------------
+
+WebCore::LinkHash ChromiumBridge::visitedLinkHash(const UChar* url,
+                                                  unsigned length) {
+  url_canon::RawCanonOutput<2048> buffer;

[macdome_gmail.com, 2008/12/13 01:36:27]

Pixels?  Edict?  Sound?  Yes, we speak a crazy language.

+  url_parse::Parsed parsed;
+  if (!url_util::Canonicalize(url, length, NULL, &buffer, &parsed))
+    return false;  // Invalid URLs are unvisited.
+  return webkit_glue::VisitedLinkHash(buffer.data(), buffer.length());
+}
+
+WebCore::LinkHash ChromiumBridge::visitedLinkHash(
+    const WebCore::KURL& base,
+    const WebCore::AtomicString& attributeURL) {
+  // Resolve the relative URL using googleurl and pass the absolute URL up to
+  // the embedder. We could create a GURL object from the base and resolve the
+  // relative URL that way, but calling the lower-level functions directly
+  // saves us the std::string allocation in most cases.
+  url_canon::RawCanonOutput<2048> buffer;
+  url_parse::Parsed parsed;
+
+  WebCore::CString cstr = base.utf8String();
+  if (!url_util::ResolveRelative(cstr.data(), cstr.length(), base.parsed(),
+                                 attributeURL.characters(),
+                                 attributeURL.length(), NULL,
+                                 &buffer, &parsed))
+    return false;  // Invalid resolved URL.
+
+  return webkit_glue::VisitedLinkHash(buffer.data(), buffer.length());
+}
+
+bool ChromiumBridge::isLinkVisited(WebCore::LinkHash visitedLinkHash) {
+  return webkit_glue::IsLinkVisited(visitedLinkHash);
+}
+
 // Widget ---------------------------------------------------------------------
 
 void ChromiumBridge::widgetSetCursor(Widget* widget, const Cursor& cursor) {
   ChromeClientImpl* chrome_client = ToChromeClient(widget);
   if (chrome_client)
     chrome_client->SetCursor(WebCursor(cursor.impl()));
 }
 
 void ChromiumBridge::widgetSetFocus(Widget* widget) {
   ChromeClientImpl* chrome_client = ToChromeClient(widget);
   if (chrome_client)
     chrome_client->focus();
 }
 
-// Link history ---------------------------------------------------------------
-
-bool ChromiumBridge::isLinkVisited(LinkHash) {
-  // NOT IMPLEMENTED
-  // http://code.google.com/p/chromium/issues/detail?id=5401
-  return false;
-}
-
 }  // namespace WebCore