text selection with the mouse not working

NEW
Assigned to

Status

--
major
12 years ago
6 years ago

People

(Reporter: tomeu, Assigned: romaxa)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

12 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-GB; rv:1.8.1.2) Gecko/20060601 Firefox/2.0.0.2 (Ubuntu-edgy)
Build Identifier: 

Text in a webpage in gtkmozembed won't be selected by the mouse. Ctrl-A works. Easily reproducible in TestGtkEmbed.

Reproducible: Always

Steps to Reproduce:
Try to select some text on a webpage with the mouse.
Actual Results:  
Nothing happens.

Expected Results:  
The text would be selected and then could be copied, dragged, etc.

Updated

12 years ago
Assignee: nobody → romaxa
(Assignee)

Comment 1

12 years ago
Created attachment 259826 [details] [diff] [review]
BUGFIX
Attachment #259826 - Flags: review?
(Assignee)

Comment 2

12 years ago
Comment on attachment 259826 [details] [diff] [review]
BUGFIX

Just fixed default value state
Attachment #259826 - Flags: review? → review?(timeless)
(Just something I noticed while looking at the patch:

from EmbedEventListener:

static PRInt32 sLongPressTimer = 0
[...]
#if 1
     if (sLongPressTimer)
       g_source_remove(sLongPressTimer);
 #else  
[...]
  sLongPressTimer = g_timeout_add(mLongMPressDelay, sLongMPress, mOwner->mOwningWidget);

First, g_timeout_add returns |guint|, NOT (signed) |int32|.
Second, sLongPressTimer isn't restored to 0 after g_source_remove.)
(Assignee)

Comment 4

12 years ago
Probably we need to create some macro...
like:
#define GLIB_SOURCE_REMOVE(source) \
  PR_BEGIN_MACRO \
    if (source) { \
      g_source_remove(source); \
      source = 0; \
    } \
  PR_END_MACRO

And this stuff will be in branch very soon.

Comment 5

12 years ago
Comment on attachment 259826 [details] [diff] [review]
BUGFIX

assuming I'm understanding this code correctly, please rename return_val to handled.
Attachment #259826 - Flags: review?(timeless) → review+

Comment 6

11 years ago
Created attachment 281782 [details] [diff] [review]
mouse-text-selection-bug-373196.patch

Updated patch trying to address comments from chpe and timeless. Also fixes the bug ;)

Any ETA we could get this to tree?

Updated

11 years ago
Attachment #281782 - Flags: review?(timeless)
Duplicate of this bug: 405616

Comment 8

11 years ago
the attachment 281782 [details] [diff] [review] fixes it for me.
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 9

11 years ago
Comment on attachment 281782 [details] [diff] [review]
mouse-text-selection-bug-373196.patch

this patch is currently blocked by whatever patch that will reintroduce that code.
Attachment #281782 - Flags: review?(timeless)
Component: Embedding: GTK Widget → Embedding: GTK Widget
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.