This patch makes chromium (as well as a random selection of test binaries I tried: media_unittests net_unittests browser_tests ui_tests test_shell_tests) successfully link in shared_library mode with no workarounds necessary in the chromium codebase.  Yay!<div>

<br></div><div>-a<br><br><div class="gmail_quote">On Wed, Jan 4, 2012 at 5:25 PM, Rafael Ávila de Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The attached patch should fix both pr10113 and pr11690.<br>
<br>
It remembers if a type has its visibility set explicitly or not and<br>
centralizes the way we merge visibility. We always prefer explicit over<br>
implicit and then pick the most restrictive visibility.<br>
<br>
This is still more strict than gcc for cases like<br>
<br>
-----------------------------------------<br>
template<class T> class __attribute__((visibility("default"))) foo {<br>
  void bar() { }<br>
};<br>
class __attribute__((visibility("hidden"))) zed;<br>
template class foo<zed>;<br>
----------------------------------------<br>
<br>
but should be compatible in any case the type could be used outside of<br>
this library.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>