[DevTools] Make resource tree model optional

third_party/WebKit/Source/devtools/front_end/console/ConsoleContextSelector.js

Index: third_party/WebKit/Source/devtools/front_end/console/ConsoleContextSelector.js
diff --git a/third_party/WebKit/Source/devtools/front_end/console/ConsoleContextSelector.js b/third_party/WebKit/Source/devtools/front_end/console/ConsoleContextSelector.js
index e9fc42f18cbeff48b8ee9ad1efff0e47d21beb67..614e361943b992582224c76131f9ba12f231a86a 100644
--- a/third_party/WebKit/Source/devtools/front_end/console/ConsoleContextSelector.js
+++ b/third_party/WebKit/Source/devtools/front_end/console/ConsoleContextSelector.js
@@ -34,7 +34,8 @@ WebInspector.ConsoleContextSelector.prototype = {
         var result;
         if (executionContext.isDefault) {
             if (executionContext.frameId) {
-                var frame = executionContext.target().resourceTreeModel.frameForId(executionContext.frameId);
+                var resourceTreeModel = WebInspector.ResourceTreeModel.fromTarget(executionContext.target());

[pfeldman, 2016/07/07 17:44:14]

What we want here is a friendly executionContext name that should have been set
on the executionContext itself (executionContext.name). So it seems like we need
to fix it elsewhere.

The rule of thumb is that if this code should work in Node, it should not
mention resourceTreeModel. And if it mentions it, it is not nullable.

[eostroukhov-old, 2016/07/13 23:30:59]

Not sure if such a fix is in scope for this refactoring...

+                var frame = resourceTreeModel && resourceTreeModel.frameForId(executionContext.frameId);
                 result =  frame ? frame.displayName() : executionContext.label();
             } else {
                 result = executionContext.target().decorateLabel(executionContext.label());