test_runner: Migrate MockSpellCheck to Chromium C++ style.

test_runner: Migrate MockSpellCheck 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_spell_check.

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

[jochen (gone - plz use gerrit), 2014-07-24 08:47:05]

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

https://codereview.chromium.org/410283003/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_spell_check.cc:33: int*
misspelledOffset,
misspelled_offset

https://codereview.chromium.org/410283003/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_spell_check.cc:47: base::string16
stringText = text;
string_text (also all other variables)

[Abhishek, 2014-07-24 10:08:39]

All the variables name has been addressed as per the comment.
PTAL !

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

https://codereview.chromium.org/410283003/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_spell_check.cc:33: int*
misspelledOffset,
On 2014/07/24 08:47:04, jochen wrote:
> misspelled_offset

Done.

https://codereview.chromium.org/410283003/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/mock_spell_check.cc:47: base::string16
stringText = text;
On 2014/07/24 08:47:04, jochen wrote:
> string_text (also all other variables)

Done.

[jochen (gone - plz use gerrit), 2014-07-24 12:17:21]

lgtm with nit addressed

https://codereview.chromium.org/410283003/diff/20001/content/shell/renderer/t...
File content/shell/renderer/test_runner/mock_spell_check.h (right):

https://codereview.chromium.org/410283003/diff/20001/content/shell/renderer/t...
content/shell/renderer/test_runner/mock_spell_check.h:41: int* misspelledOffset,
mispelled_offset

[Abhishek, 2014-07-24 12:57:13]

Resolved comments inside mock_spell_check.h.
Also corrected indentation as per chromium style of 80 chars inside
mock_spell_check.cc

PTAL !

https://codereview.chromium.org/410283003/diff/20001/content/shell/renderer/t...
File content/shell/renderer/test_runner/mock_spell_check.h (right):

https://codereview.chromium.org/410283003/diff/20001/content/shell/renderer/t...
content/shell/renderer/test_runner/mock_spell_check.h:41: int* misspelledOffset,
On 2014/07/24 12:17:21, jochen wrote:
> mispelled_offset

Done.

[Abhishek, 2014-07-24 13:49:09]

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

[commit-bot: I haz the power, 2014-07-24 13:51:14]

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

[jochen (gone - plz use gerrit), 2014-07-24 14:53:07]

you'll need to update the BUILD.gn file as well

otherwise still lgtm

[commit-bot: I haz the power, 2014-07-24 15:35:08]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_gn_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_gn_rel...)

[commit-bot: I haz the power, 2014-07-24 15:44:45]

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

[commit-bot: I haz the power, 2014-07-24 15:44:46]

Try jobs failed on following builders:
  linux_chromium_gn_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_gn_rel...)

[Abhishek, 2014-07-25 07:58:47]

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

[commit-bot: I haz the power, 2014-07-25 07:59:14]

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

[commit-bot: I haz the power, 2014-07-25 12:20:07]

Change committed as 285566