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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutTheme.cpp

Issue 2447503002: Remove newMediaPlaybackUi flag from content/ and Blink. (Closed)
Patch Set: fix mistake caught by tests Created 4 years, 2 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: third_party/WebKit/Source/core/layout/LayoutTheme.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutTheme.cpp b/third_party/WebKit/Source/core/layout/LayoutTheme.cpp
index 67172468b37d91a93a6c724ba2a9d353db141f8e..9298c89c8facd84d8887391b0f39e790411b325d 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTheme.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutTheme.cpp
@@ -268,7 +268,6 @@ static String formatChromiumMediaControlsTime(float time,
duration = 0;
int seconds = static_cast<int>(fabsf(time));
int minutes = seconds / 60;
- int hours = seconds / (60 * 60);
seconds %= 60;
@@ -276,21 +275,6 @@ static String formatChromiumMediaControlsTime(float time,
int durationSecs = static_cast<int>(fabsf(duration));
int durationMins = durationSecs / 60;
- if (!RuntimeEnabledFeatures::newMediaPlaybackUiEnabled()) {
- int durationHours = durationSecs / (60 * 60);
- durationMins %= 60;
- minutes %= 60;
- if (durationHours || hours)
- return String::format("%s%01d:%02d:%02d", (time < 0 ? "-" : ""), hours,
- minutes, seconds);
- if (durationMins > 9)
- return String::format("%s%02d:%02d", (time < 0 ? "-" : ""), minutes,
- seconds);
-
- return String::format("%s%01d:%02d", (time < 0 ? "-" : ""), minutes,
- seconds);
- }
-
// New UI includes a leading "/ " before duration.
const char* separator = includeSeparator ? "/ " : "";

Powered by Google App Engine
This is Rietveld 408576698