Initial round of cleanups now all events go through StartRequest.

Initial round of cleanups now all events go through StartRequest.

The RequestType enum is no longer needed, and AddRequest can get merged
into StartRequest.

BUG=570820
Committed: https://crrev.com/72feb1c098834b8734b7caba1de75331a9d321a8
Cr-Commit-Position: refs/heads/master@{#378537}

[Marijn Kruisselbrink, 2016-03-01 00:20:53]

Description was changed from

==========
WIP Initial round of cleanups now all events go through StartRequest.

BUG=
==========

to

==========
Initial round of cleanups now all events go through StartRequest.

The RequestType enum is no longer needed, and AddRequest can get merged
into StartRequest.

BUG=570820
==========

[Marijn Kruisselbrink, 2016-03-01 00:27:25]

mek@chromium.org changed reviewers:
+ nhiroki@chromium.org

[Marijn Kruisselbrink, 2016-03-01 00:27:26]

Not sure why I let this linger for so long. There are probably more cleanups
that are possible now all event dispatching goes through StartRequest, but this
seemed like an initial round of straight-forward ones.

[nhiroki, 2016-03-01 09:21:35]

lgtm

[Marijn Kruisselbrink, 2016-03-01 17:36:36]

The CQ bit was checked by mek@chromium.org

[commit-bot: I haz the power, 2016-03-01 17:36:54]

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

[commit-bot: I haz the power, 2016-03-01 19:53:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-01 19:55:13]

Description was changed from

==========
Initial round of cleanups now all events go through StartRequest.

The RequestType enum is no longer needed, and AddRequest can get merged
into StartRequest.

BUG=570820
==========

to

==========
Initial round of cleanups now all events go through StartRequest.

The RequestType enum is no longer needed, and AddRequest can get merged
into StartRequest.

BUG=570820

Committed: https://crrev.com/72feb1c098834b8734b7caba1de75331a9d321a8
Cr-Commit-Position: refs/heads/master@{#378537}
==========

[commit-bot: I haz the power, 2016-03-01 19:55:19]

Patchset 2 (id:??) landed as
https://crrev.com/72feb1c098834b8734b7caba1de75331a9d321a8
Cr-Commit-Position: refs/heads/master@{#378537}

[Dmitry Titov, 2016-03-03 02:04:53]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1758163002/ by dimich@chromium.org.

The reason for reverting is: Speculative revert to see if it affects memory
leaks started yesterday: http://crbug.com/591596

Will reland if it is innocent..