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

Side by Side Diff: Source/bindings/v8/BindingSecurity.cpp

Issue 25403004: [oilpan] Figure out lifetime of remaining Node raw pointers (Part 2) (Closed) Base URL: svn://svn.chromium.org/blink/branches/oilpan
Patch Set: Created 7 years, 2 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 | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 bool BindingSecurity::shouldAllowAccessToDOMWindow(BindingState* state, DOMWindo w* target, SecurityReportingOption reportingOption) 63 bool BindingSecurity::shouldAllowAccessToDOMWindow(BindingState* state, DOMWindo w* target, SecurityReportingOption reportingOption)
64 { 64 {
65 return target && canAccessDocument(state, target->document(), reportingOptio n); 65 return target && canAccessDocument(state, target->document(), reportingOptio n);
66 } 66 }
67 67
68 bool BindingSecurity::shouldAllowAccessToFrame(BindingState* state, Frame* targe t, SecurityReportingOption reportingOption) 68 bool BindingSecurity::shouldAllowAccessToFrame(BindingState* state, Frame* targe t, SecurityReportingOption reportingOption)
69 { 69 {
70 return target && canAccessDocument(state, target->document(), reportingOptio n); 70 return target && canAccessDocument(state, target->document(), reportingOptio n);
71 } 71 }
72 72
73 bool BindingSecurity::shouldAllowAccessToNode(BindingState* state, Node* target)
74 {
75 return target && canAccessDocument(state, target->document());
76 }
77
78 bool BindingSecurity::shouldAllowAccessToNode(BindingState* state, const Handle< Node>& node) 73 bool BindingSecurity::shouldAllowAccessToNode(BindingState* state, const Handle< Node>& node)
79 { 74 {
80 return shouldAllowAccessToNode(state, node.raw()); 75 return node && canAccessDocument(state, node->document());
81 } 76 }
82 77
83 bool BindingSecurity::allowSettingFrameSrcToJavascriptUrl(BindingState* state, H TMLFrameElementBase* frame, const String& value) 78 bool BindingSecurity::allowSettingFrameSrcToJavascriptUrl(BindingState* state, c onst Handle<HTMLFrameElementBase>& frame, const String& value)
84 { 79 {
85 return !protocolIsJavaScript(stripLeadingAndTrailingHTMLSpaces(value)) || ca nAccessDocument(state, frame->contentDocument()); 80 return !protocolIsJavaScript(stripLeadingAndTrailingHTMLSpaces(value)) || ca nAccessDocument(state, frame->contentDocument());
86 } 81 }
87 82
88 } 83 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698