diff options
author | pfeldman@chromium.org <pfeldman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-04 14:58:36 +0000 |
---|---|---|
committer | pfeldman@chromium.org <pfeldman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-04 14:58:36 +0000 |
commit | 72c507e994e7438e710341caf537859ee3faca11 (patch) | |
tree | 05c12c509e62091aebf7f15323a78cd950495603 /content/browser/devtools/devtools_tracing_handler.cc | |
parent | a131afc860e6d841a100878eca59835a751c97c4 (diff) | |
download | chromium_src-72c507e994e7438e710341caf537859ee3faca11.zip chromium_src-72c507e994e7438e710341caf537859ee3faca11.tar.gz chromium_src-72c507e994e7438e710341caf537859ee3faca11.tar.bz2 |
DevTools: introduce devtools_protocol.* with protocol parser / serializer helpers.
Review URL: https://chromiumcodereview.appspot.com/12079103
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180407 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/devtools/devtools_tracing_handler.cc')
-rw-r--r-- | content/browser/devtools/devtools_tracing_handler.cc | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/content/browser/devtools/devtools_tracing_handler.cc b/content/browser/devtools/devtools_tracing_handler.cc index 7dc68c4..e16768f 100644 --- a/content/browser/devtools/devtools_tracing_handler.cc +++ b/content/browser/devtools/devtools_tracing_handler.cc @@ -33,11 +33,11 @@ DevToolsTracingHandler::DevToolsTracingHandler() : DevToolsBrowserTarget::DomainHandler(kTracingDomain), is_running_(false) { RegisterCommandHandler(kTracingStartCommand, - Bind(&DevToolsTracingHandler::OnStart, - base::Unretained(this))); + base::Bind(&DevToolsTracingHandler::OnStart, + base::Unretained(this))); RegisterCommandHandler(kTracingEndCommand, - Bind(&DevToolsTracingHandler::OnEnd, - base::Unretained(this))); + base::Bind(&DevToolsTracingHandler::OnEnd, + base::Unretained(this))); } DevToolsTracingHandler::~DevToolsTracingHandler() { @@ -45,7 +45,7 @@ DevToolsTracingHandler::~DevToolsTracingHandler() { void DevToolsTracingHandler::OnEndTracingComplete() { is_running_ = false; - SendNotification(kTracingCompleteNotification, NULL, NULL); + SendNotification(kTracingCompleteNotification, NULL); } void DevToolsTracingHandler::OnTraceDataCollected( @@ -53,26 +53,26 @@ void DevToolsTracingHandler::OnTraceDataCollected( if (is_running_) { base::DictionaryValue* params = new base::DictionaryValue(); params->SetString("value", trace_fragment->data()); - SendNotification(kTracingDataCollected, params, NULL); + SendNotification(kTracingDataCollected, params); } } -base::DictionaryValue* DevToolsTracingHandler::OnStart( - const base::DictionaryValue* params, - base::Value** error_out) { +scoped_ptr<DevToolsProtocol::Response> +DevToolsTracingHandler::OnStart(DevToolsProtocol::Command* command) { std::string categories; + base::DictionaryValue* params = command->params(); if (params && params->HasKey(kCategoriesParam)) params->GetString(kCategoriesParam, &categories); TraceController::GetInstance()->BeginTracing(this, categories); is_running_ = true; - return NULL; + return command->SuccessResponse(NULL); } -base::DictionaryValue* DevToolsTracingHandler::OnEnd( - const base::DictionaryValue* params, - base::Value** error_out) { + +scoped_ptr<DevToolsProtocol::Response> +DevToolsTracingHandler::OnEnd(DevToolsProtocol::Command* command) { TraceController::GetInstance()->EndTracingAsync(this); - return NULL; + return command->SuccessResponse(NULL); } } // namespace content |