RESOLVED FIXED 191347
Custom test group form should use commit set map before customization as the behavior of radio buttons.
https://bugs.webkit.org/show_bug.cgi?id=191347
Summary Custom test group form should use commit set map before customization as the ...
dewei_zhu
Reported 2018-11-06 22:24:41 PST
Custom test group form should use commit set map before customization as the behavior of radio buttons.
Attachments
Patch (14.85 KB, patch)
2018-11-06 22:28 PST, dewei_zhu
rniwa: review+
dewei_zhu
Comment 1 2018-11-06 22:28:28 PST
dewei_zhu
Comment 2 2018-11-06 22:35:09 PST
Comment on attachment 354064 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=354064&action=review > Websites/perf.webkit.org/browser-tests/customizable-test-group-form-tests.js:143 > + > + Will remove those two extra blank lines.
dewei_zhu
Comment 3 2018-11-06 22:40:09 PST
Ryosuke Niwa
Comment 4 2018-11-07 15:31:24 PST
Comment on attachment 354064 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=354064&action=review > Websites/perf.webkit.org/public/v3/components/customizable-test-group-form.js:353 > -ComponentBase.defineElement('customizable-test-group-form', CustomizableTestGroupForm); > +ComponentBase.defineElement('customizable-test-group-form', CustomizableTestGroupForm); What happened here? Adding a new line??
dewei_zhu
Comment 5 2018-11-07 16:22:18 PST
Comment on attachment 354064 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=354064&action=review >> Websites/perf.webkit.org/public/v3/components/customizable-test-group-form.js:353 >> +ComponentBase.defineElement('customizable-test-group-form', CustomizableTestGroupForm); > > What happened here? Adding a new line?? I removed a blank line after this line.
dewei_zhu
Comment 6 2018-11-07 16:30:48 PST
Landed in r237950
dewei_zhu
Comment 7 2018-11-07 16:31:50 PST
(In reply to dewei_zhu from comment #3) > Tracked in <rdar://problem/4586649> Missing one digit. The actual tracking radar is <rdar://problem/45866496>
Note You need to log in before you can comment on or make changes to this bug.