diff options
author | derekjchow <derekjchow@chromium.org> | 2016-03-07 21:38:56 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-03-08 05:40:24 +0000 |
commit | ae1c7e7f600035bfb918ba8f217e9e0e5b505702 (patch) | |
tree | ae7ba9c5f7d36be883696c7d6ea56c9a2c9c05d6 /chromecast | |
parent | 9a8e15f6e8b9de90a8dff395a68ea6212226786f (diff) | |
download | chromium_src-ae1c7e7f600035bfb918ba8f217e9e0e5b505702.zip chromium_src-ae1c7e7f600035bfb918ba8f217e9e0e5b505702.tar.gz chromium_src-ae1c7e7f600035bfb918ba8f217e9e0e5b505702.tar.bz2 |
[Chromecast] Remove unused code in cast_content_renderer_client.
The need for the AddRendererNativeBindings has been removed internally.
Thus, we can remove this interface as well as CastRenderViewObserver.
BUG=None
R=alokp@chromium.org
Review URL: https://codereview.chromium.org/1769973003
Cr-Commit-Position: refs/heads/master@{#379758}
Diffstat (limited to 'chromecast')
-rw-r--r-- | chromecast/renderer/cast_content_renderer_client.cc | 33 | ||||
-rw-r--r-- | chromecast/renderer/cast_content_renderer_client.h | 3 |
2 files changed, 0 insertions, 36 deletions
diff --git a/chromecast/renderer/cast_content_renderer_client.cc b/chromecast/renderer/cast_content_renderer_client.cc index f6b7a77..c06c618 100644 --- a/chromecast/renderer/cast_content_renderer_client.cc +++ b/chromecast/renderer/cast_content_renderer_client.cc @@ -25,7 +25,6 @@ #include "content/public/common/content_switches.h" #include "content/public/renderer/render_frame.h" #include "content/public/renderer/render_view.h" -#include "content/public/renderer/render_view_observer.h" #include "third_party/WebKit/public/platform/WebColor.h" #include "third_party/WebKit/public/web/WebFrameWidget.h" #include "third_party/WebKit/public/web/WebSettings.h" @@ -72,31 +71,6 @@ void PlatformPollFreemem(void) { // though the comment of WebColor says it is in RGBA. const blink::WebColor kColorBlack = 0xFF000000; -class CastRenderViewObserver : content::RenderViewObserver { - public: - CastRenderViewObserver(CastContentRendererClient* client, - content::RenderView* render_view); - ~CastRenderViewObserver() override {} - - void DidClearWindowObject(blink::WebLocalFrame* frame) override; - - private: - CastContentRendererClient* const client_; - - DISALLOW_COPY_AND_ASSIGN(CastRenderViewObserver); -}; - -CastRenderViewObserver::CastRenderViewObserver( - CastContentRendererClient* client, - content::RenderView* render_view) - : content::RenderViewObserver(render_view), - client_(client) { -} - -void CastRenderViewObserver::DidClearWindowObject(blink::WebLocalFrame* frame) { - client_->AddRendererNativeBindings(frame); -} - } // namespace CastContentRendererClient::CastContentRendererClient() @@ -108,10 +82,6 @@ CastContentRendererClient::CastContentRendererClient() CastContentRendererClient::~CastContentRendererClient() { } -void CastContentRendererClient::AddRendererNativeBindings( - blink::WebLocalFrame* frame) { -} - void CastContentRendererClient::RenderThreadStarted() { base::CommandLine* command_line = base::CommandLine::ForCurrentProcess(); @@ -168,9 +138,6 @@ void CastContentRendererClient::RenderViewCreated( // application running. webview->settings()->setOfflineWebApplicationCacheEnabled(false); } - - // Note: RenderView will own the lifetime of its observer. - new CastRenderViewObserver(this, render_view); } void CastContentRendererClient::AddKeySystems( diff --git a/chromecast/renderer/cast_content_renderer_client.h b/chromecast/renderer/cast_content_renderer_client.h index 6350038..470e8a8 100644 --- a/chromecast/renderer/cast_content_renderer_client.h +++ b/chromecast/renderer/cast_content_renderer_client.h @@ -31,9 +31,6 @@ class CastContentRendererClient : public content::ContentRendererClient { ~CastContentRendererClient() override; - // Adds any platform-specific bindings to the current frame. - virtual void AddRendererNativeBindings(blink::WebLocalFrame* frame); - // ContentRendererClient implementation: void RenderThreadStarted() override; void RenderViewCreated(content::RenderView* render_view) override; |