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

Side by Side Diff: chrome/test/chromedriver/capabilities.cc

Issue 2682793002: [chromedriver] Force DevTools screenshots by default. (Closed)
Patch Set: Created 3 years, 10 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 "chrome/test/chromedriver/capabilities.h" 5 #include "chrome/test/chromedriver/capabilities.h"
6 6
7 #include <map> 7 #include <map>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 594 matching lines...) Expand 10 before | Expand all | Expand 10 after
605 page(InspectorDomainStatus::kDefaultEnabled), 605 page(InspectorDomainStatus::kDefaultEnabled),
606 timeline(InspectorDomainStatus::kDefaultDisabled), 606 timeline(InspectorDomainStatus::kDefaultDisabled),
607 trace_categories(), 607 trace_categories(),
608 buffer_usage_reporting_interval(1000) {} 608 buffer_usage_reporting_interval(1000) {}
609 609
610 PerfLoggingPrefs::~PerfLoggingPrefs() {} 610 PerfLoggingPrefs::~PerfLoggingPrefs() {}
611 611
612 Capabilities::Capabilities() 612 Capabilities::Capabilities()
613 : android_use_running_app(false), 613 : android_use_running_app(false),
614 detach(false), 614 detach(false),
615 force_devtools_screenshot(false), 615 force_devtools_screenshot(true),
616 page_load_strategy(PageLoadStrategy::kNormal), 616 page_load_strategy(PageLoadStrategy::kNormal),
617 network_emulation_enabled(false) {} 617 network_emulation_enabled(false) {}
618 618
619 Capabilities::~Capabilities() {} 619 Capabilities::~Capabilities() {}
620 620
621 bool Capabilities::IsAndroid() const { 621 bool Capabilities::IsAndroid() const {
622 return !android_package.empty(); 622 return !android_package.empty();
623 } 623 }
624 624
625 bool Capabilities::IsRemoteBrowser() const { 625 bool Capabilities::IsRemoteBrowser() const {
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
657 if (iter == logging_prefs.end() || iter->second == Log::kOff) { 657 if (iter == logging_prefs.end() || iter->second == Log::kOff) {
658 const base::DictionaryValue* chrome_options = NULL; 658 const base::DictionaryValue* chrome_options = NULL;
659 if (desired_caps.GetDictionary("chromeOptions", &chrome_options) && 659 if (desired_caps.GetDictionary("chromeOptions", &chrome_options) &&
660 chrome_options->HasKey("perfLoggingPrefs")) { 660 chrome_options->HasKey("perfLoggingPrefs")) {
661 return Status(kUnknownError, "perfLoggingPrefs specified, " 661 return Status(kUnknownError, "perfLoggingPrefs specified, "
662 "but performance logging was not enabled"); 662 "but performance logging was not enabled");
663 } 663 }
664 } 664 }
665 return Status(kOk); 665 return Status(kOk);
666 } 666 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698