Revert of Added code to fix visibility of select all option in context menu....

Revert of Added code to fix visibility of select all option in context menu. (patchset #3 id:40001 of https://codereview.chromium.org/402403003/)

Reason for revert:
This CL is causing Issue 438135 and Issue 450540. So reverting this.

Original issue's description:
> Added code to fix visibility of select all option in context menu.
> 
> If context menu is appearing on some input field. Select all option
> should appear only if some text is present inside input field.
> 
> BUG=395994
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179636

TBR=darin@chromium.org,aelias@chromium.org,abarth@chromium.org,esprehn@chromium.org,ankit2.kumar@samsung.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=395994

Review URL: https://codereview.chromium.org/942183005

git-svn-id: svn://svn.chromium.org/blink/trunk@190714 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent 5708971d
...@@ -218,10 +218,8 @@ void ContextMenuClientImpl::showContextMenu(const ContextMenu* defaultMenu) ...@@ -218,10 +218,8 @@ void ContextMenuClientImpl::showContextMenu(const ContextMenu* defaultMenu)
data.editFlags |= WebContextMenuData::CanPaste; data.editFlags |= WebContextMenuData::CanPaste;
if (toLocalFrame(m_webView->focusedCoreFrame())->editor().canDelete()) if (toLocalFrame(m_webView->focusedCoreFrame())->editor().canDelete())
data.editFlags |= WebContextMenuData::CanDelete; data.editFlags |= WebContextMenuData::CanDelete;
if (isHTMLTextFormControlElement(r.innerNonSharedNode())) { // We can always select all...
if (!toHTMLTextFormControlElement(r.innerNonSharedNode())->value().isEmpty()) data.editFlags |= WebContextMenuData::CanSelectAll;
data.editFlags |= WebContextMenuData::CanSelectAll;
}
data.editFlags |= WebContextMenuData::CanTranslate; data.editFlags |= WebContextMenuData::CanTranslate;
// Links, Images, Media tags, and Image/Media-Links take preference over // Links, Images, Media tags, and Image/Media-Links take preference over
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment