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

Side by Side Diff: components/content_settings/core/common/permission_request_id.cc

Issue 1158813002: Use RenderFrameHost for ::RequestPermission() and ::CancelPermission(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: build fixes 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/content_settings/core/common/permission_request_id.h" 5 #include "components/content_settings/core/common/permission_request_id.h"
6 6
7 #include "base/strings/stringprintf.h" 7 #include "base/strings/stringprintf.h"
8 8
9 PermissionRequestID::PermissionRequestID(int render_process_id, 9 PermissionRequestID::PermissionRequestID(int render_process_id,
10 int render_view_id, 10 int render_frame_id,
11 int bridge_id, 11 int bridge_id,
12 const GURL& origin) 12 const GURL& origin)
13 : render_process_id_(render_process_id), 13 : render_process_id_(render_process_id),
14 render_view_id_(render_view_id), 14 render_frame_id_(render_frame_id),
15 bridge_id_(bridge_id), 15 bridge_id_(bridge_id),
16 origin_(origin) { 16 origin_(origin) {
17 } 17 }
18 18
19 PermissionRequestID::~PermissionRequestID() { 19 PermissionRequestID::~PermissionRequestID() {
20 } 20 }
21 21
22 bool PermissionRequestID::Equals(const PermissionRequestID& other) const { 22 bool PermissionRequestID::Equals(const PermissionRequestID& other) const {
23 return IsForSameTabAs(other) && (bridge_id_ == other.bridge_id_) && 23 return render_process_id_ == other.render_process_id_ &&
24 (origin_ == other.origin()); 24 render_frame_id_ == other.render_frame_id_ &&
25 } 25 bridge_id_ == other.bridge_id_ &&
26 26 origin_ == other.origin_;
27 bool PermissionRequestID::IsForSameTabAs(
28 const PermissionRequestID& other) const {
29 return (render_process_id_ == other.render_process_id_) &&
30 (render_view_id_ == other.render_view_id_);
31 } 27 }
32 28
33 std::string PermissionRequestID::ToString() const { 29 std::string PermissionRequestID::ToString() const {
34 return base::StringPrintf("%d,%d,%d,%s", 30 return base::StringPrintf("%d,%d,%d,%s",
35 render_process_id_, 31 render_process_id_,
36 render_view_id_, 32 render_frame_id_,
37 bridge_id_, 33 bridge_id_,
38 origin_.spec().c_str()); 34 origin_.spec().c_str());
39 } 35 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698