Shape Detection: Implement FaceDetection on Mac as out-of-process service

content/public/app/mojo/content_browser_manifest.json

 {
   "name": "content_browser",
   "display_name": "Content (browser process)",
   "interface_provider_specs": {
     "service_manager:connector": {
       "provides": {
         "gpu": [
           "device::mojom::PowerMonitor"
         ],
         "plugin": [
           "device::mojom::PowerMonitor"
         ],
         "renderer": [
           "blink::mojom::BackgroundSyncService",
-          "blink::mojom::BarcodeDetection",
           "blink::mojom::BroadcastChannelProvider",
           "blink::mojom::BudgetService",
-          "blink::mojom::FaceDetection",

[Ken Rockot(use gerrit already), 2016/12/01 17:54:31]

It looks like there is still renderer code trying to connect to these interfaces
on the RenderFrameImpl's interface provider. At least in tests, there are some
failures related to this.

[xianglu, 2016/12/02 18:11:23]

Done.

           "blink::mojom::Hyphenation",
           "blink::mojom::MimeRegistry",
           "blink::mojom::NotificationService",
           "blink::mojom::OffscreenCanvasCompositorFrameSinkProvider",
           "blink::mojom::OffscreenCanvasSurface",
           "blink::mojom::PermissionService",
           "blink::mojom::WebSocket",
           "content::mojom::MemoryCoordinatorHandle",
           "content::mojom::ServiceWorkerDispatcherHost",
           "content::mojom::StoragePartitionService",
@@ skipping 20 matching lines @@
         "content_gpu": [ "browser" ],
         "content_plugin": [ "browser" ],
         "content_renderer": [ "browser" ],
         "content_utility": [ "browser" ],
         "service_manager": [
           "service_manager:client_process",
           "service_manager:instance_name",
           "service_manager:user_id"
         ],
         "file": [ "file:filesystem", "file:leveldb" ],
-        "media": [ "media:media" ]
+        "media": [ "media:media" ],
+        "shape_detection": [ "face_detection", "barcode_detection" ]

[Ken Rockot(use gerrit already), 2016/12/01 17:54:30]

This is unnecessary since the browser itself does not need to connect to
shape_detection.

[xianglu, 2016/12/02 18:11:23]

Done.

       }
     },
     "navigation:frame": {
       "provides": {
         "renderer": [
           // TODO(beng): these belong in //chrome's overlay, but there are some
           // issues with mash browser_tests's manifest config that make this
           // impossible this week. Remove once sky/ken fix this.
           "autofill::mojom::AutofillDriver",
           "autofill::mojom::PasswordManagerDriver",
-          "blink::mojom::BarcodeDetection",
-          "blink::mojom::FaceDetection",
           "blink::mojom::MediaSessionService",
           "blink::mojom::PermissionService",
           "blink::mojom::PresentationService",
           "blink::mojom::SensitiveInputVisibilityService",
           "blink::mojom::WebBluetoothService",
           "blink::mojom::WebSocket",
           // TODO(beng): figure out how to overlay test interfaces like this.
           "content::mojom::BrowserTarget",
           "device::mojom::VRService",
           "device::VibrationManager",
           "device::mojom::GeolocationService",
           "device::mojom::SensorProvider",
           "device::mojom::WakeLockService",
           "device::nfc::mojom::NFC",
           "device::usb::DeviceManager",
           "media::mojom::InterfaceFactory",
           "media::mojom::RemoterFactory",
           "media::mojom::Renderer",
           "mojom::MediaDevicesDispatcherHost",
           "payments::mojom::PaymentRequest"
         ]
       },
       "requires": {
         "content_renderer": [ "browser" ]
       }
     }
   }
 }