Enable storing and restoring sessionStorage by default.

Enable storing and restoring sessionStorage by default.

BUG=104292
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=152072

[marja, 2012-08-09 12:11:53]

Hi michaeln, we could try enabling this feature by default now, wdyt?

sail, can you rubberstamp this as an OWNER of chrome/browser/profiles/?

[awong, 2012-08-09 17:35:00]

Drive-by -- if it factors into the decision at all, these two CLs:

  http://codereview.chromium.org/10831116/
  http://codereview.chromium.org/10850010/

will cause the SSN to get lost at some point because the Pickle will get
rejected.  I think this is "okay" since I'm hoping they go in within the next 10
days, but just wanted to make sure it's highlighted.

[marja, 2012-08-09 19:27:19]

Yes, indeed, the data will be lost for one restart after these CLs go in. I also
think it's ok if they go in so soon.

[michaeln, 2012-08-09 20:33:52]

Nice timing, the m22 branch was just cut so flipping the switch now max's out
the time to see 'what could possibly go wrong' w/o impacting paying customers
(so to speak). Let's keep an eye out for problems around
dom_storage::SessionStorageDatabase and the about-to-be-turned-on code paths
walking up to that.

Also, there's some other complicated changes afoot around session storage (and
storage apis in general) on behalf of the isolated app support, we should
probably be ready for some bumps in the road coming from that direction too too.
Seems like the combination of these two things going on in parallel has the
potential for some surprising results ;)

lgtm

[rpetterson, 2012-08-09 20:59:40]

LGTM

[commit-bot: I haz the power, 2012-08-09 21:01:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/marja@chromium.org/10830216/1

[commit-bot: I haz the power, 2012-08-09 22:35:24]

Try job failure for 10830216-1 (retry) on mac_rel for steps "nacl_integration,
interactive_ui_tests, browser_tests".
It's a second try, previously, steps "nacl_integration, interactive_ui_tests,
browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[marja, 2012-08-17 07:04:26]

The ProfileImpl fixes got in, so this CL is simple again. The current patch set
is essentially changing the command line switch check the same way as the
previous version, even though the code is in a different place. I'll commit this
now.

[commit-bot: I haz the power, 2012-08-17 07:04:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/marja@chromium.org/10830216/10006