Disable ChromeContentRendererClientTest.NaClRestriction on ARM.

Disable ChromeContentRendererClientTest.NaClRestriction on ARM.

This tests the NaCl content-type rules, which were modified for ARM
only in patch 154043 (https://chromiumcodereview.appspot.com/10898034).
As this is a temporary change with TODO(dschuff), I've just disabled
the whole test with the same TODO.

R=dschuff@chromium.org,thakis@chromium.org,sehr@chromium.org
BUG=145694
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=154276

[Iain Merrick, 2012-08-30 13:54:51]

Needed to keep the Android buildbot green.

[Nico, 2012-08-30 15:23:48]

Since it's now in two places, can you file a bug for reenabling this and
reference that bug # in the comment you added and in the original comment?

[Derek Schuff, 2012-08-30 15:59:01]

Sorry about that, and thanks.
Filed bug https://code.google.com/p/chromium/issues/detail?id=145694 and updated
BUG= in issue
On 2012/08/30 15:23:48, Nico wrote:
> Since it's now in two places, can you file a bug for reenabling this and
> reference that bug # in the comment you added and in the original comment?

[Iain Merrick, 2012-08-30 16:01:15]

I'll update the comments

[Nico, 2012-08-30 16:14:58]

Say just "BUG=145694" in the CL description, then rietveld will auto-linkify it
(if you want to be very explicit, "BUG=chromium:145694" works too). With that,
lgtm.

http://codereview.chromium.org/10910004/diff/4002/chrome/renderer/chrome_cont...
File chrome/renderer/chrome_content_renderer_client.cc (right):

http://codereview.chromium.org/10910004/diff/4002/chrome/renderer/chrome_cont...
chrome/renderer/chrome_content_renderer_client.cc:612: //
https://code.google.com/p/chromium/issues/detail?id=145694
nit: http://crbug.com/145694 is a bit shorter

[Iain Merrick, 2012-08-30 16:53:39]

http://codereview.chromium.org/10910004/diff/4002/chrome/renderer/chrome_cont...
File chrome/renderer/chrome_content_renderer_client.cc (right):

http://codereview.chromium.org/10910004/diff/4002/chrome/renderer/chrome_cont...
chrome/renderer/chrome_content_renderer_client.cc:612: //
https://code.google.com/p/chromium/issues/detail?id=145694
On 2012/08/30 16:14:58, Nico wrote:
> nit: http://crbug.com/145694 is a bit shorter

Done.

[Derek Schuff, 2012-08-30 16:55:01]

LGTM btw, not that it means anything.
On 2012/08/30 16:53:39, Iain Merrick wrote:
>
http://codereview.chromium.org/10910004/diff/4002/chrome/renderer/chrome_cont...
> File chrome/renderer/chrome_content_renderer_client.cc (right):
> 
>
http://codereview.chromium.org/10910004/diff/4002/chrome/renderer/chrome_cont...
> chrome/renderer/chrome_content_renderer_client.cc:612: //
> https://code.google.com/p/chromium/issues/detail?id=145694
> On 2012/08/30 16:14:58, Nico wrote:
> > nit: http://crbug.com/145694 is a bit shorter
> 
> Done.

[commit-bot: I haz the power, 2012-08-30 16:55:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/husky@chromium.org/10910004/8002

[commit-bot: I haz the power, 2012-08-30 21:48:58]

Change committed as 154276