diff options
Diffstat (limited to 'content/renderer/devtools')
-rw-r--r-- | content/renderer/devtools/devtools_agent.cc | 51 |
1 files changed, 15 insertions, 36 deletions
diff --git a/content/renderer/devtools/devtools_agent.cc b/content/renderer/devtools/devtools_agent.cc index b5d6f9b..459a2b5 100644 --- a/content/renderer/devtools/devtools_agent.cc +++ b/content/renderer/devtools/devtools_agent.cc @@ -207,60 +207,42 @@ void DevToolsAgent::SendChunkedProtocolMessage(IPC::Sender* sender, } void DevToolsAgent::OnAttach(const std::string& host_id, int session_id) { - WebDevToolsAgent* web_agent = GetWebAgent(); - if (web_agent) { - web_agent->attach(WebString::fromUTF8(host_id), session_id); - is_attached_ = true; - } + GetWebAgent()->attach(WebString::fromUTF8(host_id), session_id); + is_attached_ = true; } void DevToolsAgent::OnReattach(const std::string& host_id, int session_id, const std::string& agent_state) { - WebDevToolsAgent* web_agent = GetWebAgent(); - if (web_agent) { - web_agent->reattach(WebString::fromUTF8(host_id), session_id, - WebString::fromUTF8(agent_state)); - is_attached_ = true; - } + GetWebAgent()->reattach(WebString::fromUTF8(host_id), session_id, + WebString::fromUTF8(agent_state)); + is_attached_ = true; } void DevToolsAgent::OnDetach() { - WebDevToolsAgent* web_agent = GetWebAgent(); - if (web_agent) { - web_agent->detach(); - is_attached_ = false; - } + GetWebAgent()->detach(); + is_attached_ = false; } void DevToolsAgent::OnDispatchOnInspectorBackend(int session_id, const std::string& message) { TRACE_EVENT0("devtools", "DevToolsAgent::OnDispatchOnInspectorBackend"); - WebDevToolsAgent* web_agent = GetWebAgent(); - if (web_agent) - web_agent->dispatchOnInspectorBackend(session_id, - WebString::fromUTF8(message)); + GetWebAgent()->dispatchOnInspectorBackend(session_id, + WebString::fromUTF8(message)); } void DevToolsAgent::OnInspectElement(int x, int y) { - WebDevToolsAgent* web_agent = GetWebAgent(); - if (web_agent) { - DCHECK(is_attached_); - web_agent->inspectElementAt(WebPoint(x, y)); - } + GetWebAgent()->inspectElementAt(WebPoint(x, y)); } void DevToolsAgent::OnRequestNewWindowACK(bool success) { - WebDevToolsAgent* web_agent = GetWebAgent(); - if (web_agent && !success) - web_agent->failedToRequestDevTools(); + if (!success) + GetWebAgent()->failedToRequestDevTools(); } void DevToolsAgent::AddMessageToConsole(ConsoleMessageLevel level, const std::string& message) { WebLocalFrame* web_frame = frame_->GetWebFrame(); - if (!web_frame) - return; WebConsoleMessage::Level target_level = WebConsoleMessage::LevelLog; switch (level) { @@ -282,15 +264,13 @@ void DevToolsAgent::AddMessageToConsole(ConsoleMessageLevel level, } void DevToolsAgent::ContinueProgram() { - WebDevToolsAgent* web_agent = GetWebAgent(); - if (web_agent) - web_agent->continueProgram(); + GetWebAgent()->continueProgram(); } void DevToolsAgent::OnSetupDevToolsClient( const std::string& compatibility_script) { // We only want to register once; and only in main frame. - DCHECK(!frame_->GetWebFrame() || !frame_->GetWebFrame()->parent()); + DCHECK(!frame_->GetWebFrame()->parent()); if (is_devtools_client_) return; is_devtools_client_ = true; @@ -298,8 +278,7 @@ void DevToolsAgent::OnSetupDevToolsClient( } WebDevToolsAgent* DevToolsAgent::GetWebAgent() { - WebLocalFrame* web_frame = frame_->GetWebFrame(); - return web_frame ? web_frame->devToolsAgent() : nullptr; + return frame_->GetWebFrame()->devToolsAgent(); } bool DevToolsAgent::IsAttached() { |