[Webkit-unassigned] [Bug 147800] Use python 'with' statement to make cleanup more robust and minor code cleaning

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=147800

--- Comment #4 from dewei_zhu at apple.com ---
Comment on attachment 258543
  --> https://bugs.webkit.org/attachment.cgi?id=258543
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=258543&action=review

>> Tools/Scripts/webkitpy/benchmark_runner/benchmark_runner.py:108
>>                      result = self._http_server_driver.fetch_result()
> 
> It looks like _browser_driver is not accessed anywhere else so I don't think it needs to be be on _runner at all.
> Ideally, I'd like see the code like this here:
> 
> with TestEnvironment(benchmark) as env, timeout(self._plan['timeout'])
>     result = env.fetch_results()

Browser driver is specified by platform and browser which are provided by the parameter of BenchmarkRunner.__init__. The solution I can think of currently is storing those values in a structure, and supplying this structure as  parameter of TextEnvironment.__init__ . Does that sounds good?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150808/bfc490ee/attachment-0001.html>