Reorganized new sources from recently applied patch into third_party directory.

Reorganized new sources from recently applied patch into third_party directory.
Committed: https://code.google.com/p/angleproject/source/detail?r=1719

[apatrick1, 2013-01-12 00:35:00]

https://codereview.appspot.com/7105049/diff/2001/src/compiler/Compiler.cpp
File src/compiler/Compiler.cpp (right):

https://codereview.appspot.com/7105049/diff/2001/src/compiler/Compiler.cpp#ne...
src/compiler/Compiler.cpp:22: #include
"third_party/compiler/ArrayBoundsClamper.h"
#include "../third_party/compiler/ArrayBoundsClamper.h"

https://codereview.appspot.com/7105049/diff/2001/src/compiler/ShHandle.h
File src/compiler/ShHandle.h (right):

https://codereview.appspot.com/7105049/diff/2001/src/compiler/ShHandle.h#newc...
src/compiler/ShHandle.h:25: #include "third_party/compiler/ArrayBoundsClamper.h"
#include "../third_party/compiler/ArrayBoundsClamper.h"

https://codereview.appspot.com/7105049/diff/2001/src/compiler/translator_comm...
File src/compiler/translator_common.vcxproj (right):

https://codereview.appspot.com/7105049/diff/2001/src/compiler/translator_comm...
src/compiler/translator_common.vcxproj:176: <ClCompile
Include="..\third_party\compiler\ArrayBoundsClamper.cpp" />
    <ClCompile Include="..\..\third_party\compiler\ArrayBoundsClamper.cpp" />

https://codereview.appspot.com/7105049/diff/2001/src/compiler/translator_comm...
src/compiler/translator_common.vcxproj:269: <ClInclude
Include="..\third_party\compiler\ArrayBoundsClamper.h" />
    <ClInclude Include="..\..\third_party\compiler\ArrayBoundsClamper.h" />

https://codereview.appspot.com/7105049/diff/2001/src/third_party/compiler/Arr...
File src/third_party/compiler/ArrayBoundsClamper.cpp (right):

https://codereview.appspot.com/7105049/diff/2001/src/third_party/compiler/Arr...
src/third_party/compiler/ArrayBoundsClamper.cpp:26: #include
"third_party/compiler/ArrayBoundsClamper.h"
#include "../third_party/compiler/ArrayBoundsClamper.h"

[vangelis, 2013-01-12 01:42:20]

Thanks, Ken!

In addition you should follow the guidelines here:
http://www.chromium.org/developers/adding-3rd-party-libraries

and add a README and LICENSE file in the same third_party directory to let folks
know where the code came from.

[kbr1, 2013-01-14 21:06:51]

README and LICENSE added. They pass the presubmit checks. Please review.

[vangelis, 2013-01-14 21:52:22]

On 2013/01/14 21:06:51, kbr1 wrote:
> README and LICENSE added. They pass the presubmit checks. Please review.

The way this is typically done is that the README and LICENSE files are in the
same directory (or a parent) of where the code is. That way it's clear what
files the license covers and they don't need to be mentioned in LICENSE. 

You can put all files under:
src/third_party/compiler 

or (to be more clear):
src/third_party/array_bounds

[vangelis, 2013-01-14 22:39:41]

LGTM.  Just one nit (sorry I forgot to mention it before).  Would you mind
renaming:

README.chromium

to something like

README.angle

?

[kbr1, 2013-01-14 22:45:35]

On 2013/01/14 22:39:41, vangelis wrote:
> LGTM.  Just one nit (sorry I forgot to mention it before).  Would you mind
> renaming:
> 
> README.chromium
> 
> to something like
> 
> README.angle
> 
> ?

Done.

[vangelis, 2013-01-14 23:43:42]

On 2013/01/14 22:45:35, kbr1 wrote:
> On 2013/01/14 22:39:41, vangelis wrote:
> > LGTM.  Just one nit (sorry I forgot to mention it before).  Would you mind
> > renaming:
> > 
> > README.chromium
> > 
> > to something like
> > 
> > README.angle
> > 
> > ?
> 
> Done.

LGTM again

Thanks!