Bug 73836 - AX: web search mechanism does not work with frames
Summary: AX: web search mechanism does not work with frames
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: chris fleizach
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-05 09:12 PST by chris fleizach
Modified: 2011-12-08 09:23 PST (History)
2 users (show)

See Also:


Attachments
patch (9.68 KB, patch)
2011-12-05 09:18 PST, chris fleizach
bdakin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description chris fleizach 2011-12-05 09:12:13 PST
The searching mechanism built into WK does not search within frames and does not find frames when searching just for frames
Comment 1 chris fleizach 2011-12-05 09:18:30 PST
Created attachment 117888 [details]
patch
Comment 2 chris fleizach 2011-12-05 14:42:17 PST
http://trac.webkit.org/changeset/102049
Comment 4 chris fleizach 2011-12-07 12:01:29 PST
(In reply to comment #3)
> It seems like the test added by this patch is very crashy (~30%):
> 
> http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=platform%2Fmac%2Faccessibility%2Fsearch-with-frames.html&group=%40ToT%20-%20webkit.org

can we get a crash log. this webpage is inscrutable
Comment 5 Ryosuke Niwa 2011-12-07 12:10:54 PST
(In reply to comment #4)
> (In reply to comment #3)
> > It seems like the test added by this patch is very crashy (~30%):
> > 
> > http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=platform%2Fmac%2Faccessibility%2Fsearch-with-frames.html&group=%40ToT%20-%20webkit.org
> 
> can we get a crash log. this webpage is inscrutable

http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r102242%20(35272)/platform/mac/accessibility/search-with-frames-crash-log.txt

Unfortunately the crash log isn't helpful :( Hopefully you can reproduce this crash locally.
Comment 6 chris fleizach 2011-12-07 12:17:15 PST
(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > It seems like the test added by this patch is very crashy (~30%):
> > > 
> > > http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=platform%2Fmac%2Faccessibility%2Fsearch-with-frames.html&group=%40ToT%20-%20webkit.org
> > 
> > can we get a crash log. this webpage is inscrutable
> 
> http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r102242%20(35272)/platform/mac/accessibility/search-with-frames-crash-log.txt
> 
> Unfortunately the crash log isn't helpful :( Hopefully you can reproduce this crash locally.

Why is this crash log not useful?

What leads you to believe it's this test causing the issues?
Comment 7 chris fleizach 2011-12-07 12:18:58 PST
(In reply to comment #6)
> (In reply to comment #5)
> > (In reply to comment #4)
> > > (In reply to comment #3)
> > > > It seems like the test added by this patch is very crashy (~30%):
> > > > 
> > > > http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=platform%2Fmac%2Faccessibility%2Fsearch-with-frames.html&group=%40ToT%20-%20webkit.org
> > > 
> > > can we get a crash log. this webpage is inscrutable
> > 
> > http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r102242%20(35272)/platform/mac/accessibility/search-with-frames-crash-log.txt
> > 
> > Unfortunately the crash log isn't helpful :( Hopefully you can reproduce this crash locally.
> 
> Why is this crash log not useful?
> 
> What leads you to believe it's this test causing the issues?

Rather, I mean, why do you think this is not a real bug that needs to be fixed independent of this test
Comment 8 Ryosuke Niwa 2011-12-07 12:20:08 PST
(In reply to comment #7)
> (In reply to comment #6)
> > Why is this crash log not useful?
> > 
> > What leads you to believe it's this test causing the issues?
> 
> Rather, I mean, why do you think this is not a real bug that needs to be fixed independent of this test

Oh, I'm not saying that we should fix the test. We should fix WebKit not to crash.
Comment 9 chris fleizach 2011-12-07 12:23:37 PST
(In reply to comment #8)
> (In reply to comment #7)
> > (In reply to comment #6)
> > > Why is this crash log not useful?
> > > 
> > > What leads you to believe it's this test causing the issues?
> > 
> > Rather, I mean, why do you think this is not a real bug that needs to be fixed independent of this test
> 
> Oh, I'm not saying that we should fix the test. We should fix WebKit not to crash.

Will see if i can repro... we should open a new bug... 
are all the crash logs the same or are they different?
Comment 10 Ryosuke Niwa 2011-12-07 12:25:41 PST
(In reply to comment #9)
> Will see if i can repro... we should open a new bug... 
> are all the crash logs the same or are they different?

I haven't been able to figure that out. If you go to the flakiness dashboard, you can click on one of those orange bars, you can get to build result pages.
Comment 11 chris fleizach 2011-12-08 09:23:10 PST
(In reply to comment #10)
> (In reply to comment #9)
> > Will see if i can repro... we should open a new bug... 
> > are all the crash logs the same or are they different?
> 
> I haven't been able to figure that out. If you go to the flakiness dashboard, you can click on one of those orange bars, you can get to build result pages.

We have a bug for this with a reasonable looking backtrace
https://bugs.webkit.org/show_bug.cgi?id=74093