Bug 105537 - Web Inspector: remove redundant calls to InspectorState::setObject()
Summary: Web Inspector: remove redundant calls to InspectorState::setObject()
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Andrey Kosyakov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-12-20 07:55 PST by Andrey Kosyakov
Modified: 2012-12-20 08:03 PST (History)
8 users (show)

See Also:

Attachments
Patch (4.67 KB, patch)
2012-12-20 08:00 PST, Andrey Kosyakov 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Kosyakov 2012-12-20 07:55:50 PST 
There's no point in calling InspectorState::setObject(prop, object) after getting object with InspectorState::getObject().
Comment 1 Andrey Kosyakov 2012-12-20 08:00:27 PST 
Created attachment 180342 [details]
Patch
Comment 2 Andrey Kosyakov 2012-12-20 08:03:37 PST 
Never mind. We need those setObjects() to update cookie. This might be a bit confusing, though.