Commit fe10ee60 authored by Tricia Crichton's avatar Tricia Crichton Committed by Commit Bot

[ChromeDriver] Remove obsolete code lines

Previously, an additional call to Debugger.enable was needed to take a
heap snapshot. But this is no longer needed.

Bug: chromedriver:3299
Change-Id: I0c5fd66390ea84370e4a408fe1d44f0a916700a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1988986Reviewed-by: default avatarJohn Chen <johnchen@chromium.org>
Commit-Queue: Tricia Crichton <triciac@chromium.org>
Cr-Commit-Position: refs/heads/master@{#732426}
parent 2c0963e2
......@@ -972,23 +972,7 @@ Status WebViewImpl::TakeHeapSnapshot(std::unique_ptr<base::Value>* snapshot) {
Status WebViewImpl::InitProfileInternal() {
base::DictionaryValue params;
// TODO: Remove Debugger.enable after Chrome 36 stable is released.
Status status_debug = client_->SendCommand("Debugger.enable", params);
if (status_debug.IsError())
return status_debug;
Status status_profiler = client_->SendCommand("Profiler.enable", params);
if (status_profiler.IsError()) {
Status status_debugger = client_->SendCommand("Debugger.disable", params);
if (status_debugger.IsError())
return status_debugger;
return status_profiler;
}
return Status(kOk);
return client_->SendCommand("Profiler.enable", params);
}
Status WebViewImpl::StopProfileInternal() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment