84143 - GetScrapFlavorSize very slow when text copied from classic is on clipboard

Status: RESOLVED FIXED

(Core :: XUL, defect)

Description

[Henri Sivonen (:hsivonen)]

Build ID: 2001-05-30 FizzillaCFM

Reproducible: Most of the time

Steps to reproduce:
1) Put the caret in a text field (URL field or form field)
2) If the problem didn't occur, try again.
3) If the problem still didn't occur, the problem has gone away and probably
won't reappear in the current session. Restart Mozilla to see the problem.

Actual results:
The already infamous color wheel cursor shows up.

Expected results:
Expected the color wheel cursor to not show up and expected the UI to be
responsive right away after focusing or unfocusing a field.

Additional information:
The problem sometimes just goes away for the rest of the session. It'll reappear
when Mozilla is run again.

I didn't see the problem with earlier builds.

Comment 1

[Mike Pinkerton (not reading bugmail)]

i see this too with the 5/30 builds, i suspect it has to do with smfr's NSPR
polling work.

Comment 2

[Simon Fraser [no longer active]]

huh? why would focussing a text field do anything network-related? Can you 
reproduce this bug whcn clicking if you haven't done any networking?

Comment 3

[Simon Fraser [no longer active]]

0.9.3

Comment 4

[Simon Fraser [no longer active]]

pink: I bet this bug is hang when getting clipboard data (since we do that to 
enable/disable Edit menu commands). So back to you  :)

Comment 5

[Mike Pinkerton (not reading bugmail)]

The delay is only when text from classic is on the clipboard and we call
GetScrapFlavorSize(). the workaround is to copy some text from a non-classic app.

i asked apple about this, let's see what they say. i'm not sure this is really
our bug.

Comment 6

[Mike Pinkerton (not reading bugmail)]

Attachment: [patch] fixes problem

Comment 7

[Mike Pinkerton (not reading bugmail)]

patch r=pchen, tested on os9 and osx.

the trick is to use ::GetScrapFlavorInfoList() instead, which is always 
inexpensive. the patch also cleans up some debugging ifdefs and fixes a leak in 
an error case.

Comment 8

[David Hyatt]

sr=hyatt

Comment 9

[Mike Pinkerton (not reading bugmail)]

f

Comment 10

[John Morrison]

May God have mercy on us all. The 212 bug spam-o-rama is Now!