Skip tests that has syntax errors in them

Skip tests that has syntax errors in them

This is in preparation for a fix to V8

BUG=383596
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176055

[arv (Not doing code reviews), 2014-06-11 23:37:29]

PTAL

[esprehn, 2014-06-11 23:39:05]

lgtm, how long until we can land the fix?

[arv (Not doing code reviews), 2014-06-11 23:47:36]

On 2014/06/11 23:39:05, esprehn wrote:
> lgtm, how long until we can land the fix?

Depends on V8 reviews and V8 roll. Might be a week. I'll wait to commit this
until the V8 fix is ready to land.

[ojan, 2014-06-12 03:52:38]

Why skip instead of failure? Won't you need to rebaseline these once the fix is
in?

[arv (Not doing code reviews), 2014-06-12 04:01:44]

On 2014/06/12 at 03:52:38, ojan wrote:
> Why skip instead of failure? Won't you need to rebaseline these once the fix
is in?

Yes. I guess failure is better because it would catch crashes. I'll update it.

[arv (Not doing code reviews), 2014-06-12 17:25:40]

Use Failure instead of Skip

[arv (Not doing code reviews), 2014-06-12 18:56:53]

The CQ bit was checked by arv@chromium.org

[commit-bot: I haz the power, 2014-06-12 18:57:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/arv@chromium.org/329403002/20001

[commit-bot: I haz the power, 2014-06-12 18:57:49]

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

[commit-bot: I haz the power, 2014-06-12 18:57:50]

Failed to apply patch for LayoutTests/TestExpectations:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file LayoutTests/TestExpectations
  Hunk #1 FAILED at 1075.
  1 out of 1 hunk FAILED -- saving rejects to file
LayoutTests/TestExpectations.rej

Patch:       LayoutTests/TestExpectations
Index: LayoutTests/TestExpectations
diff --git a/LayoutTests/TestExpectations b/LayoutTests/TestExpectations
index
6e05bae358c6983d899295da4e3455c9fe794682..12c9b7c5de8b6df702e8b0453ef497c2c94b9fdb
100644
--- a/LayoutTests/TestExpectations
+++ b/LayoutTests/TestExpectations
@@ -1075,3 +1075,9 @@ crbug.com/308768
http/tests/xmlhttprequest/xmlhttprequest-data-url.html [ Skip ]
 crbug.com/308768 inspector/network/network-status-non-http.html [ Skip ]
 
 crbug.com/381102
inspector/sources/debugger/debugger-step-into-custom-element-callbacks.html [
Skip ]
+
+crbug.com/383596 fast/js/implicit-call-with-global-reentry.html [ Failure ]
+crbug.com/383596 fast/js/reserved-words-as-property.html [ Failure ]
+crbug.com/383596 inspector-protocol/runtime/runtime-getProperties.html [
Failure ]
+crbug.com/383596 inspector/console/console-eval-scoped.html [ Failure ]
+crbug.com/383596 inspector/console/console-object-preview-accessors.html [
Failure ]

[arv (Not doing code reviews), 2014-06-12 19:45:50]

merge

[arv (Not doing code reviews), 2014-06-12 19:46:17]

The CQ bit was checked by arv@chromium.org

[commit-bot: I haz the power, 2014-06-12 19:47:12]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/arv@chromium.org/329403002/40001

[commit-bot: I haz the power, 2014-06-12 21:43:22]

Change committed as 176055