Simplify TestRunnerBindings::SimulateWebNotificationClick param handling

Simplify TestRunnerBindings::SimulateWebNotificationClick param handling

This CL reduces complexity on the
TestRunnerBindings::SimulateWebNotificationClick method by having it
receive the actual parameters instead of unpacking them from a
gin::Arguments object.

BUG=582124
TEST=LayoutTests/http/tests/notifications/*

Committed: https://crrev.com/c0939ba76ea40b78a7599cfa7915dac7df8c669b
Cr-Commit-Position: refs/heads/master@{#373304}

[Nina, 2016-01-28 17:17:14]

nsatragno@chromium.org changed reviewers:
+ johnme@chromium.org

[Nina, 2016-01-28 17:17:14]

nsatragno@chromium.org changed required reviewers:
+ johnme@chromium.org

[Nina, 2016-01-28 17:17:14]

Hi John,

Would you minding giving this patch a look? Thanks.

Nico.

[Nina, 2016-01-28 17:17:42]

nsatragno@chromium.org changed reviewers:
+ peter@chromium.org

[Nina, 2016-01-28 17:17:42]

CC Peter - sorry for the spam :)

[johnme, 2016-01-28 19:12:12]

lgtm, thanks :)

[Nina, 2016-02-01 11:36:05]

nsatragno@chromium.org changed reviewers:
+ rbyers@chromium.org

[Nina, 2016-02-01 11:36:05]

nsatragno@chromium.org changed required reviewers:
+ rbyers@chromium.org

[Nina, 2016-02-01 11:36:06]

Hi Rick,

Could you give test_runner.cc a review?

Thanks,
Nico.

[Rick Byers, 2016-02-02 04:46:35]

test_runner LGTM - thanks for the cleanup!

[Nina, 2016-02-03 10:46:03]

The CQ bit was checked by nsatragno@chromium.org

[commit-bot: I haz the power, 2016-02-03 10:46:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1644063002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1644063002/1

[commit-bot: I haz the power, 2016-02-03 10:46:28]

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

[commit-bot: I haz the power, 2016-02-03 10:46:29]

The author nsatragno@chromium.org has not signed Google Contributor License
Agreement. Please visit https://cla.developers.google.com to sign and manage
CLA.

[Nina, 2016-02-03 17:57:55]

The CQ bit was checked by nsatragno@chromium.org

[commit-bot: I haz the power, 2016-02-03 18:00:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1644063002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1644063002/1

[commit-bot: I haz the power, 2016-02-03 19:37:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-03 19:38:33]

Description was changed from

==========
Simplify TestRunnerBindings::SimulateWebNotificationClick param handling

This CL reduces complexity on the
TestRunnerBindings::SimulateWebNotificationClick method by having it
receive the actual parameters instead of unpacking them from a
gin::Arguments object.

BUG=582124
TEST=LayoutTests/http/tests/notifications/*
==========

to

==========
Simplify TestRunnerBindings::SimulateWebNotificationClick param handling

This CL reduces complexity on the
TestRunnerBindings::SimulateWebNotificationClick method by having it
receive the actual parameters instead of unpacking them from a
gin::Arguments object.

BUG=582124
TEST=LayoutTests/http/tests/notifications/*

Committed: https://crrev.com/c0939ba76ea40b78a7599cfa7915dac7df8c669b
Cr-Commit-Position: refs/heads/master@{#373304}
==========

[commit-bot: I haz the power, 2016-02-03 19:38:34]

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