Processing groups with nextPollSeconds === undefined

Processing groups with nextPollSeconds === undefined.

In theory, the server can generate a group with requested == true, but nextPollSeconds === undefined.

This change makes the client correctly process this.

BUG=164227
TEST=No

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

[vadimt, 2013-10-23 17:11:45]

This is a second attempt to land this CL.
The first one landed via CQ, but broke a unit tests added by another CL.
This CL adds a fix in the unit test.

[vadimt, 2013-10-23 17:12:15]

On 2013/10/23 17:11:45, vadimt wrote:
> This is a second attempt to land this CL.
> The first one landed via CQ, but broke a unit tests added by another CL.
> This CL adds a fix in the unit test.

P.S. The first CL was reverted.

[robliao, 2013-10-23 17:15:29]

https://codereview.chromium.org/31033004/diff/110001/chrome/browser/resources...
File chrome/browser/resources/google_now/background_unittest.gtestjs (left):

https://codereview.chromium.org/31033004/diff/110001/chrome/browser/resources...
chrome/browser/resources/google_now/background_unittest.gtestjs:411:
nextPollTime: Date.now()
Is this not critical to the test? Seems like nextPollTime is always set.

[vadimt, 2013-10-23 17:19:28]

https://codereview.chromium.org/31033004/diff/110001/chrome/browser/resources...
File chrome/browser/resources/google_now/background_unittest.gtestjs (left):

https://codereview.chromium.org/31033004/diff/110001/chrome/browser/resources...
chrome/browser/resources/google_now/background_unittest.gtestjs:411:
nextPollTime: Date.now()
On 2013/10/23 17:15:29, Robert Liao wrote:
> Is this not critical to the test? Seems like nextPollTime is always set.

I started developing tests for parseAndShowNotificationCards, created a first
one, and switched to something else. There will be more tests for it, and
they'll test everything. This test concentrates on the logic of adding/removing
groups.

[robliao, 2013-10-23 17:22:03]

lgtm

[commit-bot: I haz the power, 2013-10-23 17:28:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vadimt@chromium.org/31033004/110001

[commit-bot: I haz the power, 2013-10-23 23:29:17]

Change committed as 230551