Closed
Bug 401531
Opened 18 years ago
Closed 18 years ago
Look for urln on the pastboard if urld isn't available
Categories
(Camino Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Camino1.6
People
(Reporter: stuart.morgan+bugzilla, Assigned: stuart.morgan+bugzilla)
Details
(Keywords: fixed1.8.1.10)
Attachments
(1 file)
|
2.59 KB,
patch
|
froodian
:
review+
mikepinkerton
:
superreview+
|
Details | Diff | Splinter Review |
We learned in debugging the URL drag-and-drop title breakage we had a while back from core's pasteboard rewrite that Camino doesn't look for urln (in theory, iirc, urld would be the link text if applicable, and urln would be the page title if applicable).
This patch adds urln to the list of things we'll check when handling a url drop, and also makes the title extraction code more robust against failures by trying backups based on whether reading a preferred type succeeded, rather than just whether we tried.
Attachment #286530 -
Flags: review?(froodian)
|
||
Comment 1•18 years ago
|
||
Comment on attachment 286530 [details] [diff] [review]
fix
r=me
Attachment #286530 -
Flags: superreview?(mikepinkerton)
Attachment #286530 -
Flags: review?(froodian)
Attachment #286530 -
Flags: review+
|
||
Comment 2•18 years ago
|
||
Comment on attachment 286530 [details] [diff] [review]
fix
sr=pink
Attachment #286530 -
Flags: superreview?(mikepinkerton) → superreview+
|
Assignee | |
Comment 3•18 years ago
|
||
Landed on trunk and MOZILLA_1_8_BRANCH.
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1.10
Resolution: --- → FIXED
Target Milestone: --- → Camino1.6
You need to log in
before you can comment on or make changes to this bug.
Description
•