Remove duplicated provideMediaKeysTo() in MediaKeysClient.h

Source/web/WebViewImpl.h

Index: Source/web/WebViewImpl.h
diff --git a/Source/web/WebViewImpl.h b/Source/web/WebViewImpl.h
index 66ec1029bba981baf92ced263105eaa17f6f26db..0ec4102aaf31b36d11feb1d03ea2badb91c16fa4 100644
--- a/Source/web/WebViewImpl.h
+++ b/Source/web/WebViewImpl.h
@@ -50,7 +50,6 @@
 #include "web/DragClientImpl.h"
 #include "web/EditorClientImpl.h"
 #include "web/InspectorClientImpl.h"
-#include "web/MediaKeysClientImpl.h"
 #include "web/PageOverlayList.h"
 #include "web/PageScaleConstraintsSet.h"
 #include "web/PageWidgetDelegate.h"
@@ -691,7 +690,6 @@ private:
     bool m_recreatingGraphicsContext;
     static const WebInputEvent* m_currentInputEvent;
 
-    MediaKeysClientImpl m_mediaKeysClientImpl;

[gyuyoung-inactive, 2014/06/23 06:33:07]

m_mediaKeysClientImpl instance is only used by MediaKeyController. So, I thought
that it would be good to move ownership to MediaKeyController. However, I agree
that this is not big worth to do because this is trivial refactoring. To be
honest, I don't understand where memory leak happens in my last patch. Last one
is to pass OwnPtr to MediaKeyController's member variable, then it will be free
when going out of MediaKeyController. Anyway, now I just would like to remove
unused provideMediaKeysTo() in MediaKeyClient.h. Because MediaKeyController has
already same function and it is only being used by WebViewImpl.cpp. It looks the
function is duplicated.

     OwnPtr<WebActiveGestureAnimation> m_gestureAnimation;
     WebPoint m_positionOnFlingStart;
     WebPoint m_globalPositionOnFlingStart;