Last Comment Bug 687204 - android.text.ClipboardManager in android.text has been deprecated
: android.text.ClipboardManager in android.text has been deprecated
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Widget: Android (show other bugs)
: unspecified
: All Android
: -- normal (vote)
: mozilla9
Assigned To: Alex Pakhotin (:alexp)
:
: Jim Chen [:jchen] [:darchons]
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-09-16 18:54 PDT by Doug Turner (:dougt)
Modified: 2011-09-27 03:50 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch (3.74 KB, patch)
2011-09-21 18:29 PDT, Alex Pakhotin (:alexp)
blassey.bugs: review+
Details | Diff | Splinter Review

Description Doug Turner (:dougt) 2011-09-16 18:54:34 PDT
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?
Comment 1 Alex Pakhotin (:alexp) 2011-09-21 18:25:56 PDT
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.
Comment 2 Alex Pakhotin (:alexp) 2011-09-21 18:29:23 PDT
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.
Comment 4 Ed Morley [:emorley] 2011-09-27 03:50:28 PDT
https://hg.mozilla.org/mozilla-central/rev/a2f97b48ed3e

Note You need to log in before you can comment on or make changes to this bug.