|
|
Line 90: |
Line 90: |
| = Mochichrome = | | = Mochichrome = |
|
| |
|
| 13 failing tests out of 14051, in these files:
| | <pre> |
| | | 7353 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/layout/base/test/chrome/test_printpreview.xul | Timers should have run! - didn't expect "0 timers", but got it |
| chrome://mochikit/content/chrome/js/src/xpconnect/tests/chrome/test_evalInSandbox.xul | | 7354 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/layout/base/test/chrome/test_printpreview.xul | Timers should have run! - didn't expect 0, but got it |
| -> add a function that the sandbox can call which forwards to getClassName
| | </pre> |
| | |
| No idea what that comment about getClassName means. What is happening here is that the test is using exactly the same technique GreaseMonkey uses to set up the sandbox, and that doesn't work anymore.
| |
| | |
| The specific issue is with code that does <code>sandbox.__proto__ = new XPCNativeWrapper(window);</code> where the caller is chrome, but ''sandbox'' and ''window'' are same-domain. When we rewrap the XPCNativeWrapper, it becomes a simple cross-compartment wrapper without X-ray behavior.
| |
| | |
| The whole rewrapping mechanism works as designed right up to WrapperFactory::Rewrap where it's actually time to create the new wrapper. We have a ''flags'' parameter that indicates we want X-ray behavior. But we create a wrapper without it.
| |
| | |
| We could just make that work, but mrbkap thinks we should instead provide a better sandboxing API and require GreaseMonkey (and any similar extensions) to switch to that.
| |
| | |
| -jorendorff, 10/5/2010
| |
|
| |
|
| chrome://mochikit/content/chrome/js/src/xpconnect/tests/chrome/test_wrappers-2.xul | | chrome://mochikit/content/chrome/js/src/xpconnect/tests/chrome/test_wrappers-2.xul |
Line 159: |
Line 149: |
|
| |
|
| -jorendorff, 10/5/2010 | | -jorendorff, 10/5/2010 |
|
| |
| I am also getting these failures:
| |
|
| |
| 7353 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/layout/base/test/chrome/test_printpreview.xul | Timers should have run! - didn't expect "0 timers", but got it
| |
| 7354 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/layout/base/test/chrome/test_printpreview.xul | Timers should have run! - didn't expect 0, but got it
| |
|
| |
|
| = Browserchrome = | | = Browserchrome = |