Add SpellingServiceClient::IsAvailable().

Add SpellingServiceClient::IsAvailable().
This change adds a static function SpellingServiceClient::IsAvailable() and use it instead of checking conditions directly.

BUG=none
TEST=SpellingServiceClientTest.AvailableServices

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

[Hironori Bono, 2012-06-13 11:01:38]

Greetings Avi and Morita-san,

Thanks for your help in advance.
Would it be possible to review this refactoring change? This change adds a new
function and moves complicated checks which service is available for what locale
there.

Regards,

Hironori Bono

[Avi (use Gerrit), 2012-06-13 14:46:46]

lgtm

http://codereview.chromium.org/10536141/diff/5/chrome/browser/spellchecker/sp...
File chrome/browser/spellchecker/spelling_service_client.h (right):

http://codereview.chromium.org/10536141/diff/5/chrome/browser/spellchecker/sp...
chrome/browser/spellchecker/spelling_service_client.h:92: // Returns whether the
specified service is avilable for the given profile.
s/avilable/available/

[commit-bot: I haz the power, 2012-06-14 03:27:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hbono@chromium.org/10536141/13002

[Hironori Bono, 2012-06-14 03:28:38]

Greetings Avi,

Many thanks for your quick review. :)

Regards,

Hironori Bono

http://codereview.chromium.org/10536141/diff/5/chrome/browser/spellchecker/sp...
File chrome/browser/spellchecker/spelling_service_client.h (right):

http://codereview.chromium.org/10536141/diff/5/chrome/browser/spellchecker/sp...
chrome/browser/spellchecker/spelling_service_client.h:92: // Returns whether the
specified service is avilable for the given profile.
On 2012/06/13 14:46:46, Avi wrote:
> s/avilable/available/

Done.

[commit-bot: I haz the power, 2012-06-14 05:38:12]

Change committed as 142102