Summary: | webkitpy: clean up some port._filesystem references to not reference private members | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dirk Pranke <dpranke> | ||||||
Component: | New Bugs | Assignee: | Dirk Pranke <dpranke> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, eric, ojan, tony, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 71380 | ||||||||
Attachments: |
|
Description
Dirk Pranke
2012-03-19 16:54:54 PDT
Created attachment 132722 [details]
Patch
Created attachment 132725 [details]
fix nit in write_image_diff_files() as well
Comment on attachment 132725 [details] fix nit in write_image_diff_files() as well View in context: https://bugs.webkit.org/attachment.cgi?id=132725&action=review OK. > Tools/Scripts/webkitpy/layout_tests/controllers/single_test_runner.py:71 > - if port.host.filesystem.exists(expected_filename): > + if self._filesystem.exists(expected_filename): In general getting away from using "port" for these sorts of things seems good. The goal being to make "port" less of the hub/god object. Committed r111289: <http://trac.webkit.org/changeset/111289> |