[Webkit-unassigned] [Bug 56425] WebKit2 Icon Database
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Mar 15 18:31:30 PDT 2011
https://bugs.webkit.org/show_bug.cgi?id=56425
Sam Weinig <sam at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #85882|review? |review+
Flag| |
--- Comment #4 from Sam Weinig <sam at webkit.org> 2011-03-15 18:31:30 PST ---
(From update of attachment 85882)
View in context: https://bugs.webkit.org/attachment.cgi?id=85882&action=review
> Source/WebKit2/WebProcess/IconDatabase/WebIconDatabaseProxy.cpp:109
> +String WebIconDatabaseProxy::iconURLForPageURL(const String& pageURL)
> +{
> + String result;
> + if (!m_process->connection()->sendSync(Messages::WebIconDatabase::IconURLForPageURL(pageURL), Messages::WebIconDatabase::IconURLForPageURL::Reply(result), 0))
> + return String();
> +
> + return result;
> +}
> +
> +bool WebIconDatabaseProxy::iconDataKnownForIconURL(const String& iconURL)
> +{
> + bool result;
> + if (!m_process->connection()->sendSync(Messages::WebIconDatabase::IconDataKnownForIconURL(iconURL), Messages::WebIconDatabase::IconDataKnownForIconURL::Reply(result), 0))
> + return false;
> +
> + return result;
> +}
> +
> +IconLoadDecision WebIconDatabaseProxy::loadDecisionForIconURL(const String& iconURL, DocumentLoader* documentLoader)
> +{
> + IconLoadDecision result;
> + if (!m_process->connection()->sendSync(Messages::WebIconDatabase::LoadDecisionForIconURL(iconURL), Messages::WebIconDatabase::LoadDecisionForIconURL::Reply((int&)result), 0))
> + return IconLoadNo;
> +
> + return result;
> +}
These should not be sync messages. Please don't land this part.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list