android.text.ClipboardManager in android.text has been deprecated

RESOLVED FIXED in mozilla9

Status

()

Core
Widget: Android
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dougt, Assigned: alexp)

Tracking

unspecified
mozilla9
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
I see this when using jlint on android.  I am not sure if we need to make a change.  we should figure this out.


/builds/mozilla-central/embedding/android/GeckoAppShell.java:865: warning: [deprecation] android.text.ClipboardManager in android.text has been deprecated
                android.text.ClipboardManager cm = (android.text.ClipboardManager)
                            ^
/builds/mozilla-central/embedding/android/GeckoAppShell.java:865: warning: [deprecation] android.text.ClipboardManager in android.text has been deprecated
                android.text.ClipboardManager cm = (android.text.ClipboardManager)
                                                                ^
/builds/mozilla-central/embedding/android/GeckoAppShell.java:882: warning: [deprecation] android.text.ClipboardManager in android.text has been deprecated
                android.text.ClipboardManager cm = (android.text.ClipboardManager)
                            ^
/builds/mozilla-central/embedding/android/GeckoAppShell.java:882: warning: [deprecation] android.text.ClipboardManager in android.text has been deprecated
                android.text.ClipboardManager cm = (android.text.ClipboardManager)


Alex, is there something else we should be doing here?
(Assignee)

Comment 1

6 years ago
The android.text.ClipboardManager class is marked as deprecated, as in the API level 11 it is extended by android.content.ClipboardManager class, which has more functionality, supporting not only plain text, but different types of data.
It might make sense for us to use the new class on Honeycomb to make sure we work with clipboard properly, and are able to handle non-text data.
Status: NEW → ASSIGNED
(Assignee)

Comment 2

6 years ago
Created attachment 561637 [details] [diff] [review]
Patch

Use the new ClipboardManager class on the latest API levels.
Please note: a proper support for different types of clipboard data is a separate bug 649371.
Attachment #561637 - Flags: review?(blassey.bugs)
Attachment #561637 - Flags: review?(blassey.bugs) → review+
(Assignee)

Comment 3

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/a2f97b48ed3e
Whiteboard: [inbound]

Comment 4

6 years ago
https://hg.mozilla.org/mozilla-central/rev/a2f97b48ed3e
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla9
You need to log in before you can comment on or make changes to this bug.