RESOLVED FIXED 143107
Web Inspector: Convert some View classes to ES6 classes 
https://bugs.webkit.org/show_bug.cgi?id=143107
Summary Web Inspector: Convert some View classes to ES6 classes
Timothy Hatcher
Reported 2015-03-26 11:05:59 PDT
Some of the simple View classes can be converted now. The rest are harder.
Attachments
Patch (81.17 KB, patch)
2015-03-26 11:11 PDT, Timothy Hatcher 		no flags
DetailsFormatted DiffDiff
Patch (175.47 KB, patch)
2015-03-26 11:11 PDT, Timothy Hatcher 		no flags
DetailsFormatted DiffDiff
Patch (175.46 KB, patch)
2015-03-26 11:19 PDT, Timothy Hatcher 		joepeck: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2015-03-26 11:06:40 PDT
<rdar://problem/20311871>
Timothy Hatcher
Comment 2 2015-03-26 11:11:29 PDT
Created attachment 249497 [details] Patch
Timothy Hatcher
Comment 3 2015-03-26 11:11:53 PDT
Created attachment 249498 [details] Patch
Timothy Hatcher
Comment 4 2015-03-26 11:19:07 PDT
Created attachment 249501 [details] Patch
Joseph Pecoraro
Comment 5 2015-03-26 11:49:44 PDT
Comment on attachment 249501 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=249501&action=review r=me > Source/WebInspectorUI/ChangeLog:41 > + Converted to ES6 ckasses. Typo: "ckasses" => "classes" > Source/WebInspectorUI/UserInterface/Views/ContextMenu.js:150 > + constructor(event) { Nit: Style, brace should be on newline. > Source/WebInspectorUI/UserInterface/Views/EditingSupport.js:81 > + this.commitHandler = commitHandler; Could make these _foo. > Source/WebInspectorUI/UserInterface/Views/GoToLineDialog.js:122 > + event.preventDefault(); Is this fixing a bug? > Source/WebInspectorUI/UserInterface/Views/SearchBar.js:47 > + this._searchInput.addEventListener("search", this._handleSearchEvent.bind(this), false); > + this._searchInput.addEventListener("keydown", this._handleKeydownEvent.bind(this), false); Style: I think we have been dropping the false for useCapture for addEventListener. We can update that here.
Timothy Hatcher
Comment 6 2015-03-26 16:40:07 PDT
Comment on attachment 249501 [details] Patch https://trac.webkit.org/r182040
Note You need to log in before you can comment on or make changes to this bug.