Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(521)

Side by Side Diff: chrome/browser/ui/pdf/pdf_unsupported_feature.cc

Issue 24649002: Clean up a few more unused globals. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: thestig comment Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/pdf/pdf_unsupported_feature.h" 5 #include "chrome/browser/ui/pdf/pdf_unsupported_feature.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/prefs/pref_service.h" 9 #include "base/prefs/pref_service.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 using content::InterstitialPage; 47 using content::InterstitialPage;
48 using content::OpenURLParams; 48 using content::OpenURLParams;
49 using content::PluginService; 49 using content::PluginService;
50 using content::Referrer; 50 using content::Referrer;
51 using content::UserMetricsAction; 51 using content::UserMetricsAction;
52 using content::WebContents; 52 using content::WebContents;
53 using content::WebPluginInfo; 53 using content::WebPluginInfo;
54 54
55 namespace { 55 namespace {
56 56
57 static const char kAdobeReaderIdentifier[] = "adobe-reader"; 57 const char kAdobeReaderUpdateUrl[] = "http://www.adobe.com/go/getreader_chrome";
58 static const char kAdobeReaderUpdateUrl[] = 58
59 "http://www.adobe.com/go/getreader_chrome"; 59 #if defined(OS_WIN) && defined(ENABLE_PLUGIN_INSTALLATION)
60 const char kAdobeReaderIdentifier[] = "adobe-reader";
61 #endif
60 62
61 // The prompt delegate used to ask the user if they want to use Adobe Reader 63 // The prompt delegate used to ask the user if they want to use Adobe Reader
62 // by default. 64 // by default.
63 class PDFEnableAdobeReaderPromptDelegate 65 class PDFEnableAdobeReaderPromptDelegate
64 : public OpenPDFInReaderPromptDelegate { 66 : public OpenPDFInReaderPromptDelegate {
65 public: 67 public:
66 explicit PDFEnableAdobeReaderPromptDelegate(Profile* profile); 68 explicit PDFEnableAdobeReaderPromptDelegate(Profile* profile);
67 virtual ~PDFEnableAdobeReaderPromptDelegate(); 69 virtual ~PDFEnableAdobeReaderPromptDelegate();
68 70
69 // OpenPDFInReaderPromptDelegate 71 // OpenPDFInReaderPromptDelegate
(...skipping 327 matching lines...) Expand 10 before | Expand all | Expand 10 after
397 399
398 void PDFHasUnsupportedFeature(content::WebContents* web_contents) { 400 void PDFHasUnsupportedFeature(content::WebContents* web_contents) {
399 #if defined(OS_WIN) && defined(ENABLE_PLUGIN_INSTALLATION) 401 #if defined(OS_WIN) && defined(ENABLE_PLUGIN_INSTALLATION)
400 // Only works for Windows for now. For Mac, we'll have to launch the file 402 // Only works for Windows for now. For Mac, we'll have to launch the file
401 // externally since Adobe Reader doesn't work inside Chrome. 403 // externally since Adobe Reader doesn't work inside Chrome.
402 PluginService::GetInstance()->GetPlugins(base::Bind(&GotPluginsCallback, 404 PluginService::GetInstance()->GetPlugins(base::Bind(&GotPluginsCallback,
403 web_contents->GetRenderProcessHost()->GetID(), 405 web_contents->GetRenderProcessHost()->GetID(),
404 web_contents->GetRenderViewHost()->GetRoutingID())); 406 web_contents->GetRenderViewHost()->GetRoutingID()));
405 #endif 407 #endif
406 } 408 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698