<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>146457</bug_id>
          
          <creation_ts>2015-06-30 09:14:02 -0700</creation_ts>
          <short_desc>[Win] webViewAddMessageToConsole always gets 1 for isError</short_desc>
          <delta_ts>2015-06-30 10:34:07 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Brent Fulgham">bfulgham</reporter>
          <assigned_to name="Brent Fulgham">bfulgham</assigned_to>
          <cc>achristensen</cc>
    
    <cc>bfulgham</cc>
    
    <cc>commit-queue</cc>
    
    <cc>mkwst</cc>
    
    <cc>peavo</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1105684</commentid>
    <comment_count>0</comment_count>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2015-06-30 09:14:02 -0700</bug_when>
    <thetext>The code that calls the Windows API &quot;webViewAddMessageToConsole&quot; has a hard coded &quot;isError = True&quot; argument. This causes console output on Windows to improperly show errors in the many cases where the log content is informational.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1105685</commentid>
    <comment_count>1</comment_count>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2015-06-30 09:14:31 -0700</bug_when>
    <thetext>&lt;rdar://problem/21606395&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1105689</commentid>
    <comment_count>2</comment_count>
      <attachid>255823</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2015-06-30 09:24:43 -0700</bug_when>
    <thetext>Created attachment 255823
Correct argument to delegate method.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1105717</commentid>
    <comment_count>3</comment_count>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2015-06-30 10:34:07 -0700</bug_when>
    <thetext>Committed in r186121. &lt;http://trac.webkit.org/changeset/186121&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>255823</attachid>
            <date>2015-06-30 09:24:43 -0700</date>
            <delta_ts>2015-06-30 10:24:38 -0700</delta_ts>
            <desc>Correct argument to delegate method.</desc>
            <filename>isError.patch</filename>
            <type>text/plain</type>
            <size>2000</size>
            <attacher name="Brent Fulgham">bfulgham</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJLaXQvd2luL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2Uv
V2ViS2l0L3dpbi9DaGFuZ2VMb2cJKHJldmlzaW9uIDE4NjExNCkKKysrIFNvdXJjZS9XZWJLaXQv
d2luL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDE1LTA2LTMw
ICBCcmVudCBGdWxnaGFtICA8YmZ1bGdoYW1AYXBwbGUuY29tPgorCisgICAgICAgIFtXaW5dIHdl
YlZpZXdBZGRNZXNzYWdlVG9Db25zb2xlIGFsd2F5cyBnZXRzIDEgZm9yIGlzRXJyb3IKKyAgICAg
ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE0NjQ1NworICAgICAg
ICA8cmRhcjovL3Byb2JsZW0vMjE2MDYzOTU+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgKiBXZWJDb3JlU3VwcG9ydC9XZWJDaHJvbWVDbGllbnQuY3Bw
OgorICAgICAgICAobWVzc2FnZUlzRXJyb3IpOiBBZGRlZC4KKyAgICAgICAgKFdlYkNocm9tZUNs
aWVudDo6YWRkTWVzc2FnZVRvQ29uc29sZSk6IE9ubHkgcGFzcyB0cnVlIGZvcgorICAgICAgICBp
c0Vycm9yIGlmIHRoZSBtZXNzYWdlIGlzIGFuIGVycm9yIG1lc3NhZ2UuCisKIDIwMTUtMDYtMzAg
IFBlciBBcm5lIFZvbGxhbiAgPHBlYXZvQG91dGxvb2suY29tPgogCiAgICAgICAgIFtXaW5DYWly
b10gQ29tcGlsZSBlcnJvciwgV2ViRWRpdG9yQ2xpZW50OjpkaWRBcHBseVN0eWxlKCkgc2hvdWxk
IG5vdCBoYXZlIGFueSBwYXJhbWV0ZXJzLgpJbmRleDogU291cmNlL1dlYktpdC93aW4vV2ViQ29y
ZVN1cHBvcnQvV2ViQ2hyb21lQ2xpZW50LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViS2l0
L3dpbi9XZWJDb3JlU3VwcG9ydC9XZWJDaHJvbWVDbGllbnQuY3BwCShyZXZpc2lvbiAxODYwMDIp
CisrKyBTb3VyY2UvV2ViS2l0L3dpbi9XZWJDb3JlU3VwcG9ydC9XZWJDaHJvbWVDbGllbnQuY3Bw
CSh3b3JraW5nIGNvcHkpCkBAIC0zMzQsNiArMzM0LDExIEBAIHZvaWQgV2ViQ2hyb21lQ2xpZW50
OjpzZXRSZXNpemFibGUoYm9vbAogICAgIH0KIH0KIAorc3RhdGljIEJPT0wgbWVzc2FnZUlzRXJy
b3IoTWVzc2FnZUxldmVsIGxldmVsKQoreworICAgIHJldHVybiBsZXZlbCA9PSBNZXNzYWdlTGV2
ZWw6OkVycm9yOworfQorCiB2b2lkIFdlYkNocm9tZUNsaWVudDo6YWRkTWVzc2FnZVRvQ29uc29s
ZShNZXNzYWdlU291cmNlIHNvdXJjZSwgTWVzc2FnZUxldmVsIGxldmVsLCBjb25zdCBTdHJpbmcm
IG1lc3NhZ2UsIHVuc2lnbmVkIGxpbmVOdW1iZXIsIHVuc2lnbmVkIGNvbHVtbk51bWJlciwgY29u
c3QgU3RyaW5nJiB1cmwpCiB7CiAgICAgVU5VU0VEX1BBUkFNKGNvbHVtbk51bWJlcik7CkBAIC0z
NDIsNyArMzQ3LDcgQEAgdm9pZCBXZWJDaHJvbWVDbGllbnQ6OmFkZE1lc3NhZ2VUb0NvbnNvbAog
ICAgIGlmIChTVUNDRUVERUQobV93ZWJWaWV3LT51aURlbGVnYXRlKCZ1aURlbGVnYXRlKSkpIHsK
ICAgICAgICAgQ09NUHRyPElXZWJVSURlbGVnYXRlUHJpdmF0ZT4gdWlQcml2YXRlOwogICAgICAg
ICBpZiAoU1VDQ0VFREVEKHVpRGVsZWdhdGUtPlF1ZXJ5SW50ZXJmYWNlKElJRF9JV2ViVUlEZWxl
Z2F0ZVByaXZhdGUsICh2b2lkKiopJnVpUHJpdmF0ZSkpKQotICAgICAgICAgICAgdWlQcml2YXRl
LT53ZWJWaWV3QWRkTWVzc2FnZVRvQ29uc29sZShtX3dlYlZpZXcsIEJTdHJpbmcobWVzc2FnZSks
IGxpbmVOdW1iZXIsIEJTdHJpbmcodXJsKSwgdHJ1ZSk7CisgICAgICAgICAgICB1aVByaXZhdGUt
PndlYlZpZXdBZGRNZXNzYWdlVG9Db25zb2xlKG1fd2ViVmlldywgQlN0cmluZyhtZXNzYWdlKSwg
bGluZU51bWJlciwgQlN0cmluZyh1cmwpLCBtZXNzYWdlSXNFcnJvcihsZXZlbCkpOwogICAgIH0K
IH0KIAo=
</data>
<flag name="review"
          id="280913"
          type_id="1"
          status="+"
          setter="zalan"
    />
          </attachment>
      

    </bug>

</bugzilla>