Ticket #2119 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Measure waveform capture has missing area due to palette redraw

Reported by: quozl Owned by: walter
Priority: Unspecified by Maintainer Milestone:
Component: Measure Version:
Severity: Trivial Keywords:
Cc: Distribution/OS: OLPC
Bug Status: Unconfirmed

Description

There is a small rectangle of screen that is missing from the waveform capture. This is where the palette for the button was.

Measure-30, XO-1 os301, XO-1.5 os206.

Attachments

Waveform 1.png Download (38.7 KB) - added by quozl 3 years ago.
whistle capture on XO-1 showing missing rectangle below capture button
0001-avoid-race-that-leads-to-waveform-capture-image-corr.patch Download (4.6 KB) - added by quozl 3 years ago.

Change History

Changed 3 years ago by quozl

whistle capture on XO-1 showing missing rectangle below capture button

Changed 3 years ago by walter

  • status changed from new to closed
  • resolution set to wontfix

From the attached picture, it seems that the toolbar is not refreshing properly from the tooltip of the capture button. I am guessing it is a problem with either the Sugar toolbar or gdk. It doesn't seem to be a problem with Sugar 0.88 toolbars and it doesn't seem to show up when running Sugar 0.84 in the emulator. Not sure I can do anything about this.

Changed 3 years ago by quozl

  • status changed from closed to reopened
  • resolution wontfix deleted

It will eventually happen with Sugar 0.88 given sufficiently slow hardware, and is not a problem with the toolbar implementation, but rather due to taking a screenshot by reading the root window without first ensuring the display is quiescent. The attached fix has been tested on XO-1 and XO-1.5 and fixes the symptom.

Changed 3 years ago by walter

Sorry. I had misread the original ticket. I thought it was a problem with the display, which I couldn't reproduce, not the screenshot. I'll look at your patch. Thanks.

Changed 3 years ago by quozl

Tested Measure git hash 74d81ce, the symptom is solved.

Changed 3 years ago by walter

  • status changed from reopened to closed
  • resolution set to fixed

fixed in v31

Changed 3 years ago by quozl

Tested Measure-31, is fixed.

Note: See TracTickets for help on using tickets.