[Webkit-unassigned] [Bug 80702] [Qt][WK2] Implement web notifications support

Wed Jun 13 20:18:13 PDT 2012

https://bugs.webkit.org/show_bug.cgi?id=80702

--- Comment #11 from Adenilson Cavalcanti Silva <savagobr at yahoo.com>  2012-06-13 20:18:11 PST ---
Kenneth

Thanks a lot for your comments, I will upload a patch with fixes.

I have just 1 question: about the indentation in QWebPermissionRequestPrivate constructor. Do you want all parameters to be in 1 single line?

If I understood correctly, from the webkit coding sytle, it says: "Each member (and superclass) should be indented on a separate line, with the colon or comma preceding the member on that line." and has an example similar to the patch.

Next, about the constructor: currently it has 4 parameters. I can see that if in future we need to treat more permission types, it would make sense to have multiple constructors. But ATM, there are only 2 permission types, so I feel that the code will be simpler (i.e. less member functions) in this way.

Best regards

Adenilson

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.