Plugin Power Saver: Add comprehensive browser tests.

ppapi/tests/power_saver_test_plugin.cc

-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright 2015 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 <algorithm>
 
 #include "ppapi/cpp/instance.h"
 #include "ppapi/cpp/module.h"
 #include "ppapi/cpp/var.h"
+#include "ppapi/cpp/var_dictionary.h"
+#include "ppapi/tests/test_utils.h"
 
 // When compiling natively on Windows, PostMessage can be #define-d to
 // something else.
 #ifdef PostMessage
 #undef PostMessage
 #endif
 
 // This is a simple C++ Pepper plugin that demonstrates HandleMessage and
 // PostMessage.

[raymes, 2015/04/20 01:05:32]

nit: please change the comment

[tommycli, 2015/04/20 18:04:00]

Done.

 
 // This object represents one time the page says <embed>.

[raymes, 2015/04/20 01:05:31]

nit remove the comment

[tommycli, 2015/04/20 18:04:00]

Done.

-class MyInstance : public pp::Instance {
+class PowerSaverTestInstance : public pp::Instance {
  public:
-  explicit MyInstance(PP_Instance instance) : pp::Instance(instance) {}
-  virtual ~MyInstance() {}
+  explicit PowerSaverTestInstance(PP_Instance instance)
+      : pp::Instance(instance) {}
+  virtual ~PowerSaverTestInstance() {}
   virtual void HandleMessage(const pp::Var& message_data);
 };
 
 // HandleMessage gets invoked when postMessage is called on the DOM element
 // associated with this plugin instance.
 // In this case, if we are given a string, we'll post a message back to
 // JavaScript indicating whether or not that string is a palindrome.

[raymes, 2015/04/20 01:05:31]

nit: please change the comment

[tommycli, 2015/04/20 18:04:00]

Done.

-void MyInstance::HandleMessage(const pp::Var& message_data) {
-  if (message_data.is_string()) {
-    std::string string_copy(message_data.AsString());
-    std::reverse(string_copy.begin(), string_copy.end());
-    bool is_palindrome(message_data.AsString() == string_copy);
-
-    PostMessage(pp::Var(is_palindrome));
+void PowerSaverTestInstance::HandleMessage(const pp::Var& message_data) {
+  if (message_data.is_string() && (message_data.AsString() == "isPeripheral")) {
+    pp::VarDictionary message;
+    message.Set(
+        "isPeripheral",
+        pp::Var(PP_ToBool(GetTestingInterface()->IsPeripheral(pp_instance()))));
+    PostMessage(message);
   }
 }
 
 // This object is the global object representing this plugin library as long
 // as it is loaded.

[raymes, 2015/04/20 01:05:32]

nit remove the comment

[tommycli, 2015/04/20 18:04:00]

Done.

-class MyModule : public pp::Module {
+class PowerSaverTestModule : public pp::Module {
  public:
-  MyModule() : pp::Module() {}
-  virtual ~MyModule() {}
+  PowerSaverTestModule() : pp::Module() {}
+  virtual ~PowerSaverTestModule() {}
 
   // Override CreateInstance to create your customized Instance object.

[raymes, 2015/04/20 01:05:31]

nit remove the comment

[tommycli, 2015/04/20 18:04:00]

Done.

   virtual pp::Instance* CreateInstance(PP_Instance instance) {
-    return new MyInstance(instance);
+    return new PowerSaverTestInstance(instance);
   }
 };
 
 namespace pp {
 
 // Factory function for your specialization of the Module object.

[raymes, 2015/04/20 01:05:31]

nit: remove the comment

[tommycli, 2015/04/20 18:04:01]

Done.

 Module* CreateModule() {
-  return new MyModule();
+  return new PowerSaverTestModule();
 }
 
 }  // namespace pp
-