From d8803adf21c9f41f0388e6ecadae10472a012395 Mon Sep 17 00:00:00 2001 From: Yuni Jeong Date: Mon, 20 May 2013 23:26:38 +0900 Subject: [PATCH] Added "Copy" context menu in case that selected node has imageURL. [Title] Added "Copy" context menu in case that selected node has imageURL. [Issue#] P130517-3062 [Problem] when "SelectMode" context menu is selected after longtap on image in reader view, context menu(ex - Copy...) for text selection is not displayed. [Cause] If selected node has linkURL, "Copy" context menu is added. In this case, selected node has only imageURL and does not has linkURL. So, "Copy" context menu is not added. [Solution] Added "Copy" context menu in case that selected node has imageURL. Change-Id: Ifedc0aa5a8b91a176653d7add6637fa9e669975b --- Source/WebCore/page/ContextMenuController.cpp | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/Source/WebCore/page/ContextMenuController.cpp b/Source/WebCore/page/ContextMenuController.cpp index 983c1cd..04097ec 100755 --- a/Source/WebCore/page/ContextMenuController.cpp +++ b/Source/WebCore/page/ContextMenuController.cpp @@ -854,9 +854,8 @@ void ContextMenuController::populate() if (!linkURL.isEmpty()) { #if ENABLE(TIZEN_WEBKIT2_TEXT_SELECTION) if (m_hitTestResult.isSelected() || frame->selection()->isRange()) { - if (selectionContainsPossibleWord(frame)) { + if (selectionContainsPossibleWord(frame)) appendItem(SearchWebItem, m_contextMenu.get()); - } appendItem(CopyItem, m_contextMenu.get()); return; } @@ -880,6 +879,12 @@ void ContextMenuController::populate() KURL imageURL = m_hitTestResult.absoluteImageURL(); if (!imageURL.isEmpty()) { +#if ENABLE(TIZEN_WEBKIT2_TEXT_SELECTION) + if (m_hitTestResult.isSelected() || frame->selection()->isRange()) { + appendItem(CopyItem, m_contextMenu.get()); + return; + } +#endif if (!linkURL.isEmpty()) appendItem(*separatorItem(), m_contextMenu.get()); -- 2.7.4