mac: Remove a few symbols from framework.order

mac: Remove a few symbols from framework.order

We don't support NPAPI any more, so we don't export the NPAPI symbols.

I'm not sure why we needed the two narrowPrecision lines, but they've
been here since this file got added and I don't see them anymore in my local
build, so they're probably safe to remove. (If not, please revert this CL and
point me to a bot where they're needed.)

The NaCl symbols have been marked as hidden since
https://codereview.chromium.org/293009 (which landed before the CL that added
the two NaCl lines to this file, https://codereview.chromium.org/464046
– but the motivation in that CL is a coverage build, which we don't have anymore
anyways).

In the same vein, remove all the gcov symbols since we don't have a code coverage
build any more. If needed, we can add them back later.

Also remove _catch_exception_raise which was needed for breakpad. Now that
we use crashpad, this shouldn't be necessary.

No intended behavior change.

BUG=none

Committed: https://crrev.com/221bdb216bfeb84445ba9d90652a00e470c4690b
Cr-Commit-Position: refs/heads/master@{#349761}

[Nico, 2015-09-18 19:40:52]

thakis@chromium.org changed reviewers:
+ mark@chromium.org

[Mark Mentovai, 2015-09-18 20:15:11]

LGTM, but maybe you can be even more aggressive. OK to split it up if you’re
unsure about any of these.

https://codereview.chromium.org/1351143005/diff/20001/chrome/app/framework.order
File chrome/app/framework.order (right):

https://codereview.chromium.org/1351143005/diff/20001/chrome/app/framework.or...
chrome/app/framework.order:21: __ZnwmPv
I wonder if we need these two at all.

https://codereview.chromium.org/1351143005/diff/20001/chrome/app/framework.or...
chrome/app/framework.order:23: _catch_exception_raise
We can lose this now too. It was for Breakpad, but Crashpad’s better than that.

https://codereview.chromium.org/1351143005/diff/20001/chrome/app/framework.or...
chrome/app/framework.order:25: # List the gcov symbols so the code coverage bot
doesn't trip up in
Since you mentioned that we don’t have a coverage build anymore, we ought to be
able to kill this section.

[Nico, 2015-09-18 20:23:09]

Thanks, I removed those too.

[Nico, 2015-09-18 20:24:08]

https://codereview.chromium.org/1351143005/diff/20001/chrome/app/framework.order
File chrome/app/framework.order (right):

https://codereview.chromium.org/1351143005/diff/20001/chrome/app/framework.or...
chrome/app/framework.order:21: __ZnwmPv
On 2015/09/18 20:15:10, Mark Mentovai - August is over wrote:
> I wonder if we need these two at all.

I didn't remove these two. They might be needed on some instrumented build. I'll
try these in a different CL, they feel more risky than the rest. (I remove the
other two things you pointed out.)

[Mark Mentovai, 2015-09-18 20:24:43]

Cool. I’m less certain about operator new and operator delete also.

[Nico, 2015-09-18 20:27:33]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2015-09-18 20:27:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1351143005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1351143005/20001

[Nico, 2015-09-18 20:28:42]

The CQ bit was unchecked by thakis@chromium.org

[Nico, 2015-09-18 20:35:55]

The CQ bit was checked by thakis@chromium.org

[Nico, 2015-09-18 20:35:55]

The patchset sent to the CQ was uploaded after l-g-t-m from mark@chromium.org
Link to the patchset: https://codereview.chromium.org/1351143005/#ps40001
(title: "actually upload")

[Nico, 2015-09-18 20:36:01]

The CQ bit was unchecked by thakis@chromium.org

[Nico, 2015-09-18 20:37:22]

The CQ bit was checked by thakis@chromium.org

[Nico, 2015-09-18 20:37:22]

The patchset sent to the CQ was uploaded after l-g-t-m from mark@chromium.org
Link to the patchset: https://codereview.chromium.org/1351143005/#ps60001
(title: "bleh")

[commit-bot: I haz the power, 2015-09-18 20:37:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1351143005/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1351143005/60001

[commit-bot: I haz the power, 2015-09-18 20:49:26]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-09-18 20:50:06]

Patchset 4 (id:??) landed as
https://crrev.com/221bdb216bfeb84445ba9d90652a00e470c4690b
Cr-Commit-Position: refs/heads/master@{#349761}