Closed Bug 690120 Opened 8 years ago Closed 8 years ago

Firefox 7 - Warning on textarea (Warning: Use of getAttributeNode() is deprecated. Use getAttribute() instead.)

Categories

(Core :: DOM: Editor, defect)

defect
Not set

Tracking

()

VERIFIED FIXED
mozilla10
Tracking Status
firefox8 --- wontfix
firefox9 --- wontfix
firefox10 --- verified

People

(Reporter: enzo_nanni, Assigned: Ms2ger)

References

Details

(Keywords: regression, verified-beta, Whiteboard: [qa!])

Attachments

(2 files)

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0) Gecko/20100101 Firefox/7.0
Build ID: 20110922153450

Steps to reproduce:

I created a textarea


Actual results:

Error Console return:
Warning: Use of getAttributeNode() is deprecated. Use getAttribute() instead.

With Firefox 6 everything was ok.
So, you got a warning telling you that what your site was doing was no longer going to be supported, and it pointed you towards what to do to fix it.  Exactly what is your question here?
Attachment #563191 - Attachment mime type: text/plain → text/html
I know that the warning message was added by Bug 661327.
However, no getAttributeNode()is  in the attachement html.
So,I think the warning message should not expose in this case.
Status: UNCONFIRMED → NEW
Ever confirmed: true
The warning is triggered by entering a value into the textarea.

<textarea autocomplete="off"></textarea>

no warning on page load but when accessing the textarea.

without autocomplete="off" the warning is triggered on every reload

So the problem is INSIDE Firefox and see also

https://developer.mozilla.org/En/DOM/Attr#Deprecated_properties_and_methods

for a NOGO situation soon to come
Component: General → Editor
Product: Firefox → Core
QA Contact: general → editor
Version: 7 Branch → unspecified
Blocks: 661327
Attached patch Patch v1Splinter Review
Assignee: nobody → Ms2ger
Status: NEW → ASSIGNED
Attachment #568939 - Flags: review?(ehsan)
Comment on attachment 568939 [details] [diff] [review]
Patch v1

Beautiful!
Attachment #568939 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/e2ac48ba0569
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
OS: Windows 7 → All
Hardware: x86_64 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
Version: unspecified → Trunk
ms2ger, could you please also fix bug 674437 in the same way?
Looks like: https://bugzilla.mozilla.org/show_bug.cgi?id=669400

Also, you should Reopen this bug: I'm able to reproduce in Firefox 8: 
"Mozilla/5.0 (Windows NT 6.1; WOW64; rv:8.0) Gecko/20100101 Firefox/8.0 Firefox/8.0"

Warning: Use of getAttributeNode() is deprecated. Use getAttribute() instead.
Source File: chrome://browser/content/scratchpad.xul
Line: 0

AND

Warning: Use of getAttributeNode() is deprecated. Use getAttribute() instead.
Source File: chrome://browser/content/places/places.xul
Line: 0
Duplicate of this bug: 669400
(In reply to Rhett Trappman from comment #9)
> Also, you should Reopen this bug: I'm able to reproduce in Firefox 8: 
This fix will not be release until Firefox 10 (see Target Milestone field).
To reproduce the bug:
Click inside an empty text box, to set the keyboard focus.
Like the one that I am writing this comment in. ;)

Start typing: bug reproduced!
Note that getAttributeNode() and getAttribute() may not even be preset in the source of the web page. The bug is still reproduced.

I tried to disable all extensions, but that didn't resolve it.

Versions tested: Firefox 7:reproduced every time and Firefox 8:reproduced every time.

DATE: 2011-November-16, AM 4:26 GMT+2
STATUS: NOT RESOLVED!
(In reply to George from comment #12)
> Versions tested: Firefox 7:reproduced every time and Firefox 8:reproduced
> every time.
> 
> DATE: 2011-November-16, AM 4:26 GMT+2
> STATUS: NOT RESOLVED!

The Target Milestone is Firefox 10 as already stated in comment 11.

Are you able to reproduce it in Firefox 10 or later as well?
Could this land on Aurora and Beta? Would save some bugzilla activity.
Saving some bugzilla activity is no reason to backport patches.

That said, the patch is perfectly safe to backport, but IMO not worthwhile enough to do so.
> Saving some bugzilla activity 
Includes saving time and nerve-shattering of x-thousands of confused web developers who care about console warnings what we always tell them they should but most of them care not because some warnings make no sense for them
Keywords: regression
Not reproduced on version 10.
On the other hand SGV rendering is completely broken on Aurora 10!
This is a pre-alpha version and is not ready for the public.

Firefox console is a good tool for to checking a web site under development.
> On the other hand SGV rendering is completely broken on Aurora 10!
Please file a new bug about that.
Whiteboard: [qa+]
This is not the sort of fix which is suitable for branches.
Mozilla/5.0 (Windows NT 6.1; rv:10.0) Gecko/20100101 Firefox/10.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:10.0) Gecko/20100101 Firefox/10.0
Mozilla/5.0 (X11; Linux i686; rv:10.0) Gecko/20100101 Firefox/10.0

Verified the fix using the test page from comment #0 and no message was displayed in the error console at page load nor when typing in the text box.
Status: RESOLVED → VERIFIED
Keywords: verified-beta
Whiteboard: [qa+] → [qa!]
You need to log in before you can comment on or make changes to this bug.