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

Side by Side Diff: components/test_runner/web_frame_test_client.cc

Issue 2449553002: Remove dead DocumentState metrics tracking code (Closed)
Patch Set: fix tests Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/test_runner/web_frame_test_client.h" 5 #include "components/test_runner/web_frame_test_client.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/strings/string_piece.h" 10 #include "base/strings/string_piece.h"
(...skipping 379 matching lines...) Expand 10 before | Expand all | Expand 10 after
390 std::string("Downloading URL with suggested filename \"") + 390 std::string("Downloading URL with suggested filename \"") +
391 suggested_name.utf8() + "\"\n"); 391 suggested_name.utf8() + "\"\n");
392 } else { 392 } else {
393 delegate_->PrintMessage(std::string("Loading URL externally - \"") + 393 delegate_->PrintMessage(std::string("Loading URL externally - \"") +
394 URLDescription(request.url()) + "\"\n"); 394 URLDescription(request.url()) + "\"\n");
395 } 395 }
396 delegate_->TestFinished(); 396 delegate_->TestFinished();
397 } 397 }
398 } 398 }
399 399
400 void WebFrameTestClient::didStartProvisionalLoad(blink::WebLocalFrame* frame, 400 void WebFrameTestClient::didStartProvisionalLoad(blink::WebLocalFrame* frame) {
401 double trigering_event_time) {
402 test_runner()->tryToSetTopLoadingFrame(frame); 401 test_runner()->tryToSetTopLoadingFrame(frame);
403 402
404 if (test_runner()->shouldDumpFrameLoadCallbacks()) { 403 if (test_runner()->shouldDumpFrameLoadCallbacks()) {
405 PrintFrameDescription(delegate_, frame); 404 PrintFrameDescription(delegate_, frame);
406 delegate_->PrintMessage(" - didStartProvisionalLoadForFrame\n"); 405 delegate_->PrintMessage(" - didStartProvisionalLoadForFrame\n");
407 } 406 }
408 407
409 if (test_runner()->shouldDumpUserGestureInFrameLoadCallbacks()) { 408 if (test_runner()->shouldDumpUserGestureInFrameLoadCallbacks()) {
410 PrintFrameuserGestureStatus(delegate_, frame, 409 PrintFrameuserGestureStatus(delegate_, frame,
411 " - in didStartProvisionalLoadForFrame\n"); 410 " - in didStartProvisionalLoadForFrame\n");
(...skipping 287 matching lines...) Expand 10 before | Expand all | Expand 10 after
699 blink::WebEffectiveConnectionType 698 blink::WebEffectiveConnectionType
700 WebFrameTestClient::getEffectiveConnectionType() { 699 WebFrameTestClient::getEffectiveConnectionType() {
701 return test_runner()->effective_connection_type(); 700 return test_runner()->effective_connection_type();
702 } 701 }
703 702
704 TestRunner* WebFrameTestClient::test_runner() { 703 TestRunner* WebFrameTestClient::test_runner() {
705 return web_view_test_proxy_base_->test_interfaces()->GetTestRunner(); 704 return web_view_test_proxy_base_->test_interfaces()->GetTestRunner();
706 } 705 }
707 706
708 } // namespace test_runner 707 } // namespace test_runner
OLDNEW
« no previous file with comments | « components/test_runner/web_frame_test_client.h ('k') | components/test_runner/web_frame_test_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698