Vibration API : migrate to device/vibration using mojo.

content/public/browser/content_browser_client.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 #define CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 
 #include <map>
 #include <string>
 #include <utility>
@@ skipping 83 matching lines @@
 class ExternalVideoSurfaceContainer;
 class LocationProvider;
 class MediaObserver;
 class QuotaPermissionContext;
 class RenderFrameHost;
 class RenderProcessHost;
 class RenderViewHost;
 class ResourceContext;
 class SiteInstance;
 class SpeechRecognitionManagerDelegate;
-class VibrationProvider;
 class WebContents;
 class WebContentsViewDelegate;
 struct MainFunctionParams;
 struct Referrer;
 struct ShowDesktopNotificationHostMsgParams;
 struct WebPreferences;
 
 // A mapping from the scheme name to the protocol handler that services its
 // content.
 typedef std::map<
@@ skipping 442 matching lines @@
       const base::FilePath& storage_partition_path,
       ScopedVector<storage::FileSystemBackend>* additional_backends) {}
 
   // Allows an embedder to return its own LocationProvider implementation.
   // Return NULL to use the default one for the platform to be created.
   // FYI: Used by an external project; please don't remove.
   // Contact Viatcheslav Ostapenko at sl.ostapenko@samsung.com for more
   // information.
   virtual LocationProvider* OverrideSystemLocationProvider();
 
-  // Allows an embedder to return its own VibrationProvider implementation.
-  // Return NULL to use the default one for the platform to be created.
-  // FYI: Used by an external project; please don't remove.

[ppi, 2014/12/01 12:03:55]

What's up with that - have we reached out to Viatcheslav about the change?

I can land a patch for Mojo overrides in the client API
(http://crbug.com/421796), but we need a confirmation that it will be indeed
used by someone.

[timvolodine, 2014/12/01 12:11:42]

yes I have communicated with Viatcheslav about this change he was ok with it
(more details in crbug.com/416426).
In the longer run we could use the approach in your patch, but I don't think
this should block this patch at this point.

[ppi, 2014/12/01 12:25:13]

Sounds good. I will put together a standalone CL for the overrides and send it
for review to Viatcheslav.

-  // Contact Viatcheslav Ostapenko at sl.ostapenko@samsung.com for more
-  // information.
-  virtual VibrationProvider* OverrideVibrationProvider();
-
   // Creates a new DevToolsManagerDelegate. The caller owns the returned value.
   // It's valid to return NULL.
   virtual DevToolsManagerDelegate* GetDevToolsManagerDelegate();
 
   // Returns true if plugin referred to by the url can use
   // pp::FileIO::RequestOSFileHandle.
   virtual bool IsPluginAllowedToCallRequestOSFileHandle(
       BrowserContext* browser_context,
       const GURL& url);
 
@@ skipping 39 matching lines @@
 // camera. Note that this does not query the user. |type| must be
 // MEDIA_DEVICE_AUDIO_CAPTURE or MEDIA_DEVICE_VIDEO_CAPTURE.
 virtual bool CheckMediaAccessPermission(BrowserContext* browser_context,
                                         const GURL& security_origin,
                                         MediaStreamType type);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_