Closed
Bug 562032
Opened 14 years ago
Closed 14 years ago
Upstream safety fix to NSPasteboard+Utils
Categories
(Camino Graveyard :: OS Integration, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: stuart.morgan+bugzilla, Assigned: alqahira)
Details
(Whiteboard: [camino-2.0.3])
Attachments
(1 file)
2.30 KB,
patch
|
stuart.morgan+bugzilla
:
superreview+
|
Details | Diff | Splinter Review |
There was a recent fix in Chromium to prevent a possible inserting-nil-into-array exception in our pasteboard code: http://codereview.chromium.org/1750016/show We should upstream it.
Assignee | ||
Comment 1•14 years ago
|
||
Stuart, thanks for the heads-up; care to review this again? ;) Also fyi, http://src.chromium.org/viewvc/chrome/trunk/src/third_party/mozilla/README.chromium?revision=44666&view=markup&pathrev=45697 is out-of-date; either it is missing mentioning the local modification that is this patch, or it is missing that we've upstreamed the local modification it currently references (bug 538433) ;)
Assignee: nobody → alqahira
Status: NEW → ASSIGNED
Attachment #441881 -
Flags: superreview?(stuart.morgan+bugzilla)
Reporter | ||
Comment 2•14 years ago
|
||
Comment on attachment 441881 [details] [diff] [review] Upstreams the fix from Chromium sr=smorgan
Attachment #441881 -
Flags: superreview?(stuart.morgan+bugzilla) → superreview+
Assignee | ||
Comment 3•14 years ago
|
||
http://hg.mozilla.org/camino/rev/faad3bf52e93 and cvs trunk+CAMINO_2_0_BRANCH.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [camino-2.0.3]
You need to log in
before you can comment on or make changes to this bug.
Description
•