Disable strict mode for now until V8 bug 1423 is fixed.

Disable strict mode for now until V8 bug 1423 is fixed.

BUG=http://code.google.com/p/v8/issues/detail?id=1423
TEST=Click the OAuth and email buttons on a 64-bit Linux machine. Chrome should not crash.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=87920

[Jamie, 2011-06-03 17:53:08]

A minuscule code review for you.

[dmaclach1, 2011-06-03 18:26:34]

LGTM

On Fri, Jun 3, 2011 at 10:53,  <jamiewalch@chromium.org> wrote:
> Reviewers: dmac,
>
> Message:
> A minuscule code review for you.
>
> Description:
> Disable strict mode for now until V8 bug 1423 is fixed.
>
> BUG=http://code.google.com/p/v8/issues/detail?id=1423
> TEST=Click the OAuth and email buttons on a 64-bit Linux machine. Chrome
> should
> not crash.
>
>
> Please review this at http://codereview.chromium.org/7112012/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files:
>  M remoting/webapp/me2mom/remoting.js
>
>
> Index: remoting/webapp/me2mom/remoting.js
> diff --git a/remoting/webapp/me2mom/remoting.js
> b/remoting/webapp/me2mom/remoting.js
> index
>
5bc8cf6820741042818f8da9cdd683edfeb95cf1..fb254f0475e50a720f1f55748594b2d4dbe698c4
> 100644
> --- a/remoting/webapp/me2mom/remoting.js
> +++ b/remoting/webapp/me2mom/remoting.js
> @@ -2,7 +2,10 @@
>  // Use of this source code is governed by a BSD-style license that can be
>  // found in the LICENSE file.
>
> -"use strict";
> +// TODO(jamiewalch): strict mode causes the page to crash so it's disabled
> for
> +// now. Reinstate this when the associated bug is fixed.
> +// http://code.google.com/p/v8/issues/detail?id=1423
> +//"use strict";
>
>  // TODO(ajwong): This seems like a bad idea to share the exact same object
>  // with the background page.  Why are we doing it like this?
>
>
>

[commit-bot: I haz the power, 2011-06-04 08:04:24]

Change committed as 87920