Remove dead code: builder_path_from_name, builder_path_from_port_name

Remove dead code: builder_path_from_name, builder_path_from_port_name

This is a follow-up to http://crrev.com/2006383003.

Turns out builder_name_from_port_name and builder_name_from_path
aren't use anywhere, (the actual way that the builder names get
changed when getting the Google Storage path is by a function in
webkitpy.common.config.urls:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit/Tools/Scripts/webkitpy/common/config/urls.py&l=40

Since the only test in the unit test was testing build_path_from_name,
I removed the unit test for now, but I think it would be good to re-add
it with a test for builder_name_for_port_name in another CL.

This also removes rebaseline_override_dir and _ports_without_builders (both unused)

Committed: https://crrev.com/f66b53aae8a91a6814150b6971dd4f875370c32c
Cr-Commit-Position: refs/heads/master@{#395997}

[qyearsley, 2016-05-25 17:42:58]

qyearsley@chromium.org changed reviewers:
+ dpranke@chromium.org, prasadv@chromium.org

[Dirk Pranke, 2016-05-25 19:33:01]

lgtm

[qyearsley, 2016-05-25 20:40:35]

Description was changed from

==========
Remove dead code: builder_path_from_name, builder_path_from_port_name

This is a follow-up to http://crrev.com/2006383003.

Turns out builder_name_from_port_name and builder_name_from_path
aren't use anywhere, (the actual way that the builder names get
changed when getting the Google Storage path is by a function in
webkitpy.common.config.urls:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

Since the only test in the unit test was testing build_path_from_name,
I removed the unit test for now, but I think it would be good to re-add
it with a test for builder_name_for_port_name in another CL.
==========

to

==========
Remove dead code: builder_path_from_name, builder_path_from_port_name

This is a follow-up to http://crrev.com/2006383003.

Turns out builder_name_from_port_name and builder_name_from_path
aren't use anywhere, (the actual way that the builder names get
changed when getting the Google Storage path is by a function in
webkitpy.common.config.urls:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

Since the only test in the unit test was testing build_path_from_name,
I removed the unit test for now, but I think it would be good to re-add
it with a test for builder_name_for_port_name in another CL.

This also removes rebaseline_override_dir and _ports_without_builders (both
unused)
==========

[qyearsley, 2016-05-25 20:40:40]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-05-25 20:41:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2006333003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2006333003/20001

[commit-bot: I haz the power, 2016-05-25 22:01:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-25 22:03:48]

Description was changed from

==========
Remove dead code: builder_path_from_name, builder_path_from_port_name

This is a follow-up to http://crrev.com/2006383003.

Turns out builder_name_from_port_name and builder_name_from_path
aren't use anywhere, (the actual way that the builder names get
changed when getting the Google Storage path is by a function in
webkitpy.common.config.urls:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

Since the only test in the unit test was testing build_path_from_name,
I removed the unit test for now, but I think it would be good to re-add
it with a test for builder_name_for_port_name in another CL.

This also removes rebaseline_override_dir and _ports_without_builders (both
unused)
==========

to

==========
Remove dead code: builder_path_from_name, builder_path_from_port_name

This is a follow-up to http://crrev.com/2006383003.

Turns out builder_name_from_port_name and builder_name_from_path
aren't use anywhere, (the actual way that the builder names get
changed when getting the Google Storage path is by a function in
webkitpy.common.config.urls:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

Since the only test in the unit test was testing build_path_from_name,
I removed the unit test for now, but I think it would be good to re-add
it with a test for builder_name_for_port_name in another CL.

This also removes rebaseline_override_dir and _ports_without_builders (both
unused)

Committed: https://crrev.com/f66b53aae8a91a6814150b6971dd4f875370c32c
Cr-Commit-Position: refs/heads/master@{#395997}
==========

[commit-bot: I haz the power, 2016-05-25 22:03:49]

Patchset 2 (id:??) landed as
https://crrev.com/f66b53aae8a91a6814150b6971dd4f875370c32c
Cr-Commit-Position: refs/heads/master@{#395997}