When we detect a PDF with an unsupported feature, ask the user if they want t...

webkit/plugins/ppapi/ppapi_plugin_instance.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/plugins/ppapi/ppapi_plugin_instance.h"
 
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/scoped_ptr.h"
 #include "base/utf_string_conversions.h"
@@ skipping 232 matching lines @@
   &IsFullscreen,
   &SetFullscreen,
 };
 
 void ZoomChanged(PP_Instance instance_id, double factor) {
   PluginInstance* instance = ResourceTracker::Get()->GetInstance(instance_id);
   if (!instance)
     return;
 
   // We only want to tell the page to change its zoom if the whole page is the
   // PDF.  If we're in an iframe, then don't do anything.

[brettw, 2011/01/21 21:42:30]

Can you remove the reference to PDF in this comment?

-  WebFrame* frame = instance->container()->element().document().frame();
-  if (!frame->view()->mainFrame()->document().isPluginDocument())
+  if (!instance->IsFullPagePlugin())
     return;
 
   double zoom_level = WebView::zoomFactorToZoomLevel(factor);
   // The conversino from zoom level to factor, and back, can introduce rounding
   // errors.  i.e. WebKit originally tells us 3.0, but by the time we tell the
   // plugin and it tells us back, the level becomes 3.000000000004.  Need to
   // round or else otherwise if the user zooms out, it will go to 3.0 instead of
   // 2.0.
   int rounded =
       static_cast<int>(zoom_level + (zoom_level > 0 ? 0.001 : -0.001));
@@ skipping 958 matching lines @@
 }
 
 ObjectVar* PluginInstance::ObjectVarForNPObject(NPObject* np_object) const {
   NPObjectToObjectVarMap::const_iterator found =
       np_object_to_object_var_.find(np_object);
   if (found == np_object_to_object_var_.end())
     return NULL;
   return found->second;
 }
 
+bool PluginInstance::IsFullPagePlugin() const {
+  WebFrame* frame = container()->element().document().frame();
+  return frame->view()->mainFrame()->document().isPluginDocument();
+}
+
 }  // namespace ppapi
 }  // namespace webkit