Updating third-party to latest to pull in update SQLite3.8.6
Rework hphp/test/zend/.gitattributes

Summary: - if any of these are touched, you need to edit both good/ and bad/ (-bad +good hopefully)
- might as well sort them

Reviewed By: @ptarjan

Differential Revision: D1497848
Revert "Update and lock framework test dependencies"

Reviewed By: @JoelMarcey
Flakeylist another phpbb3 test

Summary: PR sent to fix the test.

Reviewed By: @ptarjan

Differential Revision: D1504905
Dump the unit in assert in refineType

Reviewed By: @ottoni

Differential Revision: D1504928
Revert "Revert "Update and lock framework test dependencies""

Summary: Also:
 - Make the runner recognize risky tests
 - const-ify the static strings
 - re-record slim

The HHVM Warnning/fatal patterns need more work - they've been broken for months (since we made warnings/fatals match PHP5). These should be in a separate diff.

Reviewed By: @JoelMarcey

Differential Revision: D1505766
ini-ize the EvalFlags

Summary: Hopefully simple change to get the EvalFlags working with ini.

Reviewed By: @ptarjan

Differential Revision: D1505739
Have FrameState drop state when hitting an unprocessed predecessor.

Summary: Adds a bitvector to FrameState. When FrameState::startBlock is called
with a block that has a predecessor that wasn't previously visited by
this FrameState, then the FrameState resets its state. The entry block
gets marked in IRBuilder::IRBuilder, because we don't call startBlock
when constructing IR in selectRegion.

Also adds a BCMarker field to FrameState::startBlock to match
IRBuilder::startBlock. This helps with building IR, when we can't get
the BCMarker from the block's first instruction.

Reviewed By: @ottoni

Differential Revision: D1499056
Update NEWS for HHVM release "Queen Latifah"

Summary: news

Reviewed By: @ptarjan

Differential Revision: D1503623
Cover third version of freetype include

Summary: On Homebrew, we want

  #include <freetyp2/config/ftheader.h>

Because... Freetype.

Extend CMake/FindFreetype.cmake to check all three places
and set defines accordingly, then use those defines to
include the correct header.

Reviewed By: @ptarjan

Differential Revision: D1504378
Remove use of non-threadsafe strtok()

Summary: Interrestingly, someone decided to use strtok_r() because
the inner call to strtok() would collide.

That's true, of course, but they didn't seem to consider that
strtok() would also collide with any other strtok() in the
same process which might be running at the same time.

For instance, another request in another thread running the same
PHP script.

Let's just go ahead and make both loops be threadsafe, shall we?

Reviewed By: @ptarjan

Differential Revision: D1504531
remove duplicate json test dir

Summary: We had slow/ext_json and slow/json. Keep everything in ext_json
Closes #3471

Reviewed By: @JoelMarcey

Differential Revision: D1501008
Update warning/fatal detection in test runner

Summary: These changed a few months back to match php5

Reviewed By: @JoelMarcey

Differential Revision: D1505836
Updated guzzle framework tests

Summary: Closes https://github.com/facebook/hhvm/pull/3526

Reviewed By: @ptarjan

Differential Revision: D1506193
Updated idiorm tests to 1.5.1

Summary: Closes https://github.com/facebook/hhvm/pull/3527

Reviewed By: @fredemmott

Differential Revision: D1506205
Tweaked the json_schema framework config

Summary: Closes https://github.com/facebook/hhvm/pull/3528

Reviewed By: @ptarjan

Differential Revision: D1506211
Implemented xdebug server jit connecting

Summary: This diff adds an `XDebugHookHandler` that is intended to respond to debugger
hooks. This depends on D1474540. The handler is used used to start the xdebug
server on exceptions if "jit" mode is enabled.

This required exposing the server attaching code. To do this, I've merged the
request local data with the other xdebug globals as thread local members on
`XDebugExtension`. It didn't make sense to have two places for thread local
globals.

Reviewed By: @ptarjan

Differential Revision: D1474688
Fix normalize for `JitA` style config options

Summary: `normalize()` didn't handle the `JitA` case, such as `JitAHotSize`. It would output `hhvm.jit_ahot_size`
instead of `jit_a_hot_size`.

Since this is sort of specialized, and `normalize()` works pretty well for everything else, just
special case this like we did for `mysql`.

Reviewed By: @paulbiss

Differential Revision: D1506492
Merge pull request #3516 from MaideCa/thirdparty_sqlite_386

Updating third-party to latest to pull in update SQLite3.8.6
Implemented xdebug remote command infra, command parsing, and status and run commands

Summary: This diff primarily implements command parsing and the command infra. The
status and run commands were implemented for purposes of testing.

The parsing code (`XDebugServer::parseInput`) was translated from xdebug. This
way there are no inconsistencies between implementations dealing with command
parsing.

This also pulls in the xdebug error codes and stores them in an enum. Error
handling output should be identical to php5 xdebug.

Commands are abstracted into a class `XDebugCommand`. All valid xdebug commands
will be represented as subclasses of this class. They can be constructed from
the client input via `XDebugCommand::fromString`. Subclasses primarily override
`XDebugCommand::handleImpl` to perform the command and output the response,
but there are other functions that can be overridden to control options.

For testing purposes, I implemented the `status` command and the `run` command.
The status command just returns the status of the remote debug server. The
`run` command continues until the next breakpoint or program shutdown.

The server should also now respond properly if the client disconnects.

Reviewed By: @ptarjan

Differential Revision: D1479467