Move visited link handling to VisitedLinkTableController and VisitedLinkProvider
Created attachment 226648 [details] Patch
Comment on attachment 226648 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=226648&action=review > Source/WebKit2/ChangeLog:26 > + add the VisitedLinkProvider object as a message receiver and send the visited link table to the process. extra space > Source/WebKit2/UIProcess/VisitedLinkProvider.cpp:180 > + ASSERT(currentTableMemory->size() == m_tableSize * sizeof(LinkHash)); This isn’t right > Source/WebKit2/UIProcess/VisitedLinkProvider.cpp:192 > + if (!m_table.addLinkHash(linkHash)) > + ASSERT_NOT_REACHED(); It’s nicer to use a local bool and ASSERT_UNUSED in such cases. > Source/WebKit2/UIProcess/VisitedLinkProvider.messages.in:1 > +# Copyright (C) 2011 Apple Inc. All rights reserved. 2011? > Source/WebKit2/WebProcess/WebPage/VisitedLinkTableController.h:52 > + void visitedLinkStateChanged(const Vector<WebCore::LinkHash>& linkHashes); Don’t need to name this parameter. > Source/WebKit2/WebProcess/WebPage/VisitedLinkTableController.messages.in:1 > +# Copyright (C) 2011 Apple Inc. All rights reserved. 2011 again?
Committed r165619: <http://trac.webkit.org/changeset/165619>