Add an IPC method for Blink to call RapporService::RecordSample()

content/renderer/renderer_blink_platform_impl.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_RENDERER_RENDERER_BLINK_PLATFORM_IMPL_H_
 #define CONTENT_RENDERER_RENDERER_BLINK_PLATFORM_IMPL_H_
 
 #include "base/compiler_specific.h"
 #include "base/id_map.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 141 matching lines @@
                               blink::WebPlatformEventListener*);
   virtual void stopListening(blink::WebPlatformEventType);
   virtual void queryStorageUsageAndQuota(
       const blink::WebURL& storage_partition,
       blink::WebStorageQuotaType,
       blink::WebStorageQuotaCallbacks);
   virtual void vibrate(unsigned int milliseconds);
   virtual void cancelVibration();
   virtual blink::WebScheduler* scheduler();
   virtual blink::WebThread* currentThread();
+  virtual void recordRappor(const char* metric, const blink::WebString& sample);

[Alexei Svitkine (slow), 2015/04/08 14:57:11]

Maybe add the same comment about Rappor here that I suggest above.

[kojii, 2015/04/08 17:31:46]

Hm, these two are virtual override of platform.h in Blink, and jam@ suggested no
blank lines for all override methods. Maybe comments should be there?
timvolodine@ has a CL for Platform to support other features, so I can suggest
there.
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

[Alexei Svitkine (slow), 2015/04/08 17:53:50]

Fair enough.

+  virtual void recordRapporURL(const char* metric, const blink::WebURL& url);
 
   // Set the PlatformEventObserverBase in |platform_event_observers_| associated
   // with |type| to |observer|. If there was already an observer associated to
   // the given |type|, it will be replaced.
   // Note that |observer| will be owned by this object after the call.
   void SetPlatformEventObserverForTesting(
       blink::WebPlatformEventType type,
       scoped_ptr<PlatformEventObserverBase> observer);
 
   // Disables the WebSandboxSupport implementation for testing.
@@ skipping 89 matching lines @@
   device::VibrationManagerPtr vibration_manager_;
 
   IDMap<PlatformEventObserverBase, IDMapOwnPointer> platform_event_observers_;
 
   DISALLOW_COPY_AND_ASSIGN(RendererBlinkPlatformImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_RENDERER_BLINK_PLATFORM_IMPL_H_