Avoid duplicate functions/code in core/inspector.

third_party/WebKit/Source/core/inspector/IdentifiersFactory.cpp

Index: third_party/WebKit/Source/core/inspector/IdentifiersFactory.cpp
diff --git a/third_party/WebKit/Source/core/inspector/IdentifiersFactory.cpp b/third_party/WebKit/Source/core/inspector/IdentifiersFactory.cpp
index f2cb7b982db5aa102552110daf00a9539febe857..d2390354dbf79ddb99f3855c92629a2473a6682c 100644
--- a/third_party/WebKit/Source/core/inspector/IdentifiersFactory.cpp
+++ b/third_party/WebKit/Source/core/inspector/IdentifiersFactory.cpp
@@ -58,6 +58,11 @@ String IdentifiersFactory::FrameId(LocalFrame* frame) {
 }
 
 // static
+String IdentifiersFactory::FrameIdSafe(LocalFrame* frame) {
+  return frame ? FrameId(frame) : "";

[pfeldman, 2017/07/21 18:10:59]

Bake it into FrameId(), return String() if nullptr is passed.

[Daniel Bratell, 2017/07/23 20:14:12]

Done.

+}
+
+// static
 LocalFrame* IdentifiersFactory::FrameById(InspectedFrames* inspected_frames,
                                           const String& frame_id) {
   bool ok;