test_runner: Migrate MockColorChooser to Chromium C++ style.

test_runner: Migrate MockColorChooser to Chromium C++ style.

Changes:
1) Run clang-format through source and header files.
2) Rename data member variables to use unix_hacker_ style.
3) Rename methods to use CamelCase style.
4) Rename file name to mock_color_chooser.

BUG=331299
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284689

[jochen (gone - plz use gerrit), 2014-07-22 07:26:45]

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
File content/shell/renderer/test_runner/mock_color_chooser.cc (right):

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_color_chooser.cc:10: using namespace
blink;
don't use namespace blink, but either use the types needed, or add blink::
everywhere

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_color_chooser.cc:20: ,
callback_(callback)
, should go on previous line

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_color_chooser.cc:34: ,
delegate_(delegate)
same here

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_color_chooser.cc:45: void
MockColorChooser::setSelectedColor(const blink::WebColor)
add parameter name also for unused args

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
File content/shell/renderer/test_runner/mock_color_chooser.h (right):

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_color_chooser.h:21:
MockColorChooser(blink::WebColorChooserClient*,
add parameter names

[Abhishek, 2014-07-22 10:49:57]

All comments are incorporated.
PTAL!

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
File content/shell/renderer/test_runner/mock_color_chooser.cc (right):

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_color_chooser.cc:10: using namespace
blink;
On 2014/07/22 07:26:45, jochen wrote:
> don't use namespace blink, but either use the types needed, or add blink::
> everywhere

Done.

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_color_chooser.cc:20: ,
callback_(callback)
On 2014/07/22 07:26:45, jochen wrote:
> , should go on previous line

Done.

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_color_chooser.cc:34: ,
delegate_(delegate)
On 2014/07/22 07:26:45, jochen wrote:
> same here

Done.

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_color_chooser.cc:45: void
MockColorChooser::setSelectedColor(const blink::WebColor)
On 2014/07/22 07:26:45, jochen wrote:
> add parameter name also for unused args

Done.

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
File content/shell/renderer/test_runner/mock_color_chooser.h (right):

https://codereview.chromium.org/404373002/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_color_chooser.h:21:
MockColorChooser(blink::WebColorChooserClient*,
On 2014/07/22 07:26:45, jochen wrote:
> add parameter names

Done.

[jochen (gone - plz use gerrit), 2014-07-22 10:54:36]

lgtm

[Abhishek, 2014-07-22 10:54:59]

The CQ bit was checked by abhishek.a21@samsung.com

[commit-bot: I haz the power, 2014-07-22 10:56:07]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/abhishek.a21@samsung.com/404373002/20001

[commit-bot: I haz the power, 2014-07-22 16:03:59]

Change committed as 284689