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

Unified Diff: third_party/WebKit/Source/devtools/front_end/perf_ui/FilmStripView.js

Issue 2912563003: DevTools: cleanup button styles (Closed)
Patch Set: fix test Created 3 years, 7 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/devtools/front_end/perf_ui/FilmStripView.js
diff --git a/third_party/WebKit/Source/devtools/front_end/perf_ui/FilmStripView.js b/third_party/WebKit/Source/devtools/front_end/perf_ui/FilmStripView.js
index ee0774cec6b99cc8274943a15f752e6fbeef9a96..049bf3e1d5ac7eb08a77e026fb4cdd63122220bb 100644
--- a/third_party/WebKit/Source/devtools/front_end/perf_ui/FilmStripView.js
+++ b/third_party/WebKit/Source/devtools/front_end/perf_ui/FilmStripView.js
@@ -223,12 +223,12 @@ PerfUI.FilmStripView.Dialog = class extends UI.VBox {
this._imageElement = this.contentElement.createChild('img');
var footerElement = this.contentElement.createChild('div', 'filmstrip-dialog-footer');
footerElement.createChild('div', 'flex-auto');
- var prevButton =
- UI.createTextButton('\u25C0', this._onPrevFrame.bind(this), undefined, Common.UIString('Previous frame'));
+ var prevButton = UI.createTextButton('\u25C0', this._onPrevFrame.bind(this));
+ prevButton.title = Common.UIString('Previous frame');
footerElement.appendChild(prevButton);
this._timeLabel = footerElement.createChild('div', 'filmstrip-dialog-label');
- var nextButton =
- UI.createTextButton('\u25B6', this._onNextFrame.bind(this), undefined, Common.UIString('Next frame'));
+ var nextButton = UI.createTextButton('\u25B6', this._onNextFrame.bind(this));
+ nextButton.title = Common.UIString('Next frame');
footerElement.appendChild(nextButton);
footerElement.createChild('div', 'flex-auto');

Powered by Google App Engine
This is Rietveld 408576698