MIPS: Skip mozilla/js1_5/extensions/regress-355497 in simulator.

MIPS: Skip mozilla/js1_5/extensions/regress-355497 in simulator.

BUG=

Committed: https://crrev.com/7cb12037d30b255090b0ff0f01d7221cacc86c1a
Cr-Commit-Position: refs/heads/master@{#26392}

[balazs.kilvady, 2015-02-02 18:41:48]

balazs.kilvady@imgtec.com changed reviewers:
+ akos.palfi@imgtec.com, bmeurer@chromium.org, danno@chromium.org,
dusan.milosavljevic@imgtec.com, gergely.kis@imgtec.com, paul.lind@imgtec.com

[paul.l..., 2015-02-02 19:07:43]

https://codereview.chromium.org/894123002/diff/1/test/mozilla/mozilla.status
File test/mozilla/mozilla.status (right):

https://codereview.chromium.org/894123002/diff/1/test/mozilla/mozilla.status#...
test/mozilla/mozilla.status:881: 'js1_5/extensions/regress-355497': [SKIP],
I'd rather not change mips64 test, unless you are aware of a case where the
reduced stack size does not work for it. I have seen no failures on the mips64
bots.

[balazs.kilvady, 2015-02-02 19:35:05]

https://codereview.chromium.org/894123002/diff/1/test/mozilla/mozilla.status
File test/mozilla/mozilla.status (right):

https://codereview.chromium.org/894123002/diff/1/test/mozilla/mozilla.status#...
test/mozilla/mozilla.status:881: 'js1_5/extensions/regress-355497': [SKIP],
On 2015/02/02 19:07:43, paul.l... wrote:
> I'd rather not change mips64 test, unless you are aware of a case where the
> reduced stack size does not work for it. I have seen no failures on the mips64
> bots.

Done.

[paul.l..., 2015-02-02 19:47:21]

LGTM. We do need v8 team review for this.

Benedikt, we considered adding FLAG_sim_stack_size to mips & arm (32-bit)
simulators, but the default stack sizes are different, and likely
test-thresholds would be also. And this is kind of a cheat: failures are due to
separated C & JS stacks on the sims. But we would be happy to implement it that
way, if you prefer.

[Benedikt Meurer, 2015-02-03 06:01:02]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-02-03 06:01:03]

I'm fine with that.

[Benedikt Meurer, 2015-02-03 06:01:07]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-02-03 06:01:07]

lgtm

[commit-bot: I haz the power, 2015-02-03 06:01:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/894123002/20001

[commit-bot: I haz the power, 2015-02-03 06:31:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-03 06:32:12]

Patchset 2 (id:??) landed as
https://crrev.com/7cb12037d30b255090b0ff0f01d7221cacc86c1a
Cr-Commit-Position: refs/heads/master@{#26392}