The default bug view has changed. See this FAQ.

[nsIDOMXULLabelElement::accessKey] errors in console at yahoo.com

RESOLVED FIXED

Status

()

Core
DOM: Events
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: marcia, Assigned: smaug)

Tracking

({qawanted, verified1.8.1.8})

Trunk
x86
Mac OS X
qawanted, verified1.8.1.8
Points:
---
Bug Flags:
blocking1.8.1.8 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
I am getting a number of these errors in the console today when I am visiting yahoo.com - haven't seen them previously and a bugzilla search yield nothing.

Error: [Exception... "'Permission denied to get property XULElement.accessKey' when calling method: [nsIDOMXULLabelElement::accessKey]"  nsresult: "0x8057001e (NS_ERROR_XPC_JS_THREW_STRING)"  location: "JS frame :: http://www.yahoo.com/ :: <TOP_LEVEL> :: line 15"  data: no]
Source File: http://www.yahoo.com/
Line: 15
Is this a regression from Bug 387033?
What are the steps to reproduce this? Or should visiting yahoo.com be enough (I don't get any errors)?
(Reporter)

Comment 3

10 years ago
smaug: My experience on the mac is I immediately get three of these errors when I simply visit yahoo.com. This happens using my existing profile or a new profile. I can check to see if it occurs on other platforms.
I can reproduce it.
Blocks: 387033
Created attachment 276672 [details] [diff] [review]
possible patch

I'll still debug to see what is actually causing the error.
Assignee: nobody → Olli.Pettay
Status: NEW → ASSIGNED
So it is just that script causes a flush. I think the patch is ok.

#0  nsTextBoxFrame::UpdateAccesskey (this=0x8dbe358, aWeakThis=@0x90cc074)
    at /home/smaug/mozilla/mozilla_cvs/mozilla/layout/xul/base/src/nsTextBoxFrame.cpp:242
#1  0x02b639e4 in nsAsyncAccesskeyUpdate::ReflowFinished (this=0x90cc070)
    at /home/smaug/mozilla/mozilla_cvs/mozilla/layout/xul/base/src/nsTextBoxFrame.cpp:224
#2  0x0295c52a in PresShell::HandlePostedReflowCallbacks (this=0x881af08) at /home/smaug/mozilla/mozilla_cvs/mozilla/layout/base/nsPresShell.cpp:4339
#3  0x02961587 in PresShell::DidDoReflow (this=0x881af08) at /home/smaug/mozilla/mozilla_cvs/mozilla/layout/base/nsPresShell.cpp:6018
#4  0x0296ea4b in PresShell::ProcessReflowCommands (this=0x881af08, aInterruptible=0)
    at /home/smaug/mozilla/mozilla_cvs/mozilla/layout/base/nsPresShell.cpp:6211
#5  0x0296efd8 in PresShell::FlushPendingNotifications (this=0x881af08, aType=Flush_Layout)
    at /home/smaug/mozilla/mozilla_cvs/mozilla/layout/base/nsPresShell.cpp:4405
#6  0x02be9e70 in nsDocument::FlushPendingNotifications (this=0x8423f40, aType=Flush_Layout)
    at /home/smaug/mozilla/mozilla_cvs/mozilla/content/base/src/nsDocument.cpp:4840
#7  0x02e8d986 in nsGlobalWindow::FlushPendingNotifications (this=0x87394e8, aType=Flush_Layout)
    at /home/smaug/mozilla/mozilla_cvs/mozilla/dom/src/base/nsGlobalWindow.cpp:7607
#8  0x02ea19e7 in nsGlobalWindow::EnsureSizeUpToDate (this=0x8e5fd68) at /home/smaug/mozilla/mozilla_cvs/mozilla/dom/src/base/nsGlobalWindow.cpp:7620
#9  0x02ec2888 in nsScreen::GetDeviceContext (this=0x8f97298) at /home/smaug/mozilla/mozilla_cvs/mozilla/dom/src/base/nsScreen.cpp:214
#10 0x02ec2cf5 in nsScreen::GetAvailRect (this=0x8f97298, aRect=@0xbfe73658) at /home/smaug/mozilla/mozilla_cvs/mozilla/dom/src/base/nsScreen.cpp:256
#11 0x02ec3057 in nsScreen::GetAvailWidth (this=0x8f97298, aAvailWidth=0xbfe73828)
    at /home/smaug/mozilla/mozilla_cvs/mozilla/dom/src/base/nsScreen.cpp:159
#12 0x0052d2b9 in NS_InvokeByIndex_P () at ../../../../dist/include/xpcom/nsISupportsImpl.h:205
#13 0x027ed05a in XPCWrappedNative::CallMethod (ccx=@0xbfe73968, mode=XPCWrappedNative::CALL_GETTER)
Attachment #276672 - Flags: superreview?(jst)
Attachment #276672 - Flags: review?(jst)
Comment on attachment 276672 [details] [diff] [review]
possible patch

r+sr=jst, but it'd be good to add a comment explaining why this context pusher is needed here.
Attachment #276672 - Flags: superreview?(jst)
Attachment #276672 - Flags: superreview+
Attachment #276672 - Flags: review?(jst)
Attachment #276672 - Flags: review+
Attachment #276672 - Flags: approval1.9?
This is a regression from a 1.9+ blocker, so asking approval.

Updated

10 years ago
Attachment #276672 - Flags: approval1.9? → approval1.9+
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
qawanted: verify this doesn't regress on the 1.8 branch after bug 387033 lands
Flags: blocking1.8.1.8+
Keywords: qawanted
Keywords: fixed1.8.1.8
(In reply to comment #9)
> qawanted: verify this doesn't regress on the 1.8 branch after bug 387033 lands
> 

Patch from 387033 has landed. No error Message in the EC on yahoo.com 

verified fixed 1.8.1.8 using Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.8pre)Gecko/2007093003 BonEcho/2.0.0.8pre and Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.8.1.8pre) Gecko/20070929 BonEcho/2.0.0.8pre ID:2007092904

-> adding verified keyword


Keywords: fixed1.8.1.8 → verified1.8.1.8
You need to log in before you can comment on or make changes to this bug.