cc: Add HasPendingNotification to delayed unique notifier.

cc: Add HasPendingNotification to delayed unique notifier.

It would be nice to know if a notification is pending, so this patch
adds this ability. This would be used in RenewTreePriority to detect
whether we should be in smoothness mode or same priority mode. That is,
if we have an expiration from smoothness pending, then we should still
be in smoothness mode.

R=reveman

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

[vmpstr, 2014-05-29 22:58:21]

Please take a look.

[reveman, 2014-05-30 00:19:28]

lgtm with one small test request

https://codereview.chromium.org/302003004/diff/1/cc/base/delayed_unique_notif...
File cc/base/delayed_unique_notifier_unittest.cc (right):

https://codereview.chromium.org/302003004/diff/1/cc/base/delayed_unique_notif...
cc/base/delayed_unique_notifier_unittest.cc:247: notifier.Cancel();
While here, can you move this Cancel() call to the loop above to make sure that
calling Schedule() and Cancel() repetitively result in only one pending task.

[vmpstr, 2014-05-30 00:38:33]

https://codereview.chromium.org/302003004/diff/1/cc/base/delayed_unique_notif...
File cc/base/delayed_unique_notifier_unittest.cc (right):

https://codereview.chromium.org/302003004/diff/1/cc/base/delayed_unique_notif...
cc/base/delayed_unique_notifier_unittest.cc:247: notifier.Cancel();
On 2014/05/30 00:19:28, reveman wrote:
> While here, can you move this Cancel() call to the loop above to make sure
that
> calling Schedule() and Cancel() repetitively result in only one pending task.

Done.

[vmpstr, 2014-05-30 00:38:37]

The CQ bit was checked by vmpstr@chromium.org

[commit-bot: I haz the power, 2014-05-30 00:40:11]

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

[commit-bot: I haz the power, 2014-05-30 10:08:06]

Change committed as 273780