<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Make preview inline navigation work API"
href="https://bugs.webkit.org/show_bug.cgi?id=155383#c11">Comment # 11</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Make preview inline navigation work API"
href="https://bugs.webkit.org/show_bug.cgi?id=155383">bug 155383</a>
from <span class="vcard"><a class="email" href="mailto:bdakin@apple.com" title="Beth Dakin <bdakin@apple.com>"> <span class="fn">Beth Dakin</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=155383#c10">comment #10</a>)
<span class="quote">> (In reply to <a href="show_bug.cgi?id=155383#c8">comment #8</a>)
> > (In reply to <a href="show_bug.cgi?id=155383#c6">comment #6</a>)
> >
> > > > > Source/WebKit2/UIProcess/API/Cocoa/WKElementInfoInternal.h:39
> > > > > +#endif // WK_API_ENABLED
> > > >
> > > > I think we normally don’t // WK_API_ENABLED.
> > > >
> > >
> > > I found 173 occurrences of '// WK_API_ENABLED' when I searched for it in the
> > > WebKit2 xcoeproj, so I left it in for now. :-O
> >
> > I should have been more specific. I think we normally avoid //
> > WK_API_ENABLED in our API and SPI headers, or at least in our API headers :-P
>
> OOoooOooOooOOoO. I have a bunch of them in the API headers. Will remove.</span >
Although we do seem to have one in WKUIDelegate.h, but I will still remove the ones I added.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>