Remove a unused parameter in the Linux FilePathWatcher implementation.

Remove a unused parameter in the Linux FilePathWatcher implementation.

BUG=none
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=126359

[Lei Zhang, 2012-03-09 21:24:04]

ping?

[Mattias Nissler (ping if slow), 2012-03-12 08:50:04]

Sorry for the delay, I was on vacation. LGTM, and thanks for the cleanup.

http://codereview.chromium.org/9553007/diff/3001/base/files/file_path_watcher...
File base/files/file_path_watcher_linux.cc (right):

http://codereview.chromium.org/9553007/diff/3001/base/files/file_path_watcher...
base/files/file_path_watcher_linux.cc:337: // 'is_directory' set in the event
masks. As a result we may
suggestion: s/'is_directory'/IN_ISDIR/

[Lei Zhang, 2012-03-12 20:42:56]

http://codereview.chromium.org/9553007/diff/3001/base/files/file_path_watcher...
File base/files/file_path_watcher_linux.cc (right):

http://codereview.chromium.org/9553007/diff/3001/base/files/file_path_watcher...
base/files/file_path_watcher_linux.cc:337: // 'is_directory' set in the event
masks. As a result we may
On 2012/03/12 08:50:04, Mattias Nissler wrote:
> suggestion: s/'is_directory'/IN_ISDIR/

Done.

[commit-bot: I haz the power, 2012-03-12 20:42:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/9553007/7001

[Lei Zhang, 2012-03-12 20:43:31]

Hmm, CQ's going to fail without an OWNER.

[Lei Zhang, 2012-03-12 20:47:32]

Oh but wait, base/files has its own OWNERS.

[commit-bot: I haz the power, 2012-03-12 22:30:56]

The commit queue went berserk retrying too often for a
seemingly flaky test. Builder is linux_rel, revision is 126247, job name
was 9553007-7001 (retry) (retry) (retry) (retry).

[commit-bot: I haz the power, 2012-03-12 22:33:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/9553007/7001

[commit-bot: I haz the power, 2012-03-13 00:43:30]

Try job failure for 9553007-7001 (retry) on win_rel for steps "browser_tests,
check_deps".
It's a second try, previously, steps "browser_tests, check_deps" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-03-13 07:10:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/9553007/7001

[commit-bot: I haz the power, 2012-03-13 08:34:00]

Change committed as 126359