Make DummyPageHolder to have all empty clients

Make DummyPageHolder to have all empty clients

This patch changes DummyPageHolder to use fillWithEmptyClients() instead of making specific empty clients for writing tests depending on client objects not created in current DumpPageHolder class, e.g. spell checker client for editing test.
 
BUG=n/a
TEST=n/a

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

[yosin_UTC9, 2014-06-05 07:43:25]

Could you review this patch?
Thanks in advance.

[tkent, 2014-06-05 07:57:32]

https://codereview.chromium.org/311263007/diff/1/Source/core/testing/DummyPag...
File Source/core/testing/DummyPageHolder.cpp (right):

https://codereview.chromium.org/311263007/diff/1/Source/core/testing/DummyPag...
Source/core/testing/DummyPageHolder.cpp:55: m_pageClients.spellCheckerClient =
&m_spellCheckerClient;
Can you use fillWithEmptyClients()?

[yosin_UTC9, 2014-06-05 09:04:13]

PTAL

https://codereview.chromium.org/311263007/diff/1/Source/core/testing/DummyPag...
File Source/core/testing/DummyPageHolder.cpp (right):

https://codereview.chromium.org/311263007/diff/1/Source/core/testing/DummyPag...
Source/core/testing/DummyPageHolder.cpp:55: m_pageClients.spellCheckerClient =
&m_spellCheckerClient;
On 2014/06/05 07:57:32, tkent wrote:
> Can you use fillWithEmptyClients()?

Done.

[tkent, 2014-06-05 09:39:10]

lgtm

[commit-bot: I haz the power, 2014-06-05 09:40:46]

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

[commit-bot: I haz the power, 2014-06-05 12:51:39]

Change committed as 175563