Transfer date/time value to chooser as double

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index b21d9e047b9a4e914326cd43acb012f2db1e9439..1f36176e7390ee3eeed91aced512a7b21d1b0795 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -6378,11 +6378,18 @@ void RenderViewImpl::LaunchAndroidContentIntent(const GURL& intent,
 bool RenderViewImpl::openDateTimeChooser(
     const blink::WebDateTimeChooserParams& params,
     blink::WebDateTimeChooserCompletion* completion) {
+  if (date_time_picker_client_)

[Miguel Garcia, 2013/11/26 16:07:08]

Please add a comment stating when can this happen, I can't see how you can have
more than one dialog open since they are full screen. Specially within the same
renderer.

[keishi, 2013/11/27 06:58:38]

Done.
JavaScript runs while the dialog is open, and JavaScript can open date/time
dialogs.

+    return false;
   date_time_picker_client_.reset(
       new RendererDateTimePicker(this, params, completion));
   return date_time_picker_client_->Open();
 }
 
+void RenderViewImpl::DidDismissDateTimeDialog() {

[Miguel Garcia, 2013/11/26 16:07:08]

This is such a simple implementation that perhaps you can inline it in the .h
file. See my comment about the signature as well.

[keishi, 2013/11/27 06:58:38]

Done.

+  DCHECK(date_time_picker_client_);
+  date_time_picker_client_.reset(NULL);
+}
+
 WebMediaPlayer* RenderViewImpl::CreateAndroidWebMediaPlayer(
       WebFrame* frame,
       const blink::WebURL& url,