Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(614)

Unified Diff: content/browser/devtools/renderer_overrides_handler.h

Issue 482503002: DevTools: follow up to r288297, check rvh for being null in all branches. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/devtools/renderer_overrides_handler.h
diff --git a/content/browser/devtools/renderer_overrides_handler.h b/content/browser/devtools/renderer_overrides_handler.h
index 4f5076ec46746d758aab97cb9e7a87f519240214..774a6838cb8157e02467f728f8f17ae4a76097c4 100644
--- a/content/browser/devtools/renderer_overrides_handler.h
+++ b/content/browser/devtools/renderer_overrides_handler.h
@@ -23,7 +23,6 @@ class Message;
namespace content {
-class DevToolsAgentHost;
class DevToolsTracingHandler;
class RenderViewHostImpl;
@@ -33,13 +32,14 @@ class RenderViewHostImpl;
class CONTENT_EXPORT RendererOverridesHandler
: public DevToolsProtocol::Handler {
public:
- explicit RendererOverridesHandler(DevToolsAgentHost* agent);
+ RendererOverridesHandler();
virtual ~RendererOverridesHandler();
void OnClientDetached();
void OnSwapCompositorFrame(const cc::CompositorFrameMetadata& frame_metadata);
void OnVisibilityChanged(bool visible);
- void OnRenderViewHostChanged();
+ void SetRenderViewHost(RenderViewHostImpl* host);
+ void ClearRenderViewHost();
bool OnSetTouchEventEmulationEnabled();
private:
@@ -102,9 +102,7 @@ class CONTENT_EXPORT RendererOverridesHandler
scoped_refptr<DevToolsProtocol::Response> InputEmulateTouchFromMouseEvent(
scoped_refptr<DevToolsProtocol::Command> command);
- RenderViewHostImpl* GetRenderViewHostImpl();
-
- DevToolsAgentHost* agent_;
+ RenderViewHostImpl* host_;
scoped_refptr<DevToolsProtocol::Command> screencast_command_;
bool has_last_compositor_frame_metadata_;
cc::CompositorFrameMetadata last_compositor_frame_metadata_;

Powered by Google App Engine
This is Rietveld 408576698