Fix file path notification filtering on Windows.

Fix file path notification filtering on Windows.

first_notification_ was getting reset on each notification in UpdateWatch,
causing the filtering to be less effective than it should have been.


BUG=NONE

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271414

[Mattias Nissler (ping if slow), 2014-05-14 10:27:52]

LGTM with a nit.

https://codereview.chromium.org/273843004/diff/1/base/files/file_path_watcher...
File base/files/file_path_watcher_win.cc (right):

https://codereview.chromium.org/273843004/diff/1/base/files/file_path_watcher...
base/files/file_path_watcher_win.cc:169: last_modified_ !=
file_info.last_modified)) {
nit: align at ( for readability

[vandebo (ex-Chrome), 2014-05-14 16:55:40]

https://codereview.chromium.org/273843004/diff/1/base/files/file_path_watcher...
File base/files/file_path_watcher_win.cc (right):

https://codereview.chromium.org/273843004/diff/1/base/files/file_path_watcher...
base/files/file_path_watcher_win.cc:169: last_modified_ !=
file_info.last_modified)) {
On 2014/05/14 10:27:52, Mattias Nissler wrote:
> nit: align at ( for readability

Oops, fixed.

[vandebo (ex-Chrome), 2014-05-14 16:55:44]

The CQ bit was checked by vandebo@chromium.org

[commit-bot: I haz the power, 2014-05-14 16:56:13]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vandebo@chromium.org/273843004/20001

[commit-bot: I haz the power, 2014-05-14 18:37:13]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)

[commit-bot: I haz the power, 2014-05-14 18:53:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-14 18:53:32]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)

[vandebo (ex-Chrome), 2014-05-14 20:04:45]

The CQ bit was checked by vandebo@chromium.org

[commit-bot: I haz the power, 2014-05-14 20:06:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vandebo@chromium.org/273843004/20001

[commit-bot: I haz the power, 2014-05-14 20:56:59]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)

[commit-bot: I haz the power, 2014-05-14 21:12:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-14 21:12:39]

Try jobs failed on following builders:
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)

[vandebo (ex-Chrome), 2014-05-19 16:27:56]

The CQ bit was checked by vandebo@chromium.org

[commit-bot: I haz the power, 2014-05-19 16:28:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vandebo@chromium.org/273843004/20001

[commit-bot: I haz the power, 2014-05-19 17:38:28]

Change committed as 271414