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]
:
: Patrick McManus [:mcmanus]
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
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

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.