When sanitizing serialized navigation entries also take iframes into account

When sanitizing serialized navigation entries also take iframes into account

The state of the iframes is encoded in the page state, so we need to
decode the page state and recursively check it.

BUG=422871
R=marja@chromium.org

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

[marja, 2015-02-23 15:30:04]

lgtm

https://codereview.chromium.org/948013002/diff/1/content/public/common/page_s...
File content/public/common/page_state.cc (right):

https://codereview.chromium.org/948013002/diff/1/content/public/common/page_s...
content/public/common/page_state.cc:136: if (!DecodePageState(data_, &state))
What happens to data_ here? Should this branch set it to empty?

[jochen (gone - plz use gerrit), 2015-02-23 16:20:45]

https://codereview.chromium.org/948013002/diff/1/content/public/common/page_s...
File content/public/common/page_state.cc (right):

https://codereview.chromium.org/948013002/diff/1/content/public/common/page_s...
content/public/common/page_state.cc:136: if (!DecodePageState(data_, &state))
On 2015/02/23 15:30:04, marja wrote:
> What happens to data_ here? Should this branch set it to empty?

IsValid() is not supposed to modify itself (it's const)

[jochen (gone - plz use gerrit), 2015-02-23 16:25:47]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2015-02-23 16:26:54]

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

[commit-bot: I haz the power, 2015-02-23 16:29:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-23 16:29:46]

Try jobs failed on following builders:
  ios_dbg_simulator on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)

[jochen (gone - plz use gerrit), 2015-02-23 16:42:27]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2015-02-23 16:43:05]

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

[commit-bot: I haz the power, 2015-02-23 16:55:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-23 16:56:36]

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

[jochen (gone - plz use gerrit), 2015-02-24 09:43:58]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/952783003/ by jochen@chromium.org.

The reason for reverting is: regresses startup performance too much.