NEW 179047
OwnedCommitViewer should include the preceding commit. 
https://bugs.webkit.org/show_bug.cgi?id=179047
Summary OwnedCommitViewer should include the preceding commit.
dewei_zhu
Reported 2017-10-31 00:37:33 PDT
OwnedCommitViewer should include the preceding commit.
Attachments
Patch (4.45 KB, patch)
2017-10-31 00:44 PDT, dewei_zhu 		rniwa: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
dewei_zhu
Comment 1 2017-10-31 00:44:21 PDT
Created attachment 325426 [details] Patch
Ryosuke Niwa
Comment 2 2017-10-31 00:57:45 PDT
Comment on attachment 325426 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=325426&action=review > Websites/perf.webkit.org/public/v3/components/commit-log-viewer.js:39 > + let fetchSingleCommit = !precedingRevision || precedingRevision == lastRevision; Use const. > Websites/perf.webkit.org/public/v3/components/commit-log-viewer.js:59 > + this._precedingCommit = precedingCommit[0]; You should check this._fetchingPromise != promise here to avoid showing results for the old range. > Websites/perf.webkit.org/public/v3/components/commit-log-viewer.js:88 > + let commits_to_process = commits && precedingCommit && precedingCommit.ownsCommits() ? [precedingCommit].concat(commits) : commits; Use camelCase. In fact, I think we can just assign directly to commits in the case we have precedingCommit.
Note You need to log in before you can comment on or make changes to this bug.