Start displaying title text for HTML elements without links.

Start displaying title text for HTML elements without links.

This CL depends on Blink change https://chromiumcodereview.appspot.com/1128933011

BUG=487782
Committed: https://crrev.com/f04695585e3c0e532aa7c6f3dda04832b6c6fdc5
Cr-Commit-Position: refs/heads/master@{#330282}

[aurimas (slooooooooow), 2015-05-15 23:14:44]

aurimas@chromium.org changed reviewers:
+ tedchoc@chromium.org

[aurimas (slooooooooow), 2015-05-15 23:14:45]

PTAL

[Ted C, 2015-05-15 23:24:55]

lgtm

https://chromiumcodereview.appspot.com/1126203011/diff/20001/chrome/android/j...
File
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuParams.java
(right):

https://chromiumcodereview.appspot.com/1126203011/diff/20001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuParams.java:114:
* @return The title or alt atribute (if title is not available).
two ts in attribute

https://chromiumcodereview.appspot.com/1126203011/diff/20001/content/browser/...
File content/browser/frame_host/render_frame_host_impl.cc (right):

https://chromiumcodereview.appspot.com/1126203011/diff/20001/content/browser/...
content/browser/frame_host/render_frame_host_impl.cc:1160: 
?

https://chromiumcodereview.appspot.com/1126203011/diff/20001/content/public/c...
File content/public/common/context_menu_params.h (right):

https://chromiumcodereview.appspot.com/1126203011/diff/20001/content/public/c...
content/public/common/context_menu_params.h:107: // This is the title or alt (it
title not available) text of the selection
"if title"

[aurimas (slooooooooow), 2015-05-15 23:34:11]

https://chromiumcodereview.appspot.com/1126203011/diff/20001/chrome/android/j...
File
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuParams.java
(right):

https://chromiumcodereview.appspot.com/1126203011/diff/20001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuParams.java:114:
* @return The title or alt atribute (if title is not available).
On 2015/05/15 at 23:24:55, Ted C wrote:
> two ts in attribute

Done

https://chromiumcodereview.appspot.com/1126203011/diff/20001/content/browser/...
File content/browser/frame_host/render_frame_host_impl.cc (right):

https://chromiumcodereview.appspot.com/1126203011/diff/20001/content/browser/...
content/browser/frame_host/render_frame_host_impl.cc:1160: 
On 2015/05/15 at 23:24:55, Ted C wrote:
> ?

Had logs in there. Done.

[aurimas (slooooooooow), 2015-05-15 23:36:06]

aurimas@chromium.org changed reviewers:
+ palmer@chromium.org, sievers@chromium.org

[aurimas (slooooooooow), 2015-05-15 23:36:06]

+palmer for content/common/frame_messages.h
+sievers for rest of content/

[palmer, 2015-05-15 23:44:30]

lgtm

[no sievers, 2015-05-15 23:53:26]

lgtm

[aurimas (slooooooooow), 2015-05-16 04:52:24]

The CQ bit was checked by aurimas@chromium.org

[aurimas (slooooooooow), 2015-05-16 04:52:24]

The patchset sent to the CQ was uploaded after l-g-t-m from tedchoc@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/1126203011/#ps40001
(title: "Ted's comments addressed")

[commit-bot: I haz the power, 2015-05-16 04:52:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1126203011/40001

[commit-bot: I haz the power, 2015-05-16 04:55:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-05-16 04:55:07]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[aurimas (slooooooooow), 2015-05-16 05:16:23]

The CQ bit was checked by aurimas@chromium.org

[aurimas (slooooooooow), 2015-05-16 05:16:24]

The patchset sent to the CQ was uploaded after l-g-t-m from
tedchoc@chromium.org, palmer@chromium.org, sievers@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/1126203011/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2015-05-16 05:16:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1126203011/60001

[commit-bot: I haz the power, 2015-05-16 06:13:36]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-18 11:31:40]

Patchset 4 (id:??) landed as
https://crrev.com/f04695585e3c0e532aa7c6f3dda04832b6c6fdc5
Cr-Commit-Position: refs/heads/master@{#330282}