|Summary:||[garden-o-matic] Builder names without underscores cause incorrect BuildSelector behavior|
|Product:||WebKit||Reporter:||Zan Dobersek <zan>|
|Component:||Tools / Tests||Assignee:||Zan Dobersek <zan>|
|Severity:||Normal||CC:||abarth, dglazkov, simon.fraser, webkit.review.bot|
|Version:||528+ (Nightly build)|
Description Zan Dobersek 2012-06-18 10:56:51 PDT
Builder names without underscores cause the BuildSelector object to not store the containers in the proper way, displaying all the containers instead of just the one the selected tab is linked with. An example of this is when viewing results for the Apple builders - the container id and hash that's used in the associated tab are just encoded as URI components (the result of the current platform's resultsDirectoryNameFromBuilderName method), which doesn't work. Chromium doesn't have such problems as the builders' results directories are already replacing whitespace, braces and dots with underscores. JQuery documentation suggests something similar. I think it's safe to convert whitespace, braces and dots to underscores for every builder name in ui.results.BuilderSelector.init and use that to link the container with the associated tab.  - http://docs.jquery.com/UI/Tabs#Ajax_mode
Comment 1 Simon Fraser (smfr) 2012-06-18 11:05:27 PDT
I'm not quite following how to see the actual bug in the garden-o-matic UI.
Comment 3 Simon Fraser (smfr) 2012-06-18 11:10:49 PDT
Comment on attachment 148130 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=148130&action=review > Tools/ChangeLog:12 > + (.): Remove this. > Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/results.js:370 > + var builderHash = builderName.replace(/[ .()]/g, '_'); This looks very similar to config.resultsDirectoryNameFromBuilderName() for chromium. Maybe put a function in base and call it in both places.
Comment 4 Zan Dobersek 2012-06-18 12:04:01 PDT
(In reply to comment #3) > > Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/results.js:370 > > + var builderHash = builderName.replace(/[ .()]/g, '_'); > > This looks very similar to config.resultsDirectoryNameFromBuilderName() for chromium. Maybe put a function in base and call it in both places. Created base.underscoredBuilderName. Committed http://trac.webkit.org/changeset/120614.