Add scroll restoration type to page state

Add scroll restoration type to page state

The scroll restoration type is added to page state and serialized. It is used
by blink to control the scroll/scale restoration behaviour for history
entries. Its value may be provided by user via web-exposed API.

Blink side CL: http://crrev.com/927213004

BUG=444094
Committed: https://crrev.com/bbac25a678deb0148e6f8a553e451a6a6ab1a6ac
Cr-Commit-Position: refs/heads/master@{#329632}

[majidvp, 2015-02-26 17:44:21]

majidvp@chromium.org changed reviewers:
+ tdresser@chromium.org

[tdresser, 2015-02-26 18:21:16]

LGTM, but make sure to get someone more familiar with this area of the code to
take a look as well.

[majidvp, 2015-03-02 14:39:14]

majidvp@chromium.org changed reviewers:
+ sievers@chromium.org

[majidvp, 2015-03-02 14:39:14]

+sievers@ for OWNER's review.

[no sievers, 2015-03-02 19:51:06]

sievers@chromium.org changed reviewers:
+ japhet@chromium.org

[no sievers, 2015-03-02 19:53:20]

On 2015/03/02 14:39:14, majidvp wrote:
> +sievers@ for OWNER's review.

looks fine if that is what we want to do.

Adding Nate also though to stamp blink-side change and this.

[majidvp, 2015-05-07 18:13:04]

On 2015/03/02 19:53:20, sievers wrote:
> On 2015/03/02 14:39:14, majidvp wrote:
> > +sievers@ for OWNER's review.
> 
> looks fine if that is what we want to do.
> 
> Adding Nate also though to stamp blink-side change and this.


Nate: PTAL. 

The only update is |s/is_created_by_history_api/should_restore_scroll/| to match
the blink side change.

[Nate Chapin, 2015-05-07 20:27:55]

On 2015/05/07 18:13:04, majidvp wrote:
> On 2015/03/02 19:53:20, sievers wrote:
> > On 2015/03/02 14:39:14, majidvp wrote:
> > > +sievers@ for OWNER's review.
> > 
> > looks fine if that is what we want to do.
> > 
> > Adding Nate also though to stamp blink-side change and this.
> 
> 
> Nate: PTAL. 
> 
> The only update is |s/is_created_by_history_api/should_restore_scroll/| to
match
> the blink side change.

lgtm once the blink side lands.

[majidvp, 2015-05-12 18:02:47]

On 2015/05/07 20:27:55, Nate Chapin wrote:
> On 2015/05/07 18:13:04, majidvp wrote:
> > On 2015/03/02 19:53:20, sievers wrote:
> > > On 2015/03/02 14:39:14, majidvp wrote:
> > > > +sievers@ for OWNER's review.
> > > 
> > > looks fine if that is what we want to do.
> > > 
> > > Adding Nate also though to stamp blink-side change and this.
> > 
> > 
> > Nate: PTAL. 
> > 
> > The only update is |s/is_created_by_history_api/should_restore_scroll/| to
> match
> > the blink side change.
> 
> lgtm once the blink side lands.

sievers: PTAL.

[no sievers, 2015-05-12 18:16:55]

lgtm

[majidvp, 2015-05-13 13:00:14]

The CQ bit was checked by majidvp@chromium.org

[majidvp, 2015-05-13 13:00:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdresser@chromium.org, japhet@chromium.org
Link to the patchset: https://codereview.chromium.org/959783003/#ps100001
(title: "Use enum for scroll restoration type")

[majidvp, 2015-05-13 13:00:27]

The CQ bit was unchecked by majidvp@chromium.org

[majidvp, 2015-05-13 13:28:24]

The CQ bit was checked by majidvp@chromium.org

[commit-bot: I haz the power, 2015-05-13 13:28:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/959783003/100001

[commit-bot: I haz the power, 2015-05-13 14:26:20]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-05-13 14:27:04]

Patchset 6 (id:??) landed as
https://crrev.com/bbac25a678deb0148e6f8a553e451a6a6ab1a6ac
Cr-Commit-Position: refs/heads/master@{#329632}