[unseasoned-pdf] Implement content rotation through the context menu

Meanwhile, change blink::WebPlugin::RotationType to an enum class.

Change-Id: Iccd2782c5d57efa242c4364afd60d6efaa790f36
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3021686
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Bill Budge <bbudge@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Daniel Hosseinian <dhoss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#901326}

Author: Daniel Hosseinian

content/renderer/pepper/pepper_plugin_instance_impl.cc

@@ -1945,7 +1945,7 @@ void PepperPluginInstanceImpl::RotateView(WebPlugin::RotationType type) {
   if (!LoadPdfInterface())
     return;
   PP_PrivatePageTransformType transform_type =
-      type == WebPlugin::kRotationType90Clockwise
+      type == WebPlugin::RotationType::k90Clockwise
           ? PP_PRIVATEPAGETRANSFORMTYPE_ROTATE_90_CW
           : PP_PRIVATEPAGETRANSFORMTYPE_ROTATE_90_CCW;
   plugin_pdf_interface_->Transform(pp_instance(), transform_type);

content/renderer/pepper/pepper_webplugin_impl.cc

@@ -461,7 +461,7 @@ bool PepperWebPluginImpl::CanRotateView() {
   return instance_->CanRotateView();
 }
 
-void PepperWebPluginImpl::RotateView(RotationType type) {
+void PepperWebPluginImpl::RotateView(blink::WebPlugin::RotationType type) {
   // Re-entrancy may cause JS to try to execute script on the plugin before it
   // is fully initialized. See: crbug.com/715747.
   if (instance_)

content/renderer/pepper/pepper_webplugin_impl.h

@@ -81,7 +81,7 @@ class PepperWebPluginImpl : public blink::WebPlugin {
   void PrintEnd() override;
 
   bool CanRotateView() override;
-  void RotateView(RotationType type) override;
+  void RotateView(blink::WebPlugin::RotationType type) override;
   bool IsPlaceholder() override;
   void DidLoseMouseLock() override;
   void DidReceiveMouseLockResult(bool success) override;

pdf/pdf_view_web_plugin.cc

@@ -496,6 +496,23 @@ void PdfViewWebPlugin::StopFind() {
   // dismissed.
 }
 
+bool PdfViewWebPlugin::CanRotateView() {
+  return !IsPrintPreview();
+}
+
+void PdfViewWebPlugin::RotateView(blink::WebPlugin::RotationType type) {
+  DCHECK(CanRotateView());
+
+  switch (type) {
+    case blink::WebPlugin::RotationType::k90Clockwise:
+      engine()->RotateClockwise();
+      break;
+    case blink::WebPlugin::RotationType::k90Counterclockwise:
+      engine()->RotateCounterclockwise();
+      break;
+  }
+}
+
 blink::WebTextInputType PdfViewWebPlugin::GetPluginTextInputType() {
   return text_input_type_;
 }

pdf/pdf_view_web_plugin.h

@@ -167,6 +167,8 @@ class PdfViewWebPlugin final : public PdfViewPluginBase,
                  int identifier) override;
   void SelectFindResult(bool forward, int identifier) override;
   void StopFind() override;
+  bool CanRotateView() override;
+  void RotateView(blink::WebPlugin::RotationType type) override;
   blink::WebTextInputType GetPluginTextInputType() override;
 
   // PdfViewPluginBase:

third_party/blink/public/web/web_plugin.h

@@ -236,10 +236,7 @@ class WebPlugin {
   virtual void StopFind() {}
 
   // View rotation types.
-  enum RotationType {
-    kRotationType90Clockwise,
-    kRotationType90Counterclockwise
-  };
+  enum class RotationType { k90Clockwise, k90Counterclockwise };
   // Whether the plugin can rotate the view of its content.
   virtual bool CanRotateView() { return false; }
   // Rotates the plugin's view of its content.

third_party/blink/renderer/core/frame/local_frame_mojo_receiver.cc

@@ -205,11 +205,11 @@ void LocalFrameMojoReceiver::PluginActionAt(
 
   switch (action) {
     case mojom::blink::PluginActionType::kRotate90Clockwise:
-      plugin_view->Plugin()->RotateView(WebPlugin::kRotationType90Clockwise);
+      plugin_view->Plugin()->RotateView(WebPlugin::RotationType::k90Clockwise);
       return;
     case mojom::blink::PluginActionType::kRotate90Counterclockwise:
       plugin_view->Plugin()->RotateView(
-          WebPlugin::kRotationType90Counterclockwise);
+          WebPlugin::RotationType::k90Counterclockwise);
       return;
   }
   NOTREACHED();