Closed Bug 687204 Opened 8 years ago Closed 8 years ago

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

Categories

(Core :: Widget: Android, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla9

People

(Reporter: dougt, Assigned: alexp)

Details

Attachments

(1 file)

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?
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
Attached patch PatchSplinter Review
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+
https://hg.mozilla.org/mozilla-central/rev/a2f97b48ed3e
Status: ASSIGNED → RESOLVED
Closed: 8 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.