Grant access to files in PageState, which have already been validated.

Grant access to files in PageState, which have already been validated.

BUG=262948
TEST=Go back (cross-process) to a page with a file selected to upload.

Committed: https://crrev.com/e1d3e5b067465db7fccdfa0bfa5f5ffd9784c53b
Cr-Commit-Position: refs/heads/master@{#306209}

[Charlie Reis, 2014-11-25 20:57:34]

Patchset #3 (id:40001) has been deleted

[Charlie Reis, 2014-11-25 21:00:54]

creis@chromium.org changed reviewers:
+ nasko@chromium.org

[Charlie Reis, 2014-11-25 21:00:55]

https://codereview.chromium.org/745053002/diff/60001/chrome/browser/sessions/...
File chrome/browser/sessions/session_restore.cc (left):

https://codereview.chromium.org/745053002/diff/60001/chrome/browser/sessions/...
chrome/browser/sessions/session_restore.cc:1085: // Set up the file access
rights for the selected navigation entry.
This is now redundant with RenderFrameHostImpl::OnNavigate.

https://codereview.chromium.org/745053002/diff/60001/content/browser/web_cont...
File content/browser/web_contents/web_contents_impl.cc (left):

https://codereview.chromium.org/745053002/diff/60001/content/browser/web_cont...
content/browser/web_contents/web_contents_impl.cc:3620:
static_cast<RenderViewHostImpl*>(rvh)))
This was buggy and causing the test to fail.  It meant we were ignoring state
updates from the old RVH if the process was otherwise unused and wasn't kept on
the swapped out list.

[nasko, 2014-11-25 22:56:20]

LGTM

[Charlie Reis, 2014-11-25 23:10:49]

Thanks!

+sky@ for OWNERS for session_restore.cc.

[Charlie Reis, 2014-11-25 23:11:03]

creis@chromium.org changed reviewers:
+ sky@chromium.org

[Charlie Reis, 2014-11-25 23:11:04]

Actually +sky this time.

[sky, 2014-12-01 16:49:00]

LGTM

[Charlie Reis, 2014-12-01 17:02:20]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2014-12-01 17:02:41]

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

[commit-bot: I haz the power, 2014-12-01 18:22:05]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2014-12-01 18:22:55]

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