Make it possible to override the tickmarks for a WebFrame.

Make it possible to override the tickmarks for a WebFrame.

This makes it possible to override the default tickmark behavior for a frame.
The default behavior is to display tickmarks for find results. However the PDF
plugin handles finding itself but still wants to set tickmarks. This CL makes
it possible to specify an array of tickmarks to display.

BUG=303491
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169423

[raymes, 2014-03-13 07:03:22]

ptal. Thanks!

[jamesr, 2014-03-13 22:53:05]

lgtm, but i think you can add some tests to make sure these map to the WebFrame
correctly and that the default tickmarks are restored by passing an empty
vector. take a look at WebFrameTests.cpp (which is very messy but should be
powerful enough for you)

[raymes, 2014-03-17 00:31:06]

Thanks James! I added the test. Could you please take a quick look at it?

[jamesr, 2014-03-17 05:32:02]

On 2014/03/17 00:31:06, raymes wrote:
> Thanks James! I added the test. Could you please take a quick look at it?

Looks good - thanks!

[raymes, 2014-03-17 05:45:31]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2014-03-17 05:45:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/raymes@chromium.org/198683002/20001

[commit-bot: I haz the power, 2014-03-17 05:57:13]

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

[commit-bot: I haz the power, 2014-03-17 05:57:13]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_dbg

[raymes, 2014-03-18 06:08:51]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2014-03-18 06:08:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/raymes@chromium.org/198683002/40001

[commit-bot: I haz the power, 2014-03-18 07:13:49]

Change committed as 169423