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

Unified Diff: third_party/WebKit/Source/web/WebDevToolsAgentImpl.cpp

Issue 2389633002: reflow comments in web/ (Closed)
Patch Set: . Created 4 years, 2 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
« no previous file with comments | « third_party/WebKit/Source/web/WebDataSourceImpl.cpp ('k') | third_party/WebKit/Source/web/WebElement.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/web/WebDevToolsAgentImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebDevToolsAgentImpl.cpp b/third_party/WebKit/Source/web/WebDevToolsAgentImpl.cpp
index 0f98611f421859b60658c9824f76049b0ec61c67..324bb9a489d8e3927200d8a69b9d6d5ca7a15477 100644
--- a/third_party/WebKit/Source/web/WebDevToolsAgentImpl.cpp
+++ b/third_party/WebKit/Source/web/WebDevToolsAgentImpl.cpp
@@ -98,8 +98,8 @@ namespace blink {
namespace {
bool isMainFrame(WebLocalFrameImpl* frame) {
- // TODO(dgozman): sometimes view->mainFrameImpl() does return null, even though |frame| is meant to be main frame.
- // See http://crbug.com/526162.
+ // TODO(dgozman): sometimes view->mainFrameImpl() does return null, even
+ // though |frame| is meant to be main frame. See http://crbug.com/526162.
return frame->viewImpl() && !frame->parent();
}
}
@@ -434,14 +434,15 @@ void WebDevToolsAgentImpl::initializeSession(int sessionId,
m_networkAgent->setHostId(hostId);
if (m_includeViewAgents) {
- // TODO(dgozman): we should actually pass the view instead of frame, but during
- // remote->local transition we cannot access mainFrameImpl() yet, so we have to store the
- // frame which will become the main frame later.
+ // TODO(dgozman): we should actually pass the view instead of frame, but
+ // during remote->local transition we cannot access mainFrameImpl() yet, so
+ // we have to store the frame which will become the main frame later.
m_session->append(
InspectorRenderingAgent::create(m_webLocalFrameImpl, m_overlay.get()));
m_session->append(
InspectorEmulationAgent::create(m_webLocalFrameImpl, this));
- // TODO(dgozman): migrate each of the following agents to frame once module is ready.
+ // TODO(dgozman): migrate each of the following agents to frame once module
+ // is ready.
Page* page = m_webLocalFrameImpl->viewImpl()->page();
m_session->append(InspectorDatabaseAgent::create(page));
m_session->append(DeviceOrientationInspectorAgent::create(page));
@@ -688,7 +689,8 @@ void WebDevToolsAgentImpl::runDebuggerTask(
void WebDevToolsAgent::interruptAndDispatch(int sessionId,
MessageDescriptor* rawDescriptor) {
- // rawDescriptor can't be a std::unique_ptr because interruptAndDispatch is a WebKit API function.
+ // rawDescriptor can't be a std::unique_ptr because interruptAndDispatch is a
+ // WebKit API function.
MainThreadDebugger::interruptMainThreadAndRun(
crossThreadBind(WebDevToolsAgentImpl::runDebuggerTask, sessionId,
passed(wrapUnique(rawDescriptor))));
« no previous file with comments | « third_party/WebKit/Source/web/WebDataSourceImpl.cpp ('k') | third_party/WebKit/Source/web/WebElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698