Skip to site navigation (Press enter)

Re: [webkit-dev] Feature Announcement: Adding </span></a></span> </h1> <p class="darkgray font13"> <span class="sender pipe"><a href="/search?l=webkit-dev@lists.webkit.org&q=from:%22Alexey+Proskuryakov%22" rel="nofollow"><span itemprop="author" itemscope itemtype="http://schema.org/Person"><span itemprop="name">Alexey Proskuryakov</span></span></a></span> <span class="date"><a href="/search?l=webkit-dev@lists.webkit.org&q=date:20120331" rel="nofollow">Sat, 31 Mar 2012 11:37:35 -0700</a></span> </p> </div> <div itemprop="articleBody" class="msgBody">  <pre> 30.03.2012, в 20:11, Darin Fisher написал(а): > I think it is a risky practice for WebKit to have half-baked "webkit" prefixed > features enabled by default on trunk. </pre><pre> I agree. The patch that exposed this "to be removed" attribute (<<a rel="nofollow" href="https://bugs.webkit.org/show_bug.cgi?id=82795">https://bugs.webkit.org/show_bug.cgi?id=82795</a>>) also added a number of tests that are expected to fail because there is simply no implementation behind them. This too, I find unhelpful and undesirable. Landing tests with failing subtests is useful in some cases, e.g. when the test is exhaustive, and the feature is mostly working fine. But in this case, I can't see any benefit of landing tests first. The downside is quite material - it's much more difficult for reviewer of actual code change to see what test coverage is. Additional bug/commit noise has its costs, too. - WBR, Alexey Proskuryakov </pre><pre>_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org <a rel="nofollow" href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a> </pre> </div> <div class="msgButtons margintopdouble"> <ul class="overflow"> <li class="msgButtonItems"><a class="button buttonleft " accesskey="p" href="msg18588.html">Previous message</a></li> <li class="msgButtonItems textaligncenter"><a class="button" accesskey="c" href="thrd2.html#18592">View by thread</a></li> <li class="msgButtonItems textaligncenter"><a class="button" accesskey="i" href="mail2.html#18592">View by date</a></li> <li class="msgButtonItems textalignright"><a class="button buttonright " accesskey="n" href="msg18594.html">Next message</a></li> </ul> </div> <a name="tslice"></a> <div class="tSliceList margintopdouble"> <ul class="icons monospace"> <li class="icons-email"><span class="subject"><a href="msg18586.html">[webkit-dev] Feature Announcement: Adding <iframe s...</a></span> <span class="sender italic">Eric Seidel</span></li> <li><ul> <li class="icons-email"><span class="subject"><a href="msg18588.html">Re: [webkit-dev] Feature Announcement: Adding &lt...</a></span> <span class="sender italic">Darin Fisher</span></li> <li><ul> <li class="icons-email tSliceCur"><span class="subject">Re: [webkit-dev] Feature Announcement: Adding...</span> <span class="sender italic">Alexey Proskuryakov</span></li> <li class="icons-email"><span class="subject"><a href="msg18594.html">Re: [webkit-dev] Feature Announcement: Adding...</a></span> <span class="sender italic">Eric Seidel</span></li> <li><ul> <li class="icons-email"><span class="subject"><a href="msg18954.html">Re: [webkit-dev] Feature Announcement: Ad...</a></span> <span class="sender italic">Eric Seidel</span></li> <li><ul> <li class="icons-email"><span class="subject"><a href="msg18955.html">Re: [webkit-dev] Feature Announcement...</a></span> <span class="sender italic">Maciej Stachowiak</span></li> <li><ul> <li class="icons-email"><span class="subject"><a href="msg18956.html">Re: [webkit-dev] Feature Announc...</a></span> <span class="sender italic">Dimitri Glazkov</span></li> <li class="icons-email"><span class="subject"><a href="msg18957.html">Re: [webkit-dev] Feature Announc...</a></span> <span class="sender italic">Eric Seidel</span></li> <li><ul> <li class="icons-email"><span class="subject"><a href="msg18958.html">Re: [webkit-dev] Feature Ann...</a></span> <span class="sender italic">Maciej Stachowiak</span></li> <li><ul> <li class="icons-email"><span class="subject"><a href="msg18959.html">Re: [webkit-dev] Feature...</a></span> <span class="sender italic">Adam Barth</span></li> <li class="icons-email"><span class="subject"><a href="msg18960.html">Re: [webkit-dev] Feature...</a></span> <span class="sender italic">Maciej Stachowiak</span></li> <li class="icons-email"><span class="subject"><a href="msg18963.html">Re: [webkit-dev] Feature...</a></span> <span class="sender italic">Jarred Nicholls</span></li> <li class="icons-email"><span class="subject"><a href="msg18967.html">Re: [webkit-dev] Feature...</a></span> <span class="sender italic">Maciej Stachowiak</span></li> </ul> </ul> </ul> </ul> </ul> </ul> </ul> </ul> </div> <div class="overflow msgActions margintopdouble"> <div class="msgReply" > <h2> Reply via email to </h2> <form method="POST" action="/mailto.php"> <input type="hidden" name="subject" value="Re: [webkit-dev] Feature Announcement: Adding <iframe seamless>"> <input type="hidden" name="msgid" value="9E9250C6-FF6C-4EAD-AD7A-94C82B6A7F57@webkit.org"> <input type="hidden" name="relpath" value="webkit-dev@lists.webkit.org/msg18592.html"> <input type="submit" value=" Alexey Proskuryakov "> </form> </div> </div> </div> <div class="aside" role="complementary"> <div class="logo"> <a href="/"><img src="/logo.png" width=247 height=88 alt="The Mail Archive"></a> </div> <form class="overflow" action="/search" method="get"> <input type="hidden" name="l" value="webkit-dev@lists.webkit.org"> <label class="hidden" for="q">Search the site</label> <input class="submittext" type="text" id="q" name="q" placeholder="Search webkit-dev"> <input class="submitbutton" name="submit" type="image" src="/submit.png" alt="Submit"> </form> <div class="nav margintop" id="nav" role="navigation"> <ul class="icons font16"> <li class="icons-home"><a href="/">The Mail Archive home</a></li> <li class="icons-list"><a href="/webkit-dev@lists.webkit.org/">webkit-dev - all messages</a></li> <li class="icons-about"><a href="/webkit-dev@lists.webkit.org/info.html">webkit-dev - about the list</a></li> <li class="icons-expand"><a href="/search?l=webkit-dev@lists.webkit.org&q=subject:%22Re%5C%3A+%5C%5Bwebkit%5C-dev%5C%5D+Feature+Announcement%5C%3A+Adding+%3Ciframe+seamless%3E%22&o=newest&f=1" title="e" id="e">Expand</a></li> <li class="icons-prev"><a href="msg18588.html" title="p">Previous message</a></li> <li class="icons-next"><a href="msg18594.html" title="n">Next message</a></li> </ul> </div> <div class="listlogo margintopdouble"> <a href="#"><img src="/webkit-dev@lists.webkit.org/logo.png" alt="webkit-dev"></a> </div> <div class="margintopdouble"> </div> </div> </div> <div class="footer" role="contentinfo"> <ul> <li><a href="/">The Mail Archive home</a></li> <li><a href="/faq.html#newlist">Add your mailing list</a></li> <li><a href="/faq.html">FAQ</a></li> <li><a href="/faq.html#support">Support</a></li> <li><a href="/faq.html#privacy">Privacy</a></li> <li class="darkgray">9E9250C6-FF6C-4EAD-AD7A-94C82B6A7F57@webkit.org</li> </ul> </div> </body> </html>