Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The everything gyp target now depends on the skiaserve gyp target.
This means that when using skia_build_server=1, building 'most' will
build skiaserve, but when skia_build_server is not defined the 
skiaserve target will still be available if specified manually.

The old json.gyp is removed as it currently does not build anything.
All of the files currently referenced by it as sources no longer exist.

Committed: https://skia.googlesource.com/skia/+/51190df040c5a81bb82b9cd5b95f07a39d0e602b

[bungeman-skia, 2016-03-08 23:29:51]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bungeman-skia, 2016-03-09 00:32:50]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
==========

[bungeman-skia, 2016-03-09 03:03:30]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bungeman-skia, 2016-03-09 03:05:57]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
==========

[bungeman-skia, 2016-03-09 03:43:37]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bungeman-skia, 2016-03-09 03:43:59]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
==========

[bungeman-skia, 2016-03-09 03:55:16]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The  microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The old json.gyp is removed as it currently does no build anything.
All of the files currently referenced by it as sources no longer exist.
==========

[bungeman-skia, 2016-03-09 04:02:26]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The  microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The old json.gyp is removed as it currently does no build anything.
All of the files currently referenced by it as sources no longer exist.
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The  microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The everything gyp target now depends on the skiaserve gyp
target. This means that when using skia_build_server=1, building
'most' will build skiaserve, but when skia_build_server is not defined
the skiaserve target will still be available if specified manually.

The old json.gyp is removed as it currently does no build anything.
All of the files currently referenced by it as sources no longer exist.
==========

[bungeman-skia, 2016-03-09 04:02:34]

The CQ bit was checked by bungeman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-09 04:02:43]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1775203002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1775203002/40001

[bungeman-skia, 2016-03-09 04:03:34]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The  microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The everything gyp target now depends on the skiaserve gyp
target. This means that when using skia_build_server=1, building
'most' will build skiaserve, but when skia_build_server is not defined
the skiaserve target will still be available if specified manually.

The old json.gyp is removed as it currently does no build anything.
All of the files currently referenced by it as sources no longer exist.
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The  microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The everything gyp target now depends on the skiaserve gyp target.
This means that when using skia_build_server=1, building 'most' will
build skiaserve, but when skia_build_server is not defined the 
skiaserve target will still be available if specified manually.

The old json.gyp is removed as it currently does no build anything.
All of the files currently referenced by it as sources no longer exist.
==========

[commit-bot: I haz the power, 2016-03-09 04:17:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-09 04:17:14]

Dry run: This issue passed the CQ dry run.

[bungeman-skia, 2016-03-09 04:18:25]

bungeman@google.com changed reviewers:
+ ethannicholas@google.com, jcgregorio@google.com, joshualitt@google.com

[bungeman-skia, 2016-03-09 04:18:25]

I used skiaserve today and started fixing things...

[bungeman-skia, 2016-03-09 04:19:10]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The  microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The everything gyp target now depends on the skiaserve gyp target.
This means that when using skia_build_server=1, building 'most' will
build skiaserve, but when skia_build_server is not defined the 
skiaserve target will still be available if specified manually.

The old json.gyp is removed as it currently does no build anything.
All of the files currently referenced by it as sources no longer exist.
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The everything gyp target now depends on the skiaserve gyp target.
This means that when using skia_build_server=1, building 'most' will
build skiaserve, but when skia_build_server is not defined the 
skiaserve target will still be available if specified manually.

The old json.gyp is removed as it currently does no build anything.
All of the files currently referenced by it as sources no longer exist.
==========

[bungeman-skia, 2016-03-09 04:20:00]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The everything gyp target now depends on the skiaserve gyp target.
This means that when using skia_build_server=1, building 'most' will
build skiaserve, but when skia_build_server is not defined the 
skiaserve target will still be available if specified manually.

The old json.gyp is removed as it currently does no build anything.
All of the files currently referenced by it as sources no longer exist.
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The everything gyp target now depends on the skiaserve gyp target.
This means that when using skia_build_server=1, building 'most' will
build skiaserve, but when skia_build_server is not defined the 
skiaserve target will still be available if specified manually.

The old json.gyp is removed as it currently does not build anything.
All of the files currently referenced by it as sources no longer exist.
==========

[joshualitt, 2016-03-09 14:09:46]

On 2016/03/09 04:18:25, bungeman1 wrote:
> I used skiaserve today and started fixing things...

LGTM, thanks for sweeping up a bit

[bungeman-skia, 2016-03-09 15:41:49]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2016-03-09 15:41:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1775203002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1775203002/40001

[commit-bot: I haz the power, 2016-03-09 15:42:56]

Description was changed from

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The everything gyp target now depends on the skiaserve gyp target.
This means that when using skia_build_server=1, building 'most' will
build skiaserve, but when skia_build_server is not defined the 
skiaserve target will still be available if specified manually.

The old json.gyp is removed as it currently does not build anything.
All of the files currently referenced by it as sources no longer exist.
==========

to

==========
Add SkDrawPosTextHCommand JSON, fix skiaserve build.

Adds SkDrawPosTextHCommand ::fromJSON and ::toJSON.

Both SkDrawPosTextCommand and SkDrawPosTextHCommand's
::toJSON write the correct number of positions, preventing reading
uninitialized memory.

The microhttpd build is now done in the build tree as opposed to in a
temporary directory.

The microhttpd build script uses os.path.join so that absolute paths
do not confuse the build. This allows compatibility with the cmake
gyp generator as CMake likes to pass absolute paths.

The microhttpd gyp target is now marked as 'none' since it is not
a 'static_library' target (which directs gyp to compile sources into
a static library). The dependencies to the action are updated to the
minimum required for sane re-building.

The everything gyp target now depends on the skiaserve gyp target.
This means that when using skia_build_server=1, building 'most' will
build skiaserve, but when skia_build_server is not defined the 
skiaserve target will still be available if specified manually.

The old json.gyp is removed as it currently does not build anything.
All of the files currently referenced by it as sources no longer exist.

Committed:
https://skia.googlesource.com/skia/+/51190df040c5a81bb82b9cd5b95f07a39d0e602b
==========

[commit-bot: I haz the power, 2016-03-09 15:42:57]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/51190df040c5a81bb82b9cd5b95f07a39d0e602b