[Chromoting] Hook up host talkgadget policy checks.

remoting/host/constants.h

Index: remoting/host/constants.h
diff --git a/remoting/host/constants.h b/remoting/host/constants.h
index 54aa2280c657a5ec1fdc210838a88f1c20071179..02f2f206f0fbe265646a8947dfa408bbf3b14f18 100644
--- a/remoting/host/constants.h
+++ b/remoting/host/constants.h
@@ -9,10 +9,13 @@
 
 namespace remoting {
 
-// This is the default prefix that is prepended to ".talkgadget.google.com"
-// to form the complete talkgadget domain name. Policy settings allow admins
+// This is the default prefix that is prepended to the kTalkGadgetURL to form
+// the complete talkgadget URL used by the host. Policy settings allow admins
 // to change the prefix that is used.
-extern const char kDefaultTalkGadgetPrefix[];
+extern const char kDefaultHostTalkGadgetPrefix[];

[Sergey Ulanov, 2012/08/24 19:43:33]

We should avoid adding more values to this file (ideally this file shouldn't
exist at all). 
kDefaultHostTalkGadgetPrefix is used only in policy_watcher.cc, so it might be
better to move it there.
kTalkGadgetURL is also used in only one file.

[garykac, 2012/08/27 22:34:57]

I would rather merge constants_mac.cc and constants_win.cc into this file.
The _unittest file uses it as well.

[Sergey Ulanov, 2012/08/28 00:33:42]

This has been discussed some time ago:
https://chromiumcodereview.appspot.com/10579002/
It's best to avoid dumping all shared constants in one file.
maybe put them in connection_block_checker.h ?

+
+// The base talkgadget URL.
+extern const char kTalkGadgetURL[];

[Sergey Ulanov, 2012/08/24 19:43:33]

This should be kTalkGadgetUrl. From chromium code style guide:
  Prefer to capitalize well-known abbreviations as if they were words: HttpUrl
instead of HTTPURL.

[garykac, 2012/08/27 22:34:57]

Done.

 
 // Known host exit codes.
 // Please keep this enum in sync with: