webbrowser-app

Code review comment for lp:~abreu-alexandre/webbrowser-app/webapp-navigation-handling

  1. webapp-navigation-handling
  2. Merge into trunk
Revision history for this message
Olivier Tilloy (osomon) wrote :

82 + var matched = false;

Instead of using an intermediate boolean var, how about initially setting action to WebView.IgnoreRequest just before the for loop, and if a given include pattern matches, set it to WebView.AcceptRequest ?

« Back to merge proposal