Bug 45666 - [Qt] Click's count is limited to three continuous click
Summary: [Qt] Click's count is limited to three continuous click
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P3 Normal
Assignee: Ragner Magalhaes
URL:
Keywords: Qt
: 61795 (view as bug list)
Depends on: 73366
Blocks:
  Show dependency treegraph
 
Reported: 2010-09-13 06:38 PDT by Ragner Magalhaes
Modified: 2011-12-09 15:53 PST (History)
9 users (show)

See Also:


Attachments
Proposed patch (18.89 KB, patch)
2010-09-13 12:26 PDT, Ragner Magalhaes
tonikitoo: review-
Details | Formatted Diff | Diff
Patch (11.36 KB, patch)
2011-12-06 07:32 PST, Hugo Parente Lima
no flags Details | Formatted Diff | Diff
Patch (11.35 KB, patch)
2011-12-06 07:55 PST, Hugo Parente Lima
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ragner Magalhaes 2010-09-13 06:38:23 PDT
How the click's count is limited to three continuous click
the test LayoutTests/fast/events/click-count.html is failing.
Comment 1 Ragner Magalhaes 2010-09-13 12:26:35 PDT
Created attachment 67453 [details]
Proposed patch
Comment 2 Ragner Magalhaes 2010-09-13 12:33:22 PDT
Comment on attachment 67453 [details]
Proposed patch

Adds a click's count on WebKit/qt/qwebpage.[h,cpp] to enable support for more than three continuous click.
Comment 3 Antonio Gomes 2010-09-21 07:18:41 PDT
Comment on attachment 67453 [details]
Proposed patch

View in context: https://bugs.webkit.org/attachment.cgi?id=67453&action=review

Good patch! I like the way you took the click couting control out of EventSender class.

A few nits, questions and requests:

1) How many consecutive clicks we support now? (triple clicking used to be the limite, right?)
2) You're removing a bunch of methods from WebPagePrivate class, Please, explain in our changelog why and how it is working now.

It would be great if we could make sure that, after your change, when the routine DlbClick was suposed to be called, it clickCould would be '2'. Same for the triple clicking routine.

Any way to assert of this case as a sanity check?

> WebKit/qt/Api/qwebpage.cpp:628
> +    Frame *focusedFrame;

Make it Frame *focusedFrame = 0;

> WebKit/qt/Api/qwebpage.cpp:672
> +    Frame *focusedFrame;

Make it Frame *focusedFrame = 0;

> WebKit/qt/ChangeLog:7
> +

Add here why you are removing the double and triple click handler routines, and how it is working not.
Comment 4 Antonio Gomes 2011-05-31 12:51:56 PDT
*** Bug 61795 has been marked as a duplicate of this bug. ***
Comment 5 Robert Hogan 2011-06-21 10:54:24 PDT
Ragner, are you going to clean up the patch for landing?
Comment 6 Caio Marcelo de Oliveira Filho 2011-07-06 07:16:20 PDT
(In reply to comment #5)
> Ragner, are you going to clean up the patch for landing?

AFAIK Ragner is not working on the project anymore. (I'm guessing you want to clean it up then, right? :-)
Comment 7 Hugo Parente Lima 2011-12-01 12:45:39 PST
I have a patch to fix this bug on wk2, but it depends on bug#45666, I'll upload it when the patch for bug#45666 lands on mainline.
Comment 8 Caio Marcelo de Oliveira Filho 2011-12-01 12:49:40 PST
(In reply to comment #7)
> I have a patch to fix this bug on wk2, but it depends on bug#45666, I'll upload it when the patch for bug#45666 lands on mainline.

Hugo, maybe you made a typo here. *This* is bug 45666. :-)
Comment 9 Hugo Parente Lima 2011-12-01 12:59:15 PST
(In reply to comment #8)
> (In reply to comment #7)
> > I have a patch to fix this bug on wk2, but it depends on bug#45666, I'll upload it when the patch for bug#45666 lands on mainline.
> 
> Hugo, maybe you made a typo here. *This* is bug 45666. :-)

Then I have a self circular dependency!

The right bug is bug#73366 =]
Comment 10 Hugo Parente Lima 2011-12-06 07:32:25 PST
Created attachment 118049 [details]
Patch
Comment 11 Kenneth Rohde Christiansen 2011-12-06 07:36:04 PST
Comment on attachment 118049 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=118049&action=review

> Source/WebKit2/UIProcess/qt/QtWebPageEventHandler.cpp:162
> +    if (m_clickTimer.isActive()
> +            && m_previousClickButton == ev->button()
> +            && (ev->pos() - m_lastClick).manhattanLength() < qApp->styleHints()->startDragDistance()) {

wrong indentation according to the style guide... (hint, it is indented too much)

> Tools/ChangeLog:8
> +        Update m_time at every call to leapForward, so dbl clicks

write out "double"
Comment 12 Hugo Parente Lima 2011-12-06 07:52:50 PST
Comment on attachment 118049 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=118049&action=review

>> Source/WebKit2/UIProcess/qt/QtWebPageEventHandler.cpp:162
>> +            && (ev->pos() - m_lastClick).manhattanLength() < qApp->styleHints()->startDragDistance()) {
> 
> wrong indentation according to the style guide... (hint, it is indented too much)

hmmm... check-webkit-style isn't perfect :-(

>> Tools/ChangeLog:8
>> +        Update m_time at every call to leapForward, so dbl clicks
> 
> write out "double"

Ok, fixing and re-uploading...
Comment 13 Hugo Parente Lima 2011-12-06 07:55:54 PST
Created attachment 118052 [details]
Patch
Comment 14 Hugo Parente Lima 2011-12-09 04:54:19 PST
I'm not a commiter, can someone commit this for me? :-)

Sorry for not upload the patch asking for the commit queue flag, I did learn it just minutes before sending the patch.
Comment 15 WebKit Review Bot 2011-12-09 04:56:08 PST
Comment on attachment 118052 [details]
Patch

Rejecting attachment 118052 [details] from commit-queue.

hugo.lima@openbossa.org does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 16 Igor Trindade Oliveira 2011-12-09 05:55:17 PST
Comment on attachment 118052 [details]
Patch

Adding to commit queue
Comment 17 WebKit Review Bot 2011-12-09 15:53:50 PST
Comment on attachment 118052 [details]
Patch

Clearing flags on attachment: 118052

Committed r102488: <http://trac.webkit.org/changeset/102488>
Comment 18 WebKit Review Bot 2011-12-09 15:53:58 PST
All reviewed patches have been landed.  Closing bug.