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

Side by Side Diff: third_party/WebKit/Source/core/layout/LayoutTheme.cpp

Issue 2017053003: Remove StringBuilder::appendLiteral. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase. Created 4 years, 6 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
OLDNEW
1 /** 1 /**
2 * This file is part of the theme implementation for form controls in WebCore. 2 * This file is part of the theme implementation for form controls in WebCore.
3 * 3 *
4 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Computer, Inc. 4 * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Computer, Inc.
5 * 5 *
6 * This library is free software; you can redistribute it and/or 6 * This library is free software; you can redistribute it and/or
7 * modify it under the terms of the GNU Library General Public 7 * modify it under the terms of the GNU Library General Public
8 * License as published by the Free Software Foundation; either 8 * License as published by the Free Software Foundation; either
9 * version 2 of the License, or (at your option) any later version. 9 * version 2 of the License, or (at your option) any later version.
10 * 10 *
(...skipping 208 matching lines...) Expand 10 before | Expand all | Expand 10 after
219 return adjustSearchFieldCancelButtonStyle(style); 219 return adjustSearchFieldCancelButtonStyle(style);
220 default: 220 default:
221 break; 221 break;
222 } 222 }
223 } 223 }
224 224
225 String LayoutTheme::extraDefaultStyleSheet() 225 String LayoutTheme::extraDefaultStyleSheet()
226 { 226 {
227 StringBuilder runtimeCSS; 227 StringBuilder runtimeCSS;
228 if (RuntimeEnabledFeatures::contextMenuEnabled()) 228 if (RuntimeEnabledFeatures::contextMenuEnabled())
229 runtimeCSS.appendLiteral("menu[type=\"popup\" i] { display: none; }"); 229 runtimeCSS.append("menu[type=\"popup\" i] { display: none; }");
230 return runtimeCSS.toString(); 230 return runtimeCSS.toString();
231 } 231 }
232 232
233 static String formatChromiumMediaControlsTime(float time, float duration, bool i ncludeSeparator) 233 static String formatChromiumMediaControlsTime(float time, float duration, bool i ncludeSeparator)
234 { 234 {
235 if (!std::isfinite(time)) 235 if (!std::isfinite(time))
236 time = 0; 236 time = 0;
237 if (!std::isfinite(duration)) 237 if (!std::isfinite(duration))
238 duration = 0; 238 duration = 0;
239 int seconds = static_cast<int>(fabsf(time)); 239 int seconds = static_cast<int>(fabsf(time));
(...skipping 665 matching lines...) Expand 10 before | Expand all | Expand 10 after
905 905
906 // padding - not honored by WinIE, needs to be removed. 906 // padding - not honored by WinIE, needs to be removed.
907 style.resetPadding(); 907 style.resetPadding();
908 908
909 // border - honored by WinIE, but looks terrible (just paints in the control box and turns off the Windows XP theme) 909 // border - honored by WinIE, but looks terrible (just paints in the control box and turns off the Windows XP theme)
910 // for now, we will not honor it. 910 // for now, we will not honor it.
911 style.resetBorder(); 911 style.resetBorder();
912 } 912 }
913 913
914 } // namespace blink 914 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698