[Webkit-unassigned] [Bug 88094] Web Inspector: Add a WebInspectorServer on Linux using the GSocket API for the GTK port

[bugzilla-daemon at webkit.org]

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


Anton Obzhirov <a.obzhirov at samsung.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|WebGL                       |Web Inspector
                 CC|                            |vsevik at chromium.org,
                   |                            |web-inspector-bugs at googlegr
                   |                            |oups.com




--- Comment #44 from Anton Obzhirov <a.obzhirov at samsung.com>  2012-10-19 05:47:51 PST ---
(In reply to comment #43)
> (In reply to comment #42)
> > Hi all, I am going to make new patch in few days, there is one question I want to clarify about refactoring the code to use SOUP compilation flag. I guess I can just follow Comment #40 From Jesus Sanchez-Palencia for that. Does anybody have any additional comments about it?
> 
> This is just my opinion and not everybody might agree with this, but I think that it would be less wasted reviewing efforts if you apply the modifications needed to land this patch first and then do further tuning in different patches (e.g. for EFL support).
> If you refactor this code, it might be difficult to do a review on only what changed.
> 
> This patch is already larger than the average and splitting in smaller patches is usually encouraged.

Makes sense for me. I will fix now all remaining bits except EFL support which will be addressed later probably with another bug.

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