TestShell: Backported EventSender.contextClick() improvement

TestShell: Backported EventSender.contextClick() improvement
to mock menu item strings.

BUG=63460
TEST=LayoutTest/editing/spelling/context-menu-suggestions.html

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=66603

[gmorrita, 2010-11-17 12:38:17]

Hi Kent-san, could you take a look?

[tkent, 2010-11-18 01:38:46]

r- because of some style errors.

http://codereview.chromium.org/5120002/diff/2001/webkit/tools/test_shell/even...
File webkit/tools/test_shell/event_sending_controller.cc (right):

http://codereview.chromium.org/5120002/diff/2001/webkit/tools/test_shell/even...
webkit/tools/test_shell/event_sending_controller.cc:786: {
"{" should be put at the last of the previous line.

http://codereview.chromium.org/5120002/diff/2001/webkit/tools/test_shell/even...
webkit/tools/test_shell/event_sending_controller.cc:790: "Back", "Reload Page",
"Open in Dashbaord", "<separator>", "View Source",
We usually use +4 spaces for line break in an expression.

[tkent, 2010-11-18 01:39:37]

Also, you should use try servers for layout tests.

[gmorrita, 2010-11-18 03:33:38]

Hi, thank you for reviewing!
I updated patch, sending a request to trybots.

http://codereview.chromium.org/5120002/diff/2001/webkit/tools/test_shell/even...
File webkit/tools/test_shell/event_sending_controller.cc (right):

http://codereview.chromium.org/5120002/diff/2001/webkit/tools/test_shell/even...
webkit/tools/test_shell/event_sending_controller.cc:786: {
On 2010/11/18 01:38:47, Kent Tamura wrote:
> "{" should be put at the last of the previous line.

Done.

http://codereview.chromium.org/5120002/diff/2001/webkit/tools/test_shell/even...
webkit/tools/test_shell/event_sending_controller.cc:790: "Back", "Reload Page",
"Open in Dashbaord", "<separator>", "View Source",
On 2010/11/18 01:38:47, Kent Tamura wrote:
> We usually use +4 spaces for line break in an expression.

Done.

[tkent, 2010-11-18 09:42:19]

LGTM