Rewriting loop in accessibility code to silence analyzer.

The VS static analyzer thought the code was a mistake by the author
(starting value less than that limit in a decreasing loop)
and judging by the comment, humans also find the loop non-trvial.

So make it more normal.

BUG=

Review URL: https://codereview.chromium.org/190033006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256086 0039d316-1c4b-4281-b951-d872f2087c98
scoped_ptr<DEVMODE[]> replaced with scoped_ptr<DEVMODE, base::FreeDeleter>.

NOTRY=true

Review URL: https://codereview.chromium.org/191913002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256087 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 256087
Move webkit_child_helpers to content/child/

BUG=338338
TBR=darin@chromium.org

Review URL: https://codereview.chromium.org/185553017

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256088 0039d316-1c4b-4281-b951-d872f2087c98
Don't serialize frame id or original url history parameters. They are obsolete.

BUG=341425,340277
TEST=PageStateSerializationTest.BackwardsCompat_v18

Review URL: https://codereview.chromium.org/192813004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256089 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 256089