Parameterize OnError method.

third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp

Index: third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp
diff --git a/third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp b/third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp
index 1d2293ef7873e3b0d35642af25933e4ff78a6c85..78eb279927005826d1b7845c41953ba2a742db55 100644
--- a/third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp
+++ b/third_party/WebKit/Source/modules/payments/PaymentTestHelper.cpp
@@ -153,24 +153,36 @@ PaymentRequestMockFunctionScope::~PaymentRequestMockFunctionScope()
     }
 }
 
+v8::Local<v8::Function> PaymentRequestMockFunctionScope::expectCall(String* captor)
+{
+    m_mockFunctions.append(new MockFunction(m_scriptState, captor));
+    return m_mockFunctions.last()->bind();
+}
+
 v8::Local<v8::Function> PaymentRequestMockFunctionScope::expectCall()
 {
     m_mockFunctions.append(new MockFunction(m_scriptState));
-    EXPECT_CALL(*m_mockFunctions.last(), call(testing::_));
+    // EXPECT_CALL(*m_mockFunctions.last(), call(testing::_));

[pals, 2016/07/12 11:42:56]

As we have overridden the "call" method, EXPECT_CALL does not work. We can set a
string (called/not called) here, but in that case we need to in many places in
test files. Is there any better way to do it? Shall I create a separate class to
test this change similar to [1] instead of modifying
PaymentRequestMockFunctionScope class?

[pals, 2016/07/12 11:45:01]

[1]
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/streams/R...

[please use gerrit instead, 2016/07/12 15:57:35]

Uncomment

[pals, 2016/07/13 05:54:27]

Done.

     return m_mockFunctions.last()->bind();
 }
 
 v8::Local<v8::Function> PaymentRequestMockFunctionScope::expectNoCall()
 {
     m_mockFunctions.append(new MockFunction(m_scriptState));
-    EXPECT_CALL(*m_mockFunctions.last(), call(testing::_)).Times(0);
+    // EXPECT_CALL(*m_mockFunctions.last(), call(testing::_)).Times(0);

[please use gerrit instead, 2016/07/12 15:57:35]

Uncomment

[pals, 2016/07/13 05:54:28]

Done.

     return m_mockFunctions.last()->bind();
 }
 

[please use gerrit instead, 2016/07/12 15:57:35]

Put the following code before MockFunction() constructor here:

ACTION_P(SaveVaueIn, captor) {
  *captor = arg0;
}

https://github.com/google/googlemock/blob/master/googlemock/docs/CheatSheet.m...

[pals, 2016/07/13 05:54:28]

Nice. Thank you.

Done. Thanks

 PaymentRequestMockFunctionScope::MockFunction::MockFunction(ScriptState* scriptState)
     : ScriptFunction(scriptState)
 {
-    ON_CALL(*this, call(testing::_)).WillByDefault(testing::ReturnArg<0>());
+    // ON_CALL(*this, call(testing::_)).WillByDefault(testing::ReturnArg<0>());

[please use gerrit instead, 2016/07/12 15:57:35]

Change this line to the following:

ON_CALL(*this, call(testing::_)).WillByDefault(
        testing::DoAll(SaveValueIn(m_value), testing::ReturnArg<0>()));

[pals, 2016/07/13 05:54:27]

Done.

+}
+
+PaymentRequestMockFunctionScope::MockFunction::MockFunction(ScriptState* scriptState, String* captor)
+    : ScriptFunction(scriptState)
+    , m_value(captor)
+{
 }
 
 v8::Local<v8::Function> PaymentRequestMockFunctionScope::MockFunction::bind()