Sat, 03 Jan 2015 20:18:00 +0100
Conditionally enable double key logic according to:
private browsing mode or privacy.thirdparty.isolate preference and
implement in GetCookieStringCommon and FindCookie where it counts...
With some reservations of how to convince FindCookie users to test
condition and pass a nullptr when disabling double key logic.
michael@0 | 1 | /* -*- Mode: C++; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */ |
michael@0 | 2 | /* This Source Code Form is subject to the terms of the Mozilla Public |
michael@0 | 3 | * License, v. 2.0. If a copy of the MPL was not distributed with this |
michael@0 | 4 | * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ |
michael@0 | 5 | |
michael@0 | 6 | /* |
michael@0 | 7 | * |
michael@0 | 8 | * Date: 18 Jun 2002 |
michael@0 | 9 | * SUMMARY: Shouldn't crash when catch parameter is "hidden" by varX |
michael@0 | 10 | * See http://bugzilla.mozilla.org/show_bug.cgi?id=146596 |
michael@0 | 11 | * |
michael@0 | 12 | */ |
michael@0 | 13 | //----------------------------------------------------------------------------- |
michael@0 | 14 | var UBound = 0; |
michael@0 | 15 | var BUGNUMBER = 146596; |
michael@0 | 16 | var summary = "Shouldn't crash when catch parameter is 'hidden' by varX"; |
michael@0 | 17 | var status = ''; |
michael@0 | 18 | var statusitems = []; |
michael@0 | 19 | var actual = ''; |
michael@0 | 20 | var actualvalues = []; |
michael@0 | 21 | var expect= ''; |
michael@0 | 22 | var expectedvalues = []; |
michael@0 | 23 | |
michael@0 | 24 | |
michael@0 | 25 | /* |
michael@0 | 26 | * Just seeing we don't crash when executing this function - |
michael@0 | 27 | * This example provided by jim-patterson@ncf.ca |
michael@0 | 28 | * |
michael@0 | 29 | * Brendan: "Jim, thanks for the testcase. But note that |var| |
michael@0 | 30 | * in a JS function makes a function-scoped variable -- JS lacks |
michael@0 | 31 | * block scope apart from for catch variables within catch blocks. |
michael@0 | 32 | * |
michael@0 | 33 | * Therefore the catch variable hides the function-local variable." |
michael@0 | 34 | */ |
michael@0 | 35 | function F() |
michael@0 | 36 | { |
michael@0 | 37 | try |
michael@0 | 38 | { |
michael@0 | 39 | return "A simple exception"; |
michael@0 | 40 | } |
michael@0 | 41 | catch(e) |
michael@0 | 42 | { |
michael@0 | 43 | var e = "Another exception"; |
michael@0 | 44 | } |
michael@0 | 45 | |
michael@0 | 46 | return 'XYZ'; |
michael@0 | 47 | } |
michael@0 | 48 | |
michael@0 | 49 | status = inSection(1); |
michael@0 | 50 | actual = F(); |
michael@0 | 51 | expect = "A simple exception"; |
michael@0 | 52 | addThis(); |
michael@0 | 53 | |
michael@0 | 54 | |
michael@0 | 55 | |
michael@0 | 56 | /* |
michael@0 | 57 | * Sanity check by Brendan: "This should output |
michael@0 | 58 | * |
michael@0 | 59 | * 24 |
michael@0 | 60 | * 42 |
michael@0 | 61 | * undefined |
michael@0 | 62 | * |
michael@0 | 63 | * and throw no uncaught exception." |
michael@0 | 64 | * |
michael@0 | 65 | */ |
michael@0 | 66 | function f(obj) |
michael@0 | 67 | { |
michael@0 | 68 | var res = []; |
michael@0 | 69 | |
michael@0 | 70 | try |
michael@0 | 71 | { |
michael@0 | 72 | throw 42; |
michael@0 | 73 | } |
michael@0 | 74 | catch(e) |
michael@0 | 75 | { |
michael@0 | 76 | with(obj) |
michael@0 | 77 | { |
michael@0 | 78 | var e; |
michael@0 | 79 | res[0] = e; // |with| binds tighter than |catch|; s/b |obj.e| |
michael@0 | 80 | } |
michael@0 | 81 | |
michael@0 | 82 | res[1] = e; // |catch| binds tighter than function scope; s/b 42 |
michael@0 | 83 | } |
michael@0 | 84 | |
michael@0 | 85 | res[2] = e; // |var e| has function scope; s/b visible but contain |undefined| |
michael@0 | 86 | return res; |
michael@0 | 87 | } |
michael@0 | 88 | |
michael@0 | 89 | status = inSection(2); |
michael@0 | 90 | actual = f({e:24}); |
michael@0 | 91 | expect = [24, 42, undefined]; |
michael@0 | 92 | addThis(); |
michael@0 | 93 | |
michael@0 | 94 | |
michael@0 | 95 | |
michael@0 | 96 | |
michael@0 | 97 | //----------------------------------------------------------------------------- |
michael@0 | 98 | test(); |
michael@0 | 99 | //----------------------------------------------------------------------------- |
michael@0 | 100 | |
michael@0 | 101 | |
michael@0 | 102 | |
michael@0 | 103 | function addThis() |
michael@0 | 104 | { |
michael@0 | 105 | statusitems[UBound] = status; |
michael@0 | 106 | actualvalues[UBound] = actual.toString(); |
michael@0 | 107 | expectedvalues[UBound] = expect.toString(); |
michael@0 | 108 | UBound++; |
michael@0 | 109 | } |
michael@0 | 110 | |
michael@0 | 111 | |
michael@0 | 112 | function test() |
michael@0 | 113 | { |
michael@0 | 114 | enterFunc('test'); |
michael@0 | 115 | printBugNumber(BUGNUMBER); |
michael@0 | 116 | printStatus(summary); |
michael@0 | 117 | |
michael@0 | 118 | for (var i=0; i<UBound; i++) |
michael@0 | 119 | { |
michael@0 | 120 | reportCompare(expectedvalues[i], actualvalues[i], statusitems[i]); |
michael@0 | 121 | } |
michael@0 | 122 | |
michael@0 | 123 | exitFunc ('test'); |
michael@0 | 124 | } |