Title: [92267] branches/chromium/835/Source/WebCore/inspector/InspectorPageAgent.cpp
- Revision
- 92267
- Author
- [email protected]
- Date
- 2011-08-03 03:18:58 -0700 (Wed, 03 Aug 2011)
Log Message
Merge 91901 - Web Inspector: [REGRESSION] [Chromium] Opening link in new tab does not work properly.
https://bugs.webkit.org/show_bug.cgi?id=65279
Reviewed by Pavel Feldman.
* inspector/InspectorPageAgent.cpp:
(WebCore::InspectorPageAgent::open):
[email protected]
BUG=87345
Review URL: http://codereview.chromium.org/7565004
Modified Paths
Diff
Modified: branches/chromium/835/Source/WebCore/inspector/InspectorPageAgent.cpp (92266 => 92267)
--- branches/chromium/835/Source/WebCore/inspector/InspectorPageAgent.cpp 2011-08-03 10:17:31 UTC (rev 92266)
+++ branches/chromium/835/Source/WebCore/inspector/InspectorPageAgent.cpp 2011-08-03 10:18:58 UTC (rev 92267)
@@ -299,6 +299,8 @@
void InspectorPageAgent::open(ErrorString*, const String& url, const bool* const inNewWindow)
{
+ UserGestureIndicator indicator(DefinitelyProcessingUserGesture);
+
Frame* mainFrame = m_page->mainFrame();
Frame* frame;
if (inNewWindow && *inNewWindow) {
@@ -315,7 +317,6 @@
} else
frame = mainFrame;
- UserGestureIndicator indicator(DefinitelyProcessingUserGesture);
// FIXME: Why does one use mainFrame and the other frame?
frame->loader()->changeLocation(mainFrame->document()->securityOrigin(), frame->document()->completeURL(url), "", false, false);
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
