Bug 115054

Summary:

Remove redundant code in ViewportArguments::resolve

Product:

WebKit

Reporter:

Jacky Jiang <jkjiang>

Component:

DOM

Assignee:

Jacky Jiang <jkjiang>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, esprehn+autocc, kenneth, rwlbuis, staikos, tmpsantos

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	kenneth: review+

Jacky Jiang

Reported 2013-04-23 13:20:06 PDT

In ViewportArguments.cpp ViewportArguments::resolve(), "result.orientation = orientation" appears twice which looks redundant to me, removing one of them.

Attachments
Patch (1.29 KB, patch) 2013-04-23 13:36 PDT, Jacky Jiang	kenneth: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Jacky Jiang

Comment 1 2013-04-23 13:36:38 PDT

Created attachment 199322 [details] Patch

Jacky Jiang

Comment 2 2013-04-23 14:00:38 PDT

Committed r148990: <http://trac.webkit.org/changeset/148990>

Note You need to log in before you can comment on or make changes to this bug.