Revert of Implement ScopedFD in terms of ScopedGeneric.

Revert of Implement ScopedFD in terms of ScopedGeneric. (https://codereview.chromium.org/191673003/)

Reason for revert:
Doesn't correctly link

/mnt/data/b/build/slave/Chromium_Linux_Codesearch/build/src/third_party/gold/gold64: warning: hidden symbol 'base::internal::ScopedFDCloseTraits::Free(int)' in obj/base/files/nacl_helper.scoped_file.o is referenced by DSO lib/libipc.so

Original issue's description:
> Implement ScopedFD in terms of ScopedGeneric.
> 
> Move to a new file base/files/scoped_file.h. I will also add ScopedFILE to here (currently in file_util.h) later.
> 
> I think there is a crash in the old code in content/browser/zygote_host/zygote_host_impl_linux.cc that this patch should fix. The old ScopedFD took the address of something in a vector that is being modified.
> 
> I removed SafeScopedFD from content/common/sandbox_linux/sandbox_linux.cc since base's ScopedFD not CHECKs on close failure (this is a more recent addition).
> 
> BUG=
> R=agl@chromium.org, viettrungluu@chromium.org
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=257001
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=257179

TBR=viettrungluu@chromium.org,agl@chromium.org,brettw@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=

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

[jochen (gone - plz use gerrit), 2014-03-15 13:33:59]

Created Revert of Implement ScopedFD in terms of ScopedGeneric.

[commit-bot: I haz the power, 2014-03-15 13:34:07]

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

[commit-bot: I haz the power, 2014-03-15 13:35:10]

Change committed as 257323

[brettw, 2014-03-16 02:48:30]

Wasn't that a nonfatal warning? It was landed fine most of the day.

[jochen (gone - plz use gerrit), 2014-03-16 12:26:38]

On 2014/03/16 02:48:30, brettw wrote:
> Wasn't that a nonfatal warning? It was landed fine most of the day.

Yeah, that's a bit of a problem. We compile with -Werror, however, gold ignores
this option. So the buildbot monitoring systems start screaming while the
compile step doesn't fail :-/

I file a bug to fix this inconsistency

[brettw, 2014-03-17 16:54:35]

I link to the failing bot would have been helpful. I can't reproduce the
warning.

[jochen (gone - plz use gerrit), 2014-03-17 16:59:44]

On 2014/03/17 16:54:35, brettw wrote:
> I link to the failing bot would have been helpful. I can't reproduce the
> warning.

I meanwhile ripped out the offending gyp stuff, so we should just revert the
revert.

[brettw, 2014-03-17 17:02:43]

On 2014/03/17 16:59:44, jochen wrote:
> On 2014/03/17 16:54:35, brettw wrote:
> > I link to the failing bot would have been helpful. I can't reproduce the
> > warning.
> 
> I meanwhile ripped out the offending gyp stuff, so we should just revert the
> revert.

Oh, no wonder it's working for me. Thanks.