Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(220)

Side by Side Diff: components/guest_view/renderer/guest_view_request.cc

Issue 1159553007: Move Tuple to base namespace. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/guest_view/renderer/guest_view_request.h" 5 #include "components/guest_view/renderer/guest_view_request.h"
6 6
7 #include "components/guest_view/common/guest_view_messages.h" 7 #include "components/guest_view/common/guest_view_messages.h"
8 #include "components/guest_view/renderer/guest_view_container.h" 8 #include "components/guest_view/renderer/guest_view_container.h"
9 #include "content/public/renderer/render_frame.h" 9 #include "content/public/renderer/render_frame.h"
10 #include "content/public/renderer/render_view.h" 10 #include "content/public/renderer/render_view.h"
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 74
75 void GuestViewAttachRequest::HandleResponse(const IPC::Message& message) { 75 void GuestViewAttachRequest::HandleResponse(const IPC::Message& message) {
76 // TODO(fsamuel): Rename this message so that it's apparent that this is a 76 // TODO(fsamuel): Rename this message so that it's apparent that this is a
77 // response to GuestViewHostMsg_AttachGuest. Perhaps 77 // response to GuestViewHostMsg_AttachGuest. Perhaps
78 // GuestViewMsg_AttachGuest_ACK? 78 // GuestViewMsg_AttachGuest_ACK?
79 GuestViewMsg_GuestAttached::Param param; 79 GuestViewMsg_GuestAttached::Param param;
80 if (!GuestViewMsg_GuestAttached::Read(&message, &param)) 80 if (!GuestViewMsg_GuestAttached::Read(&message, &param))
81 return; 81 return;
82 82
83 content::RenderView* guest_proxy_render_view = 83 content::RenderView* guest_proxy_render_view =
84 content::RenderView::FromRoutingID(get<1>(param)); 84 content::RenderView::FromRoutingID(base::get<1>(param));
85 // TODO(fsamuel): Should we be reporting an error to JavaScript or DCHECKing? 85 // TODO(fsamuel): Should we be reporting an error to JavaScript or DCHECKing?
86 if (!guest_proxy_render_view) 86 if (!guest_proxy_render_view)
87 return; 87 return;
88 88
89 v8::HandleScope handle_scope(isolate()); 89 v8::HandleScope handle_scope(isolate());
90 blink::WebFrame* frame = guest_proxy_render_view->GetWebView()->mainFrame(); 90 blink::WebFrame* frame = guest_proxy_render_view->GetWebView()->mainFrame();
91 v8::Local<v8::Value> window = frame->mainWorldScriptContext()->Global(); 91 v8::Local<v8::Value> window = frame->mainWorldScriptContext()->Global();
92 92
93 const int argc = 1; 93 const int argc = 1;
94 scoped_ptr<v8::Local<v8::Value>[]> argv(new v8::Local<v8::Value>[argc]); 94 scoped_ptr<v8::Local<v8::Value>[]> argv(new v8::Local<v8::Value>[argc]);
(...skipping 19 matching lines...) Expand all
114 return; 114 return;
115 115
116 container()->render_frame()->DetachGuest(container()->element_instance_id()); 116 container()->render_frame()->DetachGuest(container()->element_instance_id());
117 } 117 }
118 118
119 void GuestViewDetachRequest::HandleResponse(const IPC::Message& message) { 119 void GuestViewDetachRequest::HandleResponse(const IPC::Message& message) {
120 ExecuteCallbackIfAvailable(0 /* argc */, nullptr); 120 ExecuteCallbackIfAvailable(0 /* argc */, nullptr);
121 } 121 }
122 122
123 } // namespace guest_view 123 } // namespace guest_view
OLDNEW
« no previous file with comments | « components/gcm_driver/gcm_driver_desktop.cc ('k') | components/login/base_screen_handler_utils.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698