Simplify synchronous fetching in WebsitePermissionsFetcher.

Simplify synchronous fetching in WebsitePermissionsFetcher.

This makes it easier to implement a synchronous
WebsitePermissionsFetcher.Task by simply overriding Task.run(), instead
of having to call TaskQueue.next() at the end.

Committed: https://crrev.com/7c43624a5f2eeae4d5e6e2b4cfd8ed2ef72618b0
Cr-Commit-Position: refs/heads/master@{#366404}

[newt (away), 2015-12-18 02:13:57]

newt@chromium.org changed reviewers:
+ finnur@chromium.org

[newt (away), 2015-12-18 02:13:57]

What do you think?

[Finnur, 2015-12-18 10:49:51]

I'm not against this, but when is a synchronous fetcher needed? Is it for new
code?

[newt (away), 2015-12-18 16:39:04]

On 2015/12/18 10:49:51, Finnur wrote:
> I'm not against this, but when is a synchronous fetcher needed? Is it for new
> code?

All of the Fetcher classes except for LocalStorageInfoFetcher and
WebStorageInfoFetcher run synchronously. (This CL doesn't change that behavior.)

[Finnur, 2015-12-21 09:46:26]

Ah, yeah. You are right. LGTM.

[newt (away), 2015-12-21 16:38:48]

The CQ bit was checked by newt@chromium.org

[commit-bot: I haz the power, 2015-12-21 16:39:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1536893002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1536893002/1

[commit-bot: I haz the power, 2015-12-21 17:28:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-21 17:29:16]

Description was changed from

==========
Simplify synchronous fetching in WebsitePermissionsFetcher.

This makes it easier to implement a synchronous
WebsitePermissionsFetcher.Task by simply overriding Task.run(), instead
of having to call TaskQueue.next() at the end.
==========

to

==========
Simplify synchronous fetching in WebsitePermissionsFetcher.

This makes it easier to implement a synchronous
WebsitePermissionsFetcher.Task by simply overriding Task.run(), instead
of having to call TaskQueue.next() at the end.

Committed: https://crrev.com/7c43624a5f2eeae4d5e6e2b4cfd8ed2ef72618b0
Cr-Commit-Position: refs/heads/master@{#366404}
==========

[commit-bot: I haz the power, 2015-12-21 17:29:17]

Patchset 1 (id:??) landed as
https://crrev.com/7c43624a5f2eeae4d5e6e2b4cfd8ed2ef72618b0
Cr-Commit-Position: refs/heads/master@{#366404}