Change tested version of facebookphpsdk

Summary: Broken by Facebook API change in Tuesday push. PR sent to v3 SDK

Reviewed By: @JoelMarcey

Differential Revision: D1409061
Flakeylist another assetic test

Summary: Found with --isolate

Reviewed By: @JoelMarcey

Differential Revision: D1408915
Don't add branch-over edge if pre-conditions are not satisfied

Summary: Currently, we add branch-over edges without checking if the
pre-conditions are satisfied for the target block. Later, if the
ir-builder finds that the block-state is incompatible at the merge-point
because of the branch-over edge, it terminates the main-trace block
in an exit block. Ideally, we should choose the main-trace branch over
the branch-over edge.

This diff changes it so that we check that pre-conditions are
satisfied at the time the branch-over is created, and skips the
branch-over edge if they are not.

Reviewed By: @ottoni

Differential Revision: D1391291
Allow merging FrameStates with different FPs

Summary: Trace through FPs for instructions that do not modify
the actual FramePtr.

Reviewed By: @ottoni

Differential Revision: D1391581
implement get_required_files

Summary: It is just a clone of `get_included_files`.

Reviewed By: @JoelMarcey

Differential Revision: D1408381
Introduce foreach (... await as ...)

Summary: Introduce foreach (... await as ...) which performs asynchronous
iteration of object that implement AsyncIterator interface, such
as AsyncGenerator.

Reviewed By: @billf

Differential Revision: D1406457
Update third-party hash after osx updates

Summary:

Test Plan:

Reviewers:

CC:

Task ID: #

Blame Rev:
Update third-party hash after time fix

Summary:

Test Plan:

Reviewers:

CC:

Task ID: #

Blame Rev:
Changes needed for FastCGI on OSX

Summary: The rest of the changes have been PRed in hhvm-third-party, @ptarjan would look at those. They must land first.
Closes #2944

Reviewed By: @ptarjan

Differential Revision: D1401477

Pulled By: @JoelMarcey
remove travis from readme

Summary: Travis hasn't been very high signal for us. It can't compile things in time, and we can't seem to keep all the tests passing on it. Our internal CI works great, and the move to open phabricator will expose this.

I'll leave it on for PRs, but I think showing the badge so priminantly is mis-selling out much we actually test this thing.

Reviewed By: @JoelMarcey

Differential Revision: D1409835
make shell_exec() return NULL

Summary: Matching PHP5

Reviewed By: @JoelMarcey

Differential Revision: D1408385
Fix assert in ini_get

Summary: Check to make sure the SSATemp is a constant string literal before
attempting to optimize the call to ini_get

Reviewed By: @ottoni

Differential Revision: D1407892
Fix hphp/test/frameworks under the typechecker again

Reviewed By: @JoelMarcey

Differential Revision: D1403949
Add sequential + test find mode options to frameworks.yaml

Reviewed By: @JoelMarcey

Differential Revision: D1404194
Add isolation mode for OSS tests

Summary: An alternative approach to finding flakey tests. Finds a few issues, eg that assetic does shell_exec('php '..., so those tests aren't actually tested against the right build of HHVM.

Not on by default.

Reviewed By: @JoelMarcey

Differential Revision: D1403101
SplFileObject->fgetc() properly increases linenumber

Summary: That zendtest describes the problem pretty well.
Closes #3052

Reviewed By: @ptarjan

Differential Revision: D1408180

Pulled By: @JoelMarcey
The notice in the mysqli::error_list test doesn't occur with the test runner

Summary: Remove it

Reviewed By: @ptarjan

Differential Revision: D1406141
class_parents/class_implements raise a warning if class doesnt exist

Summary: Closes #3057

Reviewed By: @ptarjan

Differential Revision: D1409003

Pulled By: @JoelMarcey
Fix isset-related behaviour for ArrayObject subclasses

Summary: Unlike most objects, using isset on an ArrayObject will return false
if the element is null. However, `offsetExists` will return true in
this case, so we need to special case ArrayObject.

Commit 31104c56 introduced this special case handling, however it
would cause unexpected infinite recursion if `offsetGet` was
overridden in a child class and that child class used an expression
of the form `isset($this[$offset])`. Users were not expecting
`offsetGet` to be called as the result of an `isset` expression.

This commit fixes the behaviour by calling the `offsetGet` method
from the `ArrayObject` class, instead of the `offsetGet` method on
the supplied object.

Fixes #2871
Fixes #2495
Closes #2986

Reviewed By: ptarjan, jdelong

Differential Revision: D1397482

Pulled By: @JoelMarcey
refactor ext_hotprofiler.cpp into new file ext_hotprofiler.h

Summary: Following existing coding conventions, split out a .h file from ext_hotprofiler.cpp.  In this way, 3rd party profilers can be written and loaded as DSOs.
Closes #3027

Reviewed By: @ptarjan

Differential Revision: D1401272

Pulled By: @JoelMarcey