Last Comment Bug 708305 - Fix 2 PR_* to false/true in spdysession
: Fix 2 PR_* to false/true in spdysession
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Networking: HTTP (show other bugs)
: 11 Branch
: x86_64 Linux
: -- normal (vote)
: mozilla11
Assigned To: Patrick McManus [:mcmanus]
:
:
Mentors:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2011-12-07 10:33 PST by Patrick McManus [:mcmanus]
Modified: 2012-02-01 13:57 PST (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Attachments
patch 0 (1.11 KB, patch)
2011-12-07 10:37 PST, Patrick McManus [:mcmanus] 		honzab.moz: review+
Details | Diff | Splinter Review
View All Add an attachment (proposed patch, testcase, etc.)

Description Patrick McManus [:mcmanus] 2011-12-07 10:33:18 PST 
2 uses of PR_TRUE and PR_FALSE snuck into spdysession.cpp. change them to true/false.
Comment 1 Patrick McManus [:mcmanus] 2011-12-07 10:37:15 PST 
Created attachment 579741 [details] [diff] [review]
patch 0
Comment 2 Patrick McManus [:mcmanus] 2011-12-07 11:27:17 PST 
https://hg.mozilla.org/integration/mozilla-inbound/rev/c170c678c9ac
Comment 3 Ed Morley [:emorley] 2011-12-08 08:54:09 PST 
https://hg.mozilla.org/mozilla-central/rev/c170c678c9ac
Comment 4 Ed Morley [:emorley] 2011-12-10 15:34:06 PST 
Backed out (along with the rest of the SPDY landing) in order to stop us hitting the MSVC virtual address limit, so we can reopen the trees (bug 709193).

Sucks, but we don't really have any other choice here :-(

https://hg.mozilla.org/integration/mozilla-inbound/rev/dc48c0992358
Comment 5 Ed Morley [:emorley] 2011-12-14 11:14:16 PST 
Relanded on mozilla-central :-)

https://hg.mozilla.org/mozilla-central/rev/cf0b31ff2b6d
Note You need to log in before you can comment on or make changes to this bug.