Bug 45680 - [WINCE] Add FrameNetworkingContextWinCE::blockedError
: [WINCE] Add FrameNetworkingContextWinCE::blockedError
Status: RESOLVED FIXED
: WebKit
WebKit Misc.
: 528+ (Nightly build)
: Other Other
: P2 Normal
Assigned To:
:
:
:
:
  Show dependency treegraph
 
Reported: 2010-09-13 09:58 PST by
Modified: 2010-09-16 08:24 PST (History)


Attachments
Patch (1.96 KB, patch)
2010-09-13 10:00 PST, Patrick R. Gansterer
eric: review-
Review Patch | Details | Formatted Diff | Diff
Patch (2.02 KB, patch)
2010-09-15 01:46 PST, Patrick R. Gansterer
no flags Review Patch | Details | Formatted Diff | Diff


Note

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


Description From 2010-09-13 09:58:12 PST
see patch
------- Comment #1 From 2010-09-13 10:00:57 PST -------
Created an attachment (id=67420) [details]
Patch
------- Comment #2 From 2010-09-14 23:51:32 PST -------
(From update of attachment 67420 [details])
I'm confused.  Is this to fix a compile error?  What is this doing?  Should this be calling notImplemented()? since this seems like a dummy implementation.  Please update the ChangeLog to explain.
------- Comment #3 From 2010-09-15 01:46:27 PST -------
Created an attachment (id=67657) [details]
Patch
------- Comment #4 From 2010-09-16 07:46:07 PST -------
(From update of attachment 67657 [details])
View in context: https://bugs.webkit.org/attachment.cgi?id=67657&action=prettypatch

> WebKit/wince/WebCoreSupport/FrameNetworkingContextWinCE.cpp:55
> +WebCore::ResourceError FrameNetworkingContextWinCE::blockedError(const WebCore::ResourceRequest&) const
> +{
> +    notImplemented();
> +    return WebCore::ResourceError();
> +}

There's no need for any of the "WebCore::"s here, since you have "using namespace WebCore;" near the top of the file.
------- Comment #5 From 2010-09-16 07:48:50 PST -------
(From update of attachment 67657 [details])
(In reply to comment #4)
> There's no need for any of the "WebCore::"s here, since you have "using namespace WebCore;" near the top of the file.
You're wrong. ;-) E.g: http://msdn.microsoft.com/en-us/library/ms919861.aspx
------- Comment #6 From 2010-09-16 07:50:45 PST -------
(In reply to comment #4)
> (From update of attachment 67657 [details] [details])
> > WebKit/wince/WebCoreSupport/FrameNetworkingContextWinCE.cpp:55
> > +WebCore::ResourceError FrameNetworkingContextWinCE::blockedError(const WebCore::ResourceRequest&) const
> > +{
> > +    notImplemented();
> > +    return WebCore::ResourceError();
> > +}
> 
> There's no need for any of the "WebCore::"s here, since you have "using namespace WebCore;" near the top of the file.
I can remove it at WebCore::ResourceError, but it's strange to have it only on some places...
------- Comment #7 From 2010-09-16 07:56:16 PST -------
(In reply to comment #5)
> (From update of attachment 67657 [details] [details])
> (In reply to comment #4)
> > There's no need for any of the "WebCore::"s here, since you have "using namespace WebCore;" near the top of the file.
> You're wrong. ;-) E.g: http://msdn.microsoft.com/en-us/library/ms919861.aspx

Oh my! I've never heard of that function.

> I can remove it at WebCore::ResourceError, but it's strange to have it only on some places...

OK.
------- Comment #8 From 2010-09-16 08:24:14 PST -------
(From update of attachment 67657 [details])
Clearing flags on attachment: 67657

Committed r67620: <http://trac.webkit.org/changeset/67620>
------- Comment #9 From 2010-09-16 08:24:19 PST -------
All reviewed patches have been landed.  Closing bug.