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

Unified Diff: cc/top_controls_manager_unittest.cc

Issue 12210050: Expose FS show/hide thresholds to Command Line (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
Index: cc/top_controls_manager_unittest.cc
diff --git a/cc/top_controls_manager_unittest.cc b/cc/top_controls_manager_unittest.cc
index cdce1bdc4d89aed9d9e576ce5a2bb8fae81205bd..5fac210dae2608dd5c0e76b5905fbd87e34d75cd 100644
--- a/cc/top_controls_manager_unittest.cc
+++ b/cc/top_controls_manager_unittest.cc
@@ -18,6 +18,8 @@ namespace cc {
namespace {
static const float kTopControlsHeight = 100;
+static const float kTopControlsShowThreshold = 0.5f;
+static const float kTopControlsHideThreshold = 0.5f;
class MockTopControlsManagerClient : public TopControlsManagerClient {
public:
@@ -54,7 +56,10 @@ class MockTopControlsManagerClient : public TopControlsManagerClient {
TopControlsManager* manager() {
if (!manager_)
Ted C 2013/02/07 00:40:26 need to use braces if the statement is more than o
David Trainor- moved to gerrit 2013/02/11 19:38:28 Done.
- manager_ = TopControlsManager::Create(this, kTopControlsHeight);
+ manager_ = TopControlsManager::Create(this,
+ kTopControlsHeight,
+ kTopControlsShowThreshold,
+ kTopControlsHideThreshold);
return manager_.get();
}

Powered by Google App Engine
This is Rietveld 408576698