remoting: use VerifyHostPinHash() in place on IsPinValid()

remoting/remoting_host_mac.gypi

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'variables': {
     'remoting_host_installer_mac_roots': [
       'host/installer/mac/',
       '<(DEPTH)/chrome/installer/mac/',
     ],
@@ skipping 160 matching lines @@
       'target_name': 'remoting_host_prefpane',
       'type': 'loadable_module',
       'mac_bundle': 1,
       'product_extension': 'prefPane',
       'variables': {
         'bundle_id': '<!(python <(version_py_path) -f <(branding_path) -t "@MAC_PREFPANE_BUNDLE_ID@")',
         'host_bundle_name': '<!(python <(version_py_path) -f <(branding_path) -t "@MAC_HOST_BUNDLE_NAME@")',
         'prefpane_bundle_name': '<!(python <(version_py_path) -f <(branding_path) -t "@MAC_PREFPANE_BUNDLE_NAME@")',
       },
       'dependencies': [
+        'remoting_base',
+        'remoting_host',
         'remoting_infoplist_strings',
+        '<(DEPTH)/third_party/jsoncpp/jsoncpp.gyp:jsoncpp',
       ],
       'defines': [
         'HOST_BUNDLE_NAME="<(host_bundle_name)"',
         'PREFPANE_BUNDLE_NAME="<(prefpane_bundle_name)"',
         'JSON_USE_EXCEPTION=0',
       ],
       'include_dirs': [
         '../third_party/jsoncpp/overrides/include/',
         '../third_party/jsoncpp/source/include/',
         '../third_party/jsoncpp/source/src/lib_json/',
       ],
-
-      # These source files are included directly, instead of adding target
-      # dependencies, because the targets are not yet built for 64-bit on
-      # Mac OS X - http://crbug.com/125116.
-      #
-      # TODO(lambroslambrou): Fix this when Chrome supports building for
-      # Mac OS X 64-bit - http://crbug.com/128122.
       'sources': [
-        '../third_party/jsoncpp/overrides/src/lib_json/json_value.cpp',
-        '../third_party/jsoncpp/overrides/src/lib_json/json_reader.cpp',
-        '../third_party/jsoncpp/source/src/lib_json/json_writer.cpp',
-        '../third_party/modp_b64/modp_b64.cc',
-        'host/constants_mac.cc',
-        'host/constants_mac.h',
-        'host/host_config_constants.cc',
         'host/mac/me2me_preference_pane.h',
         'host/mac/me2me_preference_pane.mm',
         'host/mac/me2me_preference_pane_confirm_pin.h',
         'host/mac/me2me_preference_pane_confirm_pin.mm',
         'host/mac/me2me_preference_pane_disable.h',
         'host/mac/me2me_preference_pane_disable.mm',
       ],
       'link_settings': {
         'libraries': [
           '$(SDKROOT)/System/Library/Frameworks/Cocoa.framework',
           '$(SDKROOT)/System/Library/Frameworks/CoreFoundation.framework',
           '$(SDKROOT)/System/Library/Frameworks/PreferencePanes.framework',
           '$(SDKROOT)/System/Library/Frameworks/Security.framework',
         ],
       },
       'xcode_settings': {
-        'ARCHS': ['i386', 'x86_64'],

[Nico, 2016/01/04 16:28:42]

Doesn't this break using the remoting prefpane on older versions of OS X where
System Preferences is still a 32-bit app?

[Sergey Ulanov, 2016/01/04 18:12:24]

The host is compiled for 64-bit only. I assumed that System Preferences is
64-bit app by default on systems that support 64-bit. Is that not true?

         'GCC_ENABLE_OBJC_GC': 'supported',
         'INFOPLIST_FILE': 'host/mac/me2me_preference_pane-Info.plist',
         'INFOPLIST_PREPROCESS': 'YES',
         'INFOPLIST_PREPROCESSOR_DEFINITIONS': 'VERSION_FULL="<(version_full)" VERSION_SHORT="<(version_short)" BUNDLE_ID="<(bundle_id)"',
       },
       'mac_bundle_resources': [
         'host/mac/me2me_preference_pane.xib',
         'host/mac/me2me_preference_pane_confirm_pin.xib',
         'host/mac/me2me_preference_pane_disable.xib',
         'host/mac/me2me_preference_pane-Info.plist',
@@ skipping 12 matching lines @@
           'variables': {
             # A real .dSYM is needed for dump_syms to operate on.
             'mac_real_dsym': 1,
           },
         }],  # 'mac_breakpad==1'
       ],  # conditions
     },  # end of target 'remoting_host_prefpane'
   ],  # end of 'targets'
 
 }