chromeos: Remove GViewRequestInterceptor.

chromeos: Remove GViewRequestInterceptor.

This removes the code for sending the user to GView for PDF
and .ppt files.  The --enable-gview flag has been passed
unconditionally on Chrome OS for a long time, but we ignore
it in official builds.  For Chromium OS, I believe that
https://chrome.google.com/webstore/detail/docs-pdfpowerpoint-viewer/nnbmlagghjjcbdhgmkedmbmedengocbn
provides similar functionality.

BUG=161786
TBR=ben@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=170875

[zel, 2012-11-19 17:31:16]

lgtm

[willchan no longer on Chromium, 2012-11-19 17:31:41]

chrome/browser/profiles LGTM

[Ben Goodger (Google), 2012-11-19 17:35:47]

lgtm

[commit-bot: I haz the power, 2012-11-19 19:27:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/11411071/1

[commit-bot: I haz the power, 2012-11-19 22:00:12]

Retried try job too often for step(s) browser_tests

[commit-bot: I haz the power, 2012-11-19 22:06:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/11411071/1

[commit-bot: I haz the power, 2012-11-20 00:43:08]

Retried try job too often for step(s) browser_tests

[commit-bot: I haz the power, 2012-11-20 02:49:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/11411071/1

[commit-bot: I haz the power, 2012-11-20 05:10:31]

Retried try job too often for step(s) browser_tests

[Daniel Erat, 2012-11-21 00:56:53]

On 2012/11/20 05:10:31, I haz the power (commit-bot) wrote:
> Retried try job too often for step(s) browser_tests

This fails, presumably because ExtensionWebRequestApiTest.WebRequestDeclarative
doesn't expect any interceptors to be running:

Note: Google Test filter = ExtensionWebRequestApiTest.WebRequestDeclarative
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ExtensionWebRequestApiTest, where TypeParam = 
[ RUN      ] ExtensionWebRequestApiTest.WebRequestDeclarative
HTTP server started on 127.0.0.1:41985...
sending server_data: {"host": "127.0.0.1", "port": 41985} (36 bytes)
[14293:14532:1119/210843:4604658950:ERROR:nss_util.cc(491)] Error initializing
NSS with a persistent database (sql:/etc/fake_root_ca/nssdb): NSS error code:
-8174
Xlib:  extension "GLX" missing on display ":9.0".
[14293:14293:1119/210843:4604666755:ERROR:gl_surface_glx.cc(378)]
glxQueryVersion failed
[14293:14293:1119/210843:4604666811:ERROR:gl_surface_linux.cc(58)]
GLSurfaceGLX::InitializeOneOff failed.
[14293:14293:1119/210843:4604666847:ERROR:compositor.cc(144)] Could not load the
GL bindings
[14293:14293:1119/210843:4604762923:ERROR:cros_user_policy_cache.cc(147)] Failed
to decode policy
Xlib:  extension "RANDR" missing on display ":9.0".
[14293:14532:1119/210846:4607226001:FATAL:url_request_http_job.cc(965)] Check
failed: transaction_.get(). 
[14293:14532:1119/210846:4607226001:FATAL:url_request_http_job.cc(965)] Check
failed: transaction_.get(). 
Received signal 6
	base::debug::StackTrace::StackTrace() [0x7ffa3a91c04e]
	base::debug::(anonymous namespace)::StackDumpSignalHandler() [0x7ffa3a91c185]
	<unknown> [0x7ffa32e85af0]
	<unknown> [0x7ffa32e85a75]
	<unknown> [0x7ffa32e895c0]
	base::debug::BreakDebugger() [0x7ffa3a91b499]
	logging::LogMessage::~LogMessage() [0x7ffa3a93e6b6]
	net::URLRequestHttpJob::GetMimeType() [0x7ffa3a646b7a]
	net::URLRequest::GetMimeType() [0x7ffa3a6367f4]
	chromeos::GViewRequestInterceptor::MaybeInterceptResponse() [0x1dbd20e]
	net::URLRequestJobFactoryImpl::MaybeInterceptResponse() [0x7ffa3a653326]
	net::URLRequestJobManager::MaybeInterceptResponse() [0x7ffa3a654143]
	net::URLRequest::NotifyResponseStarted() [0x7ffa3a63af45]
	net::URLRequestJob::NotifyStartError() [0x7ffa3a651053]
	net::URLRequestHttpJob::MaybeStartTransactionInternal() [0x7ffa3a6491ae]
	net::URLRequestHttpJob::OnCookiesLoaded() [0x7ffa3a649321]
	net::CookieMonster::CookieMonsterTask::InvokeCallback() [0x7ffa3a48ed47]
	net::CookieMonster::GetCookiesWithInfoTask::Run() [0x7ffa3a49832b]
	net::CookieMonster::DoCookieTaskForURL() [0x7ffa3a499238]
	net::CookieMonster::GetCookiesWithInfoAsync() [0x7ffa3a499871]
	net::URLRequestHttpJob::DoLoadCookies() [0x7ffa3a648dc5]
	net::CookieMonster::CookieMonsterTask::InvokeCallback() [0x7ffa3a48ed47]
	net::CookieMonster::GetAllCookiesForURLWithOptionsTask::Run() [0x7ffa3a49b0a6]
	net::CookieMonster::DoCookieTaskForURL() [0x7ffa3a499238]
	net::CookieMonster::GetAllCookiesForURLAsync() [0x7ffa3a499ccf]
	net::URLRequestHttpJob::AddCookieHeaderAndStart() [0x7ffa3a6494f5]
	net::URLRequestHttpJob::Start() [0x7ffa3a64c3cb]
	net::URLRequest::StartJob() [0x7ffa3a63ae14]
	net::URLRequest::RestartWithJob() [0x7ffa3a63aedc]
	net::URLRequest::Restart() [0x7ffa3a63b13a]
	net::URLRequestJob::NotifyRestartRequired() [0x7ffa3a650fb0]
	appcache::AppCacheURLRequestJob::BeginDelivery() [0x7ffa37974970]
	MessageLoop::RunTask() [0x7ffa3a945e32]
	MessageLoop::DeferOrRunPendingTask() [0x7ffa3a9464bb]
	MessageLoop::DoWork() [0x7ffa3a946878]
	base::MessagePumpLibevent::Run() [0x7ffa3a907b4a]
	MessageLoop::RunInternal() [0x7ffa3a9417c8]
	base::RunLoop::Run() [0x7ffa3a96561d]
	MessageLoop::Run() [0x7ffa3a9408e5]
	content::BrowserThreadImpl::IOThreadRun() [0x7ffa38431408]
	content::BrowserThreadImpl::Run() [0x7ffa384321cb]
	base::Thread::ThreadMain() [0x7ffa3a98a459]
	base::(anonymous namespace)::ThreadFunc() [0x7ffa3a982f32]
	start_thread [0x7ffa343839ca]
	<unknown> [0x7ffa32f38cdd]

[Daniel Erat, 2012-12-03 23:55:39]

Another look?  I'm just deleting GViewRequestInterceptor now.

[willchan no longer on Chromium, 2012-12-04 00:20:23]

profiles/ lgtm still

[commit-bot: I haz the power, 2012-12-04 00:43:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/11411071/16001

[commit-bot: I haz the power, 2012-12-04 03:55:38]

Change committed as 170875