<div dir="ltr">Nice!, I quickly looked at the patch and I urged try to encapsulate the all logic of webkit2 and the webkit1 in separate modules, using a luck of strategy pattern or do something polymorphic, but I notice that it is a decision not yet taken (as Daniel says), I doubt whether to spend time on this for that reason, you will say, however when this decision is finally decided, I would like to make note of this style refactor to have clean webactivity module, which is relatively new and keep small, you think?<br>

<div><br></div><div>Regards!</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/11/27 Gonzalo Odiard <span dir="ltr"><<a href="mailto:gonzalo@laptop.org" target="_blank">gonzalo@laptop.org</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><div class="im">On Wed, Nov 27, 2013 at 1:04 PM, Daniel Narvaez <span dir="ltr"><<a href="mailto:dwnarvaez@gmail.com" target="_blank">dwnarvaez@gmail.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think it's good to have this upstream because otherwise changes to sugar-web will easily break webkit1 support.<div>


<br></div><div>I'm not completely convinced we should support webkit1 yet. I'm worried it will be pretty painful to maintain. It should be possible to port webkitgtk 2.2 to Fedora 18... </div>
<div><br></div></blockquote><div><br></div></div><div>Maybe, but have many other dependencies to update.</div><div><br></div><div>Another alternative is find what is crashing in webkit2 and solve it, but is out of my knowledge.</div>

<span class="HOEnZb"><font color="#888888">
<div><br></div><div>Gonzalo</div><div><br></div></font></span></div></div></div>
<br>_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Roger<div><br><div><a href="http://activitycentral.com/" target="_blank">Activity Central</a></div></div>
</div>