Bug 64885 - Move HttpLock to using a FileSystem object
Summary: Move HttpLock to using a FileSystem object
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Eric Seidel (no email)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-20 12:56 PDT by Eric Seidel (no email)
Modified: 2011-07-20 15:06 PDT (History)
5 users (show)

See Also:


Attachments
Patch (7.09 KB, patch)
2011-07-20 12:58 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff
Patch (11.04 KB, patch)
2011-07-20 15:01 PDT, Eric Seidel (no email)
abarth: review+
abarth: commit-queue+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2011-07-20 12:56:32 PDT
Move HttpLock to using a FileSystem object
Comment 1 Eric Seidel (no email) 2011-07-20 12:58:28 PDT
Created attachment 101498 [details]
Patch
Comment 2 WebKit Review Bot 2011-07-20 13:46:00 PDT
Comment on attachment 101498 [details]
Patch

Clearing flags on attachment: 101498

Committed r91398: <http://trac.webkit.org/changeset/91398>
Comment 3 WebKit Review Bot 2011-07-20 13:46:05 PDT
All reviewed patches have been landed.  Closing bug.
Comment 4 Stephen White 2011-07-20 14:27:10 PDT
(In reply to comment #3)
> All reviewed patches have been landed.  Closing bug.

The Chrome WebKit Linux canaries seem to be unhappy with this change:

2011-07-20 13:54:27,112 10770 http_lock.py:117 DEBUG Creating lock file: /tmp/WebKitHttpd.lock.0
Traceback (most recent call last):
  File "/mnt/data/b/build/slave/Webkit_Linux_32/build/src/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py", line 436, in <module>
    sys.exit(main())
  File "/mnt/data/b/build/slave/Webkit_Linux_32/build/src/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py", line 431, in main
    return run(port, options, args)
  File "/mnt/data/b/build/slave/Webkit_Linux_32/build/src/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py", line 107, in run
    num_unexpected_results = manager.run(result_summary)
  File "/mnt/data/b/build/slave/Webkit_Linux_32/build/src/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py", line 873, in run
    interrupted, keyboard_interrupted, thread_timings, test_timings, individual_test_timings = self._run_tests(self._test_files_list, result_summary)
  File "/mnt/data/b/build/slave/Webkit_Linux_32/build/src/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py", line 730, in _run_tests
    self.start_servers_with_lock()
  File "/mnt/data/b/build/slave/Webkit_Linux_32/build/src/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py", line 927, in start_servers_with_lock
    self._port.acquire_http_lock()
  File "/mnt/data/b/build/slave/Webkit_Linux_32/build/src/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py", line 715, in acquire_http_lock
    self._http_lock.wait_for_httpd_lock()
  File "/mnt/data/b/build/slave/Webkit_Linux_32/build/src/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/http_lock.py", line 129, in wait_for_httpd_lock
    while self._curent_lock_pid() != os.getpid():
  File "/mnt/data/b/build/slave/Webkit_Linux_32/build/src/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/http_lock.py", line 92, in _curent_lock_pid
    if not (current_pid and self._executive.check_running_pid(int(current_pid))):
NameError: global name 'current_pid' is not defined

http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Linux%2032/builds/3079/steps/webkit_tests/logs/stdio
Comment 5 Eric Seidel (no email) 2011-07-20 14:34:38 PDT
Fixing.
Comment 6 Eric Seidel (no email) 2011-07-20 15:01:59 PDT
Created attachment 101517 [details]
Patch
Comment 7 Eric Seidel (no email) 2011-07-20 15:03:07 PDT
Reopening to fix.
Comment 8 Eric Seidel (no email) 2011-07-20 15:03:28 PDT
Comment on attachment 101517 [details]
Patch

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

> Tools/Scripts/webkitpy/layout_tests/port/http_lock.py:92
> -            current_lock_file = self._filesystem.read_text_file(lock_list[0])
> +            current_pid = self._filesystem.read_text_file(lock_list[0])

This is the fix, btw.
Comment 9 Eric Seidel (no email) 2011-07-20 15:06:17 PDT
Committed r91411: <http://trac.webkit.org/changeset/91411>