Removes crash keys for crbug.com/621730

third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 61 matching lines @@
 #include "core/page/CreateWindow.h"
 #include "core/page/Page.h"
 #include "core/page/WindowFeatures.h"
 #include "core/page/scrolling/ScrollingCoordinator.h"
 #include "platform/EventDispatchForbiddenScope.h"
 #include "platform/weborigin/SecurityOrigin.h"
 #include "platform/weborigin/Suborigin.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebFrameScheduler.h"
 #include "public/platform/WebScreenInfo.h"
 #include "wtf/debug/CrashLogging.h"

[Robert Sesek, 2016/07/08 15:01:02]

Same.

[Yuki, 2016/07/11 04:00:43]

Done.

 #include <memory>
 
 namespace blink {
 
 // Rather than simply inheriting LocalFrameLifecycleObserver like most other
 // classes, LocalDOMWindow hides its LocalFrameLifecycleObserver with
 // composition. This prevents conflicting overloads between DOMWindow, which
 // has a frame() accessor that returns Frame* for bindings code, and
 // LocalFrameLifecycleObserver, which has a frame() accessor that returns a
 // LocalFrame*.
@@ skipping 588 matching lines @@
 {
     if (!isCurrentlyDisplayedInFrame())
         return;
 
     MessageEvent* event = timer->event();
 
     UserGestureIndicator gestureIndicator(timer->userGestureToken());
 
     event->entangleMessagePorts(document());
 
-    // Temporary instrumentation for http://crbug.com/621730.
-    WTF::debug::ScopedCrashKey("postmessage_src_origin", event->origin().utf8().data());
-    WTF::debug::ScopedCrashKey("postmessage_dst_origin", document()->getSecurityOrigin()->toRawString().utf8().data());
-    WTF::debug::ScopedCrashKey("postmessage_dst_url", document()->url().getString().utf8().data());
-
     dispatchMessageEventWithOriginCheck(timer->targetOrigin(), event, timer->takeLocation());
 }
 
 void LocalDOMWindow::removePostMessageTimer(PostMessageTimer* timer)
 {
     m_postMessageTimers.remove(timer);
 }
 
 void LocalDOMWindow::dispatchMessageEventWithOriginCheck(SecurityOrigin* intendedTargetOrigin, Event* event, std::unique_ptr<SourceLocation> location)
 {
@@ skipping 850 matching lines @@
 {
     // If the LocalDOMWindow still has a frame reference, that frame must point
     // back to this LocalDOMWindow: otherwise, it's easy to get into a situation
     // where script execution leaks between different LocalDOMWindows.
     if (m_frameObserver->frame())
         ASSERT_WITH_SECURITY_IMPLICATION(m_frameObserver->frame()->domWindow() == this);
     return m_frameObserver->frame();
 }
 
 } // namespace blink